GASのif・switch文で、2回関数を動かさないと正しい結果が出てこない

Question

google Apps Scriptでif文の中にswitchの入った関数を作りました。
関数を実行してみると、1度目には間違った結果を、２回目には期待する結果を出してきます。（2回目以降はなんどやっても期待通りの結果を返してくれます）
結果の例
【case0,0になるよう入力】→　①case3,3の結果　②case0,0の結果
【case3,1になるよう入力】→　①case0,0の結果　②case3,1の結果
【case4,4になるよう入力】→　①case3,1の結果　②case4,4の結果
【case1,1になるよう入力】→　①case4,4の結果　②case1,1の結果
前回の結果が残らないよう、結果を表示するセルは毎度消して試しています。

エラーなどは出てきません。
コードでおかしいところがあるのでしょうか。

コード
  //参照するスプレッドシートシート
  var ss = SpreadsheetApp.openById('シートID');
  //参照するシート
  var sheetA = ss.getSheetByName('sheet1');
  var sheetB = ss.getSheetByName('sheet2');
  var sheetC = ss.getSheetByName('sheet3');
  //参照するセル
  var value1=sheetA.getRange("A2").getValue();
  var value2=sheetA.getRange("B2").getValue(); 
  var value3=sheetA.getRange("C2").getValue();

function myFunction() {
 if ( value1 == 'A') {
    
   switch (value2) {
      case '0:0':
      case '1:0':
      case '1:1':
        sheetB.getRange("A2").setValue(sheetC.getRange("C2").getValue());
        break;
      case '2:0':
      case '2:1':
      case '2:2':
        sheetB.getRange("A2").setValue(sheetC.getRange("C3").getValue());
        break;
      case '3:0':
        sheetB.getRange("A2").setValue(sheetC.getRange("C4").getValue());
        break;
      case '3:1':
        sheetB.getRange("A2").setValue(sheetC.getRange("C5").getValue());
        break;
      case '3:2':
        sheetB.getRange("A2").setValue(sheetC.getRange("C6").getValue());
        break;
      case '3:3':
        sheetB.getRange("A2").setValue(sheetC.getRange("C7").getValue());
        break;
      case '3:4':
        sheetB.getRange("A2").setValue(sheetC.getRange("C8").getValue());
        break;
      case '4:3':
        sheetB.getRange("A2").setValue(sheetC.getRange("C9").getValue());
        break;
      case '4:4':
      case '5:4':
        sheetB.getRange("A2").setValue(sheetC.getRange("C10").getValue());
        break;
      default:
        sheetB.getRange("A2").setValue(sheetC.getRange("C11").getValue());
      }
 } else　{
      switch (value2) {
        case '0:0':
        case '1:0':
        case '1:1':
        case '2:0':
        case '2:2':
        case '3:0':
        sheetB.getRange("A2").setValue(sheetC.getRange("C12").getValue());
          break;
        case '3:1':
        case '3:2':
        case '3:3':
        case '4:3':
          sheetB.getRange("A2").setValue(sheetC.getRange("C13").getValue());
          break;
        default:
            sheetB.getRange("A2").setValue(sheetC.getRange("C14").getValue());
        }
 }
 SpreadsheetApp.flush();
}

Comments

[Lhankor_Mhy]

「間違った結果」と「期待する結果」をご例示ください。

[nassawa]

結果の例
【case0,0になるよう入力】→　①case3,3の結果　②case0,0の結果
【case3,1になるよう入力】→　①case0,0の結果　②case3,1の結果
【case4,4になるよう入力】→　①case3,1の結果　②case4,4の結果
【case1,1になるよう入力】→　①case4,4の結果　②case1,1の結果
前回の結果が残らないよう、結果を表示するセルは毎度消して試しています。

今４回ほど試した時の結果です。

[Lhankor_Mhy]

実際のスプレッドシートの実行前の状態と、「間違った結果」の実行後の状態と、「期待する結果」の実行後の状態を、できればテーブルでご例示いただけますか。

[nassawa]

すみません、そもそもテーブルがわからないです

[Lhankor_Mhy]

では、テーブルではなくてもいいです。

Answer 1

function myFunction() {
  switch (0,1){
    case 1,2,3,4,5,1: console.log('？');
  }
}

↑上記コードは、ログに？と表示されますが、これはnassawaさんが想定している通りですか？
違うのであれば、まずそこから見直されてはいかがでしょうか？

Comments

[dodox86]

＞Lhankor_Mhyさん

> これはdodox86さんが想定している通りですか？

回答者の私(dodox86)宛の問いかけでしたでしょうか。※私自身は、質問者さんのswitch文自体の使い方から懸念があると思っています。

[Lhankor_Mhy]

失礼しました。お名前を間違えていました。修正しています。

[dodox86]

承知しました。フォローとなる回答をいただきまして、どうもありがとうございます。

Answer 2

setValue()で値を更新した結果が、リアルタイムにシートに反映されていないからだと思われます。
SpreadsheetApp.flush();をmyFunction関数の最後に実行してみてください。

SpreadsheetApp#flush() - Google Apps Script

---

追記しました：

SpreadsheetApp#flush()ではダメだったということで、関数外でvar ss = SpreadsheetApp.openById('シートID');...のように宣言していることからの変数のライフサイクルの問題もあるかもしれませんが、そもそものコードで、switch文のタプル（？）指定の使い方にも問題があるようです。

以下のコードでは [20-02-14 11:51:30:539 JST] 0, 1とのようにログ出力されるので、文法エラー自体は無いもののプログラマーの意図を反映していない、不正なコードだと考えられます。（Google Apps Scriptで最近使えるようになったJavaScript V8エンジンでも一応確認しましたが、同じ結果です）

function test29() {
  var value1 = 1;
  var value2 = 1;
  switch (value1, value2) {
    case 0, 1:
      Logger.log("0, 1");
      break;
    case 1, 1:
      Logger.log("1, 1");
      break;
    default:
      Logger.log("default");
      break;
  }
}

コメントでも書きましたが、一見正しく動いているように見えるテストパターンで試されたのだと思います。再度確認してみてください。

Comments

[nassawa]

最後の}の手前に追加してみましたが、直らなかったです…

[dodox86]

myFunctionをどのように呼び出しているかご提示のコードでは分かりませんが、myFunctionを1回目で呼び出した後、2回目では再度、値をvar value1=sheetA.getRange("A2").getValue();　のように正しく読み出しているのでしょうか。

[nassawa]

myFunctionは今は都度、GASの実行を押して呼び出していました。
下記のように実行されています…
[20-02-14 10:38:24:256 JST] 実行を開始しています
[20-02-14 10:38:24:391 JST] SpreadsheetApp.openById([1qTz8p9g4sr0VOOPL0A9JDtLRKjTF_MXShrEQyxEAc8o]) [0.126 秒]
[20-02-14 10:38:24:391 JST] SpreadsheetApp.Spreadsheet.getSheetByName([サンプル]) [0 秒]
[20-02-14 10:38:24:392 JST] SpreadsheetApp.Spreadsheet.getSheetByName([吐き出し]) [0 秒]
[20-02-14 10:38:24:393 JST] SpreadsheetApp.Spreadsheet.getSheetByName([データベース]) [0 秒]
[20-02-14 10:38:24:394 JST] SpreadsheetApp.Sheet.getRange([A2]) [0 秒]
[20-02-14 10:38:24:630 JST] SpreadsheetApp.Range.getValue() [0.235 秒]
[20-02-14 10:38:24:632 JST] SpreadsheetApp.Sheet.getRange([B2]) [0 秒]
[20-02-14 10:38:24:633 JST] SpreadsheetApp.Range.getValue() [0 秒]
[20-02-14 10:38:24:634 JST] SpreadsheetApp.openById([1qTz8p9g4sr0VOOPL0A9JDtLRKjTF_MXShrEQyxEAc8o]) [0 秒]
[20-02-14 10:38:24:634 JST] SpreadsheetApp.Spreadsheet.getSheetByName([サンプル]) [0 秒]
[20-02-14 10:38:24:635 JST] SpreadsheetApp.Spreadsheet.getSheetByName([吐き出し]) [0 秒]
[20-02-14 10:38:24:635 JST] SpreadsheetApp.Spreadsheet.getSheetByName([データベース]) [0 秒]
[20-02-14 10:38:24:636 JST] SpreadsheetApp.Sheet.getRange([A2]) [0 秒]
[20-02-14 10:38:24:636 JST] SpreadsheetApp.Range.getValue() [0 秒]
[20-02-14 10:38:24:637 JST] SpreadsheetApp.Sheet.getRange([B2]) [0 秒]
[20-02-14 10:38:24:637 JST] SpreadsheetApp.Range.getValue() [0 秒]
[20-02-14 10:38:24:637 JST] SpreadsheetApp.Sheet.getRange([C2]) [0 秒]
[20-02-14 10:38:24:637 JST] SpreadsheetApp.Range.getValue() [0 秒]
[20-02-14 10:38:24:638 JST] SpreadsheetApp.Sheet.getRange([A2]) [0 秒]
[20-02-14 10:38:24:639 JST] SpreadsheetApp.Sheet.getRange([C5]) [0 秒]
[20-02-14 10:38:24:723 JST] SpreadsheetApp.Range.getValue() [0.084 秒]
[20-02-14 10:38:24:723 JST] SpreadsheetApp.Range.setValue([Dの例文です]) [0 秒]
[20-02-14 10:38:24:919 JST] SpreadsheetApp.flush() [0.195 秒]
[20-02-14 10:38:25:041 JST] 実行が無事終了しました（合計ランタイム 0.283 秒）

[nassawa]

見当違いの回答していたらすみません

[dodox86]

//参照するスプレッドシートシート
//参照するスプレッドシートシート
var ss = SpreadsheetApp.openById('シートID');
//参照するシート
var sheetA = ss.getSheetByName('sheet1');
...などのコード部分は、関数になっていないということですか？

[nassawa]

下記の部分はmyFunctionの関数の中に入れていなかったです。

//参照するスプレッドシートシート
var ss = SpreadsheetApp.openById('シートID');
//参照するシート
var sheetA = ss.getSheetByName('sheet1');
var sheetB = ss.getSheetByName('sheet2');
var sheetC = ss.getSheetByName('sheet3');
//参照するセル
var value1=sheetA.getRange("A2").getValue();
var value2=sheetA.getRange("B2").getValue();
var value3=sheetA.getRange("C2").getValue();

[dodox86]

そもそもですが、switch 文のタプル指定って、正しく動いていますか？　できないはずなのですが。
（こちらで確認したところでは動いてません GASに最近導入されたv8ランタイムでも同じ）
switch (value2,value3) {
case 0,0:

else 節でも、switch (value2) としている割にはcaseで0, 0: などとされています。
} else　{
switch (value2) {
case 0,0:
...
テストされているパターンで、一見、動いているように見えているだけ、と言うことはありませんか。

[dodox86]

尚、影響は無いようですが、elseの後に全角空白文字が入っています
} else<ココに入ってる>{
switch (value2) {
case 0,0:

[nassawa]

タプル指定やめて書き直したら、問題なく動くようになりました。2回目には正しく動いちゃってたので、できるものだと思ってしまいました…。