現在書いているコードを短縮してシンプルにする方法を教えてください

Question

実現したいこと

現在書いているコードを短縮してシンプルにしたい

詳細

よろしくお願いいたします。

function myFunction() {


//HTMLを取得
  const url = "URL";
  const response = UrlFetchApp.fetch(url);
  const html = response.getContentText("UTF-8")

//テーブルデータを取得
  const tabledatas = Parser.data(html).from('<tbody>').to("</tbody>").build();

  //テーブルデータから特定の文字列に囲まれたデータをすべて抽出.iterate();（東G)
  let tabledata1 = Parser.data(tabledatas).from('<td>東G</td>').to('</td>').iterate(); 


  //テーブルデータから特定の文字列に囲まれたデータをすべて抽出.iterate();（東S）
  let tabledata2 = Parser.data(tabledatas).from('<td>東S</td>').to('</td>').iterate();


  //テーブルデータから特定の文字列に囲まれたデータをすべて抽出.iterate();（東P）
  let tabledata3 = Parser.data(tabledatas).from('<td>東P</td>').to('</td>').iterate();


  //テーブルデータから特定の文字列に囲まれたデータをすべて抽出.iterate();（東2）
  let tabledata4 = Parser.data(tabledatas).from('<td>東2</td>').to('</td>').iterate();


  //テーブルデータから特定の文字列に囲まれたデータをすべて抽出.iterate();（東1）
  let tabledata5 = Parser.data(tabledatas).from('<td>東1</td>').to('</td>').iterate();

  //テーブルデータから特定の文字列に囲まれたデータをすべて抽出.iterate();（JQ）
  let tabledata6 = Parser.data(tabledatas).from('<td>JQ</td>').to('</td>').iterate();

  
  //テーブルデータから特定の文字列に囲まれたデータをすべて抽出.iterate();（東マ）
  let tabledata7 = Parser.data(tabledatas).from('<td>東マ</td>').to('</td>').iterate();



//スプシのデータ取得
  const spreadFile = SpreadsheetApp.getActiveSpreadsheet();

//スプシのシート名を指定してデータ取得
  const spreadSheet = spreadFile.getSheetByName("シート1");

let tabledata11,tabledata22,tabledata33,tabledata44,tabledata55,tabledata66,tabledata77

//すべてのテーブルデータをスプシに記入
  
  //テーブルデータ（東G)をスプシに記入
    for (let i = 0; i < tabledata1.length; i++) {

    //抽出した無駄な部分を削除
    tabledata11 =tabledata1[i].replace("<td>","");  
        tabledata11 =tabledata1[i].replace(/[\s\t\n]/g,""); 
        tabledata11 = tabledata1[i].replace("<td>", ""); 
      spreadSheet.getRange(spreadSheet.getLastRow()+1,2).setValues([[tabledata11]])

    }

  
  //テーブルデータ（（東S)）をスプシに記入
    for(let i=0;i<tabledata2.length;i++){  

      //抽出した無駄な部分を削除
    tabledata22 =tabledata2[i].replace("<td>","");  
        tabledata22 =tabledata2[i].replace(/[\s\t\n]/g,""); 
        tabledata22 = tabledata2[i].replace("<td>", ""); 

      spreadSheet.getRange(spreadSheet.getLastRow()+1,2).setValues([[tabledata22]])
    }


  
  //テーブルデータ（（東P)）をスプシに記入
    for(let i=0;i<tabledata3.length;i++){  

                //抽出した無駄な部分を削除
    tabledata33 =tabledata3[i].replace("<td>","");  
        tabledata33 =tabledata3[i].replace(/[\s\t\n]/g,""); 
        tabledata33 = tabledata3[i].replace("<td>", ""); 

      spreadSheet.getRange(spreadSheet.getLastRow()+1,2).setValues([[tabledata33]])

    }

  

  //テーブルデータ（（東2)）をスプシに記入
    for(let i=0;i<tabledata4.length;i++){  

                //抽出した無駄な部分を削除
    tabledata44 =tabledata4[i].replace("<td>","");  
        tabledata44 =tabledata4[i].replace(/[\s\t\n]/g,""); 
        tabledata44 = tabledata4[i].replace("<td>", ""); 

      spreadSheet.getRange(spreadSheet.getLastRow()+1,2).setValues([[tabledata44]])
    }


  //テーブルデータ（（東1)）をスプシに記入
    for(let i=0;i<tabledata5.length;i++){  

                //抽出した無駄な部分を削除
    tabledata55 =tabledata5[i].replace("<td>","");  
        tabledata55 =tabledata5[i].replace(/[\s\t\n]/g,""); 
        tabledata55 = tabledata5[i].replace("<td>", ""); 

      spreadSheet.getRange(spreadSheet.getLastRow()+1,2).setValues([[tabledata55]])

    }

  //テーブルデータ（（JQ)）をスプシに記入
    for(let i=0;i<tabledata6.length;i++){  

                //抽出した無駄な部分を削除
    tabledata66 =tabledata6[i].replace("<td>","");  
        tabledata66 =tabledata6[i].replace(/[\s\t\n]/g,""); 
        tabledata66 = tabledata6[i].replace("<td>", ""); 

      spreadSheet.getRange(spreadSheet.getLastRow()+1,2).setValues([[tabledata6[i]]])
    }

  //テーブルデータ（（東マ)）をスプシに記入
    //for(i of tabledata7){
    //spreadSheet.appendRow([tabledata7[i]]);
        for(let i=0;i<tabledata7.length;i++){  

                    //抽出した無駄な部分を削除
    tabledata77 =tabledata7[i].replace("<td>","");  
        tabledata77 =tabledata7[i].replace(/[\s\t\n]/g,""); 
        tabledata77 = tabledata7[i].replace("<td>", ""); 

      spreadSheet.getRange(spreadSheet.getLastRow()+1,2).setValues([[tabledata77]])


}

}

Comments

[TN8001]

JavaとJavaScriptは全く別の言語です。

「Java」タグは外してください。

「Google Apps Script」タグがあるので付けてください。
[タグ一覧｜Google Apps Script](https://teratail.com/tags/Google%20Apps%20Script)

「Google スプレッドシート」タグもあります。
[タグ一覧｜Google スプレッドシート](https://teratail.com/tags/Google%20%E3%82%B9%E3%83%97%E3%83%AC%E3%83%83%E3%83%89%E3%82%B7%E3%83%BC%E3%83%88)

[aaiaka]

コメントありがとうございます。質問を修正したのでご確認よろしくお願いいたします。

[ikedas]

現在書いているコードを短縮してシンプルにするために、自分でやってみたことを記してください。また、その結果自分が想定した通りにならなかったのであれば、どうなることを想定していて実際はどうなったのかを記してください。
ちなみに、このコメント欄に書くのではありません。質問文を編集して書いてください。

[TN8001]

対応ありがとうございます^^

[atlanticSalmon]

5行目、12行目、20行目はfor(... in ...)ではないのですか？

[int32_t]

tabledata1 から tabledata7 を用意する方法から改善したほうがいいと思います。そのあたりのコードも開示してはどうでしょうか。

[aaiaka]

コメントありがとうございます。質問を修正したのでご確認よろしくお願いいたします。

[atlanticSalmon]

for (... of ...) {
for (... in ...) {
は
for ( ; ; ) {
だったの？

[atlanticSalmon]

というか、
for of 文をご存知で使わないのは何故ですか？
テーブルデータとシート記入と二重にfor文を使えるのはご存知ですよね。

[ikedas]

というか、質問者さん自身がコードをいろいろ書き換えて試していらっしゃるようなんで、
> 現在書いているコードを短縮してシンプルにするために、自分でやってみたことを記してください。また、その結果自分が想定した通りにならなかったのであれば、どうなることを想定していて実際はどうなったのかを記してください。
という私の問いかけにも反応をいただきたいです。

ちなみに (繰り返しになりますが)、このコメント欄に書くのではありません。質問文を編集して書いてください。

[atlanticSalmon]

えっと、よくわかりませんが、質問者さんは質問を楽しんでいて回答は望んでいない（自分で回答を見つけたい）ということですか？

[ikedas]

私個人の印象としては、質問の動機とか自分が試したこととかを示さず、ただ答えだけを求める質問のしかたはよろしくないなーと思いますね。

[atlanticSalmon]

コードが同じことの繰り返しで重複しているのを
まとめて書く方法を知りたいだけかと思っていました。
なので配列でのfor文の使い方を知っているようなので、重ねて組み合わせればいいだけかと。
動機や何を試したのかは、私は関心がありませんでした。

[ikedas]

まあそういうことなんでしょうが、それにしてもいろいろやりかたはあるわけで、例えば「東G」とかいった部署名? なんですかね、これをパラメータ化してまとめるとか、他のやりかたとかあると思います。そういったことを何か**試した上で質問**してきているのではないのなら、本当にやりたいの? ということです。
一方、現場の感覚で言うと、ちゃんと動いてるものなら強いて単純化しようとしてバグを入れたらまずいので、そのままがいいという判断もありうるわけですよね。だから、あえてそういうことを**しようと思った動機** (たとえば「上長に指示されたから」でもいいので) を知りたいというのはあります。

[atlanticSalmon]

動機しだいで（上からの指示なら）バクありでもいいと仰りたいように聞こえますよ。

[ikedas]

上長の指示の結果バグを作り込んでしまったら、それは上長の判断ミスです。会社勤めというのはそういうものです。ていうか何の話だ。

[atlanticSalmon]

変な話になって、ごめんなさい。

ただ、
> 上長の指示の結果バグを作り込んでしまったら、それは上長の判断ミスです。
というのを責任転嫁というのではないかなぁと...一瞬思ってしまいました。
ごめんなさい。

上はどんなときにでも責任は負うものですね。

[ikedas]

上長が「お前が勝手にバグを入れたんだ」と言ったら責任転嫁です。
組織の中で上司からの指示に従って仕事をしたのに問題が生じたら、責任を取るのは組織であって、個人ではありません。そうでなきゃ誰も、会社で上司の指示に従ったりしませんでしょ。

なんか変な話になっちゃいましたね、こちらこそ失礼。

[aaiaka]

for of 文をご存知で使わないのは何故ですか？
>昨日から初めてまだ初学者のため、for ofよりfor文が自分ではコードの理解がしやすいためです。

gasでしたいことは、とあるサイトの表から特定の情報をスクレイピングすることです。

よろしくお願いいたします。

[aaiaka]

//すべてのテーブルデータをスプシに記入

これ以降のコードを短縮できそうと思っているのですが、短縮できるアイデアがありません。
どなたかアイデアありましたら教えていただけますと幸いです。

[ikedas]

なぜ短縮したいのでしょうか。別に今のままでも動くのだから問題ないではありませんか。
また、for文の3種類の構文を知っているくらいですから、短縮のために何か試してみたことがあるのではないですか。それを説明してください。

[aaiaka]

なぜ短縮したいのでしょうか。
>短縮した方が気持ち良いしかっこよくないですか？スキルアップのためでもあります。
この気持ちに共感していただけないでしょうか。

[aaiaka]

短縮のために何か試してみたことがあるのではないですか。それを説明してください。
>特にありません。ただ単に短縮したコードを書いてみたいです。

[ikedas]

スキルアップのためなら、ご自分で勉強されるのが一番です。参考書を読むなりして、短縮できるテクニックがないか調べましょう。

[aaiaka]

短縮できるかどうか調べて、私の方でも試したのですができませんでした、、

[aaiaka]

答え探すより人に聞いた方が効率良くないですか？
もちろん、いろいろ寄り道して思考を辿ることが重要なときもありますが、プログラミングは答え見て覚えた方が上達速度早いと思っています。

私は間違っていますでしょうか。玄人の方の意見を聞かせていただけますか。

[ikedas]

ですから、試したことを書いてくださいと言っています。
teratailでは、自分で問題に取り組まずに単に正解を要求するような「**丸投げの質問**」は推奨されていません。
https://teratail.com/help/avoid-asking

回答を得たいのであれば、「私の方でも試したのですができませんでした」と言うのではなく、
> 現在書いているコードを短縮してシンプルにするために、自分でやってみたことを記してください。また、その結果自分が想定した通りにならなかったのであれば、どうなることを想定していて実際はどうなったのかを記してください。

---
流石にこれだけ言えば、普通は対応すると思うのですが。しないとしたら、最初から回答を求めていない質問の可能性が出てきますね。

[hentaiman]

掲載コードから見て取れる事はforを知っていてあえて使わないことを選んだという事、関数も知っていそうだがそれもあえて使わないことを選んだのだろうという事だけです。
ならばこれ以上改善の余地はないです。これがもしforが一切書かれていないコードだったらforがありますよって教える事はできます。

> プログラミングは答え見て覚えた方が上達速度早いと思っています。

この程度なら試行錯誤考えた方がスキルアップになりそうです。
見て覚えた方が良いレベルの内容は大体書籍になっていたりするものだし、その難易度が高いものならライブラリ化されていたりします。

[atlanticSalmon]

ここまでのやり取りを読む前は、
単純にコードの文字数を少なくしたいだけかと思っていて、
なぜ質問者さんが自分で書いているのにわかっていないかと疑問でした。

何がどうなっているのかを理解できなくても、
それを使ったコードを書いてみたかったのですね。

わからなくてもいいから動くコードを欲しいのであれば、
あえて初学者には理解できないように書いたコードでもいいのでしょうか。

[ikedas]

> あえて初学者には理解できないように書いたコード

そういうコードゴルフ的なことをするのは楽しいので、奇抜なコードを回答として投稿されるのもよいと思いますよ☺️

[atlanticSalmon]

> 答え探すより人に聞いた方が効率良くないですか？
> もちろん、いろいろ寄り道して思考を辿ることが重要なときもありますが、プログラミングは答え見て覚えた方が上達速度早いと思っています。

と仰ってますが、
ご自身で提示したコード自体からいろいろ（今回の質問が不要なほど）覚えられるのになぁと思ってしまいました。

[atlanticSalmon]

> for of 文をご存知で使わないのは何故ですか？
> >昨日から初めてまだ初学者のため、for ofよりfor文が自分ではコードの理解がしやすいためです。

コードがわかりにくくなる for ( ; ; ) 文を使うのは
イテレータが必要でなおかつ途中でbreakしたい時ですね。
配列ならsomeメソッドで置き換えできる可能性もありますが。

[ikedas]

質問者さんからのレスポンスがなくなりましたが、明日の午前中まで待ってみます。

[aaiaka]

あえて初学者には理解できないように書いたコードでもいいのでしょうか。
>もし短縮できるものがあるなら見てみたいです！色々勉強させてください。

[ikedas]

> 現在書いているコードを短縮してシンプルにするために、自分でやってみたことを記してください。また、その結果自分が想定した通りにならなかったのであれば、どうなることを想定していて実際はどうなったのかを記してください。
がんば!

[YellowGreen]

> >もし短縮できるものがあるなら見てみたいです！
> 色々勉強させてください。

まずは、
ご自身で書いたコードを勉強するなり、
これまでの試行錯誤を書いてみましょう。
という話の流れになっていますよ。

Answer 1

tabledataN を配列化すれば以下のように書けると思います。

  // テーブルデータから特定の文字列に囲まれたデータをすべて抽出
  const tabledata = ['東G', '東S', '東P', '東2', '東1', 'JQ', '東マ'].
      map(e => Parser.data(tabledatas).from(`<td>${e}</td>`).to('</td>').iterate());
  ...
  // テーブルデータをシートに記入
  for (let data of tabledata) {
    for (let d of data) {
      d = d.replace(....);
      spreadSheet.getRange(spreadSheet.getLastRow()+1,2).setValues([[d]]);
    }
  }

Answer 2

int32_t様のコードを1文にすると、これでも動作しますよ。

  ['東G', '東S', '東P', '東2', '東1', 'JQ', '東マ']
    .map(e => Parser.data(tabledatas).from(`<td>${e}</td>`).to('</td>').iterate())
    .forEach(data => data.forEach(d => spreadSheet.getRange(spreadSheet.getLastRow() + 1, 2)
    .setValue(d.replace("<td>","").replace(/[\s\t\n]/g,"").replace("<td>", ""))));