google Apps Scriptコードでfor文でループさせたい

Question

gmailに届いたHTMLメールをスプレッドシートへデータで落としています。
parseXmlで下記のコードを作成しました。
E列の２行目のHTMLデータを文字に変更してF２、G２、H２、I２,J２,K２,L２,M２,N２,O２列に落としています
この作業をfor文でE列の３行目、４行目、５行目にデータが入ったらF３〜O３　F４〜O4, F5~O5 と言うように
自動で F〜Oに文字が出るようにしたいと
思い下記コードの６行目にfor文を作成しましたがうまくいきません
多分for文の中にセルの指定をしなくてはいけないのですがその方法がよくわかりません
わかる方いましたら教えていただきたいです

コード
function parseXml() {
  const ss = SpreadsheetApp.getActiveSpreadsheet();
  const sheet = ss.getSheetByName('シート1');
 
  const range = sheet.getRange('E2');
  const lastRow = sheet.getLastRow();
  for(let i = 2; i <= lastRow; i++) {
    console.log(sheet.getRange(i, 1).getValue());
  }
}
    
    
  //Logger.log(range.getValue());
  
  
  var data = Parser.data(range.getValue()).from('<table').to('</table>').build();
  
  //Logger.log(data);
  
  
  var tds = Parser.data(data).from('<td style="border-collapse:collapse;border-bottom:1px solid #efefff;word-break:break-all;vertical-align:top;padding:.5em">').to('</td>').iterate();
  
  var length = tds.length;
  
  //番号
  Logger.log(tds[0]);
  
  //取得できた番号をシートに書き込み
  sheet.getRange('F2').setValue(tds[0]);
  
  //氏名
  Logger.log(tds[1]);
  
  //取得できた氏名をシートに書き込み
  sheet.getRange('G2').setValue(tds[1]);
   //ふりがな
  Logger.log(tds[2]);
  
  //取得できたふりがなをシートに書き込み
  sheet.getRange('H2').setValue(tds[2]);
  
   //メールアドレス
  Logger.log(tds[3]);
  
  //取得できたメールアドレスをシートに書き込み
  sheet.getRange('I2').setValue(tds[3]);
  
   //年齢
  Logger.log(tds[4]);
  
  //取得できた年齢をシートに書き込み
  sheet.getRange('J2').setValue(tds[4]);
  
   //学年
  Logger.log(tds[5]);
  
  //取得できた学年をシートに書き込み
  sheet.getRange('K2').setValue(tds[5]);
  
   //種別
  Logger.log(tds[6]);
  
  //取得できた種別をシートに書き込み
  sheet.getRange('L2').setValue(tds[6]);
  
   //曜日
  Logger.log(tds[7]);
  
  //取得できた曜日をシートに書き込み
  sheet.getRange('M2').setValue(tds[7]);
  
  
   //時間
  Logger.log(tds[8]);
  
  //取得できた時間をシートに書き込み
  sheet.getRange('N2').setValue(tds[8]);
  
   //理由
  Logger.log(tds[9]);
  
  //取得できた理由をシートに書き込み
  sheet.getRange('O2').setValue(tds[9]);
  
  
  
   
}

Comments

[mkk]

・parseXml()の7行目のconsole.logでは何が出力されますか？
・そもそも　 { 　と　 } 　の数が合っておらず、parseXmlを呼び出しても
　var data = Parser.data(range.getValue()).from('<table').to('</table>').build();
　以降のコードは実行されないかと思います。

[nobinomio]

7行目のconsole.logでFからOのデータを出したいのですが、　mkkさんのおっしゃる { 　と　 } の書き方がわからず困っていました。

[mkk]

FからOのデータを出したいのは分かったのですが、今時点のプログラムを実行すると7行目のconsole.logで何が出力されますか？という質問です。

{ 　と　 } 　の数が合っていないというのは、プログラムの仕様ですが例えば

function parseXml() {
のように関数が始まった時、必ず { で始まり

}
のように } で終わるようになっています。
つまり、{ で始まって } で終わるようになっているので、プログラムの中で
必ず { と } は同じ数になるはずなのです。
添付してもらったプログラムを見てみると
開きカッコ　{　の数は
1行目：function parseXml() {
7行目：for(let i = 2; i <= lastRow; i++) {
の２つです。

対して閉じカッコ } の数は
9行目： }
10行目：}
最終行：}
の3つになっているということです。

[nobinomio]

}の数間違えてました。　申し訳ありません。　7行目のログをみると、スプレッドシートのA列に書いてある日時の文字が、ずらっと書いてあるところまででてきます。

[mkk]

謝る必要はないですよ。
ただ、間違えていると分かった状態で、どう直すべきかはわかるでしょうか？
プログラムの流れというか、基礎というか、そのあたりが分からないと
恐らく今後も詰まってしまうかと思うので、もう少しプログラムが読めるようになるといいかなと思います。

[papinianus]

> gmailに届いたHTMLメールをスプレッドシートへデータで落としています。
↑これはシートあたりたかだか一度しかおきないのでしょうか。もし追記がある場合どのようなサイクルで処理されるか、から見直す必要があると思います。
懸念しているのは、複数回実行する場合、最後の展開されていない行から読み出さないと行数が増えるにつれ実行時間がのびてしまうことです。
gmail から挿入するときに展開する処理をいれれば無意味なループで悩む必要はなくなり、かつ、実行時間も抑制されます。

Answer 1

まず回答の前に気になった点が何点かあります。
今後プログラムをしていくにあたって重要な部分にもなりますので理解しておいていただればと思います。

①質問への追記でも記載しましたが、カッコの数が合っていない件
カッコの数が合っていないことは分かったかと思いますが、それによって今どうなっているか分かるでしょうか？
添付のプログラムで結構な行数を書いていますが、parseXml関数を呼んだ時

function parseXml() {
  const ss = SpreadsheetApp.getActiveSpreadsheet();
  const sheet = ss.getSheetByName('シート1');

  const range = sheet.getRange('E2');
  const lastRow = sheet.getLastRow();
  for(let i = 2; i <= lastRow; i++) {
    console.log(sheet.getRange(i, 1).getValue());
  }
}

プログラムは上記のところまでで実行は終わり、それより下は実行されません。
function parseXml() { で始まった開きカッコが上記の最終行で閉じられているのですから。
そこより下のプログラムでHTMLデータを分解しているので、
どうやっても分解後のデータをF〜Oに入れるなんてことはできない状態なのです。

②for文を入れる位置
上記でも書きましたが、for文より下の行でHTMLデータを分解しているのです。
つまりfor文が実行されている位置ではHTMLデータは分解されておらず、出力しようにも
準備ができていない状態です。
for文をもし入れるのであれば最後の方でしょう。
そしてconsole.logで出力する際のgetRange(i, 1)の指定の仕方も違うかと思います。
getRangeの指定の仕方はgetRange(何行目か, 何列目か) です。
今の状態は1列目に固定した状態で、for文で2行目から最終行までループしてしまっています。

以上、色々書いてしまいましたが今後の参考になれば幸いです。


では本題の回答ですが、こちらで確認ができないので下記をお試しください。
なおログ出力関係のプログラムは省きますので、必要であれば追加してください。

function parseXml() {
  const ss = SpreadsheetApp.getActiveSpreadsheet();
  const sheet = ss.getSheetByName('シート1');
  const lastRow = sheet.getLastRow();

  // 2行目から最終行までループ
  for(let i=2; i<=lastRow; i++) {
    // HTMLデータの読み込み先はi行目の5列目（ E列 ）
    const range = sheet.getRange(i, 5);
    
    // HTMLデータを分解
    var data = Parser.data(range.getValue()).from('<table').to('</table>').build();
    var tds = Parser.data(data).from('<td style="border-collapse:collapse;border-bottom:1px solid #efefff;word-break:break-all;vertical-align:top;padding:.5em">').to('</td>').iterate();
    
    // tdsの数（ F～Oまでだから10個 ）だけループ
    for(let s=0; s<tds.length; s++) {
      // シートに書き込み
      sheet.getRange(i, s+6).setValue(tds[s]);
    }
  }
}

Comments

[nobinomio]

プログラミング初心者の為まだまだ分からないだらけで、とても助かりました。　動くようになりました。