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

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

ただいまの
回答率

90.48%

  • VBA

    2385questions

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

  • Excel

    1966questions

    Excelは、マイクロソフト社が開発しているデータ集計や分析を行う表計算ソフトの一つです。文書作成や表計算、資料作成などの多彩な機能を備えており、統合パッケージであるMicrosoft Officeに含まれています。

  • コピー

    29questions

    元のオブジェクトを破壊することなく、オブジェクトの複製を生成することをコピーと呼びます。

アプリケーション定義またはオブジェクト定義のエラー 範囲指定の間違い? ブック名の処理 重複を防ぐ

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 675

SnowMonkey

score 4

前提

フォルダ内に、複数のエクセルブック(被験者の人数分)があります。
それらのブックを1つのブックにまとめるコードを書きました。

例えば、
フォルダ内全ブックのSheet1を、複写先のSheet1に
フォルダ内全ブックのSheet2を、複写先のSheet2に まとめるものです

色々調べて
まとめること自体は一応できました。

ブック名に関する処理を追加しようとしたときに、皆様の
お知恵を拝借できればありがたいと思いました。

ブック名

フォルダ内のブック名には、

  • 実験条件(たとえば測定したのが室内か野外か)と、
  • 2桁の被験者ID番号
    の情報が含まれています

野外で測定した被験者12番のブックなら、
yagai12.xls とつけます。

実現したいこと・助言を欲しいこと

1.ブック名を切り分けて、A列とB列に記入する

yagai12.xls というブック名なら
yagaiと12を切り分けて、yagaiをA列に、12をB列の該当行に空白なく記入する

2.同じファイルを何度も処理しないようにする
これもブック名を使ってチェックすればいいのかなと思いますが、具体策が思いつかない。

3.ブック名に関する処理以外も含めて、もっと良い書き方があれば助言を得たい。

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

1.や2.の処理をする前段階として、ブック名を取得して、A列を埋めようとしたときに
上手くいきませんでした。

コードを実行すると以下のエラーメッセージが表示されます。

アプリケーション定義またはオブジェクト定義のエラーです

該当のソースコード

Option Explicit

'「data」フォルダにあるファイルを開いて、その内容をこのワークブックにまとめる
Sub importData()
Dim fso As FileSystemObject
Set fso = New FileSystemObject


Dim f As File
For Each f In fso.GetFolder(ThisWorkbook.Path & "\data").Files 'dataフォルダにあるファイルを1つずつ開いて処理
    With Workbooks.Open(f.Path)
         Dim bkName As String
         bkName = .Name

         Dim i   As Long
         For i = 1 To Worksheets.Count  ' 全シートを処理する
            With .Worksheets("Sheet" & i)

                 '----------データの複写先のシートの最終行を取得--------------------
                Dim wsResult As Worksheet
                Set wsResult = ThisWorkbook.Worksheets("Sheet" & i)   'データの複写先のシート

                Dim LastRow  As Long
                 LastRow = wsResult.Cells(Rows.Count, 3).End(xlUp).row  'データの複写先のシートの最終行

                '------開いたシートの使用されている範囲を、複写先シートの最終行の次行にコピーする A列とB列は空欄にしておく------------
                .UsedRange.Copy wsResult.Cells(LastRow + 1, 3)

                 '------データを追加した範囲のA列にブック名を書き込む------------
'                 wsResult.Cells(LastRow + 1, 1).Value = bkName  'エラーは出ないが、空白セルができる
                Dim LastRow2  As Long
                 LastRow2 = wsResult.Cells(Rows.Count, 2).End(xlUp).row  'データの複写先のシートの最終行を再び取得して
                 wsResult.Range(Cells(LastRow + 1, 1), Cells(LastRow2, 1)).Value = bkName

            End With
         Next i
          .Close

    End With

Next f

End Sub

上記のうち、この部分だと思います。ここを外すとエラーメッセージは出ません。

          '------データを追加した範囲のA列にブック名を書き込む------------
'                 wsResult.Cells(LastRow + 1, 1).Value = bkName  'エラーは出ないが、空白セルができる
                Dim LastRow2  As Long
                 LastRow2 = wsResult.Cells(Rows.Count, 2).End(xlUp).row  'データの複写先のシートの最終行を再び取得して
                 wsResult.Range(Cells(LastRow + 1, 1), Cells(LastRow2, 1)).Value = bkName

試したこと

1つのセルに記入できることを確認
→Rangeの使い方に問題がありそうということで、
ネットに記載されていたものをコピペしてみたが、
それでも上手くいかない。

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

Excel 2016

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

質問への追記・修正、ベストアンサー選択の依頼

  • hatena19

    2019/01/28 13:15

    エラーがでるコードがどれか明確にしてください。

    キャンセル

回答 1

checkベストアンサー

+1

おそらく下記の行でエラーになっていると思いますが、

wsResult.Range(Cells(LastRow + 1, 1), Cells(LastRow2, 1)).Value = bkName


よくある間違いですね。 Cells とするとアクティブなシートを対象にしてしまいます。下記のようにちゃんとシートを省略せずに記述しないとダメですね。。

wsResult.Range(wsResult.Cells(LastRow + 1, 1), wsResult.Cells(LastRow2, 1)).Value = bkName

下記のような記述するするとスッキリするかも(好みですが)。

wsResult.Range("A" & LastRow & ":A" & (LastRow2)).Value = bkName

1.ブック名を切り分けて、A列とB列に記入する

切り分けの条件を明確にしてください。
被験者番号は2桁固定、実験条件の桁数は可変なのですか。
また拡張子は"xls"で固定ですか。
そうだとするなら、下記のようなコードてよろしいかと。

wsResult.Range("A" & LastRow & ":A" & (LastRow2)).Value = Left(bkName, Len$(bkName)-6)
wsResult.Range("B" & LastRow & ":B" & (LastRow2)).Value = Mid(bkName, Len$(bkName)-5,2)

2.同じファイルを何度も処理しないようにする
これもブック名を使ってチェックすればいいのかなと思いますが、具体策が思いつかない。

Findメソッドを使ってファイル名が見つかったら読み込み処理をしないようにすればいいでしょう。

条件に当てはまるセルを検索する(Find/FindNext/FindPreviousメソッド) :Excel VBA|即効テクニック|Excel VBAを学ぶならmoug

ただ、自分なら、読み込み済みのファイルは、別フォルダーに移動させるようにしますね。
その方が読み込み済みかどうかが一目瞭然ですので。

ファイルを移動する(Nameステートメント):Excel VBA|即効テクニック|Excel VBAを学ぶならmoug

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/01/28 18:37

    ありがとうございます。なぜエラーメッセージがでるのかよく分かりました。
    ブック名の切り分けや、処理の重複の問題への助言もありがとうございました。

    キャンセル

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

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

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

  • VBA

    2385questions

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

  • Excel

    1966questions

    Excelは、マイクロソフト社が開発しているデータ集計や分析を行う表計算ソフトの一つです。文書作成や表計算、資料作成などの多彩な機能を備えており、統合パッケージであるMicrosoft Officeに含まれています。

  • コピー

    29questions

    元のオブジェクトを破壊することなく、オブジェクトの複製を生成することをコピーと呼びます。

  • トップ
  • VBAに関する質問
  • アプリケーション定義またはオブジェクト定義のエラー 範囲指定の間違い? ブック名の処理 重複を防ぐ