質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.48%
Google Apps Script

Google Apps ScriptはGoogleの製品と第三者のサービスでタスクを自動化するためのJavaScriptのクラウドのスクリプト言語です。

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

Q&A

解決済

3回答

3677閲覧

javascript(Google Apps Script)でのif条件分岐のor(||)が上手くいかないです

plack

総合スコア13

Google Apps Script

Google Apps ScriptはGoogleの製品と第三者のサービスでタスクを自動化するためのJavaScriptのクラウドのスクリプト言語です。

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

0グッド

0クリップ

投稿2017/12/06 09:58

###前提・実現したいこと
今月プログラミングを始めた者です。
Google Apps ScriptとGoogleスプレッドシートを使って、給与明細を作りたいと考えています。

給与明細を作る上で必要となる、社会保険料率表や源泉徴収税額表は定期的に更新となるため、更新月にアラートとして自動でメッセージを出せるようにしたいと思っています。

社会保険料率の更新月:毎年3月・9月
社会保険料率と雇用保険料率の更新月:毎年4月
源泉徴収税額の更新月:毎年1月
住民税の更新月:毎年6月・7月

これらをif文を使って条件分岐しようと考えています。
社会保険料率の「毎年3月・9月」のように年に数回更新されるものについては、if文のOR(||)を使っているのですが、どうやらここがネックになっているようです。(もしかすると他の部分が原因かもしれません)

最終的に、今月の月とif文による条件分岐がTRUEとなった時、6,13セルにメッセージを書き込むシステムです。

###発生している問題・エラーメッセージ

エラーメッセージは発生していません。 ただ、現在12月のはずなのですが、どうやら最初のif文(3月・9月)でTrueになっているようです。

###該当のソースコード

google

1function myFunction() { 2 3 4 var myActiveSpreadSheet = SpreadsheetApp.getActiveSpreadsheet(); 5 var myActiveSheet = myActiveSpreadSheet.getSheets()[0]; 6 7 //今月の月を取得 8 var date = new Date(); 9 month = date.getMonth() +1; 10 11 //3月、9月に社会保険料率が変更となる。 12 if (month == 3||9) { 13 myActiveSheet.getRange(6,13).setValue("・社会保険料率を最新版に更新してください。"); 14 } 15 16 //4月に社会保険料率と雇用保険料率が変更となる。(今月が12月なので便宜上ここに12を含めています) 17 else if (month == 4||12) { 18 myActiveSheet.getRange(6,13).setValue("・社会保険料率と雇用保険料率を最新版に更新してください。"); 19 } 20 21 //1月に源泉徴収税額表が変わる 22 else if (month == 1){ 23 myActiveSheet.getRange(6,13).setValue("・源泉徴収税額表を最新版に更新してください。"); 24 } 25 26 //6月と7月は住民税が変わる 27 else if (month == 6||7){ 28 myActiveSheet.getRange(6,13).setValue("・住民税を最新版に更新してください。"); 29 } 30 31 32}

###試したこと
if === 3||9

if == 2
などを試しました。
if == 2
だと上手くいったように思います。

###補足情報(言語/FW/ツール等のバージョンなど)
初めての投稿なので緊張しています。
質問方法などに問題がございましたら、ついでにご指摘いただけると助かります。

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答3

0

ベストアンサー

サボらずに、if(month === 3 || month === 9)と書いてください。

month === 3 || 9は、(month === 3) || 9という意味になり、常にtrueとして評価されます(month === 3ならtrue、そうでなければ9)。

投稿2017/12/06 10:02

maisumakun

総合スコア145184

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

plack

2017/12/06 10:07

とてもわかりやすい解凍をいただきありがとうございました。初歩的なミスでしたが、これでまた一つレベルアップすることができました。
defghi1977

2017/12/06 10:19

ヤバイ, 演算子の優先順位を間違えてた
guest

0

以下演算子の優先順を勘違いしていたため, 余計な箇所を削除しました.


if文の条件判定式の書き方に問題があります.

例えば「monthが3月もしくは9月である」ことを判定するのであれば

JavaScript

1if(month == 3||9){}

ではなく

JavaScript

1if(month == 3 || month == 9){}

とすべきです.

投稿2017/12/06 10:08

編集2017/12/06 10:18
defghi1977

総合スコア4756

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

||どちらかの条件式が真の時と認識してください

JavaScript

1 if (month == 3 || month == 9) {

投稿2017/12/06 10:02

退会済みユーザー

退会済みユーザー

総合スコア0

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

plack

2017/12/06 10:08

迅速なご回答をいただき感謝いたします。いつか私も回答者側になれるよう精進いたします。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問