【GAS】forの中でforを使うやり方が理解できません

Question

●やりたい事流れ
同じ日にちごとに処理を流したい
例えば、１月3日の日にちとカテゴリーの表部分をコピペして別の場所で処理
次に１月17日で同様の処理を行う
上記のように日にちが続く限り同様の処理を流したいと思っています。

![

前回の質問で、列情報を取り出すやり方を教えてもらいました。
https://teratail.com/questions/376991

これを踏まえてコードを書いてみたのですが、同じ日にちごとに処理が回りません。
なぜでしょうか。
あと、日にちが続く限り処理を回す方法も分かりません。
なので、とりあえず”j＝10”として、10回分まわるようにしています。

function myFunction() {
  //1. 現在のスプレッドシートを取得
var spreadsheet = SpreadsheetApp.getActiveSpreadsheet();
//2. フォームの回答 シートを取得
var sheet = spreadsheet.getSheetByName('計画書');
const lastRow = sheet.getLastRow();
var count = 0;
var count2 = 0;
for(var j = 0 ; j <= lastRow; ++j) {
  currentRow = 2
for(var i = currentRow ; i <= lastRow; ++i) {///////////////////////currentRowを、次の日にちに変更させたい
  var range1 = sheet.getRange(i, 1);
  var value1 = range1.getValue();
  var range2 = sheet.getRange(i+1, 1);
  var value2 = range2.getValue();
  //ログに出力
  Logger.log(value1);
  Logger.log(value2);
  count++
  if ( value1  !=  value2)
break;
}


count2++
Logger.log("i:"+i);
Logger.log("count:"+count);
Logger.log("count2:"+count2);
 var date = sheet.getRange(i-count+1, 2, count, 5);
 var sheet2 = spreadsheet.getSheetByName('指示書（サンプル）');
 sheet2.copyTo(spreadsheet);
 var sheet3 = spreadsheet.getSheetByName('指示書（サンプル） のコピー');
 sheet3.setName("指示書"+count2);
 const range_to = sheet3.getRange("A5");
 //アクティブなセルに貼付け
 date.copyTo(range_to, {contentsOnly:true});

  var cell = sheet3.getRange("A2:F10");
  cell.setHorizontalAlignment("center");
  cell.setVerticalAlignment("middle");


}}

Comments

[itagagaki]

助言ですが、まずインデントを正しく整形するべきです。
あと「日にちが続く」「同じ日にち」といった言葉の意味が私にはよくわかりません。違う言葉で説明できますか？

Answer 1

手作業でやるならどういう手順でやるかと考えてみると、だいたい

1. 2行目の日にちの値を覚える
2. 覚えた値と異なる同じ日にちの行を下方向に探す。
3. 異なる日にちか空行があったら、2行目から今見ている行の上の行までをまとめて処理する
4. 今見ている行が空行なら終わり
5. 今見ている行の日にちを覚える
6. 覚えた値と異なる同じ日にちの行を下方向に探す。
7. 異なる日にちか空行があったら、ステップ5の行から今見ている行の上の行までをまとめて処理する
8. 今見ている行が空行なら終わり
9. 今見ている行の日にちを覚える
10. ...

こんな感じになるのではないでしょうか。これをコードにするなら、4ステップ毎が外側のループで、ステップ2・6が内側のループです。

もう少しコードに近い表現にすると、

1. 変数 currentRow を 2 で初期化
2. currentRow が lastRow 以下の間 while ループ
3. 変数 currentDate を currentRow の1番目のカラムの値で初期化
4. 変数 row を currentRow + 1 から lastRow で for ループ
5. row の1番目のカラムの値が currentDate と異なるならループ終了
6. (内側のループはここまで)
7. currentRow から row - 1 の間の行をまとめて処理
8. currentRow に row を代入
9. (外側のループはここまで)

Comments

[aRyo]

ご回答ありがとうございます。
2の「currentRow が lastRow 以下の間 while ループ」部分が、理解できません。
前回の質問の方法で別の日の列を抽出し、繰り返し処理できないのでしょうか。

頭の中で整理出来ていなかったので、再度作りなしました。
その中で、どうしても分からない点として”for(var i = 2 ; i <= lastRow; ++i) {}”の部分です。
せっかく繰り返し処理で次の日にちを抽出しても初期化式に代入？する方法が分かりません。

ここの部分で、おっしゃったwhileのコードが必要なのかなと考えているのですが、合っていますでしょうか。

お手数をおかけしますが、返信いただけると幸いです。

[int32_t]

現在のコードを尊重するなら、

for(var j = 0 ; j <= lastRow; ++j) {
currentRow = 2

を

let currentRow = 2;
while (currentRow <= lastRow) {
...
currentRow = i + 1;
}

こんな感じにすると良いんじゃないでしょうか。

[aRyo]

出来ました！！
本当にいつもありがとうございます！
恐らくまだまだ未熟なため、これからもお世話になるかと思います。
その時、またよろしくお願いいたします。