VBAなんでも質問スレ Part3

**デフォルトの名無しさん** · 2021/09/09(木) 19:46:21.53

VBAを使った質問ならなんでもござれ
本来の対象であるオフィスアプリを操作する以外の話もＯＫ

ゲーム作り、Webアクセス、外部アプリの操作
COM(ActiveX)、Win32API、.NET Framework、DirectXなどなど
VBAで実行するものであればなんでも質問してください

VBAを開発環境としていろいろ作っちゃいましょう

前スレ
VBAなんでも質問スレ Part2
https://mevius.5ch.net/test/read.cgi/tech/1432173164/

VBAなんでも質問スレ
http://peace.2ch.net/test/read.cgi/tech/1342087380/

**デフォルトの名無しさん** · 2021/09/09(木) 19:50:57.85

関連スレ

Excel VBA 質問スレ Part72
https://mevius.5ch.net/test/read.cgi/tech/1626565335/

Access VBA 質問スレ Part2
https://mevius.5ch.net/test/read.cgi/tech/1544620479/

古代VBプログラマ質問スレ（Ver.6.0 まで） part65 [転載禁止]c2ch.net
https://mevius.5ch.net/test/read.cgi/tech/1440930335/

Excel総合相談所 145
https://mevius.5ch.net/test/read.cgi/bsoft/1624085733/

**デフォルトの名無しさん** · 2021/09/09(木) 19:51:44.70

前スレ落ちたままだったから建てた

**デフォルトの名無しさん** · 2021/09/17(金) 20:28:55.22

プロシージャ終わったあとに変数の中身を保持するにはどうすればいいですか
ボタンとかで変数を後で呼び出したいです

**デフォルトの名無しさん** · 2021/09/17(金) 20:58:48.95

シートのセルに値を書き込んでおく

**デフォルトの名無しさん** · 2021/09/18(土) 00:17:54.33

>>4
Static変数をつかう
プロシジャの外で変数を定義する
VBA以外のどこか（シートとか）に保存する

好きな方法えらべ

**デフォルトの名無しさん** · 2021/10/20(水) 22:37:31.96

超初心者です
すみません、他スレでもきいたのですが

以下の対応をさせたいです
①シートを支店分だけ増やす
②シート名をそれぞれ「支店一覧」A2～A18の支店名にする
③作成したシートのB5に支店名を入れる
④作成したシートの表のA列が"0"になっている行を削除する
⑤作成したシートのオートフィルタを解除する

以下のものではエラーになりますが、どこが悪いのでしょうか

Sub シート作成
Dim 支店名 As Range
For Each 名前 In Worksheets("支店一覧").Range("A2：A18")
Worksheets("原本").Copy After:=Worksheets(Worksheets.Count)
With ActiveSheet
.Name = 支店名.Value
.Range("B5") = 支店名.Value
End With
　 Range("B3").AutoFilter 1, "0"
With Range("B3").CurrentRegion.Offset(1, 0)
.Resize(.Rows.Count - 1).EntireRow.Delete
　　Range("B3").AutoFilter
End With
Next 支店名

**デフォルトの名無しさん** · 2021/10/20(水) 22:49:26.57

④作成したシートの表のB列が"0"になっている行を削除する
の誤りです

コードの3行目は
For Each 支店名 In Worksheets("支店一覧").Range("A2：A18")
の誤りです

**デフォルトの名無しさん** · 2021/10/21(木) 01:14:54.35

>>7
エラーについて質問する時は、なんていうエラーが出たか、どの部分が黄色くなってるか書いて
プログラムは間違ってないから、おかしいのは支店一覧のA列に入ってる支店名と思われる

7 · 2021/10/26(火) 13:12:31.54

>>7です。
間があいてしまい、申し訳ございません。

マクロを実施すると、「支店一覧」Ａ２：Ａ１８のＡ２，Ａ３，Ａ４のシートまでは対応が完了するのですが、
その後Ａ５名のシートが作成されず、「原本（２）」というシートが作成されたあと、以下エラーとなります。

「実行時エラー'1004':
この名前は既に使用されています。別の名前を入力してください。」

>>9さんの仰る通りなのだと思うのですが、「支店一覧」をみるかぎりではおかしなところはみあたりません
（もちろんA2:A18に重複もありません）

**デフォルトの名無しさん** · 2021/10/26(火) 18:21:16.44

>>10
隠しシートか、支店名のところに隠れた行があるのでは？
きちんとデバッグしないと分からんけど。

11 · 2021/10/26(火) 18:27:36.54

簡単なデバッグとしてら、まずはwithステートメントを全てコメントアウトして、動くかどうか。
あとは支店名のところをシンプルにアルファベットのAからQとかに変えてみる。

**デフォルトの名無しさん** · 2021/12/16(木) 14:58:51.82

reportのインスタンスを作成して、property Letでprivate変数に配列を渡した後に、docmd.openreportを行うと変数が全部初期化されるのな。
reportに表示させるものを変数で与えるのは不可能なのかなあ？
やっぱり、クエリとか一時的なテープルが必要なのかな？Openargsで出来そうだけど、数が多いと非現実的だよね。

13 · 2021/12/16(木) 15:02:05.17

グローバル変数でやるべきかなあ？でも変なときに変数の寿命が来るから不安だよなあ。

**デフォルトの名無しさん** · 2022/02/13(日) 16:56:03.94

word vbaについて質問です。
本文↓
0001234567891

コード↓
Sub findText()
With Selection.Find
.Text = "1"
Do While .Execute
Selection.Comments.Add Range:=Selection.Range
Selection.TypeText Text:="あ"
Loop
End With
End Sub

特定の文字列を検索し、コメントを追加するマクロを作っているのですが、
コードを実行すると、本文1個目の「1」にコメントがついた後，
2ループ目の「Selection.Comments.～」で「5935 コメント、脚注および文末脚注は、本文にのみ追加できます。」というエラーが出ます。
コメント入力時にカーソルが本文外に出るのが原因かと思い、
ループの最後にカーソルを本文中（コメントを付けたワードの直後）に戻すようにしたいのですが
どのように記述したらいいのでしょうか？

**デフォルトの名無しさん** · 2022/03/01(火) 18:52:06.00

Excel VBAで質問です

現在開いているシートにマクロボタンを作り、実行させると…
現在シートのB列に検索値
別のシート(ABC)のB列の5行目～400行目までを検索して一致したセル番号が欲しいのですが
For cnt=1 To 10 samsu(cnt)=WorksheetFunction.Match(Cells(cnt,1),Worksheets(”ABC”).Range(Cells(5,1),Cells(400,1)),0)
…
next
このMatch関数の正しい書き方を教えてもらいたいのですが。

**デフォルトの名無しさん** · 2022/03/01(火) 20:16:20.95

B列ってcells(?,1)で合ってる？
もうcells(?,"B")しか使わないから忘れた

**デフォルトの名無しさん** · 2022/03/01(火) 22:26:50.78

>>17
すみません。
Bは2でした…
2に訂正しても、match関数としては動かないみたいです。

**デフォルトの名無しさん** · 2022/03/12(土) 08:01:57.51

質問です。

アクセスから書き込みパスワード付きエクセルファイルに保存するためにエクスポートします。

当然、単純にエクスポートするコードを書いたらエラーで止まります。
これが5年くらい前

最近、この手順を自動化するために
エクセルファイル側の書き込みパスワードを解除し、保存、
アクセスからエクスポートと、いうコードを書きました。

パスワードの再設定をしていないのに正常にエクスポートされ、
書き込みパスワードも正しくかかっています。

パスワード絡みの動作がおかしい気がしますが
心当たりはありますでしょうか？

**デフォルトの名無しさん** · 2022/03/12(土) 08:09:55.30

WiNdowsを10にかえたことも影響あるかもしれま追記
せん。

19 · 2022/04/09(土) 10:45:51.24

ウインドウズの更新かかったらさらに挙動が変わりましたw

質問の件ですが
もともとファイル内容の変更ではなく
新規作成で問題無い事に気づいたので
パスワードなしマクロファイルを作成→
マクロファイルからパスワード付きで新規(上書き)保存
と、二段階にしました。

**デフォルトの名無しさん** · 2022/05/01(日) 14:19:41.70

2日前まで動いていた自作のマクロが、昨日からウィルスと誤検出されて実行できなくなってしまいました
内容はseleniumでchromeを操作して特定のページを表示させるだけの比較的単純な物です

Windows Defenderの定義と、chromeとseleniumと.netは最新にしましたが改善されません
どうしたら誤検出だとシステムに教えることができるのでしょうか？

**デフォルトの名無しさん** · 2022/05/01(日) 14:58:43.07

署名

**デフォルトの名無しさん** · 2022/05/01(日) 19:48:46.19

デジタル証明書を作って読み込ませたら実行できるようになりました
ありがとうございました
それにしても、こんなに簡単に証明書を作れてしまうのなら、あんまり意味がない気もしますね

**デフォルトの名無しさん** · 2022/05/01(日) 19:50:53.15

とりあえず実行はできましたが、証明書を入れたら起動がとても遅くなってしまいました
誤検出だと知らせる方法もお願いします

**デフォルトの名無しさん** · 2022/05/06(金) 17:09:58.99

Excel VBAの質問です。
初心者で申し訳ありません。
yahoo知恵袋にベストアンサーにあったものなのです。

Sub Sample1()
Dim dt As Date

dt = Range("D11").Value
Range("D12").Value = GetNENDO(dt, 3)
MsgBox dt & " は " & GetNENDO(dt, 3) & " 年度です"
End Sub

Function GetNENDO(dt, x)
GetNENDO = Year(DateAdd("m", -x, dt))
End Function

Range("D12").Value = GetNENDO(dt, 3)の(dt, 3)が理解できません。
またYear(DateAdd("m", -x, dt))の-xとはどういうことでしょうか。

抽象的ですみません

**デフォルトの名無しさん** · 2022/05/06(金) 18:24:20.55

自作のファンクションでGetNENDOというのを作っていて、
このファンクションは引数に指定された日付dtに対してxか月前（-x）の日付の年を返却するみたいだね。
今回はセルD11に入っている日付の3か月前の年をD12に設定しているような動作だと思う
例）D11に「2022/2/3」が入っていればD12には「2021」が設定される

**デフォルトの名無しさん** · 2022/05/06(金) 19:49:29.47

知恵袋さんは回答するだけしといて解説が無いから初心者にはキツイよ

**デフォルトの名無しさん** · 2022/05/06(金) 22:15:14.38

カレンダー年から会計年に変換するVBAのようだが
こんなのワークシート関数で十分なのに有り難がって使う理由が分からんよ

**デフォルトの名無しさん** · 2022/05/07(土) 05:50:57.56

>>29
プログラミング全般に言えることだけど、単なる数式と比較して、関数化して名前が付けてあると、他人が見た時に用途や動作がわかりやすくなるという大きなメリットがある

**デフォルトの名無しさん** · 2022/05/07(土) 23:47:50.28

>>29
それ言い出したら、「全部0100010101000」で表現できるのにいちいちアセンブラとかありがたがって使う意味わからん、まで遡る。

抽象化は大事だよ。

あと、地の文に「有難がって」みたいなネガティブな印象の語を持ってきて、言いたいことを暗に伝えるのはやめたほうがいいと思う。

ゲスゴミがよく使うんだけど、「・・・などと訳のわからないことを言っており」とか、「行き過ぎた○○の是正を」とかの表現を使って責任逃れしつつ人や物にレッテルを貼る卑怯者に見える。

俺がゲスゴミって表現使ったのことでなんとなく俺のスタンス伝わるだろ？そういう類。

**デフォルトの名無しさん** · 2022/05/08(日) 10:19:01.14

>>29
ライブラリとか作ったり使ったりしたことなさそう

**デフォルトの名無しさん** · 2022/05/08(日) 10:51:36.83

>>26にある知恵遅れ袋の回答者が来てるようだな

**デフォルトの名無しさん** · 2022/05/08(日) 11:57:36.38

>>26
GetNENDはなんでわざわざ引数2つにしてるんだろうな
年度を求めるなら暦年との差は必ず3な訳で、
2つめの引数に3以外を指定するならGetNENDOって名前が不適切

>>33
https://detail.chiebukuro.yahoo.co.jp/qa/question_detail/q1063327550
これだろ？
質問の要件が「エクセルのVBAを使って」なんだからおかしくはない

**デフォルトの名無しさん** · 2022/05/08(日) 13:10:38.22

>>34
年度が4月始まりじゃない場面は業界によって色々ある

**デフォルトの名無しさん** · 2022/05/08(日) 14:40:26.99

>>35
それは事業年度
年度とだけ言う場合は4月始まりだよ

**デフォルトの名無しさん** · 2022/05/08(日) 15:13:04.89

その関数を何の用途に使ってるか部外者にはわからない
デフォルトが何かも職場によって違う
https://cdn.img-asp.jp/cms/351698_1_0_0_1.jpg

**デフォルトの名無しさん** · 2022/05/10(火) 02:24:40.53

知恵袋をたまに見てるとさ、無言で変な構文やなんかのサイトを張り付けて去っていく回答者いるけど、なんか不気味じゃない？？

**デフォルトの名無しさん** · 2022/05/10(火) 08:22:39.72

事業年度や会計年度は主体・組織によって全く異なる。
会社だと3月期決算の会社は4月始まりだが、
3月期決算でない会社は世の中にたくさんある。

学校年度だって国によって違う。
4月始まりの日本は少数派。

**デフォルトの名無しさん** · 2022/05/10(火) 09:44:48.97

自転車置き場の議論

**デフォルトの名無しさん** · 2022/05/10(火) 09:47:10.09

年度が4月始まりじゃないならなおさら二つ目の引数は馬鹿丸出しだろ
普通年度始まりの月とかにしないか？

**デフォルトの名無しさん** · 2022/05/10(火) 14:23:17.79

26です

>>27
ありがとうございました。
>>34のとおりの知恵袋です。

解説が欲しかったので助かりました

**デフォルトの名無しさん** · 2022/05/11(水) 08:32:22.33

>>42
VBAなんてExcelに該当のコード貼り付けてステップ実行もすぐできるんだからためそうぜ
「Year(DateAdd("m", -x, dt))」についてはヘルプに引数の説明が載ってるんだから見て理解しないと今後が思いやられるぞ

**デフォルトの名無しさん** · 2022/06/17(金) 09:07:00.42

今日の発見
Option Explicitの有無でサブフォームの選択の仕方が変わるのな
ちゃんと書かないとエラーになるコードもOption Explicitを消すと動く

**デフォルトの名無しさん** · 2022/06/17(金) 09:41:15.90

エラーにならないようにちゃんと書け

**デフォルトの名無しさん** · 2022/06/17(金) 10:05:18.56

>>45
行数が節約できるじゃん。それにOption Explicitを書かないとちゃんと動作するんだよ。
このことはどこかに書いてないかな？

**デフォルトの名無しさん** · 2022/06/17(金) 12:05:13.97

自分は業務で使うちょっとしたツールをVBA使う事があるけど
Option Explicitを書いてもちゃんと動くようにコーディングするのが普通だと思うから外すつもりはないけど
行数削減目的で外すとかは絶対ないわ
若いもんに作らせて外されててつけたら動かないようなもの持ってきたら作り直させるね

**デフォルトの名無しさん** · 2022/06/17(金) 13:12:56.41

エラーになるってことは変数宣言されてないのが含まれてるってことだよな
ちゃんとエラーで止まるコードと何となく動くけど実はバグってる（けど気づかない）コードとどっちが良いのか良く考えなよ

**デフォルトの名無しさん** · 2022/06/17(金) 13:56:57.20

まず「ちゃんと書いてない」部分が何かわからないから議論にならないでしょ
ただ>>44のVBAはちゃんとかかなくても動いてる（目的が達成できる）ならそれでいいと思う
自分とこの業務だったら拒否するけどね

44 · 2022/06/17(金) 16:46:52.28

購入した業務用のmdbのコードが余りに酷いからリファクタリングごっこしてるうちに見つけたんですよ。1200以上のモジュールをOption Explict化したら急にオブジェクトが見つからないとか言われて驚いたんですよ。変数宣言は1週間かけてチェック済なんですよねえ。不思議ですよねえ

**デフォルトの名無しさん** · 2022/06/17(金) 17:05:38.65

それはOption Explict化をミスってるんじゃないか

広域で定義（または利用されて暗黙的に定義）されてる変数を
ローカルで定義したとかの可能性が高い

つか1000以上のモジュールがあって販売してるようなプログラムとは思えん作りだな

**デフォルトの名無しさん** · 2022/06/17(金) 17:25:09.27

「ごっこ」って言葉を免罪符にしてるんだろうけど
直せないならやらなきゃいい

**デフォルトの名無しさん** · 2022/06/17(金) 22:53:08.85

>>44
Option Explicitはあえてエラーをエラーとするようにするための記述だからな
HTMLでいうstrictだよ

44 · 2022/06/18(土) 08:43:08.28

>>51
> 広域で定義（または利用されて暗黙的に定義）されてる変数を
> ローカルで定義したとかの可能性が高い
でも、Compileは通るんですよねえ。
あとで実験してみますね。

>>つか1000以上のモジュールがあって販売してるようなプログラムとは思えん作りだな
愚痴らせて下さい。
金取るくせにOption Expliciじゃないモジュールが多数。変数がなんでもかんでもVariant。変数の使い回しし放題。インデントがめちゃくちゃ。
命名規則がメチャクチャでバラバラ。
古いコードがコメントアウトで多数残ってる。何十行ものコメントアウトがたくさんあって読みにくいたらありゃしない。
戻り値を返さないFunction。戻り値が返ったと思ったら、受け取る変数はどこでも読まれない。Subでいいだろうが。
宣言だけして利用されない変数多数。空のプロシジャーを呼び出すなど無意味なコード。

とりあえず読み書きされない変数は全部削除しましたが3日かかりました。
全部変数宣言するには1週間かかりました。
分析ツールは他にも多数のDead Codeを指摘していますが、全部はできません。
酷すぎます。

44 · 2022/06/18(土) 08:49:13.02

>>52
おいら、プログラマーじゃないんだよ。だから「ごっこ」と書いた。でも本はちゃんと買ってあるよ。
業務で利用しているAccessを改良しているうちに泥沼にはまったんだよ。
Accedssを初めて使ったときは変数という言葉も知らなかったよ。

**デフォルトの名無しさん** · 2022/06/18(土) 10:39:04.86

なんにせよAccess VBAはスレチ
Explicitの綴り間違えすぎてるのも怪しいし

**デフォルトの名無しさん** · 2022/06/18(土) 11:58:55.45

ここはVBAなら何でもありだと思うけどな
「Excel VBA」スレと間違えてないか？

**デフォルトの名無しさん** · 2022/06/18(土) 12:55:52.02

>>57
まさにそっちと間違えてた
ごめんなさい

**デフォルトの名無しさん** · 2022/06/18(土) 13:38:21.35

>>54
あるあるだね
とくに若手に作らせるとそんなもんだよ
命名規則とかインデントとか、とにかく余裕がなくて勉強してるヒマがない
納期に追われて、とりあえず動く物を作るのでせいいっぱい

まあ自分の担当になったんなら、気長に直していくこったな
俺はひどいコード直すの嫌いじゃない
最近はいいツールあるしね
インデントぐらいならほぼ一発で綺麗に直してくれるし

**デフォルトの名無しさん** · 2022/06/18(土) 18:52:07.80

購入品なら不具合出たら責任取らせればそれで済む話
リファクタリングなんて自分の勉強目的以外は無意味
自分の勉強目的ならここに愚痴だか何だかわからない書き込みをするのも無意味

44 · 2022/06/18(土) 22:31:48.14

書いてみて反応がなければよし、あれば語らう。
自由な空間で試行錯誤をすることを否定するなよ老人脳。

**デフォルトの名無しさん** · 2022/06/19(日) 10:16:32.15

スレも理解できない奴が他人を批判するのもどうかと思うけどな
そもそもソースすら満足に直せない奴も低能なのではないか

44 · 2022/06/19(日) 13:49:52.36

ソースは差し障りがないように改変して皆さんに相談してみたいと思います

**デフォルトの名無しさん** · 2022/06/19(日) 14:32:35.71

>Option Explicitの有無でサブフォームの選択の仕方
選択の仕方ってのがどういった状況を指してるのかわからんけど
正常に動作してる状況で宣言の有無でそれが変わる例が思いつかんわ
そこだけでもコード曝せ

44 · 2022/06/19(日) 14:36:56.35

私も非常に驚いたので晒したいです。
今日は家族サービスでできないので近いうちに。

**デフォルトの名無しさん** · 2022/06/19(日) 14:56:20.29

>>44
それは解決とは言えんのだな

バグり上等でとりあえず動けばいいなら

**デフォルトの名無しさん** · 2022/06/19(日) 17:19:29.91

さしあたり1200以上のモジュールってのは行数のことだよね

**デフォルトの名無しさん** · 2022/06/19(日) 17:50:53.92

オプエクで動かなくなるのって、デフォルトプロパティあたりのタイプミスとかじゃないのかなあ
晒せるように改変してたらあっさり解決したりして

**デフォルトの名無しさん** · 2022/06/20(月) 12:29:08.41

若手って何歳を指しているのよ？
まさか40代を指して若手とか定義しているのか？

**デフォルトの名無しさん** · 2022/06/20(月) 12:49:30.22

年齢でも勤続年数でもないのは確か
何ができたら一人前とか定義するのも不可能だし

**デフォルトの名無しさん** · 2022/06/20(月) 12:58:01.87

初心者の定義ならできそう

マニュアルを読まない
型を適切に使い分けない
コーディングルールを意識しない(インデントやコメント、命名規則も含む)
エラーメッセージを読まない
とにかくググらない

**デフォルトの名無しさん** · 2022/06/20(月) 13:20:31.60

いきなりあのマニュアルは敷居が高すぎる
とは思う。

**デフォルトの名無しさん** · 2022/06/20(月) 13:30:03.87

マニュアルってどれ？