googleカレンダーの予定をchatworkに通知｜予定がない日は通知しないようにする

Question

前提・実現したいこと

googleカレンダーに登録したスケジュールを当日の指定した時間にChatworkに通知設定。
予定がない（0件）のときにもChatworkに通知がいく状態です。

これを予定がない（0件）のときにはChatworkへ通知がいかないようにしたいです。

該当のソースコード

/* 指定のカレンダーの本日の予定をチャットワークに送る */
function sendSchedule() {

  var myCals=CalendarApp.getCalendarById('[e-mail]'); //特定のIDのカレンダーを取得
  var myEvents=myCals.getEventsForDay(new Date());　//カレンダーの本日のイベントを取得

  /* チャットワークに送る文字列のヘッダー */
  var strBody = "[info][title]本日の予定：" + Utilities.formatDate(new Date(), 'JST', 'yyyy/MM/dd') + " (roger)[/title]"

  /* イベントの数だけ繰り返し */
  for(var i=0;i<myEvents.length;i++){
    var strTitle=myEvents[i].getTitle(); //イベントのタイトル
    var strStart=_HHmm(myEvents[i].getStartTime()); //イベントの開始時刻
    var strEnd=_HHmm(myEvents[i].getEndTime()); //イベントの終了時刻
    strBody=strBody + strStart + ' - ' + strEnd + strTitle + 'n'; //チャットワークに送る文字列にイベント内容を追加
  }

  strBody = strBody + '[/info]';

  /* チャットワークにメッセージを送る */
  var cwClient = ChatWorkClient.factory({token: ' XXXXXXXXXXXXXXXX'});　//チャットワークAPI
  cwClient.sendMessage({
    room_id:XXXXXXXX, //ルームID
    body: strBody
  });

}

/* 時刻の表記をHH:mmに変更 */
function _HHmm(str){

  return Utilities.formatDate(str, 'JST', 'HH:mm');

}

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

下記のサイトを参考にしております。
毎朝、当日の予定をGoogleカレンダーから取得してチャットワークに送る

Comments

[YellowGreen]

for(var i=0;i<myEvents.length;i++){
の5行下の
}
を
cwClient.sendMessage({
room_id:XXXXXXXX, //ルームID
body: strBody
});
の下の行に移動してみるとどうなりますか？

}を元の場所から削除して});の下の行に挿入です。

[YAmaGNZ]

イベントがないというのはmyEvents.length==0でしょうから
myEvents.length!=0と0じゃないときに送信すればいいのでは？
もしくはmyEventsを取得した時にmyEvents.length==0なら即returnとか

[YellowGreen]

> イベントがないというのはmyEvents.length==0でしょうから

for(var i=0;i<myEvents.length;i++){
このforループはそれを踏まえてこのようにしていると思いますよ。

[YAmaGNZ]

だから0の時にループしないでstrBodyに予定の詳細のテキストが入らないままsendMessageを実行してるってことだと理解したので
それをしないようにif文で回避すればどうかという提案だったのですが・・・

[_Na]

YAmaGNZ さんからご教示いただいた

for(var i=0;i<myEvents.length;i++){
の5行下の
}
を
cwClient.sendMessage({
room_id:XXXXXXXX, //ルームID
body: strBody
});
の下の行に移動

で出来ました！

迅速ご回答いただき、ありがとうございます！

[YellowGreen]

お役に立てたようで良かったです。

> だから0の時にループしないでstrBodyに予定の詳細のテキストが入らないままsendMessageを実行してるってことだと理解したので

なのでメール送信をループ内で行えばという提案でした。

[YAmaGNZ]

1っ回のメッセージにて通知するという意図があったのかと思いましたので、無いときには送らないという提案でした。

あとその方法ですとYellowGreenさんの提案かと

[YellowGreen]

> 1っ回のメッセージにて通知するという意図があったのかと思いましたので、
↑
私が質問者様に確認してませんでしたね。

_Na様
今の修正だと同じ日に複数の予定があるときは、予定ごとにメールが届きますので、それをまとめて1通のメールで送信したいときは}の位置を戻して、元のコードをYAmaGNG様のご提案のように修正された方がいいと思います。

具体的には、2つの修正を行います。

(1つ目の修正)
}
を元の場所に戻す。

(2つ目の修正)
var myEvents=myCals.getEventsForDay(new Date());　//カレンダーの本日のイベントを取得

の次の行に

if (!myEvents.length) {
return;
}

を挿入です。

[_Na]

> YAmaGNZ さん
おっしゃる通り、1回のメッセージにて通知する方法でした。
気づいてくださりありがとうございます！

> YellowGreen さん
ご教示いただきました
=====
var myEvents=myCals.getEventsForDay(new Date());　//カレンダーの本日のイベントを取得

の次の行に

if (!myEvents.length) {
return;
}
=====
で、希望どおり
「予定がない日には通知なし」
「1回のメッセージで複数の予定を通知」
にすることができました。

ご教示いただきました皆様ありがとうございます！

[YellowGreen]

私の確認ミスですし、
解決方法のご提案はYAmaGNZ様です。

[_Na]

お二方の迅速かつ丁寧なご対応に感謝しかありまん...
本当にありがとうございます！

Answer 1

YAmaGNZ さん
YellowGreen さん
ありがとうございました！

解決方法

var myEvents=myCals.getEventsForDay(new Date());　//カレンダーの本日のイベントを取得

の次の行に

if (!myEvents.length) {
return;
}

上記の方法で希望どおりの
「予定がない日には通知なし」
「1回のメッセージで複数の予定を通知」
にすることができました。