閲覧頂きありがとうございます。
タイトルの通り、シート名が一致した場合の転記内容がうまく行かず躓いています。
教えて頂けると幸いです。
やりたい事はマスターデータのシート名と転記先のシート名が一致した場合、セルに入力されている値を転記先へコピーすることです。
よろしくお願いいたします。
Option Explicit
Sub マスターデータ取り込み02() '選択したファイルを取り込み、別のファイルに貼り付ける
Dim RC As Integer
Dim OpenFileName, fileName, Path, SetFile As String
Dim wbMoto, wbSaki As Workbook
Dim sh1name As String
Dim i As Long
Dim flag As Boolean
sh1name = ActiveSheet.Name '現在開いているシート名
Set wbMoto = ActiveWorkbook 'マスターデータ取り込み元をセット
Application.DisplayAlerts = False '応答メッセージの非表示
RC = MsgBox("Excel Bを開きますか?", vbYesNo + vbQuestion, "確認")
If RC = vbYes Then 'もしRCのメッセージにYesと答えたら
OpenFileName = Application.GetOpenFilename("Microsoft Excelブック,*.xls?")
'ダイアログボックスを表示して、マスターデータファイルを指定します。
If OpenFileName <> "Fales" Then 'もし指定したファイルがFalseではなかったら
SetFile = OpenFileName 'マスターデータファイルの指定=SetFile
Else '指定したファイルがなかったら
MsgBox "キャンセルされました" 'キャンセルされましたというメッセージボックスが表示される
Exit Sub 'マスターデータの取り込みをキャンセル
End If
Workbooks.Open fileName:=SetFile, ReadOnly:=True, UpdateLinks:=0 'マスターデータファイルを読み取り専用で開く
'ダイアログボックスで指定したExcel Bを開きます。
Set wbSaki = Workbooks.Open(Path & SetFile) '転記元のシート検索
For i = 1 To ActiveWorkbook.Worksheets.Count 'アクティブブックのシートを全てカウント
If Worksheets(i).Name = sh1name Then 'もし転記元のシートの名前と現在開いているブックのシートの名前が一緒だったら
flag = True
End If
Next i
If flag <> True Then 'フラグが"True"ではなかったら
MsgBox sh1name & "シートはありません。", vbInformation 'シートはありませんの情報メッセージアイコンが出る
wbSaki.Close False 'ExcelBのファイルを閉じる
End If
'転記内容
wbSaki.Worksheets(i).Range("G2:N32").Copy
wbMoto.Worksheets(sh1name).Range("G2:N32").PasteSpecial xlPasteFormulasAndNumberFormats
Application.CutCopyMode = False 'コピー切り取りを解除
wbSaki.Close False 'マスターデータ取り込み先のファイルを閉じる
Else
MsgBox "処理を中断します"
End If
Application.DisplayAlerts = True
End Sub
No.2ベストアンサー
- 回答日時:
こんにちは
ざっと見ただけなので、確認はしていませんけれど・・・
>転記内容がうまく行かず躓いています。
どこかで、エラーが発生しているのでしょうか?
それとも実行はできるけれども、思った結果とは違ってしまうとか?
「マスターデータ」というネーミングから想像すると、マスター(=固定)なのではないかと思いますけれど、その割にはファイル指定ダイアログで選択しているようなので、意味がよく分からないですね。
等々の疑問は置いておくとして、気になったのは、
>Workbooks.Open fileName:=SetFile,~~
>Set wbSaki = Workbooks.Open(Path & SetFile)
と、同じファイルを2回開こうとしていること。
多分、エラーにはなりませんけれど、あまり意味があるとは言えませんし、変数pathってどこにも定義されていませんよね?
(通常は空白文字列として解釈されるので、エラーにはなりませんけれど意味不明)
エラーの原因になっているのは、
>wbSaki.Worksheets(i).Range("G2:N32").Copy
ではないかと推測します。
この時の変数 i の値を確認してみてください。
多分、シート数を超えた値になっていて、シートを参照できないのではないかと想像します。
同じ名前のシートを見つけた際に、そのシート番号を保持しておきたければ、保持用の変数に値を代入しておくか、あるいは発見したときに Exit For などでループを抜けてしまうような制御にしておく必要があるように感じます。
※ デバッグ中は、Application.DisplayAlerts = False などとはせずにテストしたほうが、ちゃんと警告が表示されるので何かと便利かと思いますけれど・・・
質問が上手く伝えられず申し訳ございません。
おっしゃる通り
wbSaki.Worksheets(i).Range("G2:N32").Copy
ここでエラーが起きていました。
マスターデータは固定なのですが、共有フォルダを使用しているのでpathが変わってくるのではないかと思うのですが、どうでしょうか?
おっしゃる通りpath定義してないですサイトを参考に組んでいるので、まだまだ理解不足ですみません。。
>Workbooks.Open fileName:=SetFile,~~
>Set wbSaki = Workbooks.Open(Path & SetFile)
こちらどちらかを削除しても問題ないという事でしょうか?
Application.DisplayAlerts = False などとはせずに
⇒ご指摘ありがとうございます。
こちら削除してやってみます。
保持用の変数や、Exit Forなど、ヒントありがとうございます。
いろいろ試してみます、ありがとうございます!
No.1
- 回答日時:
何処で不具合(エラーなど)が生じるかが分かればもう少し詳しく解析できそうですが、
>Application.DisplayAlerts = False '応答メッセージの非表示
のためにエラー表示が出ないのでしょう。
開発段階では使用しないことをお勧めします。
>Workbooks.Open fileName:=SetFile, ReadOnly:=True, UpdateLinks:=0
と
>Set wbSaki = Workbooks.Open(Path & SetFile) '転記元のシート検索
は同じファイルを2回開く操作をしているように見えます。
>For i = 1 To ActiveWorkbook.Worksheets.Count '
>・・・
>Next i
>wbSaki.Worksheets(i).Range("G2:N32").Copy
は完全にアウトです。
For_Nextが終了した時点でiは最終値+1になっています。
ですから、Worksheets(i)は存在しないでしょう。
解決策は
flag = True
のところで転記操作を行えば良いのではないでしょうか?
また、このようにしないと
>If flag <> True Then
>・・・
>End If
の後に転記操作を行っていますので必ず実行しようとします。
詳しい解説ありがとうございます!
まさに
>wbSaki.Worksheets(i).Range("G2:N32").Copy
ここでエラーが起きる状態でした!
教えられたやり方で試してみます!
ありがとうございます
お探しのQ&Aが見つからない時は、教えて!gooで質問しましょう!
このQ&Aを見た人はこんなQ&Aも見ています
-
見学に行くとしたら【天国】と【地獄】どっち?
みなさんは、一度だけ見学に行けるとしたら【天国】と【地獄】どちらに行きたいですか? 理由も聞きたいです。
-
歳とったな〜〜と思ったことは?
歳とったな〜〜〜、老いたな〜〜と思った具体的な瞬間はありますか?
-
みんなの【マイ・ベスト積読2024】を教えてください。
積読、ついついしちゃいませんか?そこでみなさんの 「2024年に買ったベスト積読」を聞きたいです。
-
【選手権お題その1】これってもしかして自分だけかもしれないな…と思うあるあるを教えてください
「出身中学と出身高校が混ざったような校舎にいる夢を見る」「まぶたがピクピクしてるので鏡で確認しようとしたらピクピクが止まってしまう」など、 これってもしかして自分だけかもしれないな…と思うあるあるを教えてください
-
【大喜利】【投稿~1/20】 追い込まれた犯人が咄嗟に言った一言とは?
【お題】追い込まれた犯人が咄嗟に言った一言とは?
-
エクセルVBA Ifでシート名が合致したら別ファイルから転記する場合のElse IfとForの書き方
Visual Basic(VBA)
-
VBA セルの値と同じ名前のシートにデータを補填するやり方を教えてください エクセルのブックがありま
Excel(エクセル)
-
特定の文字を含むシートだけマクロ処理をしたい
Visual Basic(VBA)
-
-
4
エクセルマクロでシート名を条件にする方法
Excel(エクセル)
-
5
VBA セルの値と同じ名前のシートにデータを貼り付けするやり方を教えてください
Excel(エクセル)
-
6
【VBA】指定した検索条件に一致したら別シートに転記したい
Visual Basic(VBA)
-
7
【VBA】元のシート内の文字列を別シートと比較し、一致したら元のシートの別のセルへ転記する方法。
Excel(エクセル)
-
8
セルの値と同じ名前のシートをアクティブにするには?
Excel(エクセル)
-
9
vba 2つの条件が一致したらコピーして別シートに値のみ貼り付け
Visual Basic(VBA)
-
10
【VBA】シート名に特定文字が入っていたらマクロを実行したいです。
Visual Basic(VBA)
-
11
同じ作業を複数のシートに実行させるにはどうしたらいいのでしょうか
Visual Basic(VBA)
-
12
EXCEL VBAで全選択範囲の解除
Excel(エクセル)
-
13
excelのマクロで該当処理できなければ飛ばして進むにはどうすればよいのでしょうか
Visual Basic(VBA)
-
14
エクセルVBAでセルに入力したパスでブックを開く
Excel(エクセル)
-
15
VBA 別ブックから条件に合うものを転記したいです
Visual Basic(VBA)
-
16
VBA:ユーザーフォームのマルチページに色を付けたい。
Word(ワード)
-
17
ExcelVBAを使って、値がある場合は作業を繰り返し実行するプログラムを作成したい。
Visual Basic(VBA)
-
18
エクセルのマクロで検索・抽出したデータを修正及び更新して元データに反映
Excel(エクセル)
-
19
複数の条件に合う行番号を取得するには
その他(Microsoft Office)
-
20
シート名一致すれば印刷、一致しなければメッセージbox
Visual Basic(VBA)
関連するカテゴリからQ&Aを探す
おすすめ情報
- ・漫画をレンタルでお得に読める!
- ・集中するためにやっていること
- ・テレビやラジオに出たことがある人、いますか?
- ・【お題】斜め上を行くスキー場にありがちなこと
- ・人生でいちばんスベッた瞬間
- ・コーピングについて教えてください
- ・あなたの「プチ贅沢」はなんですか?
- ・コンビニでおにぎりを買うときのスタメンはどの具?
- ・おすすめの美術館・博物館、教えてください!
- ・ことしの初夢、何だった?
- ・【お題】大変な警告
- ・【大喜利】【投稿~1/20】 追い込まれた犯人が咄嗟に言った一言とは?
- ・洋服何着持ってますか?
- ・みんなの【マイ・ベスト積読2024】を教えてください。
- ・「これいらなくない?」という慣習、教えてください
- ・今から楽しみな予定はありますか?
- ・AIツールの活用方法を教えて
- ・【お題】逆襲の桃太郎
- ・自分独自の健康法はある?
- ・最強の防寒、あったか術を教えてください!
- ・【大喜利】【投稿~1/9】 忍者がやってるYouTubeが炎上してしまった理由
- ・歳とったな〜〜と思ったことは?
- ・モテ期を経験した方いらっしゃいますか?
- ・好きな人を振り向かせるためにしたこと
- ・スマホに会話を聞かれているな!?と思ったことありますか?
- ・それもChatGPT!?と驚いた使用方法を教えてください
- ・見学に行くとしたら【天国】と【地獄】どっち?
- ・これまでで一番「情けなかったとき」はいつですか?
- ・この人頭いいなと思ったエピソード
- ・あなたの「必」の書き順を教えてください
- ・14歳の自分に衝撃の事実を告げてください
- ・人生最悪の忘れ物
- ・あなたの習慣について教えてください!!
- ・都道府県穴埋めゲーム
このQ&Aを見た人がよく見るQ&A
デイリーランキングこのカテゴリの人気デイリーQ&Aランキング
-
別ブックをダイアログボックス...
-
VBA シートをコピーする際に Co...
-
ワイルドカード「*」を使うとう...
-
【ExcelVBA】zip圧縮されたCSV...
-
VBA 別ブックからコピペしたい...
-
VBAで別ブックのシートを指定し...
-
エクセルVBAが途中で止まります
-
vbaでvbaProjectのパスワード解...
-
【Excel VBA】書き込み先ブック...
-
VBS Bookを閉じるコード
-
VBAで別のブックにシートをコピ...
-
Excel にて、 リストボックスの...
-
Excel2007VBAファイルの表示に...
-
Excelファイルを開くとき、読み...
-
VBで既存エクセルシートを新規...
-
【ExcelVBA】インデックスが有...
-
ユーザーフォームの切り替えに...
-
VBA 実行時エラー 2147024893
-
Excelマクロ 該当する値の行番...
-
[Excel]ADODBでNull変換されて...
マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング
-
VBA シートをコピーする際に Co...
-
別ブックをダイアログボックス...
-
VBA 別ブックからコピペしたい...
-
エクセルVBAが途中で止まります
-
ワイルドカード「*」を使うとう...
-
Excelマクロ 該当する値の行番...
-
【Excel VBA】書き込み先ブック...
-
VBA コードを実行すると画面が...
-
【ExcelVBA】インデックスが有...
-
VBAで別ブックのシートを指定し...
-
【ExcelVBA】zip圧縮されたCSV...
-
VBAで別のブックにシートをコピ...
-
VBAで複数のブックを開かずに処...
-
VBA 実行時エラー 2147024893
-
[Excel]ADODBでNull変換されて...
-
VBS Bookを閉じるコード
-
VBA シート名が一致した場合の...
-
フォルダ内の全てのファイルに...
-
vbaでvbaProjectのパスワード解...
-
Excel2007VBAファイルの表示に...
おすすめ情報
flag = True の後に
wbSaki.Worksheets(i).Range("G2:N32").Copy
wbMoto.Worksheets(sh1name).Range("G2:N32").PasteSpecial xlPasteFormulasAndNumberFormats
を実行しましたが、オブジェクト変数またはwithブロック変数が設定されていませんとエラーが出ました。
エラーを調べてみるとsetを付ける、インスタント化すると記載されているサイトを見つけましたが、よくわかりません。
fujillin様がおっしゃるように、保持用の変数かExit Forで一回中断するのが良いのでしょうか…