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

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

ただいまの
回答率

87.37%

エクセル2010 実行時エラー1004

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 3,417

score 837

前提・実現したいこと

ここに質問の内容を詳しく書いてください。

エクセル2010にてマクロ走行中に実行時エラー1004
アプリケーション定義またはオブジェクト定義のエラーが出力される。

フォルダパスを指定した後、再帰的にフォルダの中身を(サブディレクトリを含む)を参照し
拡張子.log ファイルをエクセルの6行目、D列に書き込ませています。

階層の浅いファイルでは等エラーは発生しませんが
階層が深く、ファイル数が多いと当、現象が発生します

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

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

該当のソースコード

Dim 行 As Long
Dim cnt As Long
Dim c行 As Long
Dim d行 As Long
Dim h行 As Long
Dim i行 As Long

Sub open_dir_and_run()

Dim Folder As Object
Dim strPath As String

Set Folder = CreateObject("Shell.Application"). _
BrowseForFolder(0, "フォルダを選択してください", 0, 0)

If Not Folder Is Nothing Then
strPath = Folder.Items.Item.Path
Else
Exit Sub
End If

cnt = 5
c行 = 5
d行 = 5
h行 = 5
i行 = 5
Call re_call(strPath)

End Sub

Sub re_call(ByVal strPath)

Dim buf As String
Dim textbuf As String
Dim text_cnt As Long

buf = Dir(strPath & "\*.log")
'ターゲットのファイル名
tgt_file = strPath & "\" & buf
Do While buf <> ""

Open tgt_file For Input As #1
Do Until EOF(1)
Line Input #1, textbuf
cnt = cnt + 1
Cells(cnt, 4) = textbuf <--- ここでエラー

Loop
Close #1
buf = Dir()
Loop
With CreateObject("Scripting.FileSystemObject")
For Each f In .GetFolder(strPath).SubFolders
Call re_call(f.Path)
Next f
End With
End Sub

VBA エクセル

試したこと

グローバル変数 cnt が694になったときにエラーが出力されます
ウォッチで
Cells(cnt, 4)を監視していると
落ちたときに、中身が
: FormulaLabel : <アプリケーション定義またはオブジェクト定義のエラーです。> : XlFormulaLabel : Module1.re_call

: Name : <アプリケーション定義またはオブジェクト定義のエラーです。> : Variant : Module1.re_call

: PivotCell : <アプリケーション定義またはオブジェクト定義のエラーです。> : PivotCell : Module1.re_call

: QueryTable : <アプリケーション定義またはオブジェクト定義のエラーです。> : QueryTable : Module1.re_call

: ServerActions : <アプリケーション定義またはオブジェクト定義のエラーです。> : Actions : Module1.re_call

が、見えています

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

ここにより詳細な情報を記載してください。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 2

checkベストアンサー

0

原因は分かりませんが、この辺りを確認してみてはいかが?

1.エラーが出る時のtextbufの中身は?
2.イミディエイトでCells(694, 4).Value = "aaa"を実行してもエラーが出ますか?
3.新規ワークシートでも再現しますか?
4.一応、モジュール先頭にOption Explicitを付けて変数の宣言を強制してはいかがですか?
5.Cellsの前にシートを指定してはいかがですか?(プログラム序盤にSet WS = ActiveSheetとして、使う時はWS.Cells(・・・).Valueとしては?

ついでに書き下ろしてみました。

Option Explicit

Const 出力列 = 4

Sub open_dir_and_run()

    Dim Folder As Object
    Dim strPath As String

    Set Folder = CreateObject("Shell.Application"). _
    BrowseForFolder(0, "フォルダを選択してください", 0, 0)

    If Not Folder Is Nothing Then
        strPath = Folder.Items.Item.Path
    Else
        Exit Sub
    End If


    Dim cnt As Long
    Dim WS As Worksheet
    Set WS = ActiveSheet

    WS.Columns(出力列).NumberFormatLocal = "@"
    cnt = 5 + 1
    Call re_call(strPath, WS, cnt)
    Debug.Print "次の出力行:" & cnt

End Sub

'strPath : 検索フォルダパス
'WS      : 出力シート
'outRow  : 出力開始行
Sub re_call(ByVal strPath As String, ByVal WS As Worksheet, ByRef outRow As Long)

    Dim LogArray()  As String
    Dim objFile     As Object
    Dim objFolder   As Object

    With CreateObject("Scripting.FileSystemObject")

        'ファイル内容出力
        For Each objFile In .GetFolder(strPath).Files
            'logファイルのみ
            If objFile.Name Like "*.log" Then
                '※改行コードはCRLF
                LogArray = Split(objFile.OpenAsTextStream(1).ReadAll, vbCrLf)
                If UBound(LogArray) > 0 Then
                    'SplitのUboundは要素数-1だがログファイルの末尾が(普通は)空行なので+1され相殺のため無視してOK
                    '両方を想定するなら修正が必要。
                    WS.Cells(outRow, 出力列).Resize(UBound(LogArray) + 1, 1).Value = WorksheetFunction.Transpose(LogArray)
                    outRow = outRow + UBound(LogArray)
                End If
            End If
        Next

        '配下のフォルダを再帰呼び出し
        For Each objFolder In .GetFolder(strPath).SubFolders
            Call re_call(objFolder.Path, WS, outRow)
        Next

    End With

End Sub

追記:エラー1004が出た原因はtextbuf="=="のようです。
WS.Columns(出力列).NumberFormatLocal = "@"を追加して書式設定を「文字列」に変えることで対処できます。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/01/13 12:03

    kotori-chunchun 様、適切なご回答、ありがとうございました。
    犯人逮捕しました。
    無事、マクロが最後まで、走行することを確認しました。

    キャンセル

  • 2019/01/13 12:10

    >しかし2.Cells(694, 4).Value = "aaa"がエラーになるのが納得いきません。本当に"aaa"を代入した場合も>エラーが出るのでしょうか。

    落ちた後に値を書き換えた為、エラー表示されただけかとおもいます。
    cnt カウントが694になった直後に値を書き換え再度確認しましたが。
    走行しました。なのでkotori-chunchun様の認識はあってます。
    私のオペミスです。

    キャンセル

  • 2019/01/13 12:14

    それは幸いでした。
    結果報告ありがとうございます。

    解決ボタンを押してクローズしておいてください。

    キャンセル

0

OPENステートメントのファイルパス文字数制限でエラーが発生していると思われます。
http://mgate.info/computer/software/msoffice/excel/959/

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/01/12 18:38

    落ちる、直前のファイルパスが格納されている変数tgt_file は98バイトである事を確認しました。
    教えていただいた、URLを参照し、260バイトは超えてませんでした。
    ファイルパス文字数制限は、超えてないと思うのですが・・・・

    キャンセル

  • 2019/01/12 18:52 編集

    ファイルパス文字数制限は関係ないのですね。
    それでは、OPENステートメントは空いている番号を取得して
    使用するようにしてはどうでしょうか。
    FNo = FreeFile
    DoEvents
    Open tgt_file For Input As #FNo

    キャンセル

  • 2019/01/12 19:36

    ご指摘の通り、コーディングし、FreeFile関数で使用可能なファイル番号を取得し、実行しましたが
    グローバル変数 cnt が694になったときにエラーが出力されます

    キャンセル

  • 2019/01/12 19:38

    そうですか。問題が別のところにあるようなので、お役に立てず申し訳ありません。

    キャンセル

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

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

関連した質問

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