GAS エラーは出ていないが実行されない

Question

初心者です。
先日、沢山の方に手助けしていただいて、Line上で動作する通知を送信するメモ帳botを作成しました。
作成日には問題なく動作をしていたのですが、何らかの理由で
登録していたトリガーがエラーは発生していないのに通知を送信されない
独自でコードを変えようとすると、デプロイした後に通知の内容が記載されているのにも関わらず、予定がありませんと表示される
という問題点が見つかりました。

var ACCESS_TOKEN = "XXXXXXXXXX=";
var PUSH = "https://api.line.me/v2/bot/message/push";
var REPLY = "https://api.line.me/v2/bot/message/reply";

function getSchedTime(date, time) {
  return new Date(
    Utilities.formatDate(new Date(date), "JST", "yyyy-MM-dd") + 
    " " + Utilities.formatDate(new Date(time), "JST", "HH:mm") + "+09:00"
  );
}
 
function doPost(e) {
    var events = JSON.parse(e.postData.contents).events;
    events.forEach(function(event) {
      if(event.type == "message") {
        reply(event);
      } else if(event.type == "follow") {
        follow(event);
      } else if(event.type == "unfollow") {
        unFollow(event);
      }
    });    
}
 
function push() {
  var spreadsheet = SpreadsheetApp.openById("XXXXXXXXXX");
  var sheet = spreadsheet.getActiveSheet();

  var start = 2;
  var end = sheet.getLastRow();

  var today　=  new Date(); 　
  var formattedDate = Utilities.formatDate(today, "JST", "yyyy/MM/dd");

  var spsh = SpreadsheetApp.openById("XXXXXXXXXX");
  var sht = spsh.getActiveSheet();
  var data = sht.getDataRange().getValues();
  var userlist = [];
  for(var n = 0; n < data.length; n++){
    userlist.push(data[n][0]);
  }      

  // ユーザーリストから取得したユーザーID毎にループする。
  for (var u = 0; u < userlist.length; u++) {
    var value = [];
    for (var i = start; i <= end; i++) {
      // i行目のユーザーIDの取得
      var sell_D = "D" + i;
      var user = sheet.getRange(sell_D).getValue();
      // ユーザーIDがユーザーリストのu番目のユーザーと一致する場合
      if (user == userlist[u]) {
        var sell_C = "C" + i;
        var day_C = sheet.getRange(sell_C).getValue();
        // 日付列が空欄だとエラーになるのでcontinueする。
        if (day_C == '') continue;
        day_C = Utilities.formatDate(day_C, "JST", "yyyy/MM/dd");
        if (formattedDate !== day_C) continue;
        var sell_G = "G" + i;
        time_G = Utilities.formatDate(new Date(sheet.getRange(sell_G).getValue()), "JST", "HH:mm");
        var sell_B = "B" + i;
        value.push("・" + sheet.getRange(sell_B).getValue() + "(" + time_G + ")" + "\n\n");
      }
    }
    if (value.length > 0) {
      var postData = {
        // 個別のユーザーIDを指定
        to: userlist[u],
        messages: [
          {
            type: "text",
            text:
              "【本日の予定】\n\n" + value.join("") + "今日も一日がんばってね！",
          },
        ],
      };

      var headers = {
        "Content-Type": "application/json; charset=UTF-8",
        Authorization: "Bearer " + ACCESS_TOKEN,
      };

      var options = {
        method: "POST",
        headers: headers,
        payload: JSON.stringify(postData),
      };
      UrlFetchApp.fetch(PUSH, options);
    }
  }
}

function reply(data) {
  var postMsg = data.message.text;
  var replyToken = data.replyToken;
  var replyText = "";
  var userId = data.source.userId;

  if (postMsg == "登録") {
    // entry.zzzzzzはユーザーIDのフォームフィールドの番号。
    replyText =
      "予定の登録はこちら!\n" +
      "https://docs.google.com/forms/d/e/XXXXXXXXXX=" +userId;
  } else if (postMsg == "今後の予定") {
    var spreadsheet = SpreadsheetApp.openById("XXXXXXXXXX");
    var sheet = spreadsheet.getActiveSheet();

    var today =new Date(Utilities.formatDate(new Date(), "JST", "yyyy-MM-dd 00:00:00+09:00"))
     // スプレッドシートのデータを一括取得 (時刻データがある７列目まで取得するように修正）
    var allRecords = sheet.getRange(2, 1, sheet.getLastRow()-1, 7).getValues();
    // ユーザID が一致しかつ当日以降のものだけ抽出し、日付順に並び替え。
    var records = allRecords
      .filter(e => e[3] == userId && e[2] >= today)
      .sort((a, b) => getSchedTime(a[2], a[6]) - getSchedTime(b[2], b[6]));

    // 日付をフォーマットし、予定と一緒に配列化。
    var value = records.map(e => {
      var date = Utilities.formatDate(new Date(e[2]), "JST", "yyyy/MM/dd");
      var time = Utilities.formatDate(new Date(e[6]), "JST", "HH:mm");
      return `・${e[1]} (${date} ${time})\n\n`
    });
    if (value.length > 0) replyText = value.join("") + "忘れないようにしましょう！";
    else replyText = "予定がありません。";
  }
 
    if(replyText == ""){
        return;
    } else {
        var postData = {
          "replyToken" : replyToken,
          "messages" : [
              {
                  "type" : "text",
                  "text" : replyText
              }
          ]
        };
 
        var headers = {
            "Content-Type" : "application/json; charset=UTF-8",
            "Authorization" : "Bearer " + ACCESS_TOKEN
        };
 
        var options = {
            "method" : "POST",
            "headers" : headers,
            "payload" : JSON.stringify(postData)
        };
 
        return UrlFetchApp.fetch(REPLY, options);      
    }
}
 
function follow(e) {
  var spsh = SpreadsheetApp.openById("XXXXXXXXXX");
  var sht = spsh.getActiveSheet();
  sht.appendRow([e.source.userId]);
}
 
function unFollow(e){
  var spsh = SpreadsheetApp.openById("XXXXXXXXXX"); 
  var sht = spsh.getActiveSheet();
  var result = findRow(sht, e.source.userId, 1);
  if(result > 0){
    sht.deleteRows(result);
  }
}
 
function findRow(sht,val,col){
  var data = sht.getDataRange().getValues(); 
  for(var i = 0; i < data.length; i++){
    if(data[i][col-1] === val){
      return i+1;
    }
  }
  return 0;
}

ご指導のほどよろしくお願いします。

Comments

[Tatsunosuke]

何らかの理由を探さないことには、なんともできないので、
APIが呼び出された箇所から値を返すところまの間に色々とconsole.logを仕込んで、
どこでエラーが出ているか調べてみてください！

こちらで実際の内容に触れないため、現状だとなんともいません。。。

[SN____R]

APIが呼び出された箇所から値を返すところまでの間に
↑具体的にはどのあたりでしょうか...初歩的な質問で申し訳ありません。

[Tatsunosuke]

このプログラムですが、細かな、プログラムの動きはわからないとしても、どの関数から始まって、どの関数が呼び出されて、どの関数で終わっているのか、どのような流れで動いているか理解できていますか？

[Tatsunosuke]

関数に引数が正常に渡っているか、それぞれの関数が呼び出されたタイミングで確認してみてください。

[SN____R]

完全には理解できていませんが、なんとなくは理解できたような...という形でした。
結果、push()関数内のコードがスプレッドシートと合っていないのが原因だったみたいです。
お手数お掛けしました。

[Tatsunosuke]

解決したようで良かったです。
プログラムが思ったとおりに動かないが、原因もわからないというのは良く発生します。
なので、処理の所々にlogを吐くように設定しておき、どこでエラーが発生しているか追えるようにしておくと、今後の開発もスムーズに進むと思いますよ！

Answer 1

確認なのですが、質問文のpush()関数内のコードがスプレッドシートと合っていないように見えます。
（昨日修正したはずと思ったのですが、コードが修正前の状態になっていませんか？保存できていなかった可能性？）

【確認していただきたいこと】
回答を集計するスプレッドシートの並びが、掲載の画像の通りに
・タイムスタンプ　A列
・通知の内容：B列
・通知の日付：C列
・通知の時刻：D列
・ユーザID：E列
となっているならば、コードは下記のように修正するべきと思います。
(違っている場合は、現状のスプレッドシートの画像をアップロードしなおしてください）

それでもうまく行かない場合は、
・コードに記載されている各スプレッドシートのidが正しいか。
・フォームに紐づいているスプレッドシートは意図したシートになっているか
・スクリプトエディタ上で修正したら都度「新しいデプロイ」を行って、デプロイの管理から、最新バージョンに紐付けているか
・LINE Develop内に設定されているwebhookのURL等は正しいか

を確認してください。

var ACCESS_TOKEN = "XXXXXXXXXX=";
var PUSH = "https://api.line.me/v2/bot/message/push";
var REPLY = "https://api.line.me/v2/bot/message/reply";

function getSchedTime(date, time) {
  return new Date(
    Utilities.formatDate(new Date(date), "JST", "yyyy-MM-dd") + 
    " " + Utilities.formatDate(new Date(time), "JST", "HH:mm") + "+09:00"
  );
}

function doPost(e) {
    var events = JSON.parse(e.postData.contents).events;
    events.forEach(function(event) {
      if(event.type == "message") {
        reply(event);
      } else if(event.type == "follow") {
        follow(event);
      } else if(event.type == "unfollow") {
        unFollow(event);
      }
    });    
}

function push() {
  var spreadsheet = SpreadsheetApp.openById("XXXXXXXXXX");
  var sheet = spreadsheet.getActiveSheet();

  var start = 2;
  var end = sheet.getLastRow();

  var today　=  new Date(); 　
  var formattedDate = Utilities.formatDate(today, "JST", "yyyy/MM/dd");

  var spsh = SpreadsheetApp.openById("XXXXXXXXXX");
  var sht = spsh.getActiveSheet();
  var data = sht.getDataRange().getValues();
  var userlist = [];
  for(var n = 0; n < data.length; n++){
    userlist.push(data[n][0]);
  }      

  // ユーザーリストから取得したユーザーID毎にループする。
  for (var u = 0; u < userlist.length; u++) {
    var value = [];
    for (var i = start; i <= end; i++) {
      // i行目のユーザーIDの取得
-      var sell_D = "D" + i;
-      var user = sheet.getRange(sell_D).getValue();
+      var sell_E = "E" + i;
+      var user = sheet.getRange(sell_E).getValue();
      // ユーザーIDがユーザーリストのu番目のユーザーと一致する場合
      if (user == userlist[u]) {
        var sell_C = "C" + i;
        var day_C = sheet.getRange(sell_C).getValue();
        // 日付列が空欄だとエラーになるのでcontinueする。
        if (day_C == '') continue;
        day_C = Utilities.formatDate(day_C, "JST", "yyyy/MM/dd");
        if (formattedDate !== day_C) continue;
-        var sell_G = "G" + i;
-        time_G = Utilities.formatDate(new Date(sheet.getRange(sell_G).getValue()), "JST", "HH:mm");
+        var sell_D = "D" + i;
+        time_D = Utilities.formatDate(new Date(sheet.getRange(sell_D).getValue()), "JST", "HH:mm");

        var sell_B = "B" + i;
-        value.push("・" + sheet.getRange(sell_B).getValue() + "(" + time_G + ")" + "\n\n");
+        value.push("・" + sheet.getRange(sell_B).getValue() + "(" + time_D + ")" + "\n\n");

      }
    }
    if (value.length > 0) {
      var postData = {
        // 個別のユーザーIDを指定
        to: userlist[u],
        messages: [
          {
            type: "text",
            text:
              "【本日の予定】\n\n" + value.join("") + "今日も一日がんばってね！",
          },
        ],
      };

      var headers = {
        "Content-Type": "application/json; charset=UTF-8",
        Authorization: "Bearer " + ACCESS_TOKEN,
      };

      var options = {
        method: "POST",
        headers: headers,
        payload: JSON.stringify(postData),
      };
      UrlFetchApp.fetch(PUSH, options);
    }
  }
}

function reply(data) {
  var postMsg = data.message.text;
  var replyToken = data.replyToken;
  var replyText = "";
  var userId = data.source.userId;

  if (postMsg == "登録") {
    // entry.zzzzzzはユーザーIDのフォームフィールドの番号。
    replyText =
      "予定の登録はこちら!\n" +
      "https://docs.google.com/forms/d/e/XXXXXXXXXX=" +userId;
  } else if (postMsg == "今後の予定") {
    var spreadsheet = SpreadsheetApp.openById("XXXXXXXXXX");
    var sheet = spreadsheet.getActiveSheet();

    var today =new Date(Utilities.formatDate(new Date(), "JST", "yyyy-MM-dd 00:00:00+09:00"))
     // スプレッドシートのデータを一括取得
    var allRecords = sheet.getRange(2, 1, sheet.getLastRow()-1, 7).getValues();
    // ユーザID が一致しかつ当日以降のものだけ抽出し、日付順に並び替え。
    var records = allRecords
-      .filter(e => e[3] == userId && e[2] >= today)
-      .sort((a, b) => getSchedTime(a[2], a[6]) - getSchedTime(b[2], b[6]));
+      .filter(e => e[4] == userId && e[2] >= today)
+      .sort((a, b) => getSchedTime(a[2], a[3]) - getSchedTime(b[2], b[3]));


    // 日付をフォーマットし、予定と一緒に配列化。
    var value = records.map(e => {
      var date = Utilities.formatDate(new Date(e[2]), "JST", "yyyy/MM/dd");
-      var time = Utilities.formatDate(new Date(e[6]), "JST", "HH:mm");
+      var time = Utilities.formatDate(new Date(e[3]), "JST", "HH:mm");

      return `・${e[1]} (${date} ${time})\n\n`
    });
    if (value.length > 0) replyText = value.join("") + "忘れないようにしましょう！";
    else replyText = "予定がありません。";
  }

    if(replyText == ""){
        return;
    } else {
        var postData = {
          "replyToken" : replyToken,
          "messages" : [
              {
                  "type" : "text",
                  "text" : replyText
              }
          ]
        };

        var headers = {
            "Content-Type" : "application/json; charset=UTF-8",
            "Authorization" : "Bearer " + ACCESS_TOKEN
        };

        var options = {
            "method" : "POST",
            "headers" : headers,
            "payload" : JSON.stringify(postData)
        };

        return UrlFetchApp.fetch(REPLY, options);      
    }
}

function follow(e) {
略
}

function unFollow(e){
略
}

function findRow(sht,val,col){
略
  }
  return 0;
}

Comments

[SN____R]

保存できてなかった可能性が大きいかもしれないです...
新たに掲載して頂いたコードを元に、書き換えたところ正常に動作するようになりました！
昨日お世話になったばかりなのにすみませんでした...

[退会済みユーザー]

解決してよかったです。