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

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

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

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

Q&A

解決済

1回答

4499閲覧

vba 空白を左詰めに時間がかかる

退会済みユーザー

退会済みユーザー

総合スコア0

VBA

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

0グッド

0クリップ

投稿2020/05/19 09:42

編集2020/05/19 11:38

同じ行内で重複があれば削除し、空白を左詰めにする作業を行っています。
下記コードで動くのですが、左詰めにすごく時間がかかります。

他に方法はないでしょうか。
ちなみに、 (xlToLeft)しなければ動きは早いです。

F g1|Fg2 |Fg3 |Fg4
aaa|bbb|空白 |ccc
aaa|aaa|空白 |空白
ccc|空白 |ccc|bbb
aaa|ccc|bbb|ddd

Sub sample() Dim i As Long, j As Long For i = 2 To Cells(Rows.Count, 1).End(xlUp).Row For j = 4 To 1 Step -1 If WorksheetFunction.CountIf(Range(Cells(i, 1), Cells(i, 4)), Cells(i, j)) > 1 Or Cells(i, j) = "" Then Cells(i, j).Delete (xlToLeft) End If Next j Next i End Sub

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

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

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

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

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

satokei

2020/05/19 10:28

とりあえず、思いついた案ですが、 1. 作業用シートへ行列入れ替えで貼り付け 2. RemoveDuplicatesメソッドで削除 3. 結果を書き戻す
退会済みユーザー

退会済みユーザー

2020/05/19 10:50

RemoveDuplicatesを調べてみたのですが、重複削除するコードであっていますか? 同じ行内の列方向の重複削除まではできていて、削除したセル=空白を左詰めしたいのですが。 教えて頂いた内容が理解できず申し訳ございません。
satokei

2020/05/19 11:00

RemoveDuplicatesメソッドは、重複行を削除して上へ詰めてくれるメソッドです。 このままでは要件に合わないため、行列入れ替えればできると思った次第です。 ただ処理が速くなるかは不明です。 https://excwlvba.blogspot.com/2013/06/removeduplicates.html
退会済みユーザー

退会済みユーザー

2020/05/19 11:35

ありがとうございます。 上に詰めるにも時間かかりました。
guest

回答1

0

ベストアンサー

セル範囲を配列に入れ、セルをクリアします。
Dictionaryを利用して重複削除し、セルに戻します。
Microsoft Scripting Library参照設定してください。
(もっと早い方法はあるかもしれません)

VBA

1Sub sample() 2 3 Dim i As Long, j As Long 4 Dim ary As Variant 5 Dim maxRow As Long: maxRow = Cells(Rows.Count, 1).End(xlUp).Row 6 ary = Range(Cells(2, 1), Cells(maxRow, 4)) 7 Cells.Clear 8 For i = 1 To UBound(ary, 1) 9 Dim dic As Dictionary 10 Set dic = New Dictionary 11 12 For j = 1 To UBound(ary, 2) 13 If ary(i, j) <> "" Then 14 If Not dic.Exists(ary(i, j)) Then dic.Add ary(i, j), Null 15 End If 16 Next j 17 18 Cells(i + 1, 1).Resize(, dic.Count) = dic.Keys 19 Set dic = Nothing 20 Next i 21 22End Sub

投稿2020/05/19 14:19

編集2020/05/20 13:56
ryuno_vanilla

総合スコア119

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

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

退会済みユーザー

退会済みユーザー

2020/05/20 03:23

ありがとうございます。できました。
ryuno_vanilla

2020/05/20 13:52 編集

速度はどうだったでしょうか? セルに1つずつアクセスしているのであまり早くなかったかもしれません。 1か所訂正します。 ' For j = 0 To dic.Count - 1 ' Cells(i + 1, j + 1).Value = dic.Keys(j) ' Next j この3行を消して Cells(i + 1, 1).Resize(, dic.Count) = dic.Keys の一行に入れ替えてみてください。 もう少し早くなると思います。 回答も編集します。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.35%

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

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

質問する

関連した質問