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

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

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

for文は、様々なプログラミング言語で使われている制御構造です。for文に定義している条件から外れるまで、for文内の命令文を繰り返し実行します。

Google Apps Script

Google Apps ScriptはGoogleの製品と第三者のサービスでタスクを自動化するためのJavaScriptのクラウドのスクリプト言語です。

Q&A

解決済

1回答

804閲覧

GASのfor文で、値が1つだけしか取得されない

takoyaki87

総合スコア19

for

for文は、様々なプログラミング言語で使われている制御構造です。for文に定義している条件から外れるまで、for文内の命令文を繰り返し実行します。

Google Apps Script

Google Apps ScriptはGoogleの製品と第三者のサービスでタスクを自動化するためのJavaScriptのクラウドのスクリプト言語です。

0グッド

0クリップ

投稿2022/07/22 02:47

編集2022/07/22 02:48

前提

GASで商品販売管理システムを作っています。
注文された商品ごとの製造年月日を取得し、発送管理表に転記したいのですが、上手くいきません。

システムの一連の流れとしては、

① Googleフォームに商品データを入力
② Googleフォームと紐づいたシートにデータ転記
③ ②で転記されたデータを、別の発送管理表にさらに転記
④ 処理が終わったデータのシステム送信欄に「済」という文字をつける

です。

↑②・③の2ステップを踏む理由

なぜわざわざ「② → ③」と2ステップで転記するのかといいますと、③の転記のさい、商品ごとの注文データを1行ずつ表示したいからです。

②のデータ転記では、ひとつの取引先から一回の注文につき商品が複数注文されることを想定し、Googleフォームから入力された商品データをまとめて1行で表示しています。下記画像のとおりです。
イメージ説明
(↑Googleフォームに紐づいているシートです)

これを③の転記で各商品につき一列で表示すると、下記画像のようになります。

イメージ説明
(↑Googleフォームに紐づいたシートのデータから、GASで別のワークブックのシートに自動転記しています)

このように、発送管理表には「商品ごとにデータを1行ずつ表示したい」ので、2ステップで処理しています。

今回実現したいこと

各商品 (20g、80g、1kg、20kgの四種類) はそれぞれ別の製造年月日のため、商品ごとに割り振られた製造年月日データを、発送管理表に表示させたいです。

(製造年月日の参照先データ↓)
イメージ説明

発生している問題・エラーメッセージ

製造年月日をGASで取得するさい、現状のコードですとB3セルの「2022/07/02」というデータしか取得されず、発送管理表にも商品の種類問わず全て「2022/07/02」と表記されてしまいます。

イメージ説明

本来ですと、製造年月日の参照データから

・2022/07/02 (20gの製造年月日)
・2022/07/08 (80gの製造年月日)
・2022/07/01 (1kgの製造年月日)
・2022/07/20 (20kgの製造年月日)

を全て取得したいのですが、最初の20gの製造年月日「2022/07/02」だけが取得されている状態です。

該当のソースコード

function sendData() { var sheet1 = SpreadsheetApp.getActiveSpreadsheet().getSheetByName("必要なデータ"); var sheet2 = SpreadsheetApp.getActiveSpreadsheet().getSheetByName("回答"); var lastRow2 = sheet2.getLastRow(); var producer = sheet1.getRange(2,2).getValue(); var anotherSheet = SpreadsheetApp.openById("1jl79opzAO9jS9_O39GUb5o6xvw98RxySegrIPDMDWXQ") var sheetH = anotherSheet.getSheetByName('発送管理表');   var lastRowH = sheetH.getLastRow();                 //商品の送り先、発送日を取得 for(var r = 2; r <= lastRow2; r++){ var twoWords = sheet2.getRange(r,5,1,2).getValues(); var check = sheet2.getRange(r,4);                 //商品の種類、数量を取得 for(var c = 7; c <= 10; c++){ var gram = sheet2.getRange(1,c).getValue(); var amount = sheet2.getRange(r,c).getValue(); //商品の製造年月日を取得 (ここだけ値が1種類しか取得されていません) for(var u = 3; u <= 6; u++) { var makeDay = sheet1.getRange(u,2).getValue(); //数量の欄に値がない or システム送信が「済」なら何もせず、そうじゃなければデータ転記 if(amount === "" || check.getValue() === "済"){;}else{ lastRowH = lastRowH + 1 sheetH.getRange(lastRowH,2,1,2).setValues(twoWords); sheetH.getRange(lastRowH,4).setValue(gram); sheetH.getRange(lastRowH,5).setValue(amount); sheetH.getRange(lastRowH,7).setValue("製品"); sheetH.getRange(lastRowH,8).setValue(makeDay); sheetH.getRange(lastRowH,9).setValue(producer); } //一連の処理が繰り返されないよう、ループを止める break; } } check.setValue("済"); } }

試したこと

ソースコードでは、転記処理のさいにfor文を3つ組み合わせています。

製造年月日を取得するfor文は、ご覧のとおり3つめにあります。これを試しに1つめ or 2つめに入れ替えたところ、製造年月日が全て取得されました。
しかし、代わりに3つめに入れ替えた別のfor文で取得した値が、同じく1種類だけになってしまいます。

これはつまり、「3つめに配置したfor文だけが正常に動いていない」ということかと考え、原因を調べていますが見つからずにおります。

for文の { } の位置をあれこれ変えてみたりもしましたが、解決しません。

アドバイスを頂けないでしょうか?
よろしくお願いいたします。

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

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

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

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

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

guest

回答1

0

ベストアンサー

インデントがおかしいのでわかりにくいですが、
おそらくfor(var u = 3; u <= 6; u++) {の最後で無条件でbreakしてるのでu=3の1回しかループしていません。
とりあえずbreakをコメントアウトしてみては?

投稿2022/07/22 03:58

RiaFeed

総合スコア2701

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

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

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

質問する

関連した質問