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

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

ただいまの
回答率

90.87%

  • VBA

    1551questions

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

CSVファイルをボタンを使用してインポートしたい

解決済

回答 1

投稿 編集

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

CSVファイルをエクセルのシート(Sheet1)にインポートしたいです。
はじめマクロの記録でインポートしてボタンに登録して実行したのですが、2回目以降の実行で不具合が出ました。そこで、teratailにて質問しご教授いただきましたプログラムを実行しました。
該当箇所を自分のファイルに変更し試したところ、CSVファイルのインポートがどうもうまくいきませんでした。

今回の問題点

①期待する動き
挿入ボタンをクリックし、PC上からCSVファイルを選択しエクセルシートにインポート
②実際の動き
ボタンをクリックすると、メッセージボックスが開き”インポートしますか?”と出て、”はい”を選択しますとインポートされるはずですが、エラーメッセージが開きます。
③エラー内容
”microsoft visual basic 400”と表示されます。調べて見たところ、セルの未指定等が原因のようでしたので、セル、シートも指定して実行したところやはり同じエラーが出ました。
また、セルをしてすると”識別子が違います”と出ます。指定箇所が違うのでしょうか。

下記がボタンを押した際の処理です

Public Sub ImportCsvFile()

    ' 変数宣言
    Dim csvBook As Workbook
    Dim csvSheet As Worksheet
    Dim Sheet1 As Worksheet
    Dim activeCell As Range
    Dim csvBookPath As String

    If MsgBox("選択されたセルにCSVデータをインポートしてよろしいですか?", vbQuestion + vbYesNo, "確認") = vbNo Then
        Exit Sub
    End If


    Set Sheet1 = ActiveSheet


    On Error Resume Next
    Set activeCell = Selection
    On Error GoTo 0


    If Range("A1:D4") Is Nothing Then
        MsgBox "セルが選択されていないため処理を継続できません", vbExclamation + vbOKOnly, "処理失敗"
        Exit Sub

    End If

    csvBookPath = Application.GetOpenFilename(FileFilter:="CSV ファイル, *.csv?")

    If csvBookPath = "" Then
        MsgBox "CSVファイルが選択されなかったので処理を終了します", vbExclamation + vbOKOnly, "処理失敗"
        Exit Sub

    End If

    Application.ScreenUpdating = False

    Set csvBook = GetCsvBook(csvBookPath)

    If csvBook Is Nothing Then
        MsgBox "CSVファイルを開けなかったので処理を終了します", vbExclamation + vbOKOnly, "処理失敗"

        Application.ScreenUpdating = True
        Exit Sub

    End If

    Set csvSheet = csvBook.Worksheets(1)

    Sheet1.Unprotect

    With csvSheet.UsedRange
        csvSheet.Range(csvSheet.Cells(IIf(NEED_HEADER, 1, 2), 1), .Cells(.Cells.Count)).Copy activeCell
    End With

    csvBook.Close False

    ProtectSheet ActiveSheet

    Application.ScreenUpdating = True

End Sub


ご指摘よろしくお願いします。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • ExcelVBAer

    2018/01/11 14:32

    どうもうまくいかないって、なぞなぞですか?

    キャンセル

  • ExcelVBAer

    2018/01/11 14:38

    GetCsvBookの中身も不明です

    キャンセル

  • Zuishin

    2018/01/11 14:52

    GetCsvBook の内容も追記してください。

    キャンセル

  • 退会済みユーザー

    2018/01/11 17:54

    複数のユーザーから「やってほしいことだけを記載した丸投げの質問」という意見がありました
    「質問を編集する」ボタンから編集を行い、調査したこと・試したことを記入していただくと、回答が得られやすくなります。

回答 1

checkベストアンサー

+1

①関連リンクがあるのなら貼りましょう

TeraTailには日々たくさんの質問があがっているのですから、以前の質問が関連しているのであればせめてリンクを貼りましょう。
すべての人が質問者さんの過去の質問履歴をひとつひとつ読み返してくれるわけではありませんよ。
⇒たぶんこれ

②問題点は明確に記載しましょう

「どうもうまくいきません」では何もわかりません。
同じ環境を手元に用意し、同じコードを実行したとしても、質問者様が何を問題としているのかわからなければ再現性の確認にもなりません。

・期待する動き
・実際の動き
・問題の発生するソースコード
・エラー内容(エラーが発生している場合)

といった内容は最低限必要です。
これらの情報があれば、回答する側もある程度問題点の絞り込みができることが多いです。

あとはコード上ではどこまで期待する動きとなっており、どこから想定外の動きとなっているのかも記載してほしい内容なのですが、これが絞れる人はある程度自力で対策も立てられるのかもしれません。

再現性について

今回提示いただいたコードと、過去の質問にあったコードを組み合わせて動作確認してみましたが、当方の環境(Windows7、Excel2010)では指定したCSV内容がアクティブシート上に展開されました。
正常動作と思えますが、やはり問題点を明確に記載いただいた方がよさそうです。

追記をお待ちしております。



追記・コメントを受けて

確認メッセージしか表示されていないということは、問題のあるコードは

Set Sheet1 = ActiveSheet

On Error Resume Next
Set activeCell = Selection
On Error GoTo 0

If Range("A1:D4") Is Nothing Then
    MsgBox "セルが選択されていないため処理を継続できません", vbExclamation + vbOKOnly, "処理失敗"
    Exit Sub
End If

csvBookPath = Application.GetOpenFilename(FileFilter:="CSV ファイル, *.csv?")


この範囲に絞られますね。

変数名の問題?

このうち最初の2つの処理Set Sheet1 = ActiveSheetSet activeCell = Selectionについてですが、Sheet1activeCellという変数名はどちらもExcelVBAでは予約語です。

こちらの環境では問題なく動作しているので何とも言えませんが、コードを読む際に誤解を招く可能性もあり、変数名としてはあまりふさわしくありません。
まずは変数名を変更してみてはどうでしょうか。(宣言や利用箇所を全て変更する必要があります)

Range("A1:D4") Is Nothing?

3つめのブロックIf Range("A1:D4") Is Nothing Thenですが、Range範囲を直接指定しているのにNothingになる状況は考えにくく、あまり意味のない判定になっていると思います。
過去の質問で提供されたコードではIf activeCell Is Nothing Thenとなっていたので、デバッグ目的か何かで変更されたままになっているのだと思います。
おそらくボタン自体がフォーカスされている状態でマクロを実行した場合に、Selectionからセル範囲を取れなくなるのを回避しようとしたのではないでしょうか。

最後に選択されていたセルを取りたいのであれば、Selectionではなく予約語の方のActiveCellで取得できます。

'Dim Sheet1 As Worksheet    '廃止
'Dim activeCell As Range    '廃止
Dim shtPaste As Worksheet
Dim rngPaste As Range

'(中略)

'Set Sheet1 = ActiveSheet
Set shtPaste = ActiveSheet

'On Error Resume Next
'Set activeCell = Selection
'On Error GoTo 0
Set rngPaste = ActiveCell

'If activeCell Is Nothing Then
If rngPaste Is Nothing Then
    MsgBox "セルが選択されていないため処理を継続できません", vbExclamation + vbOKOnly, "処理失敗"
    Exit Sub
End If

'(中略)

'Sheet1.Unprotect
shtPaste.Unprotect

With csvSheet.UsedRange
    'csvSheet.Range(csvSheet.Cells(IIf(NEED_HEADER, 1, 2), 1), .Cells(.Cells.Count)).Copy activeCell
    csvSheet.Range(csvSheet.Cells(IIf(NEED_HEADER, 1, 2), 1), .Cells(.Cells.Count)).Copy rngPaste
End With

根本のエラー原因にはまだたどり着いていませんが、ひとまずお試しください。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/01/11 17:27

    ご指摘ありがとうございました!安易な考えの質問申し訳ありません。勉強になりました、
    今後気をつけます。
    ご指摘いただいた箇所の修正いたしましたので、見ていただけたら幸いです。よろしくお願いします。

    キャンセル

  • 2018/01/11 18:01

    今回の処理ではいくつかメッセージやダイアログが表示されるタイミングがあります。

    Step1:確認MSG "選択されたセルにCSVデータをインポートしてよろしいですか?"
    Step2:失敗MSG "セルが選択されていないため処理を継続できません"
    Step3:ファイル指定ダイアログ
    Step4:失敗MSG "CSVファイルが選択されなかったので処理を終了します"
    Step5:失敗MSG "CSVファイルを開けなかったので処理を終了します"

    このうちStep1とStep3は必ず表示されるはずです。
    今回、エラーが発生するまでに表示されるのはStep1の確認メッセージだけでしょうか?

    キャンセル

  • 2018/01/11 18:27

    はい、確認MSGのみ表示され、インポートしますか?に対して はい をクリックすると
    エラーになります!

    キャンセル

  • 2018/01/12 11:00

    回答に追記しました。
    まだ根本原因にはたどり着いていませんが、変数名の変更など、まずはお試しください。

    キャンセル

  • 2018/01/12 13:00

    ありがとうございます!
    修正し試してみます。

    キャンセル

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

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

関連した質問

  • 解決済

    VBA シートコピー作成後指定枚数でファイル分割したい

    いつも大変お世話になっております。 下記の、シートコピーのコードを書きました。 コピーシート数は、32枚~最大130枚まであります、 130枚コピーすると色々大変なのと重くな

  • 解決済

    セルに値を入れたら隣のセルにボタンを自動生成させる方法

    失礼しました。 詳しく用途をご説明します。 B列とC列のセルに何かしらの値をいれると A列にBC列の値を引数とした関数を呼び出すボタンを生成させます。 また、 BとC

  • 解決済

    ダイアログ画面 キャンセル

    vbaでシートを保存時にダイアログ画面から保存場所を選択させているのですが、キャンセルを押した際に”本当に保存を中止しますか?”等のアラートを出して はい/いいえ を用意し、いいえ

  • 解決済

    エクセルマクロでグラフが表示されないです。

    グラフを各シートから値をひっぱてきて重ねて表示したいのですが、 グラフを重ねるどころか真っ白になってしまいます。 いろいろ調べてみたのですが解決できませんでしたのでご教授願

  • 受付中

    VBAを用いてバッチファイルを作成するツール

    前提・実現したいこと 業務でVBAを用いてバッチファイルを作成するツールが必要となりました。 ActiveDirectoryのUID棚卸しの為に使用します。 添付画像のような

  • 解決済

    VBAを使用したセルのコピー&ペースト時のアクティブシートの移動について

    前提・実現したいこと VBAを使用したマクロを作成したのですが、 実行環境等によってアクティブシートが移動するときとしないときがあるように感じます。 マクロ自体は対策を行い、完成し

  • 解決済

    VBA高速化について

    20個のエクセルファイルを読み込み、特定のシートにあるテーブルから特定の値を探し出し、その右横にあるセルの値を取り出します。 集計用のエクセルのテーブルでも、同じ特定の値をテーブル

  • 解決済

    データ挿入、データ削除をボタンで行う。

    VBA初心者です。 現在、スコア(球技)の集計をしたいと思い 表にファイル内のデータを指定の位置に挿入する。というものを実現したいのですが 色々試したのですが、なかなかうまくいきま

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

  • VBA

    1551questions

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