複数の文字列を並列したい

Question

こんにちは。質問をさせていただきます。
開発環境はGoogle App Scriptです。

実行したいこと

毎分実行のトリガーを設定して、下記のコードを動かしたいと考えています。
"03:00" と "07:00" と "11:00" と "15:00" と"19:00" 　と "23:00" にだけシート全体の数字が入っているセルにだけ１を足すというものです。

問題点

最初はそれぞれ分けて記述していたのですが、毎分実行のトリガーだと数が増えるにつれてGASの制限にひっかかりつつあったので、まとめようと思ったのですが、学んだばかりの論理演算子を用いてやってみてもできませんでした。

何が間違っているのか、また解決策を教えていただきたく思います。
有識者の方、是非ともよろしくお願い致します。

試したコード①

var ss = SpreadsheetApp.getActiveSpreadsheet();
var sh = ss.getActiveSheet();


function plus1() {

  if(Utilities.formatDate(new Date(), "Asia/Tokyo", "HH:mm") != ("03:00" || "07:00" || "11:00" || "15:00" || "19:00" || "23:00" )) return;

  const currentValues = sh.getSheetValues(1, 1, sh.getLastRow(), sh.getLastColumn());

  const plusOneToEachCell = function(row){
    return row.map(function(cellVal){
      if (typeof cellVal == "number") return cellVal + 1;
      return cellVal;
    })
  };

  const plusOneValues = currentValues.map(plusOneToEachCell);
  sh.getRange(1, 1, plusOneValues.length, plusOneValues[0].length).setValues(plusOneValues);
}

結果
"03:00"の際は実行されたものの、"07:00"以降は実行されませんでした。

試したコード②

var ss = SpreadsheetApp.getActiveSpreadsheet();
var sh = ss.getActiveSheet();


function plus1() {

  if(Utilities.formatDate(new Date(), "Asia/Tokyo", "HH:mm") != ((((("03:00" || "07:00" ) || "11:00" ) || "15:00" ) || "19:00" ) || "23:00" )) return;

  const currentValues = sh.getSheetValues(1, 1, sh.getLastRow(), sh.getLastColumn());

  const plusOneToEachCell = function(row){
    return row.map(function(cellVal){
      if (typeof cellVal == "number") return cellVal + 1;
      return cellVal;
    })
  };

  const plusOneValues = currentValues.map(plusOneToEachCell);
  sh.getRange(1, 1, plusOneValues.length, plusOneValues[0].length).setValues(plusOneValues);
}

結果
いずれも実行されませんでした。

Comments

[退会済みユーザー]

えっとなぜ if(Utilities.formatDate(new Date(), "Asia/Tokyo", "HH:mm") != ("03:00" || "07:00" || ... )) return; がまともな構文と思ったのはどこかに書いてあったのかな?（ｗ

[nikukyu]

どこかに書いていたのではなく、既成のものを自分で学習した知識でどうにかしようとた結果です....(T_T) progateでまだ学習中なので、初心者というかド素人と思っていただければありがたいです...

Answer 1

var targetTimes = ["03:00","07:00","11:00","15:00","19:00","23:00"];

if(targetTimes.indexOf(Utilities.formatDate(new Date(), "Asia/Tokyo", "HH:mm")) == -1) return;

とりあえずif文のところはこう直してみてください。

Comments

[nikukyu]

ありがとうございます！！m(_ _)m
このようにして組むのですね！！
現在、見て調べて記述内容の意味が素早く分かるようになってきてはいるんですが、自分で自由に組めない状態です...
まだまだ精進してがんばります！
繰り返しにはなりますが、ありがとうございました。

Answer 2

フォローとタグの関係から回答を誤答しました。

Comments

[namnium1125]

？これPHPですよね？GASはjsがベースのようですが…

https://en.wikipedia.org/wiki/Google_Apps_Script

[kei344]

in_arrayはPHPの関数で、JavaScriptには実装されていないと思いますよ。
「そして || があるので 1 or 0 で返しますね 」はどの部分でしょうか。("03:00" || "07:00" || "11:00" || "15:00" || "19:00" || "23:00" )は"03:00"が返ってくると思いますよ。

[退会済みユーザー]

すまん、フォローしているタグの関係から誤認した

[退会済みユーザー]

https://ideone.com/Ry92Ui

PHP のバージョンで「"03:00" || "07:00" 」が TRUE or FALSE に代わってるようだ

[kei344]

To: asahina1979さん
PHPでの話ですね、私がコメントに書いた「"03:00"が返ってくる」はJavaScriptの場合です。

[退会済みユーザー]

しょっぱならからPHPと思い込んで誤答してるのでそのままPHPで考えてる部分の指摘かとおもってしまたよ・・