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

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

新規登録して質問してみよう
ただいま回答率
85.48%
VBA

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

Q&A

解決済

1回答

1110閲覧

vbaで重複データを削除して出力したい

maimikan

総合スコア1

VBA

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

0グッド

0クリップ

投稿2023/03/21 12:14

実現したいこと

①重複したデータ行を削除して出力したい

前提

マクロよりcsvファイルを取り込み、重複したレコードを削除する処理を実施し、重複しない出力結果を取得したいです。
ディクショナリーオブジェクトで、fruitNumが重複しているか重複していないかでaとbのディクショナリオブジェクトに分ける前に、
RemoveDuplicatesで、12列目、46列目、41列目が同一データの行を削除したいです。
ですが、エラーで「アプリケーション定義またはオブジェクト定義のエラーです」とでてしまいます
理由がわからず、どこを改善すればエラーが解消されますでしょうか。
よろしくお願いいたします。
補足ですが、ヘッダーは1~3行目までで、セルの結合を行っているので、4列目から読み込むようにしています(RemoveDuplicatesは結合したセルには使用できないかと思いますので。マクロで、レイアウトのシートを参照するように設定しています)。
また、取り込みたいcsvファイルのレコードは、様々な種類の重複があります。

Range("A4:DD" & maxRow).RemoveDuplicates (Array(12, 46, 41))を挿入する前は、うまく動きました。
前提として、下記のvbaコードは抜粋となります。

RemoveDuplicates以外の方法で可能なコードがあれば、それでもかまいませんのでご教示ください。

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

該当のソースコード

vbaのソースコード

1 2 Set aDic = CreateObject("Scripting.Dictionary") 3 Set bDic = CreateObject("Scripting.Dictionary") 4 Dim maxRow As Long 5 maxRow = Cells(Rows.Count, 4).End(xlUp).Row 6 7 'データ行まで読み飛ばし 8 Line Input #1, buf 'ヘッダー行1読込 9 Line Input #1, buf 'ヘッダー行2読込 10 Line Input #1, buf 'ヘッダー行3読込 11 Line Input #1, buf 'カラムID行読込 12 Do Until EOF(1) 13 Line Input #1, buf 14 tmp = Split(buf, ",") 15 16 '重複しているデータを削除 17 Range("A4:DD" & maxRow).RemoveDuplicates (Array(12, 46, 41)) 18 19 fruitNum = tmp(11) 20 ' aディクショナリーにfruitNumが存在しない場合、追加 21 If Not aDic.Exists(fruitNum) Then 22 aDic.Add fruitNum, fruitNum 23 ' aディクショナリーに社員番号が存在する場合、 bディクショナリーに追加 24 Else 25 bDic.Add fruitNum, fruitNum 26 End If 27 Loop 28 29 Close #1 30 31'データ行まで読み飛ばし 32 Line Input #1, buf 'ヘッダー行1読込 33 Line Input #1, buf 'ヘッダー行2読込 34 Line Input #1, buf 'ヘッダー行3読込 35 Line Input #1, buf 'カラムID行読込 36 Do Until EOF(1) '縦方向ループ 37 Line Input #1, buf '1行読込 38 buf = "," & buf 'ダミー列を追加 39 tmp = Split(buf, ",") 40 41 Case IntSyokyu_HK,_K 42 fruitNum = tmp(12) 43 Kingaku= tmp(46) 44 ShouhinnCode = tmp(41) 45 ' Kingakuまたは ShouhinnCodeが空白または0で、かつfruitNumが重複している場合、レコードをスキップする 46 If (shokusekiTeate = "") Or (shokusekiTeate = 0) And (hyoukaKyu = "") Or (hyoukaKyu = 0) And bDic.Exists(employeeNum) Then 47 GoTo Continue 48 ElseIf (Kingaku = "") Or (Kingaku = 0) And bDic.Exists(fruitNum) Then 49 GoTo Continue 50 End If 51 End Select

試したこと

ここに問題に対して試したことを記載してください。

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

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

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

tatsu99

2023/03/22 03:37

CSVファイルをオープンしている行が見当たりません。 マクロの全ての行を提示したほうが、良い回答が付くかと思います。 又、29行で#1をクローズした後、オープンせずに32行で、#1を使用しています。 これでは、正常に動作するはずがありません。
guest

回答1

0

ベストアンサー

重複削除の処理を挿入して動かなくなったということなので、その部分が問題になっているのだと思います。
そこでまずは次のように変更してみて動作するか確認してみることをお勧めします。

変更前:Range("A4:DD" & maxRow).RemoveDuplicates (Array(12, 46, 41))
変更後:Range("A4:DD" & maxRow).Select

この時点でエラーが出るか、その先でエラーになっているかが分かります。
もしこれでエラーが出た場合は Range の指定で失敗していると考えられるので、次のように修正してみると動くかもしれません。

変更前:Range("A4:DD" & maxRow)
変更後:Range("A4:DD" & cstr(maxRow))

cstr は引数のデータを文字列型に変換する関数です。
変数 maxRow が Long 型になっているので、"A4:DD" との結合を行うために maxRow を文字列型に変換しています。

投稿2023/03/21 12:51

ga3

総合スコア23

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問