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

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

ただいまの
回答率

87.50%

エクセルVBAでのブック間ペーストができない

解決済

回答 2

投稿

  • 評価
  • クリップ 1
  • VIEW 2,255

score 6

前提・実現したいこと

エクセル(VBA)で集計表を作成しています。

デスクトップにある集計フォルダ内にあるエクセルファイル(.xlsx)の特定のセルをコピーしてマクロのあるブックにペーストをしたいです。

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

実行時エラー'1004':
アプリケーション定義またはオブジェクト定義のエラーです。

該当のソースコード

Sub 集計()

'Application.ScreenUpdating = False
'ActiveSheet.Range("B3:B15") = ""        

Dim Filename    As String
Dim IsBookOpen  As Boolean
Dim OpenBook    As Workbook

With CreateObject("WScript.Shell")
    .CurrentDirectory = "C:\Desktop\集計\"
End With


Filename = Dir("*.xlsx")

Do While Filename <> "" 'フォルダ内のファイルをループ
    i = 1
    If IsBookOpen = False Then

        Workbooks.Open (Filename), UpdateLinks:=1

        ActiveWorkbook.Worksheets("10月").Range("G1").Copy
        ThisWorkbook.Worksheets("10月").Activate
        Range(2, 3).PasteSpecial Paste:=xlPasteValues                 
        Workbooks(Filename).Close SaveChanges:=False 'True

    End If

i = i + 1

Filename = Dir()

Loop

End Sub

試したこと

おそらくペーストができていないので
18行目のペーストの特性やブック・シートの指定方法を変えましたが
開いたブックのコピーができた段階でエラーが出ます。

補足情報(FW/ツールのバージョンなど)

office365

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

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

+2

Range(2, 3).PasteSpecial Paste:=xlPasteValues
ここでエラーになるのですよね。

Range("C2").PasteSpecial Paste:=xlPasteValues
とするか、
Cells(2,3).PasteSpecial Paste:=xlPasteValues
としてください。


蛇足ですが、使用していない変数があったり、無駄な処理が多いのが気になるので、リファクタリングしてみました。

Sub 集計()
    Dim Filename    As String
    Dim OpenBook    As Workbook
    Dim PasteCell As Range
    Set PasteCell = ThisWorkbook.Worksheets("10月").Cells(2, 3)

    Const FolderPath = "C:\Desktop\集計\"
    Filename = Dir(FolderPath & "*.xlsx")

    Do While Filename <> ""
        Set OpenBook = Workbooks.Open(FolderPath & Filename)

        PassteCell.Value = OpenBook.Worksheets("10月").Range("G1").Value
        OpenBook.Close SaveChanges:=False

        Set PasteCell = PasteCell.Offset(1) '次のセル
        Filename = Dir()
    Loop

End Sub

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/04/09 21:59

    ありがとうございます、Cellsにしたらできました。
    また、頂いたVBAでやったところ
    Set OpenBook = Workbooks.Open(Filename)
    でエラーが出ました・・

    キャンセル

  • 2020/04/09 22:04

    あっ、コード間違ってました。
    回答のコードを修正しました。

    キャンセル

  • 2020/04/10 19:16

    ありがとうございます。
    ただ他の方のが今後の応用のイメージがわきやすかったのでベストは今回ごめんなさい。。。

    キャンセル

checkベストアンサー

+1

オブジェクト定義のエラーです。

Rangeプロパティの引数の指定方法が間違っています。
つまり存在しないセルを指定しているということです。
Rangeプロパティは数字のみでは、セルを認識しません。
行番号、列番号で指定するのは、Cellsプロパティです。

↓参考サイト
Rangeプロパティ 

他にも

If IsBookOpen = False Then

変数「IsBookOpen」がTrueになりようがないし、
おかしな記述が多いです。
試行錯誤の中で消しちゃったのでしょうか?

Sub test()
    Dim buf As String
    Dim wb As Workbook
    Dim i As Long
    Const sDirPath As String = "C:\Desktop\集計\*"


    buf = Dir(sDirPath & ".xlsx")
    Do While Len(buf) > 0
        'ファイルをすでに開いているか確認
        For Each wb In Workbooks
            If buf = wb.Name Then Exit For
        Next

        '開いてなければ開く
        If wb Is Nothing Then
            Set wb = Workbooks.Open(sDirPath & buf)
        End If

        'コピペ
        wb.Worksheets("10月").Range("G1").Copy
        ThisWorkbook.Worksheets("10月").Cells(i, 3).PasteSpecial xlPasteValues

        '開いたファイルを閉じる
        wb.Close False

        buf = Dir()     '次のファイル
        i = i + 1       '次の行番号
    Loop
End Sub

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/04/09 21:56

    ありがとうございます、間違いを直したらできました。
    まえあ、頂いたVBAでやったところ
    Set wb = Workbooks.Open(sDirPath & buf)
    で同じエラーが出てしまいました。

    キャンセル

  • 2020/04/09 22:24

    Const sDirPath As String = "C:\Users\hiraigumit\Desktop\集計\"

    buf = Dir(sDirPath & "*.xlsx")

    失礼しました。上に修正願います。

    キャンセル

  • 2020/04/10 19:16

    ありがとうございます。

    キャンセル

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

  • ただいまの回答率 87.50%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る