スプレッドシート内の表現を正しく直したい

Question

前提・実現したいこと

スプレッドシート内の表現を正しく直したいです。

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

途中までうまく行くのですが、なぜか途中で終了してしまいます。

該当のソースコード

  var events = calendar.getEvents(starttime,endtime);
  var values = [];

events.forEach(function(event){
var title = event.getTitle();
var start = event.getStartTime();
var description = event.getDescription();
var payment = description.match(/支払い方法:(.*)/);
var coupons = description.match(/クーポン等:(.*)/);
var coupons2 = description.match(/クーポン利用額:(.*)/);
var course = description.match(/コース:(.*)/);
var site = description.match(/予約媒体:(.*)/);
var language = description.match(/対応言語:(.*)/);
var table = description.match(/相席:(.*)/);
values.push([title,start,payment,coupons,course,site,language,table,coupons2]);
}//foreach終わり
)//foreach終わり

var ss = SpreadsheetApp.getActiveSpreadsheet();
var sheet = ss.getSheetByName("ご予約")
var lastRow = sheet.getLastRow()+1;
var lastcolumn = sheet.getLastColumn();
var range = sheet.getRange(1,2,values.length,values[0].length);
var clear = sheet.clear();
var datas = range.setValues(values);
sheet.getRange(1,1,values.length,1).insertCheckboxes();
//正規表現
for(var i = 1; i <= values[1].length; i++){
  var ** = '予約媒体: '
  var nocoupon = 'クーポン等: '
  var nocoupon2 = 'クーポン利用額: ' 
  var cash = '支払い方法: ';
  var ** = '予約媒体: **';
  var ** = '予約媒体: **';

if(sheet.getRange(i,7).getValue() == **){
  sheet.getRange(i,7).setValue('予約媒体: **');
continue;
}//if done
if(sheet.getRange(i,7).getValue() == **){
  sheet.getRange(i,7).setValue('予約媒体: **');
  continue;
}
if(sheet.getRange(i,7).getValue() == gurunabi){
  sheet.getRange(i,7).setValue('予約媒体: **');
  continue;
}
if(sheet.getRange(i,4).getValue() == '支払い方法:確認のこと'){
  sheet.getRange(i,4).setValue('支払い方法: クレジット');
  continue;
}


}//if done
}//for done

試したこと

IはVALUES[0]より小さい、としていたのを[1]に変えてもダメでした。
よろしくお願いします。

Answer 1

sheetの更新箇所ですが

for(var i = 1; i <= values[1].length; i++){

とvalues[1].lengthでやるのはおかしくないですか？

values[1].lengthは列数ですよね？なので９固定だと思います。
行の数を指定するのだからvalues.lengthが妥当なのでは？

ついでに気になった個所

if(sheet.getRange(i,7).getValue() == **){
  sheet.getRange(i,7).setValue('予約媒体: **');
continue;
}//if done

と、continueしてますが、それだとその列の処理が終われば
その次の行の処理になりますが
他の列の処理もするのであれば、continueは不要じゃないのでしょうか。

Comments

[Sou23]

continueと[1]を消したらうまくいきました！列数は9固定なんですね、知りませんでした。ありがとうございます！

[xail2222]

>列数は9固定なんですね、知りませんでした。

values.push([title,start,payment,coupons,course,site,language,table,coupons2]);
とvaluesにデータを追加しているのですから
values[0]もvalues[1]もlengthの結果は9になります。

[Sou23]

なるほど……！！！勉強になりました！ありがとうございます！