[GAS]if文の条件式で、andとorを併用する方法がわかりません。

Question

前提・実現したいこと

編集時にトリガーが作動するようスクリプトを組みたいのですが、
アクティブセルがスプレッドシートのタブ名(sheet1) 、かつA,C,E列のどれかの場合
処理を行いたいです。

いくつかパターンを変えて検証しましたがわかりませんでした。

該当のソースコード

 var ss = SpreadsheetApp.getActiveSpreadsheet();
 var spreadsheet = ss.getActiveSheet();
 var cell = spreadsheet.getActiveCell();

 if(ss.getName()=="sheet1" && cell.getColumn()==1 || cell.getColumn()==3 || cell.getColumn()==5){
  処理
  }
}

複雑な条件式になるとやはりifの中にifを入れるしかないのでしょうか。
よろしくお願いいたします。

Answer 1

掛け算のほうが足し算より計算の優先順位が高いように、&& と || にも優先順位があります。質問文のコードは、

• ss.getName()=="sheet1" かつ cell.getColumn()==1
• または cell.getColumn()==3
• または cell.getColumn()==5

と解釈されています。

ので、ss.getName()=="sheet1" && (cell.getColumn()==1 || cell.getColumn()==3 || cell.getColumn()==5) と括弧で囲んでください。ss.getName()=="sheet1" && [1, 3, 5].includes(cell.getColumn()) でもいいです。

演算子の優先順位 一覧表

Answer 2

if(ss.getName()=="A" && (cell.getColumn()==1 || cell.getColumn()==3 || cell.getColumn()==5)){
  処理
  }
}

ではだめでしょうか。orの条件を（）囲めば優先度が上がるかと。

Comments

[akiraaaa]

ご返答ありがとうございます。
こちらも試してみたのですがダメなようでした。。(両方の条件に当てはまっていても動作しない模様)

また、質問内容のタブ名と列がどちらも"A"となっており表現が紛らわしかったため、タブ名は"sheet1"に書き直しました。

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

Answer 3

いただいた回答を試しても動作しないと思ったら、
spreadsheet.getName()=="sheet1"
と記載すべきところを
ss.getName()=="sheet1"
としていたのが原因でした！！

初歩的なミスで申し訳ありません。
記載方法に自信がなかったため相談しましたが、
回答いただけたおかげでそれ以外の不審点に目を向ける事が出来ました。

ご回答いただいたお二方、大変ありがとうございました！

Comments

[otn]

なんかおかしいときに、まず疑うのはタイプミスの類いです。