●やりたいこと(一応、できている)
A列1行目から最終行(1045876)まですべてのセルに1~100までの数字がランダムで入ってます。(header なし)
B列1行目から最終行(1045876)まですべてのセルに boy or girl のどちらかがランダムで入ってます。(header なし)
この中から、条件(A列50以上B列boy)に合致する行のみをD列とE列に転記したい。
●質問
条件に合致した行だけ、配列の要素数が必要になるが、その要素数がいくつになるかわからない。
その場合、配列の要素数の初期値は実際に必要となる要素数よりも少ないとエラーが出るが要素数が多い分には、問題はないのか?
下記コードのように、配列の1次元の要素数を最終行(1 to rows.count)としました。
つまり、すべての行が条件に合致した場合にも、要素数が足りるようにしています。
実際には、もっと少ない行数が条件に合致するため、この1,045,876も必要ありません。
しかし、これはこれで、間違ってはいないやり方なのでしょうか?
最初に自分で考えたやり方は、初期値を(1 to1 ,1 to2)にしておき、
条件に合致した行のみ、配列の一番上の要素数に入れる、そして、redim preserveで1次元目の要素数を1つ足すというやり方をやろうとしました。しかし、調べたら、皆さんご指摘の通り、1次元目の要素数は、変更できないということがわかり、上記のようなやり方をとりました。
もっとスマートなやり方は他にありますでしょうか?
ちなみに、転記件数は、全体の半分のおよそ52万件で30秒くらいかかりました。
十分早いのでしょうか?
Sub hairetsu() Dim i, v, j, z Dim ary() j = 1 ReDim ary(1 To Rows.Count, 1 To 2) For i = 1 To Rows.Count If Cells(i, 1) >= 50 And Cells(i, 2) = "boy" Then For v = 1 To UBound(ary, 2) ary(j, v) = Cells(i, v) Next v j = j + 1 End If Next i Cells(1, 4).Resize(UBound(ary, 1), UBound(ary, 2)) = ary End Sub
回答3件
あなたの回答
tips
プレビュー
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。
2021/06/01 13:33
2021/06/01 13:37
2021/06/02 22:16
2021/06/02 22:49
2021/06/08 22:09