【Google Apps Script/GAS】ファイル全体文字背景の色変更をしたい

前提・実現したいこと

進捗状況が変わった際に1行文字背景の変更をGoogle Apps Scriptで実行できるようにしたいです。

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

１，
https://fastclassinfo.com/entry/gas_spreadsheet_color_change/
上記サイトを参考に実行したらD列よりも左側のみ灰色になりました。
サイトだと1行全体灰色になったのですがどうにも上手くいきません。

２，
実際は1ファイルに対し複数の営業所分のシートがあるためこ、
ファイル全体にこのGASを実行したいのですがシート名を指定しないと実行エラーになってしまいます。

３，
以前こちらで質問して解決したコードと今回のコードを増やしたら、
前回のコードは実行されましたが、今回のコードは実行されませんでした。
１つのスプレッドシートに対して複数のGASは実行されないのでしょうか。

※1番目の画像は新規作成したスプレッドシートに対しGASを実行したらできました。

該当のソースコード

//プログラム0｜プロシージャ名
function changecellcolor() {    

  //プログラム1｜スプレッドシートを取得
  var spreadsheet = SpreadsheetApp.getActiveSpreadsheet();
  var sheet = spreadsheet.getSheetByName('シート1');
  
  //プログラム2｜シート1のデータを二次元配列として取得
  var myRange = sheet.getDataRange().getValues();
    
  //プログラム3｜シート1を1行ずつ調べるD列（状況）に応じて背景色を変更
  for(var i = 1; i < myRange.length; i++) {//2行目から開始するためにi=1
    var status = myRange[i][3];   //D列が状況なので、[3]とする
    var range = sheet.getRange(i+1, 1, 1, myRange[0].length);//1行を取得する
    
    //プログラム4｜シート1のD列（状況）が完了であれば背景色、そうではないなら色なし
    if(status=='6(納入完了)'){
      range.setBackground("#C0C0C0");//灰色
    }else{
      range.setBackground(null);//色なし
    }
  }
}

試したこと

~~//プログラム1｜スプレッドシートを取得
var spreadsheet = SpreadsheetApp.getActiveSpreadsheet();
var sheet = spreadsheet.getSheetByName('シート1');

//プログラム2｜シート1のデータを二次元配列として取得
var myRange = sheet.getDataRange().getValues();~~

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

itagagaki

2021/11/11 14:36

トリガー実行ではないのですね。どうやって実行するのですか？

行動規範の内容に同意します

回答1件

ベストアンサー

１，1行全体を灰色にする。

-> getMaxColumns()を用いるとシートの全列数を取得できます。
これを利用して右端まで塗りつぶします。

diff
1（略）
2  //プログラム3｜シート1を1行ずつ調べるD列（状況）に応じて背景色を変更
3  for(var i = 1; i < myRange.length; i++) {//2行目から開始するためにi=1
4    var status = myRange[i][3];   //D列が状況なので、[3]とする
5-   var range = sheet.getRange(i+1, 1, 1, myRange[0].length);//1行を取得する
6+   var range = sheet.getRange(i+1, 1, 1, sheet.getMaxColumns());  // 1行全体を取得する
7（略）

　

２．実際は1ファイルに対し複数の営業所分のシートがあるため、
ファイル全体でGASを実行したい

-> getSheetByName のかわりに getActiveSheetを使えば、
シート名に関係なく、変更したアクティブなシートに対して処理を実行することができると思います。。

diff
1//プログラム0｜プロシージャ名
2function changecellcolor() {    
3
4  //プログラム1｜スプレッドシートを取得
5  var spreadsheet = SpreadsheetApp.getActiveSpreadsheet();
6- var sheet = spreadsheet.getSheetByName('シート1');
7+ var sheet = spreadsheet.getActiveSheet();
8（以下略）

３．

以前こちらで質問して解決したコードと今回のコードを増やしたら、前回のコードは実行されましたが、今回のコードは実行されませんでした。１つのスプレッドシートに対して複数のGASは実行されないのでしょうか。

こちらで、質問文記載の画像と同じように、質問記載の関数と前回の関数両方に、「変更時」トリガーを設定したところ、セル編集時にどちらも同時並行で正常に実行されました。

画像を見る限りでは、背景色.gsのタイトルの左横に未保存のアイコンが見えています。上書き保存した場合どうでしょうか。

また、そもそも条件に合致しておらず実行されていないように見えるだけかもしれません。
・function changecellcolor() {
・ for(var i = 1; i < myRange.length; i++) {//2行目から開始するためにi=1
・ if(status=='6(納入完了)'){
・ }else{

それぞれの直下に
Logger.log(~)、を入れて、実行してみた場合、各ログが表示されるか確認してみてはいかがでしょうか。

（プリントデバッグは基本中の基本です）

たとえば条件に合致するセルを編集したつもりでも、"関数スタート"のログしか表示されていない場合は、どこかにバグがあるということになり、プログラムを見直す必要があると思います。

diff
1function changecellcolor() {    
2+ Logger.log("関数スタート");
3  //プログラム1｜スプレッドシートを取得
4  var spreadsheet = SpreadsheetApp.getActiveSpreadsheet();
5  var sheet = spreadsheet.getSheetByName('シート1');
6
7  //プログラム2｜シート1のデータを二次元配列として取得
8  var myRange = sheet.getDataRange().getValues();
9
10  //プログラム3｜シート1を1行ずつ調べるD列（状況）に応じて背景色を変更
11  for(var i = 1; i < myRange.length; i++) {//2行目から開始するためにi=1
12+   Logger.log("forループ開始");
13    var status = myRange[i][3];   //D列が状況なので、[3]とする
14    var range = sheet.getRange(i+1, 1, 1, myRange[0].length);//1行を取得する
15
16    //プログラム4｜シート1のD列（状況）が完了であれば背景色、そうではないなら色なし
17    if(status=='6(納入完了)'){
18+     Logger.log("status==6");
19      range.setBackground("#C0C0C0");//灰色
20    }else{
21+     Logger.log("status!=6");
22      range.setBackground(null);//色なし
23    }
24  }
25}