Excel VBA 質問スレ Part72

2021/07/18(日) 08:42:15.37

!extend:checked:vvvvv:1000:512
↑同じ内容を2行貼り付けるナリ

ExcelのVBAに関する質問スレナリ
コード書き込みや作成依頼もOKナリ

※前スレ
Excel VBA 質問スレ Part70
https://mevius.5ch.net/test/read.cgi/tech/1616072923/
Excel VBA 質問スレ Part71
https://mevius.5ch.net/test/read.cgi/tech/1621914481/

VIPQ2_EXTDAT: checked:vvvvv:1000:512:: EXT was configured

2021/09/10(金) 16:41:57.41

>>936
追記
あなたの考えどおりに動く、offsetって関数があるからそっち使ったらいいんじゃないかな

2021/09/10(金) 16:42:03.28

>>936
1,1にすればなるだろ
0だから行列がそれぞれマイナス1されてるんじゃね？
0入れたことないから予想だけど

2021/09/10(金) 16:52:02.92

>>937-938
Cellsの引数はそのまま1つ目2つ目と考える事にします
変数名は質問用に適当にしましたすみません
offsetが距離1つ先2つ先なのでこちらを使用しようと思います
ありがとうございました

>>939
1,1にすればなりますが論点はそこではありません

2021/09/10(金) 16:54:59.11

0から始まる配列なんかを入れたいという感じか

2021/09/10(金) 17:05:41.13

>>934
変数が全角と半角混在してんじゃね
つかコードそのまま張れよ

2021/09/10(金) 17:13:14.83

>>942
cの宣言してないだけ

2021/09/10(金) 17:47:04.78

>>920
下に書いたようにParamArrayを使ったメソッドが定義されたクラスモジュールを作って、
それに宣言した変数を渡してみたけどエラーは起きないよ。

後、「参照渡し」で有る事を気にしてるけど、ParamArrayが指定された引数は参照渡しになる。
例えばtestSub内でargs(0)に適当な値を割り当てると、呼び出し元のcaller関数のaの値も変わる。

クラスモジュール(class.cls)
Public Sub testSub(ParamArray args())
Dim arg
For Each arg In args
If VarType(arg) = vbString Then Debug.Print arg;
Next
End Sub

標準モジュール
Sub caller()
Dim a As String: a = "a"
Dim b As String: b = "b"
Dim c As String: c = "c"
Dim Class As New Class
Class.testSub a, b, c
'イミディエイトウインドウ: abc
End Sub

だからこっちが質問内容を取り違えてるんだと思うんだけど、どこが違う？
それとエラーメッセージを教えて欲しい
もし意味が分かるならコンパイルエラーか実行時エラーかも知りたい

2021/09/10(金) 18:37:19.42

>>934
Columnsで列ごとに取り出す方法で短く、分かりやすくしてみた

Sub PrintOut()
Dim Column As Range
For Each Column In Range("D1:H4").Columns　'指定範囲を一列ごと取り出す
Range("A1:A4").Value = Column.Value　'取り出した一列の値をA1:A4に張り付ける
ActiveWindow.SelectedSheets.PrintOut
Next
End Sub

ちなみにこういうのはActive***とセル番号指定じゃなくて、
シートモジュールとテーブル使った方がメンテしやすくなる。

A1:A4とD1:H4にそれぞれテーブルを作成（セルを選択して「ホーム」リボンの「テーブルとして書式設定」）し、
A1:A4のテーブル名を「コピー先」、D1:H4のテーブル名を「コピー元一覧」にする。
そのシートのシートモジュールに次の様なメソッドを定義する

Sub MyPrintOut()
Dim Column As ListColumn
For Each Column In ListObjects("コピー元一覧").ListColumns
ListObjects("コピー先").Value = Column.DataBodyRange.Value
ActiveWindow.SelectedSheets.PrintOut
Next
End Sub

やってることは一緒だけど、
・コピペする（される）セルの番地が変わっても動く
・D1:H4の範囲が増えてもテーブルを範囲を変更するだけで動く
・「コピー元一覧」から「コピー先」に張り付けてるのがソースを見ただけで分かる
とメリットだらけ

2021/09/10(金) 21:16:37.69

>>937
> 余談だけどその変数名は絶対にやめた方がいい
> VBAには既にRangeって単語があるんだから
なぜ？
VBAは型の名前空間と変数等の名前空間違うから問題なく動くぞ

2021/09/10(金) 21:38:37.34

>>946
そういう問題じゃないんですよ

2021/09/10(金) 21:40:05.19

>>947
だからどういう問題かを書けないなら黙ってなよ

2021/09/10(金) 21:43:02.76

>>948
動くかどうかという話ではないので論点はそこではありません

2021/09/10(金) 22:07:47.67

動かすだけならグローバルに配列つくればいいわな

2021/09/10(金) 22:16:10.75

>>949
ああ説明できないのかしたくないのかは知らんけどそういうことしか書けないのな
まあ君はそれでいいと思うよw

2021/09/10(金) 22:19:16.50

>>951
可読性の面もあるし、変数名の意味付けの面もあるだろう
Rangeって書かれても、どんな目的で何のRangeを変数に入れたのかが分かるように書くのが綺麗なコード
知らんけど

2021/09/10(金) 22:20:43.81

難癖つけててワロタ

2021/09/10(金) 22:37:14.80

>>952
> Rangeって書かれても、どんな目的で何のRangeを変数に入れたのかが分かるように書くのが綺麗なコード
例えば指定された範囲にいくつかの装飾を適用する
Private Sub 装飾(Range As Range)
With Range
...
End With
End Sub
君なら引数の名前をどうする？

2021/09/10(金) 22:56:49.10

Excelの装飾ってのがよくわからんけど、書式のこと？

関数っぽい書き方してるからこんなんでいいんじゃないの

Private Sub SetFormat_to_TargetRange(ByVal TargetRange As Range)
With TargetRange
...
End With
End Sub

2021/09/10(金) 23:04:12.45

文脈的に明らかならRange As Rangeいいと思うけどね。

あと、「Rangeオブジェクト」である以上の情報がない変数や引数に対して無理やり名づけようとしても、
結局TargetRange的な抽象的な名前になっちゃう事もある。

2021/09/10(金) 23:06:14.39

>>955
流石にそれは冗長すぎて逆に読みにくいだろ・・・

2021/09/10(金) 23:11:20.56

>>954
そこはtarget as rangeでいいと思う
一番読みやすい

自分しか使わない場合でもrange　as　rangeは使わないかな
vbaの場合、引数なしの関数と変数って見分けが付きにくい
ってかなんだこれエラーになるぞ

Sub foo()
Dim Calculate As Long
Calculate
End Sub

これはエラーにならない
Sub foo()
Dim Calculate As Long
End Sub

これもエラーにならない
Sub foo()
Calculate
End Sub

変数名と関数名がかぶると変数が優先される？
まぁ何にしろ、こんな事は避けたいから俺はかぶらないようにするけどね

2021/09/10(金) 23:15:36.34

多少長くても要素要素は省いちゃメンテが大変になるから多少冗長でも構わないスタンスで

単語の簡素化はしてもいいなら関数名をSetFmt_TgtRngとかにするかもね

あとそもそも、挙げられた例は不適切では？
今回話題になってるのは呼び出し先の(汎用的に使える)関数での変数名ではなくて、呼び出し元での変数名かと

2021/09/10(金) 23:25:05.99

>>958
再定義（シャドーイング）されてるだけ。
親スコープから継承されてる宣言名（今回で言えばオブジェクト名.Calculate）を再定義すると、定義が上書きされる。

Calculateは整数として再定義されたので、そのスコープではCalculateだけなら暗黙的に整数になる。
明示的に「オブジェクト名.Calculate」とすればメソッドとしてのCalculateを使用できる

2021/09/11(土) 00:00:09.96

personal.xlsbを複数の端末で共有したいので共有フォルダーにおいています．
XLSTARTのpersonal.xlsbは削除して共有フォルダーのpersonal.xlsbへのショートカットをおいています．
エクセルが起動するときにpersonal.xlsbを開くかどうか毎回聞いてきます
（セキュリティのマクロの設定は「警告を表示して全てのマクロを無効にする」です）

毎回開くを押すのが面倒なので共有フォルダーのpersonal.xlsbだけ常時開くような設定はできないでしょうか?

2021/09/11(土) 07:08:07.40

>>960
ありがとうございます
こんな機能あったのか、名前被るとエラーになると思って避けてたわ

2021/09/11(土) 08:32:29.27

>>962
スコープを持った言語ではたいてい内側のスコープが優先されるよ
中にはあえてエラーにする言語もあるけど
https://docs.microsoft.com/ja-jp/dotnet/csharp/misc/cs0136

2021/09/11(土) 09:37:25.87

>>955
関数の大きさにもよるな
小さくてすぐ上に引数設定して
型が見えてるようなのはvalueでいいと思う
少しごちゃごちゃして見辛いのは
targetRangeでいいと思うし
更にもう少し大きくなって業務的な
意味を持つようになったらその名前付ければ
いいんじゃないかな

2021/09/11(土) 10:18:08.93

>>936です
質問用に最少構成にする時点でもう少し変数名に気を付けるべきでした
変な流れになってすみません

2021/09/11(土) 10:49:48.68

>>964
Valueはないわ…

2021/09/11(土) 12:17:24.93

>>966
そうだよな。value指定ではフォーマット変更は無理だよな

2021/09/11(土) 13:14:28.26

Valueだけはねぇわ
targetとtmpとi・j・kとrくらいしか意味の薄い変数は使わないようにしないと

2021/09/11(土) 13:57:33.58

m,nも

2021/09/11(土) 14:37:20.09

i・j・k

顔文字かと思った

2021/09/12(日) 11:12:03.90

valueがダメだと言ってる奴は恐らくプロパティとか
プロパティの使えない他言語とか使ったことのない井の中の蛙という奴だな

VBAしかやってないジジイにはよくあること

2021/09/12(日) 11:42:37.67

>>971
君、韓国人？

2021/09/12(日) 11:48:24.82

韓国をばかにするな！ITに関しては日本より進んでいるぞ

2021/09/12(日) 12:10:04.57

どこ見て言ってんだか

2021/09/12(日) 12:15:32.69

ここ見ててんだよ

2021/09/12(日) 12:55:49.25

>>971
またチンケなマウント取りが来たなw
>>954の引数としてValueはないわって話
値としての用途ならValueもよく使うよ

2021/09/12(日) 13:36:49.25

オブジェクト型のRange型なのに、Valueを変数名にするのはVBAの常識的にNGなのは自明

2021/09/12(日) 14:09:17.30

>>977

> Valueを変数名にする
変数名の適否ではなくって、ByValで値引き渡しにしたらもとの範囲のフォーマットを変更できないという話じゃないのか？

2021/09/12(日) 16:11:41.17

>>978
今問題にしてたのは変数（引数）名であって引数の渡し方ではないのだが
参照型と参照渡しとちゃんと区別して理解してる？

2021/09/12(日) 16:21:24.92

元の元は>>936(私)でdim range as rangeに対してsetしたのですが
引数としてrange as rangeとされている状態でsetする事なんてあるのですか？
普通は無いなら途中から引数の話になるのもずれてません？

2021/09/12(日) 16:31:54.19

そう言えばデコレーターパターンとかでは
valueはよく見るけどreferって見ないよね
なんでだろ？

2021/09/12(日) 16:37:26.61

>>979
955の話じゃないのか？
誤解してたらすまん。

2021/09/12(日) 18:56:51.30

>>981
それは引数が何かの値という意味で
プログラム的な参照とか値とかは関係ないからさ

2021/09/12(日) 19:28:40.93

>>980
お前さんの普通がわからんけど、複数のレンジを返したいならそういうケースもあると思うよ
そもそも>>952は名前付けの話でSet云々の話に限定はしてないと思うし

2021/09/12(日) 21:17:53.40

ttps://whois.nic.ad.jp/cgi-bin/whois_gw?key=182.22.16.251&codecheck-sjis=%E3%81%AB%E3%81%BB%E3%82%93%E3%81%AD%E3%81%A3%E3%81%A8%E3%82%8F%E3%83%BC%E3%81%8F%E3%81%84%E3%82%93%E3%81%B5%E3%81%89%E3%82%81%E3%83%BC%E3%81%97%E3%82%87%E3%82%93%E3%81%9B%E3%82%93%E3%81%9F%E3%83%BC&submit=
に表示されているネットワーク名「YAHOO-NET」を取得しようとしているのですが、
Cells(1, 2) = objIE.Document.getElementsByTagName("pre").innerText
ではメソッドがサポートされいないとエラー表示されます。
何がいけないのでしょうか？

**名無し募集中。。。** (ﾜｯﾁｮｲ 9e8c-xZL4) · 2021/09/12(日) 21:29:23.70

普段使ってないから問題点がこれ意外にもあるかわからないが
Cells(1, 2) = objIE.Document.getElementsByTagName("pre")(1).innerText
これでどう？

2021/09/12(日) 21:29:29.14

VBAとJavaScriptが、混ざっちゃった感じかな

2021/09/12(日) 22:10:03.90

n時間後にエンターキーを押すプログラムってどう作るんだ？初心者にもわかりやすく誰か教えて

2021/09/12(日) 22:16:01.50

「n時間後にアラームを設定して、鳴ったらENTERを押せ！」
と、命令する

2021/09/12(日) 22:27:13.14

>>986
ありがとうございます。
「オブジェクト変数がセットされていません」というエラーメッセージが出ており、検証はできておりませんが、いけそうな気がします。
ページ遷移したらobjIE.Documentに遷移先の情報が自動で設定されないのかもしれません。

**デフォルトの名無しさん** (JP 0Hc9-dGLa) · 2021/09/12(日) 22:28:25.73

>>988
基本はこうだけど、このままだとタイマーを仕掛けたのを忘れてて、ほかの作業をしてる時に急にEnterが押されてびっくりすると思うよ

Sub n時間後にEnterを押す()
　　n = 1　'n時間を指定
　　n時間後 = Now + TimeValue(n & ":00:00")　'n時間後が何時何分か計算する
　　Do
　　　　DoEvents
　　Loop Until Now >= n時間後　'n時間誤まで待つ
　　SendKeys "~"　'Enterを押す
End Sub

2021/09/12(日) 23:27:46.85

>>990
(1)じゃなかった
(0)だった

2021/09/13(月) 01:05:28.69

>>991
せめてOnTime使えよ

**デフォルトの名無しさん** (ｽﾌｯ Sd0a-dGLa) · 2021/09/13(月) 01:09:51.01

OnTimeはあとからタイマーの設定を確認したり停始する方法がないから使い勝手が悪すぎるのがなあ

2021/09/13(月) 01:34:41.43

配列に保存しとくか
シートに書き出しておく
それを元に確認したりキャンセルする

**デフォルトの名無しさん** (ｽﾌｯ Sd0a-dGLa) · 2021/09/13(月) 02:28:08.29

>>990
URLはここまででいいんだよ
https://whois.nic.ad.jp/cgi-bin/whois_gw?key=182.22.16.251

2021/09/13(月) 07:15:43.89

>>992, 996
ありがとうございます！

2021/09/13(月) 07:30:13.42

つぎ
https://mevius.5ch.net/test/read.cgi/tech/1631485799/l50

2021/09/13(月) 07:41:47.37

>>998
うむ、ご苦労

2021/09/13(月) 09:30:38.14

1000ならコロナ収束

**1001** · Over 1000

このスレッドは１０００を超えました。
新しいスレッドを立ててください。
life time: 57日 0時間 48分 23秒