VBAについての質問です。
マクロが保存されているエクセルと同じフォルダにあるエクセルファイルを全て開くマクロをつくりたくて、以下のようにしました。
Dim filename As String
Dim openedbook As Workbook
Dim isbookopen As Boolean
filename = Dir(ThisWorkbook.Path & "*.xls")
Do While filename <> ""
If filename <> ThisWorkbook.Name Then
isbookopen = False
For Each openedbook In Workbooks
If openedbook.Name = filename Then
isbookopen = True
Exit For
End If
Next
If isbookopen = False Then
Workbooks.Open (filename)
End If
End If
filename = Dir()
Loop
エラーはでないのですが、ファイルは1つも開きません。何がだめなのかを教えていただきたいです。
よろしくお願いします。
No.6ベストアンサー
- 回答日時:
こんにちは
ファイルパスの「\」その他に関しては、既に指摘の出ている通りですね。
一方で、開いているブックとのチェックを
>For Each openedbook In Workbooks
で行っていますが、毎回ループでチェックしなければならないのと、ブックを開くごとにチェック対象が増えてゆく(実際は、新しく開いたブックのチェックは不要)という欠点があります。(別に、間違った処理にはなりませんけれど)
さらには、このチェックは、
>If filename <> ThisWorkbook.Name Then
とも重複していることになります。
(Workbooksには、ThisWorkbookも含まれているので)
最初にDictionaryオブジェクト等に、既オープンのブックをリスト化するなどしておくことで、チェックを単純にできるものと考えられます。
また、ロジックを単純化できると、IFのネストやループのネストも減るので、視認性もよくなると思われます。
以下、ご参考までに。
Sub Sample_12047309()
Dim fList, f
Dim folderPath As String, fileName As String
folderPath = ThisWorkbook.Path & "\"
Set fList = CreateObject("Scripting.Dictionary")
For Each f In Workbooks
fList.Add f.Name, ""
Next f
fileName = Dir(folderPath & "*.xls")
While fileName <> ""
If Not fList.Exists(fileName) Then _
Workbooks.Open (folderPath & fileName)
fileName = Dir()
Wend
End Sub
No.5
- 回答日時:
とりあえず、修正しておきました。
動作確認済みです。
Public Sub sample()
Dim filename As String
Dim openedbook As Workbook
Dim isbookopen As Boolean
filename = Dir(ThisWorkbook.Path & "\*.xls")
Do While filename <> ""
If filename <> ThisWorkbook.Name Then
isbookopen = False
For Each openedbook In Workbooks
If openedbook.Name = filename Then
isbookopen = True
Exit For
End If
Next
If isbookopen = False Then
Workbooks.Open (ThisWorkbook.Path & "\" & filename)
End If
End If
filename = Dir()
Loop
End Sub
ありがとうございます!
最初に私が訂正してたのは不十分だった用で、書いていただいた通りに訂正したら無事にできました
御丁寧にありがとうございました!
No.4
- 回答日時:
>filename = Dir(ThisWorkbook.Path & "*.xls")
これについては 他の方の指摘どり \ が無い
しかし この一行は ループの外にあり
”全て開く” 事は出来ないよな
基本的な事をまったく理解されていないように思われます
Dir関数をなかなか理解できずにうなっていたのを見透かされましたね笑
より正確に思ったものを作れるようになるためにご指摘の通り基本を学び直したいと思います。
お探しのQ&Aが見つからない時は、教えて!gooで質問しましょう!
このQ&Aを見た人はこんなQ&Aも見ています
-
あるあるbotに投稿したけど採用されなかったあるある募集
あるあるbotに投稿したけど採用されなかったあるあるをこちらに投稿してください
-
フォロワー20万人のアカウントであなたのあるあるを披露してみませんか?
あなたが普段思っている「これまだ誰も言ってなかったけど共感されるだろうな」というあるあるを教えてください
-
映画のエンドロール観る派?観ない派?
映画が終わった後、すぐに席を立って帰る方もちらほら見かけます。皆さんはエンドロールの最後まで観ていきますか?
-
海外旅行から帰ってきたら、まず何を食べる?
帰国して1番食べたくなるもの、食べたくなるだろうなと思うもの、皆さんはありますか?
-
天使と悪魔選手権
悪魔がこんなささやきをしていたら、天使のあなたはなんと言って止めますか?
-
Excel VBAで同じフォルダ内のファイルを開くには?
Excel(エクセル)
-
同一フォルダ内のファイルを開く
Excel(エクセル)
-
エクセルVBAでセルに入力したパスでブックを開く
Excel(エクセル)
-
-
4
VBAでファイルを開くときにファイル名でワイルドカードを使用したいです
その他(プログラミング・Web制作)
-
5
フォルダ内の全てのBookに同じ処理を繰り返す
Excel(エクセル)
-
6
EXcelマクロで同じフォルダ内のファイル全て開く。ドライブまたいでも対応
Android
-
7
【VBA】コピー&複数個所のペースト繰り返し
Excel(エクセル)
-
8
別ファイルを開かず、INDIRECT関数を使用せずに、別ファイルのデータを求めたい
Excel(エクセル)
関連するカテゴリからQ&Aを探す
おすすめ情報
- ・漫画をレンタルでお得に読める!
- ・人生のプチ美学を教えてください!!
- ・10秒目をつむったら…
- ・あなたの習慣について教えてください!!
- ・牛、豚、鶏、どれか一つ食べられなくなるとしたら?
- ・【大喜利】【投稿~9/18】 おとぎ話『桃太郎』の知られざるエピソード
- ・街中で見かけて「グッときた人」の思い出
- ・「一気に最後まで読んだ」本、教えて下さい!
- ・幼稚園時代「何組」でしたか?
- ・激凹みから立ち直る方法
- ・1つだけ過去を変えられるとしたら?
- ・【あるあるbot連動企画】あるあるbotに投稿したけど採用されなかったあるある募集
- ・【あるあるbot連動企画】フォロワー20万人のアカウントであなたのあるあるを披露してみませんか?
- ・映画のエンドロール観る派?観ない派?
- ・海外旅行から帰ってきたら、まず何を食べる?
- ・誕生日にもらった意外なもの
- ・天使と悪魔選手権
- ・ちょっと先の未来クイズ第2問
- ・【大喜利】【投稿~9/7】 ロボットの住む世界で流行ってる罰ゲームとは?
- ・推しミネラルウォーターはありますか?
- ・都道府県穴埋めゲーム
- ・この人頭いいなと思ったエピソード
- ・準・究極の選択
このQ&Aを見た人がよく見るQ&A
デイリーランキングこのカテゴリの人気デイリーQ&Aランキング
-
【VBA】写真の縦横比を変えずに...
-
Not kind:Folders
-
google Drive 容量を減らせない...
-
Cドライブにconfig msiというフ...
-
常に「すべての列のサイズを自...
-
DESKTOP.INI 内容を解読
-
Windows10のメタデータの「メ...
-
Vistaエクスプローラでファイル...
-
NAS上のBATファイル実行
-
Wordを開こうとすると『環境変...
-
貼り付けとショートカットの貼...
-
Apoint2Kってスパイウェアでし...
-
教えて!gooで、はっていいリン...
-
フォーマットした後のハードデ...
-
zip形式の動画をDVD-RWに保存し...
-
システムファイルの削除について
-
ジョブ制御言語とは
-
フォルダウインドウの表示設定...
-
ダウンロードしたフォントをOS9...
-
フォルダの属性がすべて読み取...
マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング
-
【VBA】写真の縦横比を変えずに...
-
VBAで同一フォルダにあるブック...
-
const char*のグローバル変数と...
-
フォルダ名表示をするには?
-
パイソンでエクセルマクロを動...
-
常に「すべての列のサイズを自...
-
検索したファイルの場所を調べ...
-
Not kind:Folders
-
System32内のファイルを書き換...
-
タスクスケジューラで指定フォ...
-
シャットダウン時に自動ファイ...
-
ファイル並び順がバラバラで困...
-
貼り付けとショートカットの貼...
-
VB.net 任意の例外を発生させ...
-
レジストリの変更が保存されない
-
エクセルを立ち上げるとBook2が...
-
Ububtuでファイル共有できない...
-
ショートカットファイルそのも...
-
Cドライブにconfig msiというフ...
-
google Drive 容量を減らせない...
おすすめ情報
Doの行からとばされました
そのように変更しても変わらずです泣