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

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

ただいまの
回答率

88.10%

vbaで、mwsに活用

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,106

score -123

http://mrxblog.net/2015/10/22_mws_excel_vba/

上記URLがあるのですが、
' ASINを20個づつ取り出す以降の箇所で、

cur_j = 20
For j = 1 To 20
' 空になっているか確認する
If (lowestWS.Cells(lowestRow, ASINCol) = "" And lowestWS.Cells((lowestRow + 1), ASINCol) = "") Then
loopBit = 0
cur_j = j - 1
Exit For
End If

これは、価格の一番低い、価格を抽出していると思うのですが、
これに加えて、Weight    Currency    ListPrice    Image    Image URL

これらを一編にコードで記入する場合は、arrayを使用するのでしょうか。
ひとつづ記入するのが、手間なので一度に記入する方法は、ございませんでしょうか。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

+3

本当にlowestRowは一番価格の安い行でしょうか?
変数の名前からそう思い込まれていませんか?

順を追って変数の使われ方を見ていくと、

'ASIN のスタート行
lowestRow = 2
ASINCol = 1


でASINの開始行としてlowestRowに2が設定されています。
ついでにASINColには1が設定され、これ以降変更されていません。


その後lowestRowは使用されないまま質問いただいた処理に入ります。

' ASINを20個づつ取り出す
cur_j = 20
For j = 1 To 20
    ' 空になっているか確認する
    If (lowestWS.Cells(lowestRow, ASINCol) = "" And lowestWS.Cells((lowestRow + 1), ASINCol) = "") Then
        loopBit = 0
        cur_j = j - 1
        Exit For
    End If
    ' ASINを配列に格納
    asin(j) = CStr(lowestWS.Cells(lowestRow, ASINCol).Value)
    ' 次行へ
    lowestRow = lowestRow + 1
Next


まず、jを20回ループする中で、1回ごとにlowestRowを +1しています。
そうやってlowestRowを1行ずつずらしながら、lowestWS.Cells(lowestRow, ASINCol)のセル値とlowestWS.Cells((lowestRow + 1), ASINCol)のセル値をチェックしています。

lowestWS.Cells(lowestRow, ASINCol)についてもう少し掘り下げると、
lowestWSは関数の最初で"getLowest"というシートが設定されている変数です。
このシートのlowestRow行, ASINCol列の座標のセルから値を取り出しています。
ループ1周目ならlowestRowは2、ASINColは1なので、A2セルということになります。
ループ2周目ではlowestRowは3になり、ASINColは1のままなのでA3セルを取得します。

同様にlowestWS.Cells(lowestRow, ASINCol)は、lowestRow + 1行, ASINCol列の座標のセルを指すことになります。
ループ1周目ならA3セル、ループ2周目ならA4セルということになります。


条件文は
If (lowestWS.Cells(lowestRow, ASINCol) = "" And lowestWS.Cells((lowestRow + 1), ASINCol) = "") Then
です。
ループ1周目ならA2セルが空セルで、かつA3セルも空セルの場合に条件文の中の処理を実行します。
ループ2周目ならA3セルが空セルで、かつA4セルも空セルの場合に条件文の中の処理を実行します。

つまり、lowestRow行とその次の行のASIN列に何も入力がない場合に条件文の中の処理を行うということです。

条件文の中では

loopBit = 0
cur_j = j - 1
Exit For


を行っています。

loopBitはこの20回のForループよりも上位で行っているWhileループを抜けるためのフラグのようです。
cur_jはForループに入る前に20という値をセットしていますが、ループの途中で条件を満たした場合にループ回数-1の値に再セットしています。
そしてExit ForでForループを抜けています。

条件を満たしていなければ、そのあとの処理でセルの値をASIN配列に格納し、次の行も処理していきます。


まとめると、
・A列を2行目から順にチェックする。
・20回処理するまでに2行連続で空欄が続いた場合はループ処理を中断する。
・それ以外の場合は取得したセル内容をASIN配列に格納していく。
という処理のようです。

ここまでの処理で抽出しているのは最低価格ではなく、ASINの情報です。

この処理ではASIN配列に値を格納していますが、他の情報も格納していきたいのであれば、ASINに変わる他の配列をそれぞれ用意してもいいですし、1つの多次元配列でまとめて管理する方法もあると思います。
どう管理したら扱いやすいかでご判断ください。


手早く解析するために変数名からおおよその処理を推測することもありますが、思い込みは危険です。
ひとつづつ読み解き、推測に対する裏付けとっていくことが大事です。
面倒ですが、それがコーディングというものですので頑張ってみてください。

参考になれば幸いです。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

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

  • ただいまの回答率 88.10%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

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