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

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

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

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

JavaScript

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

Q&A

解決済

2回答

825閲覧

GAS 土日祝日を除きメール下書き作成をしたい

bin.

総合スコア2

Google Apps Script

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

JavaScript

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

0グッド

1クリップ

投稿2022/05/13 08:49

編集2022/05/15 19:05

1.前提・実現したいこと
土日祝日を除いてメールの下書きが作成されるようにしたい

2.発生している問題・エラーメッセージ
条件分岐のコードを入れたところプログラムが作動しなくなりました。
(条件分岐なしだと作動します)
お力添え頂けますと幸いです。

3.該当ソースコード

function createmail() { //曜日の取得 var date = new Date(); var day = date.getDay(); //今日が土日なら処理を中断 if (day===0 || day===6) { return; } Logger.log(day); Logger.log(date); //祝日の取得 var ss = SpreadsheetApp.getActiveSpreadsheet(); var holidaysheet = ss.getSheetByName("祝日一覧"); var lastRow = holidaysheet.getLastRow() var holiday = holidaysheet.getRange(1,1,lastRow).getValues().slice(1);   //祝日一覧の日付とdateの日付が一致した場合処理を中断 for (var i=0 ; i<holiday.length ; ++i){ if(holiday[i]=date){ return; } Logger.log(holiday); //FMT詳細 var ss = SpreadsheetApp.getActiveSpreadsheet(); var sheet = ss.getSheetByName("FMT"); //toの取得 var range = sheet.getRange("C2"); var to = range.getValue(); //ccの取得 var range = sheet.getRange("C3"); var cc = range.getValue(); //subjectの取得 var range = sheet.getRange("C4"); var subject = range.getValue(); //当日すべてのイベントが格納されてる箱 var values = [];    //イベントの取得  var calendar = CalendarApp.getCalendarById("カレンダーID");  var today = new Date();  var events = calendar.getEventsForDay(today);  var length = events.length;  //繰り返し処理  for (var i=1 ; i<length ; ++i){     var event = events[i];      //eventsから配列を取り出す   var title = event.getTitle(); //eventのタイトルを取得      var start = event.getStartTime();                 //eventのスタート時間の取得   start = Utilities.formatDate(start,"Asia/Tokyo","HH:mm");    var end = event.getEndTime();                  //eventのエンド時間の取得   end = Utilities.formatDate(end,"Asia/Tokyo","HH:mm");   var time = `${start} - ${end}`                    values.push(`${time} ${title}`)    } //イベントをスプレッドシートにセット    var daily = sheet.getRange("C5"); var titles = daily.setValue(values.join("\n"));   //bodyの取得 var range = sheet.getRange("C6"); var body = range.getValue(); //optionsの指定 options = { cc:cc } //下書き作成 GmailApp.createDraft(to,subject,body,options); } }

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

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

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

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

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

退会済みユーザー

退会済みユーザー

2022/05/13 08:57

「うまく作動しない」とは具体的にどういうことでしょうか? 期待する動作と合わせて、具体的に記載してください。
bin.

2022/05/13 09:18

以下のコードをいれたところ、下書き作成もされなくなってしまいました。 (条件分岐なく、メールの下書きを作成するのみのコードだと作動しました) やりたいこととしては、土日祝日を除いてメールの下書きが作成されるようにしたいです。 ▼コード ====================================== //曜日の取得 var date = new Date(); var day = date.getDay(); //今日が土日なら処理を中断 if (day===0 || day===6) { return; } Logger.log(day); Logger.log(date); //祝日の取得 var ss = SpreadsheetApp.getActiveSpreadsheet(); var holidaysheet = ss.getSheetByName("祝日一覧"); var lastRow = holidaysheet.getLastRow() var holiday = holidaysheet.getRange(1,1,lastRow).getValues().slice(1);   //祝日一覧の日付とdateの日付が一致した場合処理を中断 for (var i=0 ; i<holiday.length ; ++i){ if(holiday[i]=date){ return; }
guest

回答2

0

ベストアンサー

下記のように
var day = date.getDay();
の直後に
Logger.log(day);
Logger.log(date);
を入れて、曜日の数字および日付が意図するものになっているか確認してください。

function createmail() { //曜日の取得 var date = new Date(); var day = date.getDay(); Logger.log(day); Logger.log(date); //今日が土日なら処理を中断 if (day===0 || day===6) { return; } (以下略)

意図する時刻になっていない場合は、設定が日本時間以外になっている可能性があります。

下の図のように、左側の歯車のマークをクリックして、
タイムゾーンを「(GMT+09:00) GMT+09:00」
に設定してください。

イメージ説明

イメージ説明


追記

js

1 if (holiday[i] = date) { 2 return; 3 }

これだと「=」が1つで、代入になっているため、常にtrueになってしまいます。

js

1 2 if (holiday[i] == date) { 3

のように、=を「==」に変えてみてください。

追記2

また、
・祝日のシートデータがわからないので推測ですが、おそらくholidayは時刻が0:00:00となるので、上記比較の際にdateの時刻も0:00:00にそろえる必要がある。

・holidayの数だけ処理をループしているため、同じ下書きが作られてしまう。
といった問題があります。

これらを修正してみたのが下記です。
(全部の情報が公開されておらず、一部推測しているため、うまく動かない可能性はあります。
可能ならスプレッドシートを共有していただいた方が解決の近道かと思います)

js

1function createmail() { 2 3 //曜日の取得 4 var date = new Date(); 5 var day = date.getDay(); 6 7 //今日が土日なら処理を中断 8 if (day === 0 || day === 6) { 9 return; 10 } 11 12 Logger.log(day); 13 Logger.log(date); 14 15 //祝日の取得 16 var ss = SpreadsheetApp.getActiveSpreadsheet(); 17 var holidaysheet = ss.getSheetByName("祝日一覧"); 18 var lastRow = holidaysheet.getLastRow() 19 var holiday = holidaysheet.getRange(1, 1, lastRow).getValues().slice(1); 20 21 // 時刻を0:00:00にする。 22 date.setHours(0, 0, 0, 0); 23 //祝日一覧の日付とdateの日付が一致した場合処理を中断 24 for (var i = 0; i < holiday.length; ++i) { 25 if (holiday[i] == date) { 26 return; 27 } 28 } 29 30 Logger.log(holiday); 31 32 33 //FMT詳細 34 var ss = SpreadsheetApp.getActiveSpreadsheet(); 35 var sheet = ss.getSheetByName("FMT"); 36 37 //toの取得 38 var range = sheet.getRange("C2"); 39 var to = range.getValue(); 40 41 //ccの取得 42 var range = sheet.getRange("C3"); 43 var cc = range.getValue(); 44 45 //subjectの取得 46 var range = sheet.getRange("C4"); 47 var subject = range.getValue(); 48 49 50 51 //当日すべてのイベントが格納されてる箱 52 var values = []; 53 54 //イベントの取得 55 var calendar = CalendarApp.getCalendarById("カレンダーID"); 56 var today = new Date(); 57 var events = calendar.getEventsForDay(today); 58 var length = events.length; 59 60 //繰り返し処理 61 for (var i = 1; i < length; ++i) { 62 63 var event = events[i];      //eventsから配列を取り出す 64 var title = event.getTitle(); //eventのタイトルを取得  65 66 67 var start = event.getStartTime();                 //eventのスタート時間の取得 68 start = Utilities.formatDate(start, "Asia/Tokyo", "HH:mm"); 69 var end = event.getEndTime();                  //eventのエンド時間の取得 70 end = Utilities.formatDate(end, "Asia/Tokyo", "HH:mm"); 71 var time = `${start} - ${end}` 72 73 values.push(`${time} ${title}`) 74 } 75 76 77 78 //イベントをスプレッドシートにセット    79 var daily = sheet.getRange("C5"); 80 var titles = daily.setValue(values.join("\n")); 81 82 83 //bodyの取得 84 var range = sheet.getRange("C6"); 85 var body = range.getValue(); 86 87 88 //optionsの指定 89 options = { 90 91 cc: cc 92 } 93 94 //下書き作成 95 GmailApp.createDraft(to, subject, body, options); 96 97}

投稿2022/05/13 11:47

編集2022/05/15 06:50
退会済みユーザー

退会済みユーザー

総合スコア0

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

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

bin.

2022/05/13 17:06

ありがとうございます>< 先程確認したのですが、ログは『5.0』『Fri May 13 07:50:59 GMT+09:00 2022』とでており タイムゾーンも『GMT+09:00』となっておりました。が下書き作成されせず。。 コードが間違っているのでしょうか・・
退会済みユーザー

退会済みユーザー

2022/05/13 20:06

追記しました。 if (holiday[i] = date) { を if (holiday[i] == date) { に修正してみてください。
bin.

2022/05/15 06:20

ご確認ありがとうございます! 追記していただいたように『holiday[i] == date』に書き換えたのですが、 今度は際限なく下書きが作成されるようになりました。。 (実行停止を押さないと止まらないほどに下書きが。。) 何度もご確認をいただいている中申し訳ございませんが、 もしまだお力添えいただけるようでしたら、どこが間違えてるのか教えていただけますと幸いです..><
退会済みユーザー

退会済みユーザー

2022/05/15 06:49

修正・追記しました。
bin.

2022/05/15 10:05

追記ありがとうございます! いただいたメソッド追加したところ無事条件分岐で作成することができました! 日付だけでなく、時間も一致しないと上手く動かないのですね..とても勉強になりました。m(_ _)m お忙しい中、親身になってご回答いただき本当にありがとうございました!!!!
guest

0

Logger.log(day);
Logger.log(date);

これをif文の前に移動させて、そいつになに出てくるのか見てみては

投稿2022/05/13 10:07

y_waiwai

総合スコア87719

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問