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

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

ただいまの
回答率

90.36%

  • VBA

    1898questions

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

  • Excel

    1624questions

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

VBA ADO 3246エラー

解決済

回答 4

投稿 編集

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

tryCSharp

score 14

Excelでcsvのデータを取り込みOracleのテーブルにADOでinsertしています。
その際、CommitTransで3246エラー「Connection オブジェクトは、トランザクション中に明示的に閉じることができません。」が出てしまいます。

データ数は約40000件です。
何か原因として考えられることはありますでしょうか。

以下のことを試しました。

  • デバッグで1件登録 → 成功
  • Execute文の前後にBeginTransとCommitTransを置いて実行 → 失敗(3246エラー発生)
    Dim sh As Worksheet
    Dim shMenu As Worksheet

    Set sh = Worksheets("Sheet1")
    Set shMenu = ActiveSheet

    '初期化
    sh.Range("A1").Select
    sh.Cells.Select
    Selection.ClearContents

    'CSVの取り込み
    With sh.QueryTables.Add(Connection:= _
        "TEXT;" & shMenu.Cells(21, 3).Value _
        , Destination:=sh.Range("A1"))
        .Name = Mid(shMenu.Cells(21, 3).Value, InStrRev(shMenu.Cells(21, 3).Value, "\") + 1)
        .FieldNames = True
        .RowNumbers = False
        .FillAdjacentFormulas = False
        .PreserveFormatting = True
        .RefreshOnFileOpen = False
        .RefreshStyle = xlInsertDeleteCells
        .SavePassword = False
        .SaveData = True
        .AdjustColumnWidth = True
        .RefreshPeriod = 0
        .TextFilePromptOnRefresh = False
        If CInt(Application.Version) > 10 Then
            .TextFilePlatform = 932
        Else
            .TextFilePlatform = xlWindows
        End If
        .TextFileStartRow = 1
        .TextFileParseType = xlDelimited
        .TextFileTextQualifier = xlTextQualifierDoubleQuote
        .TextFileConsecutiveDelimiter = False
        .TextFileTabDelimiter = False
        .TextFileSemicolonDelimiter = False
        .TextFileCommaDelimiter = True
        .TextFileSpaceDelimiter = False
        .TextFileColumnDataTypes = Array(2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 1, 2, 2, 2, 1, 1, 1, 1, 1, 1, _
        1, 2, 2, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1 _
        , 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 2, 2, 1, 1, 2, 1, 2)
        If CInt(Application.Version) > 10 Then
            .TextFileTrailingMinusNumbers = True
        End If
        .Refresh BackgroundQuery:=False
    End With
    sh.Range("A1").Select
    sh.Cells.Select
    Selection.QueryTable.Delete
    sh.Range("A1").Select

    Dim cnn As New ADODB.Connection

    cnn.Open ConStr, ConID, ConPwd

    Dim strSQL As String

    Dim lngRow As Long

    cnn.BeginTrans
    For lngRow = 2 To MaxRow
        If sh.Cells(lngRow, 1).Value = vbNullString Then
            Exit For
        End If
    strSQL = "insert into table1(COL1, COL2, COL3, COL4, COL5, COL6, COL7, COL8, COL9, COL10, COL11, COL12, COL13, COL14, COL15, COL16, COL17, COL18, COL19, COL20, COL21, COL22, COL23, COL24, COL25, COL26, COL27, COL28, COL29, COL30, COL31, COL32, COL33, COL34, COL35, COL36, COL37, COL38, COL39, COL40, COL41, COL42, COL43, COL44, COL45, COL46, COL47, COL48, COL49, COL50, COL51, COL52, COL53, COL54, COL55, COL56, COL57, COL58, COL59, COL60, COL61, COL62, COL63, COL64, COL65, COL66, COL67, COL68, COL69, COL70, COL71, COL72, COL73, COL74, COL75, COL76, COL77, COL78, COL79, COL80, COL81, COL82, COL83, COL84, COL85, COL86, COL87, COL88, COL89, COL90, COL91, COL92, COL93, COL94, COL95, COL96, COL97, COL98, COL99, COL100, COL101, COL102, COL103, COL104, COL105, COL106, COL107, COL108, COL109, COL110, COL111, COL112, COL113)values(" & _
        Trim(sh.Cells(lngRow, 1).Value) & ", '" & Trim(sh.Cells(lngRow, 2).Value) & "', " & Trim(sh.Cells(lngRow, 3).Value) & ", '" & Trim(sh.Cells(lngRow, 4).Value) & "', '" & Trim(sh.Cells(lngRow, 5).Value) & "', '" & Trim(sh.Cells(lngRow, 6).Value) & "', '" & Trim(sh.Cells(lngRow, 7).Value) & "', '" & Trim(sh.Cells(lngRow, 8).Value) & "', '" & Trim(sh.Cells(lngRow, 9).Value) & "', '" & Trim(sh.Cells(lngRow, 10).Value) & "', '" & _
        Trim(sh.Cells(lngRow, 11).Value) & "', '" & Trim(sh.Cells(lngRow, 12).Value) & "', '" & Trim(sh.Cells(lngRow, 13).Value) & "', '" & Trim(sh.Cells(lngRow, 14).Value) & "', '" & Trim(sh.Cells(lngRow, 15).Value) & "', '" & Trim(sh.Cells(lngRow, 16).Value) & "', '" & Trim(sh.Cells(lngRow, 17).Value) & "', '" & Trim(sh.Cells(lngRow, 18).Value) & "', '" & Trim(sh.Cells(lngRow, 19).Value) & "', '" & Trim(sh.Cells(lngRow, 20).Value) & "', '" & _
        Trim(sh.Cells(lngRow, 21).Value) & "', '" & Trim(sh.Cells(lngRow, 22).Value) & "', '" & Trim(sh.Cells(lngRow, 23).Value) & "', '" & Trim(sh.Cells(lngRow, 24).Value) & "', '" & Trim(sh.Cells(lngRow, 25).Value) & "', " & Trim(sh.Cells(lngRow, 26).Value) & ", '" & Trim(sh.Cells(lngRow, 27).Value) & "', '" & Trim(sh.Cells(lngRow, 28).Value) & "', '" & Trim(sh.Cells(lngRow, 29).Value) & "', '" & Trim(sh.Cells(lngRow, 30).Value) & "', '" & _
        Trim(sh.Cells(lngRow, 31).Value) & "', '" & Trim(sh.Cells(lngRow, 32).Value) & "', '" & Trim(sh.Cells(lngRow, 33).Value) & "', '" & Trim(sh.Cells(lngRow, 34).Value) & "', '" & Trim(sh.Cells(lngRow, 35).Value) & "', " & Trim(sh.Cells(lngRow, 36).Value) & ", '" & Trim(sh.Cells(lngRow, 37).Value) & "', '" & Trim(sh.Cells(lngRow, 38).Value) & "', " & Trim(sh.Cells(lngRow, 39).Value) & ", " & Trim(sh.Cells(lngRow, 40).Value) & ", " & _
        Trim(sh.Cells(lngRow, 41).Value) & ", " & Trim(sh.Cells(lngRow, 42).Value) & ", " & Trim(sh.Cells(lngRow, 43).Value) & ", " & Trim(sh.Cells(lngRow, 44).Value) & ", " & Trim(sh.Cells(lngRow, 45).Value) & ", " & Trim(sh.Cells(lngRow, 46).Value) & ", " & Trim(sh.Cells(lngRow, 47).Value) & ", " & Trim(sh.Cells(lngRow, 48).Value) & ", " & Trim(sh.Cells(lngRow, 49).Value) & ", " & Trim(sh.Cells(lngRow, 50).Value) & ", " & _
        Trim(sh.Cells(lngRow, 51).Value) & ", '" & Trim(sh.Cells(lngRow, 52).Value) & "', " & Trim(sh.Cells(lngRow, 53).Value) & ", " & Trim(sh.Cells(lngRow, 54).Value) & ", " & Trim(sh.Cells(lngRow, 55).Value) & ", " & Trim(sh.Cells(lngRow, 56).Value) & ", " & Trim(sh.Cells(lngRow, 57).Value) & ", " & Trim(sh.Cells(lngRow, 58).Value) & ", " & Trim(sh.Cells(lngRow, 59).Value) & ", " & Trim(sh.Cells(lngRow, 60).Value) & ", " & _
        Trim(sh.Cells(lngRow, 61).Value) & ", '" & Trim(sh.Cells(lngRow, 62).Value) & "', '" & Trim(sh.Cells(lngRow, 63).Value) & "', " & Trim(sh.Cells(lngRow, 64).Value) & ", " & Trim(sh.Cells(lngRow, 65).Value) & ", '" & Trim(sh.Cells(lngRow, 66).Value) & "', '" & Trim(sh.Cells(lngRow, 67).Value) & "', '" & Trim(sh.Cells(lngRow, 68).Value) & "', '" & Trim(sh.Cells(lngRow, 69).Value) & "', '" & Trim(sh.Cells(lngRow, 70).Value) & "', " & _
        Trim(sh.Cells(lngRow, 71).Value) & ", " & Trim(sh.Cells(lngRow, 72).Value) & ", " & Trim(sh.Cells(lngRow, 73).Value) & ", " & Trim(sh.Cells(lngRow, 74).Value) & ", '" & Trim(sh.Cells(lngRow, 75).Value) & "', '" & Trim(sh.Cells(lngRow, 76).Value) & "', '" & Trim(sh.Cells(lngRow, 77).Value) & "', " & Trim(sh.Cells(lngRow, 78).Value) & ", '" & Trim(sh.Cells(lngRow, 79).Value) & "', '" & Trim(sh.Cells(lngRow, 80).Value) & "', '" & _
        Trim(sh.Cells(lngRow, 81).Value) & "', '" & Trim(sh.Cells(lngRow, 82).Value) & "', " & Trim(sh.Cells(lngRow, 83).Value) & ", " & Trim(sh.Cells(lngRow, 84).Value) & ", '" & Trim(sh.Cells(lngRow, 85).Value) & "', '" & Trim(sh.Cells(lngRow, 86).Value) & "', " & Trim(sh.Cells(lngRow, 87).Value) & ", " & Trim(sh.Cells(lngRow, 88).Value) & ", '" & Trim(sh.Cells(lngRow, 89).Value) & "', " & Trim(sh.Cells(lngRow, 90).Value) & ", " & _
        Trim(sh.Cells(lngRow, 91).Value) & ", '" & Trim(sh.Cells(lngRow, 92).Value) & "', '" & Trim(sh.Cells(lngRow, 93).Value) & "', " & Trim(sh.Cells(lngRow, 94).Value) & ", " & Trim(sh.Cells(lngRow, 95).Value) & ", '" & Trim(sh.Cells(lngRow, 96).Value) & "', '" & Trim(sh.Cells(lngRow, 97).Value) & "', " & Trim(sh.Cells(lngRow, 98).Value) & ", " & Trim(sh.Cells(lngRow, 99).Value) & ", '" & Trim(sh.Cells(lngRow, 100).Value) & "', '" & _
        Trim(sh.Cells(lngRow, 101).Value) & "', " & Trim(sh.Cells(lngRow, 102).Value) & ", " & Trim(sh.Cells(lngRow, 103).Value) & ", '" & Trim(sh.Cells(lngRow, 104).Value) & "', " & Trim(sh.Cells(lngRow, 105).Value) & ", '" & Trim(sh.Cells(lngRow, 106).Value) & "', '" & Trim(sh.Cells(lngRow, 107).Value) & "', " & Trim(sh.Cells(lngRow, 108).Value) & ", " & Trim(sh.Cells(lngRow, 109).Value) & ", " & Trim(sh.Cells(lngRow, 110).Value) & ", " & _
        Trim(sh.Cells(lngRow, 111).Value) & ", '" & Trim(sh.Cells(lngRow, 112).Value) & "', '" & Trim(sh.Cells(lngRow, 113).Value) & "')"

        cnn.Execute strSQL
    Next lngRow
    cnn.CommitTrans
    cnn.Close
    Set cnn = Nothing
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • sazi

    2018/04/25 15:28 編集

    実行したいのはinsert ですよね。strSQLの中身のSQLが中途半端ですが、コードはこれが全てですか?

    キャンセル

  • m6u

    2018/04/25 15:30

    ADODB.Connection.Openメソッドで与えている接続文字列を差し支えない範囲で示してほしいのと、ODBCにOracleデータベースへの接続を定義していると思うので設定画面の画像を示してほしいところでしょうか。

    キャンセル

  • tryCSharp

    2018/04/25 15:53

    コピペミスでINSERT INTO テーブル名が抜けていました。

    キャンセル

  • tryCSharp

    2018/04/25 15:55

    >ADODB.Connection.Openメソッドで与えている接続文字列は「Provider=MSDAORA;Data Source=サーバー名」です。ODBCは使っておらずinstant clientを使っています。

    キャンセル

回答 4

check解決した方法

0

皆様ご回答ありがとうございました。
MaxRowが65536になっていた為余計なSQL文をExecuteしてしまっていたようです。
For文の終了時に条件でFor文を抜けるようにしたところ上手くいきました。

'・・・
'・・・
'・・・
        cnn.Execute strSQL
        If sh.Cells(lngRow + 1, 1).Value = vbNullString Then
            Exit For
        End If
    Next lngRow
cnn.CommitTrans
    cnn.Close
    Set cnn = Nothing

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/04/26 11:07

    余計なSQL文で実はエラーになっていたということですか?

    キャンセル

  • 2018/04/26 11:39

    実際は約40000件しかないのでそれ以上のセルは空なのでsh.Cells(lngRow, 1).Valueで取得した文字は空文字で「insert into table1(COL1, COL2, ・・・)values(,'', ・・・)」というようなExecute文が実行されていたようです。でもよくよく見たらFor文の先頭に「If sh.Cells(lngRow, 1).Value = vbNullString Then」がありますね。あれ??

    キャンセル

  • 2018/04/26 11:44 編集

    saziさん
    m6uさん
    ttyp03さん

    コピペミスがありました。For文の先頭をなぜか
    If sh.Cells(lngRow, 1).Value = vbNullString Then
    cnn.Close
    Exit For
    End If
    にしていました。過去の自分がなぜか加えた方がいいと思ったようです。
    ご面倒お掛け致して申し訳ございません。

    キャンセル

  • 2018/04/26 11:51

    開始したトランザクションを終了せずに、コネクションをクローズするロジックがあったということですね。納得しました。

    キャンセル

0

ヒント:Debug.Print strSQL

INSERT INTO テーブル名
に該当する文言がstrSQLに含まれてない不完全なSQLをCommitTransしようとしても、ね。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/04/25 15:53

    コピペミスでINSERT INTO テーブル名が抜けていました。

    キャンセル

0

コードは全てですか?
Connectionに対してopenしたままrecordsetなどがありませんか?

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/04/25 17:00

    recordsetは取込後に使用しています。

    キャンセル

0

1件がOKで40000件がNGだとすると、大量にコミットしようとして処理が追いつかずに、コミット完了前にCloseしようとしてエラー?
CommitTransが非同期で戻ってくるのか、いまいち調べても良くわかりませんでした。
仮に、例えば1000件ごとにコミットするとかしたらどうなるのでしょうか。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/04/26 10:49

    Execute文の前後にBeginTransとCommitTransを置いて実行した時に失敗していました。

    キャンセル

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

  • VBA

    1898questions

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

  • Excel

    1624questions

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