
現在、VBAで、
Find関数で行う処理を記述しています。
以下、記述------------------------
Dim LastRow, onecell As String
Dim OrderNew As String
LastRow = Range("B38").End(xlDown).Row
oldad = 38
For cnt = 38 To LastRow
OrderNo = Range("D" & oldad).Value
Range("K" & cnt).Value = Range("B" & oldad).Value
Range("M" & cnt).Value = Range("C" & oldad).Value
Range("N" & cnt).Value = Range("D" & oldad).Value
Range("O" & cnt).Value = Range("E" & oldad).Value
Range("P" & cnt).Value = Range("F" & oldad).Value
Range("P" & cnt).Value = Range("G" & oldad).Value
Range("Q" & cnt).Value = Range("H" & oldad).Value
Set OrderNoResult = Range("D" & 38 & ":D" & LastRow).Find (OrderNo, MatchByte:=False, MatchCase:=False, LookIn:=xlValues, Lookat:=xlWhole, SearchOrder:=xlByRows)
OrderResultValue = OrderNoResult.Value
Do While OrderNoResult.Value = OrderResultValue oldad = OrderNoResult.Row
Set OrderNoResult = Range("D" & 38 & ":D" &LastRow).FindNext(OrderNoResult)
If OrderNoResult Is Nothing Then
Exit Do
End If
Loop
Range("L" & cnt).Value = Range("B" & oldad).Value
Range("R" & cnt).Value = Range("I" & oldad).Value
oldad = oldad + 1
Next cnt
終わり----------------------------
30週目で必ず止まります。値はあります。Ctrl+Fで同じ値をペーストして、実行して該当したので間違いありません。
また、一度停止し、再度実行すると、その一周は動き、また停止します。これを一回ずつ繰り返す場合は、問題なく処理されます。
オブジェクトの初期化のような処理が必要なのでしょうか?
ご教授お願いいたします。
No.2ベストアンサー
- 回答日時:
根本的に Find の使用の仕方が間違ってると思います
下記で代用できませんか
Dim LastRow As String
Dim OrderNew As String
'最終行を取得
LastRow = Range("B38").End(xlDown).Row
oldad = 38
'最終行になるまでループ
For cnt = 38 To LastRow
'初期値セット(値が詰められている最初の行)。一週目は'1'
OrderNo = Range("D" & oldad).Value
'ヘッダーをセット。
Range("K" & cnt).Value = Range("B" & oldad).Value
Range("M" & cnt).Value = Range("C" & oldad).Value
Range("N" & cnt).Value = Range("D" & oldad).Value
Range("O" & cnt).Value = Range("E" & oldad).Value
Range("P" & cnt).Value = Range("F" & oldad).Value
Range("P" & cnt).Value = Range("G" & oldad).Value
Range("Q" & cnt).Value = Range("H" & oldad).Value
'D列の注文番号を検索。Rangeオブジェクトを格納。
'Rangeオブジェクトから値を格納。
OrderResultValue = Range("D" & oldad).Value
'ループ。比較。検索される値が見つからなくなったらその行番号を取得し、 _
IF文によりループエンド。その際、次の検索開始位置をoldadで確保。
Do While Range("D" & oldad + 1).Value = OrderResultValue
oldad = oldad + 1
Loop
Range("L" & cnt).Value = Range("B" & oldad).Value
Range("R" & cnt).Value = Range("I" & oldad).Value
'決済行の次行が、新たな発注番号のため、oldad+1で次行取得。
oldad = oldad + 1
Next cnt
試してみてください
解決しました。
私のやりたい処理は、そもそもループで拾える処理だったんですね・・・・。
私があまりに短絡的な発想をしていたみたいで恐縮です。
非常に助かりました。
ありがとうございます。
No.1
- 回答日時:
こんな複雑なコードを質問にコピーし、エラーを考えてくれていうのは無茶だ。
コードを読み解くことまで読者に放り投げている。
全体的なエクセルでの処理・データの成り立ち・したいことを文章で説明すること。
プログラムでやっている処理など文章ででも説明を添えたら。
>Range("K" & cnt).Value = Range("B" & oldad).Value
でCntは38から最下行まで変化させているが、右辺のoldad = 38は38で固定している。処理内容がFindならなぜ繰り返すのか一見して判らなかった。
>30週目で
週は表上ではどう変化するのか?週が変わるとデータの列が変わるのか?
突然出てきて判らない。読者・回答者に配慮した質問をしてほしい。
作った自分だけが判ることでは困る。
例えばFind("g")は私の勝手な例ですが
Sub test02()
LastRow = Range("B38").End(xlDown).Row
OrderNoResult = Range("D" & 38 & ":D" & LastRow).Find("g")
MsgBox OrderNoResult
End Sub
では、初発見のセルの「値」しか返さないが、値は探索した値に決まっている。普通は見つかった行などを後の処理に使うのではと思うが、値では何に使うのかな。
後の重複発見探求はFindNextで追求する必要が有るが、
OrderResultValue = OrderNoResult.Value
はなぜ行っているのか。
この回答への補足
現在、VBAで、
Find関数で行う処理を記述しています。
以下、記述------------------------
Dim LastRow As String
Dim OrderNew As String
'最終行を取得
LastRow = Range("B38").End(xlDown).Row
oldad = 38
'最終行になるまでループ
For cnt = 38 To LastRow
'初期値セット(値が詰められている最初の行)。一週目は'1'
OrderNo = Range("D" & oldad).Value
'ヘッダーをセット。
Range("K" & cnt).Value = Range("B" & oldad).Value
Range("M" & cnt).Value = Range("C" & oldad).Value
Range("N" & cnt).Value = Range("D" & oldad).Value
Range("O" & cnt).Value = Range("E" & oldad).Value
Range("P" & cnt).Value = Range("F" & oldad).Value
Range("P" & cnt).Value = Range("G" & oldad).Value
Range("Q" & cnt).Value = Range("H" & oldad).Value
'D列の注文番号を検索。Rangeオブジェクトを格納。
Set OrderNoResult = Range("D" & 38 & ":D" & LastRow).Find (OrderNo, MatchByte:=False, MatchCase:=False, LookIn:=xlValues, Lookat:=xlWhole, SearchOrder:=xlByRows)
'Rangeオブジェクトから値を格納。
OrderResultValue = OrderNoResult.Value
'ループ。比較。検索される値が見つからなくなったらその行番号を取得し、IF文によりループエンド。その際、次の検索開始位置をoldadで確保。
Do While OrderNoResult.Value = OrderResultValue oldad = OrderNoResult.Row
Set OrderNoResult = Range("D" & 38 & ":D" &LastRow).FindNext(OrderNoResult)
If OrderNoResult Is Nothing Then
Exit Do
End If
Loop
Range("L" & cnt).Value = Range("B" & oldad).Value
Range("R" & cnt).Value = Range("I" & oldad).Value
'決済行の次行が、新たな発注番号のため、oldad+1で次行取得。
oldad = oldad + 1
Next cnt
終わり----------------------------
ご教授お願いいたします。
@書き直しました。
・やりたいこと
ある行に、注文番号が1から連なっています。
この注文番号で構成されるレコードは、重複もあります。
発注「1」
注文変更「1」
決済「1」
のように、多レコードで構成されています。
今回やりたいのは、この発注部分と決済部分の注文番号の取得です。
乱暴な記述をしてしまい申し訳ありません。
お探しのQ&Aが見つからない時は、教えて!gooで質問しましょう!
このQ&Aを見た人はこんなQ&Aも見ています
関連するカテゴリからQ&Aを探す
今、見られている記事はコレ!
-
弁護士が解説!あなたの声を行政に届ける「パブリックコメント」制度のすべて
社会に対する意見や不満、疑問。それを発信する場所は、SNSやブログ、そしてニュースサイトのコメント欄など多岐にわたる。教えて!gooでも「ヤフコメ民について」というタイトルのトピックがあり、この投稿の通り、...
-
弁護士が語る「合法と違法を分けるオンラインカジノのシンプルな線引き」
「お金を賭けたら違法です」ーーこう答えたのは富士見坂法律事務所の井上義之弁護士。オンラインカジノが違法となるかどうかの基準は、このように非常にシンプルである。しかし2025年にはいって、違法賭博事件が相次...
-
釣りと密漁の違いは?知らなかったでは済まされない?事前にできることは?
知らなかったでは済まされないのが法律の世界であるが、全てを知ってから何かをするには少々手間がかかるし、最悪始めることすらできずに終わってしまうこともあり得る。教えてgooでも「釣りと密漁の境目はどこです...
-
カスハラとクレームの違いは?カスハラの法的責任は?企業がとるべき対応は?
東京都が、客からの迷惑行為などを称した「カスタマーハラスメント」、いわゆる「カスハラ」の防止を目的とした条例を、全国で初めて成立させた。条例に罰則はなく、2025年4月1日から施行される。 この動きは自治体...
-
なぜ批判コメントをするの?その心理と向き合い方をカウンセラーにきいた!
今や生活に必要不可欠となったインターネット。手軽に情報を得られるだけでなく、ネットを介したコミュニケーションも一般的となった。それと同時に顕在化しているのが、他者に対する辛らつな意見だ。ネットニュース...
おすすめ情報
このQ&Aを見た人がよく見るQ&A
デイリーランキングこのカテゴリの人気デイリーQ&Aランキング
-
「ご処理進めて頂きますようお...
-
エクセルで、日付を入力すると...
-
【Excel】特定の文字を含むセル...
-
UMLでの例外処理
-
switch の範囲指定
-
DoEventsがやはり分からない
-
VBAでループ内で使う変数名を可...
-
VBAでCOPYを繰り返すと、処理が...
-
EXCEL VBA マクロ 実行する度に...
-
月度は何て読みますか?
-
CStringの中から数字だけの場合...
-
リョウ・・・量?料?
-
VB.NET Excelを読み込んでDataT...
-
インタラクティブの反対語は?
-
ボタンクリックイベント 重複...
-
Do~Loopした回数をカウントしたい
-
findは動くがfindnextがマクロ...
-
セックスレスの既婚女性は自慰...
-
処理の実行中にOutLookがまとも...
-
フォーム上のすべてのTextBoxを...
マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング
-
「ご処理進めて頂きますようお...
-
エクセルで、日付を入力すると...
-
【Excel】特定の文字を含むセル...
-
switch の範囲指定
-
VBAでループ内で使う変数名を可...
-
EXCEL VBA マクロ 実行する度に...
-
UMLでの例外処理
-
銀行の窓口処理の件で知ってる...
-
VBAでCOPYを繰り返すと、処理が...
-
月度は何て読みますか?
-
インタラクティブの反対語は?
-
VB.NET DataGridViewでIsNewRo...
-
DoEventsがやはり分からない
-
VBの質問#if 0 then ってどう...
-
お家デートをしててハグを長い...
-
セックスレスの既婚女性は自慰...
-
VB.NET Excelを読み込んでDataT...
-
リョウ・・・量?料?
-
メルカリのメルカードで買い物...
-
Loadイベント中にほかのイベン...
おすすめ情報