質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.35%
VBA

VBAはオブジェクト指向プログラミング言語のひとつで、マクロを作成によりExcelなどのOffice業務を自動化することができます。

マクロ

定義された処理手続きに応じて、どのような一連の処理を行うのかを特定させるルールをマクロと呼びます。

Q&A

解決済

2回答

4640閲覧

Callステートメントが1つ目しか実行されない

mattsuncaca

総合スコア2

VBA

VBAはオブジェクト指向プログラミング言語のひとつで、マクロを作成によりExcelなどのOffice業務を自動化することができます。

マクロ

定義された処理手続きに応じて、どのような一連の処理を行うのかを特定させるルールをマクロと呼びます。

0グッド

0クリップ

投稿2021/10/29 08:29

前提・実現したいこと

複数ブックのマクロを別のブックから一括で実行したい。

・特定のブック1を開き、順番通りにクエリを更新し、最終結果を別のcsvにコピペする
・特定のブック2を開き、順番通りにクエリを更新し、最終結果を別のcsvにコピペする

・特定のブック25を開き、順番通りにクエリを更新し、最終結果を別のcsvにコピペする

という内容を、Callステートメントを使い、別ブックのマクロ一回で処理させようとしています。

発生している問題・エラーメッセージ

VBA

1Sub 一括result更新() 2 Call result更新1 3 Call result更新2 4End Sub

上記マクロを実行しても、1つ目(result更新1)しか実行されない。
(これをresult更新25まで一気にやりたいです)

該当のソースコード

result更新1は、別ブックを開き、"データ更新test"というマクロを実行します。
result更新2は、ankenNo = 2になるだけの違いです。

VBA

1Sub result更新1() 2 'Dim excelFileFullPath As String 3 Dim macroName As String 4 Dim wk As Workbook 5 6 ankenNo = 1 7 8 9 '「呼び出される用のExcelファイル」のフルパスを指定 10 excelFileFullPath = ActiveSheet.ListObjects("master").DataBodyRange.Cells(ankenNo, 1) 11 '呼び出される用のマクロ名(プロシージャ名)を指定 12 macroName = "データ更新test" 13 14 '「呼び出される用のExcelファイル」を開く 15 Set wk = Workbooks.Open(Filename:=excelFileFullPath) 16 17 '呼び出される用のマクロ(プロシージャ)を実行 18 Run "'" & excelFileFullPath & "'!'" & macroName & "'" 19 20End Sub

上記で呼び出され、特定のブック1~25内で実行される"データ更新test"のコードは下記です。

VBA

1Sub データ更新test() 2' データ更新 Macro' 3 4 ActiveWorkbook.Connections("クエリ - result_original").OLEDBConnection.BackgroundQuery = False 5 ActiveWorkbook.Connections("クエリ - result_カテゴリ変更").OLEDBConnection.BackgroundQuery = False 6 ActiveWorkbook.Connections("クエリ - result_除外").OLEDBConnection.BackgroundQuery = False 7 ActiveWorkbook.Connections("クエリ - 着予_result").OLEDBConnection.BackgroundQuery = False 8 ActiveWorkbook.Connections("クエリ - result_媒体粒度").OLEDBConnection.BackgroundQuery = False 9 ActiveWorkbook.Connections("クエリ - result_CPN粒度").OLEDBConnection.BackgroundQuery = False 10 ActiveWorkbook.Connections("クエリ - result").OLEDBConnection.BackgroundQuery = False 11 ActiveWorkbook.Connections("クエリ - 着予_plan").OLEDBConnection.BackgroundQuery = False 12 ActiveWorkbook.Connections("クエリ - shamei_retargeting").OLEDBConnection.BackgroundQuery = False 13 ActiveWorkbook.Connections("クエリ - CPN_unique").OLEDBConnection.BackgroundQuery = False 14 ActiveWorkbook.Connections("クエリ - ADG_unique").OLEDBConnection.BackgroundQuery = False 15 16 17 ActiveWorkbook.Connections("クエリ - result_original").Refresh 18 ActiveWorkbook.Connections("クエリ - result_カテゴリ変更").Refresh 19 ActiveWorkbook.Connections("クエリ - result_除外").Refresh 20 ActiveWorkbook.Connections("クエリ - 着予_result").Refresh 21 ActiveWorkbook.Connections("クエリ - result_媒体粒度").Refresh 22 ActiveWorkbook.Connections("クエリ - result_CPN粒度").Refresh 23 ActiveWorkbook.Connections("クエリ - result").Refresh 24 ActiveWorkbook.Connections("クエリ - 着予_plan").Refresh 25 ActiveWorkbook.Connections("クエリ - shamei_retargeting").Refresh 26 ActiveWorkbook.Connections("クエリ - CPN_unique").Refresh 27 ActiveWorkbook.Connections("クエリ - ADG_unique").Refresh 28 29 'クエリ更新の最終結果をテンプレcsvに貼り付け 30 Call result生成_自動 31 32 ThisWorkbook.Close SaveChanges:=True 33End Sub

result更新1は問題なく実行されるのですが、result更新2はスキップされてしまいます。

試したこと

result更新1内で、1~2のループ処理をしてみましたが、同じくresult更新1しか実行されませんでした。

データ更新test最後の

ThisWorkbook.Close SaveChanges:=True

が次のresult更新2の

'「呼び出される用のExcelファイル」を開く Set wk = Workbooks.Open(Filename:=excelFileFullPath)

とバッティングしてしまっているのかもと素人目では思います。

VBAは完全素人で、ネット上の情報をかき集めながら作っています。
1日中ここでスタックしてしまっているので、お詳しい方の目から見た助言をいただけますと幸いです。

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

ベストアンサー

呼び出されたブック内で自ブックを閉じてはいけません。

呼び出し側でブックを開いているのですから、
閉じるのも呼び出し側でやりましょう。

'「呼び出される用のExcelファイル」を開く Set wk = Workbooks.Open(Filename:=excelFileFullPath) '呼び出される用のマクロ(プロシージャ)を実行 Run "'" & excelFileFullPath & "'!'" & macroName & "'"

  wk.Close

投稿2021/10/29 13:23

iruyas

総合スコア1067

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

mattsuncaca

2021/11/01 07:01

ご回答有り難うございます! こちらで解決いたしました! 呼び出し側で閉じてしまうと、呼び出される側のクエリ更新を待たずに閉じてしまうという問題があったので、最初の段階でこの方法を排除してました。 しかし、今やってみると、問題なく更新が終わってから閉じるようになったので、少し不思議ですが、触っているうちに、問題を解決していたのかなと思います! ありがとうございました!
guest

0

私もVBAは詳しくはありませんが、おそらく読み通りかと思います。

VBA

1ThisWorkbook.Close

としていることで、「一括result更新」マクロを実行しているブックも閉じているのだと思います。
対応としてはActiveWorkbook.Closeとするのが良いと思います。
(「データ更新test」マクロでも前段でActiveWorkbookを使用していますよね。むしろ、なぜ最後にThisWorkbookにしているのかというところに違和感がありました)
こちらのページでThisWorkbookとActiveWorkbookの違いがまとめられています。この違いを理解しておいた方が良いと思います。

また、VBAでしたらデバッグ環境が整っております。デバッガで挙動を追った方がなぜそうなるのかが理解しやすいと思います。

投稿2021/10/29 12:27

archiver

総合スコア1567

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

mattsuncaca

2021/11/01 06:55

ご回答ありがとうございます! ActiveWorkbookに変更しても残念ながら解決しませんでしたが、 ThisWorkbookとActiveWorkbookの違いを改めて意識する機会となりました! デバッガで挙動を追ったところ、1つ目のCallが終わったあと、2つ目にいかず、1つ目の先頭に戻ってしまうような挙動でした。 とても興味深いですが、他回答にあった、「ブックを呼び出し側で閉じる」としたことで、次の処理に進んでくれました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.35%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問