GASで、スプレッドシートに入力した値と、現在の日（年・日時ではなく、「日」のみの値が合致しているかどうか） が同じであれば、チャットワークにその値を表示する というプログラムを作成しています。

Question

前提・実現したいこと

GASで、スプレッドシートに入力した値と、現在の日（年・日時ではなく、「日」のみの値が合致しているかどうか）
が同じであれば、チャットワークにその値を表示する
というプログラムを作成しています。

■■な機能を実装中に以下のエラーメッセージが発生しました。

#プログラム自体にエラーは出ていないのですが、スプレッドシートに「14」と入力しているけれど（本日の日と合致しているけれど）、チャットワークにelseの値（15）がに表示されてしまうため、正しく日を拾えていないかと思います。理由が分かれば教えていただきたいです。よろしくお願いします。

エラーメッセージ
エラーは出ていないです。

該当のソースコード

ffunction myFunction() {
  

//スプレッドシートの、特定の範囲を読み込む
var sheet = SpreadsheetApp.getActiveSheet();
var x = sheet.getRange(2,1).getValue();

//日付の定義（日を取得する）
var date = new Date
var day = date.getDate();
var yesterday = date.getDate(day-1);
var tomorrow = date.getDate(day+1);

//入力した日と条件が合致していれば、（CWに値を表示する）
if( x > yesterday && x < tomorrow)
{

x=x;

}

else{

x=x+1;

}

var client = ChatWorkClient.factory({token: '8314e1a8f4c4e0c5891d1d90f731e85b'});
client.sendMessage({room_id:100266334, body:x});



}

試したこと

ここに問題に対して試したことを記載してください。

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

ここにより詳細な情報を記載してください。

Answer 1

GASのdate.getDateメソッドは引数を取らない(指定しても無視する)ので、

var yesterday = date.getDate(day - 1);
var tomorrow = date.getDate(day + 1);

で得られる変数yesterdayとtomorrowの値は、変数dayと同じ値です。例えば、質問日(2021-08-14)に実行すれば、すべて14になります。

すると、条件式x > yesterday && x < tomorrowは、変数xの値が何であっても(ある値より大きくかつ小さい数は存在しないことから)成立しないので、else以下の処理が常に実行されます。

次に、質問者がひねり出したyesterday < x < tomorrowという条件式は、Pythonなら質問者の想定通りに働きますが、GASではyesterday < xの評価結果(falseかtrue)を数値(0か1)に変換後、それがtomorrowの値より小さいか比較することになります。
変数xの値が何であっても評価結果はたかだか0か1なので、質問日(2021-08-14)に実行すれば全体として常に成立することになります。

「変数xの値と変数dayの値を直接等しいか比較する」のを避ける理由がなければ、変数yesterdayもtomorrowも不要で、

if (day === x) {
  // 日が一致したときの処理
}

とすればいいでしょう。

Comments

[tanaka_444]

エラーが出ず、一応やりたいことができていたので解決したと思っていたのですが、間違ってプログラムを理解するところでした。ご回答ありがとうございます。

＝＝

GASのdate.getDateメソッドは引数を取らない(指定しても無視する)ので、

var yesterday = date.getDate(day - 1);
var tomorrow = date.getDate(day + 1);
で得られる変数yesterdayとtomorrowの値は、変数dayと同じ値です。例えば、質問日(2021-08-14)に実行すれば、すべて14になります。
＝＝

→こちらですが、今回は不要ですがもしyesterdayとtomorrowを出したい場合、
var date = new Date
var day = date.getDate();
var yesterday = day.getDate()-1;
var tomorrow = day.getDate()+1;
であれば、昨日と明日の値を出すことができますでしょうか？

＝＝

次に、質問者がひねり出したyesterday < x < tomorrowという条件式は、Pythonなら質問者の想定通りに働きますが、GASではyesterday < xの評価結果(falseかtrue)を数値(0か1)に変換後、それがtomorrowの値より小さいか比較することになります。
変数xの値が何であっても評価結果はたかだか0か1なので、質問日(2021-08-14)に実行すれば全体として常に成立することになります。
＝＝
→演算子の理解誤っておりました。危うく間違えて理解するところでした。
ありがとうございます。

[Daregada]

> var yesterday = day.getDate()-1;
> var tomorrow = day.getDate()+1;
> であれば、昨日と明日の値を出すことができますでしょうか？

まず、変数dayの値は整数なのでgetDateメソッドは用意されていません。

dateの間違いであるとして、
「昨日と明日の値」が、変数xとの比較に使うための数値ならば「はい」。ただし、すでに得られている変数dayの値を-1、+1したほうがgetDateを毎回呼び出すよりも早いでしょう。

日付として通用する値が欲しいのであれば「いいえ」。
本日が月初(1日)のときの昨日の日付、月末(31日など)のときの明日の日付が必要なら、setDateメソッドを使ってdateを更新する必要があります。

[tanaka_444]

＝＝
まず、変数dayの値は整数なのでgetDateメソッドは用意されていません。

dateの間違いであるとして、
「昨日と明日の値」が、変数xとの比較に使うための数値ならば「はい」。ただし、すでに得られている変数dayの値を-1、+1したほうがgetDateを毎回呼び出すよりも早いでしょう。
＝＝
→なるほど、よく理解できました。
変数xとの比較に使用したかったのですが、ご指摘の通り関数を呼び出すよりも、
変数dayの値を-1、+1にした方が早いかと思いますので参考にさせていただきます。

＝＝
日付として通用する値が欲しいのであれば「いいえ」。
本日が月初(1日)のときの昨日の日付、月末(31日など)のときの明日の日付が必要なら、setDateメソッドを使ってdateを更新する必要があります。
＝＝
→こちらの、日付として通用する値の出し方も、知っておきたかったので勉強になりました。
ご回答ありがとうございました！

[Daregada]

それでは、「解決した方法」を解除して、こちらを選択して下さい。

[tanaka_444]

こちらの解決方法を選択しましたm(_ _)m

Answer 2

こちら解決しました。
if文の中の条件式が誤っていたというミスでした。
yesterday<x<tomorrowにすると正しく実行されました。