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

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

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

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

Q&A

解決済

4回答

10450閲覧

VBAでのループが読み込まれない

hanabi.

総合スコア16

VBA

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

0グッド

0クリップ

投稿2019/03/28 09:43

実現したいこと

各シートの指定セル(例:A3)をコピーし、『集計』シートにループで貼り付けたい。

前提

お世話になります。VBA初心者です。
現在VBAを使ってコピペの自動化しようとしており、下記For Next文を書きました。

ステップインで確認したところ、Paste処理の部分が読み込まれず、また原因が分からず困っております。

該当のソースコード

Sub SheetRoop() Dim i As Long Dim n As Long For i = 1 To Sheets.Count Sheets(i).Range("A3").Copy For n = 2 To 3 '↓下記の一文が読み込まれない Sheets(4).Cells(n, 2).PasteSpecial Next n n = n + 1 Next i End Sub

解決方法をご存知の方がいらっしゃいましたら、教えていただけると幸いです。
よろしくお願いいたします。

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

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

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

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

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

torisan

2019/03/29 00:26

>Paste処理の部分が読み込まれず これはどういう意味でしょうか?処理がこの行を通らない?通っても貼り付かない? また、処理を動かした時の現状のエクセルと、どうなって欲しいかの理想系のエクセル、あとシートの構成(Sheet1,Sheet2,Sheet3,集計 シートの4つ 等)があると良いと思います。
hanabi.

2019/03/29 00:40

言葉足らずで申し訳ございませんでした。 『処理がこの行を通らない』の意味です。
guest

回答4

0

変数n の使い方が間違っており、思ったように動かないと思います。
変数n は、for文の変数で、値がリセットされてしまい、常に上書きしています。
さらに悪いことに、最後の出力シートもコピペしているので、推測で、空白が貼り付けて、結果は、何も出力されていないと思います。

サンプルプログラムの様に思えますが、単純に値の貼り付けであれば、Copyメソッドは使う必要はなく、
下の様なシンプルなコードで対応が可能です。

VBA

1Sub SheetRoop() 2 3 Dim i As Long 4 5 For i = 1 To Sheets.Count - 1 6 Sheets(Sheets.Count).Cells(2, 2) = Sheets(Sheets.Count).Cells(2, 2) + Sheets(i).Range("A3") 7 Next i 8 9End Sub 10

投稿2019/03/28 23:06

編集2019/03/28 23:27
kai_keitai

総合スコア344

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

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

0

変数「n」の使い方が美味くないです。
ステップ実行しながら、ローカルウィンドウで、
変数「n」の中の値がどのように変化しているか追ってみてください。

プロパティ、メソッドの探り方 マクロ記録とF1のHelpを使う

サンプル

VBA

1Sub SheetRoop2() 2 3 Dim i As Long 4 Dim n As Long 5 6 n = 2 7 For i = 1 To 3 8 Sheets(i).Range("A3").Copy Sheets(4).Cells(n, 2).Resize(2) 9 n = n + 2 10 Next 11End Sub

左から4番目のシートにデータを転記していくなら、
見て行くのは、
1番目から3番目ですかね?
ずばり書いちゃわないと、
命令された方(エクセルVBAの場合はエクセル君)は
律儀に何も考えずに作業しちゃうので、
意図した結果を返さないかも?

投稿2019/03/28 11:13

mattuwan

総合スコア2136

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

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

0

ベストアンサー

シート1~3のA3セルを、シート4のB2・B3に貼る想定なのでしょうが貼る場所をB2・B3からズラすところが上手く動いておらず、ずっとB2・B3に上書きしているのだと思います。
(シート3のA3セルが空欄なら空欄で上書きされるので何も動いてないように見えるかと)

ブレークポイントやステップインで動作確認し直してみてください。

VBA

1Sub SheetRoop() 2 3Dim i As Long 4Dim n As Long 5m = 0 6 7For i = 1 To Sheets.Count - 1 8 Sheets(i).Range("A3").Copy 9 10 For n = 2 To 3 11 '↓下記の一文が読み込まれない 12 Sheets(4).Cells(n + m, 2).PasteSpecial 13 Next n 14 15 m = m + 2 16Next i 17 18End Sub

投稿2019/03/28 10:17

kobac

総合スコア188

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

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

hanabi.

2019/03/29 00:43

皆様のいただいたコードを用いたところ、無事思い通りの機能を実現することができました。 ベストアンサーは一番早かった回答とさせていただきます。 皆様本当にありがとうございました。
guest

0

また、単純に合計だけを求めたいのであれば、VBAは不要で、ワークシート関数で対応が可能です。
例えば、

EXCEL

1=SUM(Sheet1:Sheet3!A3)

ワークシートの串刺し計算が、:演算子によって可能になります。

投稿2019/03/28 23:20

kai_keitai

総合スコア344

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

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

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

質問する

関連した質問