google apps script GAS 繰り返し処理 特定の文字に反応して別のスプに抽出

Question

function myFunction() {
  var spreadsheet = SpreadsheetApp.openById('スプレッドキー');
  var sheet = spreadsheet.getActiveSheet();
  var values = sheet.getRange('a:bf').getValues();
  var spreadsheet2 = SpreadsheetApp.getActive();
  var price = findValue(values, '変更する', 10); 
  Logger.log(price);
  spreadsheet2.getRange('A2').setValue(price);

}

// 二次元配列から指定されたカラム番号の値を探す
function findValue(values, value, column) {
  var columnIndex = column - 1; // VLOOKUPのように引数では左から何列目と渡しているが、配列キーは0から始まるので1ズレる
  for (var i in values) {
    if (values[i][0] == value) {
      return values[i][columnIndex];
    }
  }
  return false;
}

他の方のものなどを参考にして作ったものが上のコードとなります。

・実現したいこと

■ SpreadsheetApp.openById('スプレッドキー')のA列の値が”変更する”の時、その行全て(a:bf)を取得して別のスプレッド（上のコードでは：spreadsheet2）に(a:bf)の値を表示させたいです。

初心者で質問の仕方もいたりませんがどうかご教授お願いいたします。

Comments

[sawa]

質問文のコードは実行すると何が（どこまで）出来るのか、もしエラーがでるならどういったエラーが出るのか、そういった記載があると良いかもです。GASがまったくわからないのであれば、関数で抽出する方法もありますのでそっちを検討しても良いかと。

Answer 1

追記
エラーが出ていたのは、forEachの閉じカッコが足りていないためでした、すみません！
あと、var values = sheet1.getRange('a:bf').getValues(); ここのシートの指定がsheetになっていたたので、sheet1に変更してます。

これでいけると思います。

function myFunction() {
    var spreadsheet = SpreadsheetApp.openById('スプレッドキー1');
    var sheet1 = spreadsheet.getSheetByName('シート1'); //[todo]シート名を指定

    var spreadsheet2 = SpreadsheetApp.openById('スプレッドキー2');
    var sheet2 = spreadsheet2.getSheetByName('シート2'); //[todo]シート名を指定

    
　　 var values = sheet1.getRange('a:bf').getValues();

    //a1のセルはvalues[0][0]に入ってます。
    if(values[0][0] == "変更する"){
    //[todo]変更するだった場合の処理
    //setValue １セルだけの処理だと重たくなるため、極力、getValues, setValuesの複数形がおすすめ
    //a:brの値だけ取り出す setValues は２次元配列じゃないと貼り付け出来なのでこの形に
    
    var setValues = []
    values.forEach(e=>{//取得したすべての値を確認する。
      if(e[0] =="変更する"){
　　　  　setValues.push(e) // "変更する"のときの行の配列
　　   }
    })
    //シートへの貼り付け
    //[todo]必要であればシートのクリアを
      sheet2.getRange(1,1,setValues.length,setValues[0].length).setValues(setValues)
    }
}

---

追記
コメント確認しました。
そうするとこんな感じになりますね。
eがどうなってるかわからない場合は、
console.log()やデバッグを使って確認してみてくださいね！

var setValues = []
values.forEach(e=>{　//取得したすべての値を確認する。
　　if(e[0] =="変更する"){
　　　　setValues.push(e) // "変更する"のときの行の配列
　　}
}

//シートへの貼り付け
//[todo]必要であればシートのクリアを

sheet2.getRange(1,1,setValues.length,setValues[0].length).setValues(setValues)

---

全部書いてしまいましたがこんな感じです！
[todo]と書いてあるところが質問者さんが変更すべき箇所です。

function myFunction() {
    var spreadsheet = SpreadsheetApp.openById('スプレッドキー');
　　 var sheet1 = spreadsheet.getSheetByName('シート１'); //[todo]シート名を指定

    var spreadsheet2 = SpreadsheetApp.openById('スプレッドキー');
　　 var sheet2 = spreadsheet2.getSheetByName('シート２'); //[todo]シート名を指定


    var values = sheet.getRange('a:bf').getValues();　
　　 //↑これでもいいですし、面倒であれば全データを↓取得でもOKです。A：BFまでのいずれかに確実に値が入っているなら。
    var values = sheet1.getDataRange().getValues();

    //a1のセルはvalues[0][0]に入ってます。
　　 if(values[0][0] == "変更する"){
      //[todo]変更するだった場合の処理


      //setValue １セルだけの処理だと重たくなるため、極力、getValues, setValuesの複数形がおすすめ
　　　 //a:brの値だけ取り出す setValues は２次元配列じゃないと貼り付け出来なのでこの形に
      var setValues = []
　　　 setValues.push(values[0]) //values[0] は１行目のセルの配列

　　　 //すでにsheet2に値が入っている場合は、sheet2.clear()などで消した方がいいです。
　　　 
　　　　//setValuesを使うときはこの書き方。[開始する行、開始する列,貼り付ける配列の長さ,貼り付ける２次元配列の長さ]
　　　　sheet2.getRange(1,1,setValues.length,setValues[0].length).setValues(setValues)
    }
}

Comments

[dc_jiu]

ご回答ありがとうございます。

例えばですが

a1は空白
a2は変更する
a3は空白
a4は空白
a5は変更する

となっている時に
2行目と5行目のa:bfを抽出するものを作りたいです。
説明足らずで申し訳ございません。
行は最終行まで行いたいです。

[dc_jiu]

丁寧なご回答本当にありがとうごさいます。

function myFunction() {
var spreadsheet = SpreadsheetApp.openById('スプレッドキー1');
　　 var sheet1 = spreadsheet.getSheetByName('シート1'); //[todo]シート名を指定

var spreadsheet2 = SpreadsheetApp.openById('スプレッドキー2');
　　 var sheet2 = spreadsheet2.getSheetByName('シート2'); //[todo]シート名を指定

var values = sheet.getRange('a:bf').getValues();
　　 //↑これでもいいですし、面倒であれば全データを↓取得でもOKです。A：BFまでのいずれかに確実に値が入っているなら。
//var values = sheet1.getDataRange().getValues();

//a1のセルはvalues[0][0]に入ってます。
　　 if(values[0][0] == "変更する"){
//[todo]変更するだった場合の処理

//setValue １セルだけの処理だと重たくなるため、極力、getValues, setValuesの複数形がおすすめ
　　　 //a:brの値だけ取り出す setValues は２次元配列じゃないと貼り付け出来なのでこの形に
var setValues = []
values.forEach(e=>{//取得したすべての値を確認する。
　　if(e[0] =="変更する"){
　　　　setValues.push(e) // "変更する"のときの行の配列
　　}
}

//シートへの貼り付け
//[todo]必要であればシートのクリアを

sheet2.getRange(1,1,setValues.length,setValues[0].length).setValues(setValues)
}
}

こういうふうになるという認識でよろしいでしょうか？
シンタックスエラーがでてるのですが}を消す。大文字の確認等をしても解決しないのご教授いただきたいです。。。
本当に初歩的な質問で申し訳ないのですがよろしくお願いします

[dc_jiu]

できました！
長々と対応いただき本当にありがとうございました！

[Tatsunosuke]

よかったです！頑張ってください！！