前回質問させていただき解決したと思ったのですが、取得するシートが(土)のものになってしまっていて困っています。
金土日と様子を見ておりましたが、全日とも(土)のスプレッドシートを取得してきているようでした。
やりたいこととしては
・曜日別でチャットに送りたい(スプシ内シート別で曜日分けしている)
// スプレッドシートを特定
var spreadsheet = SpreadsheetApp.openById(SHEET_ID);
var ss = SpreadsheetApp.getActiveSpreadsheet();
//曜日判別・取得
var today = new Date();
var day = today.getDay();
if (day === 0) {
var sheet1 = ss.getSheetByName("日");}
else if (day === 1) {
var sheet1 = ss.getSheetByName("月");}
else if (day === 2){
var sheet1 = ss.getSheetByName("火");}
else if (day === 3){
var sheet1 = ss.getSheetByName("水");}
else if (day === 4){
var sheet1 = ss.getSheetByName("木");}
else if (day === 5){
var sheet1 = ss.getSheetByName("金");}
else (day === 6);{
var sheet1 = ss.getSheetByName("土");
}
//sheet1をアクティブシートにする
sheet1.activate();
//シートの最終行番号、最終列番号を取得
var startRow = 2;
var startColumn = 1;
var lastRow = ss.getLastRow();
var lastColumn = ss.getLastColumn();
// シートの縦横すべてを取得
var sheetData = ss.getSheetValues(startRow, startColumn, lastRow, 2);
以下、割愛させていただきます
sheet1の定義がおかしいことになっているかと思われるのですが、お分かりになる方ご教授の程宜しくお願い致します。
[追記]
誤って(土)のシートを削除してしまい、再作成したのですが「null のメソッド「activate」を呼び出せません。」とエラーになってしまいます。
こちらの解決方法もご教示いただけたらありがたいです。
-
気になる質問をクリップする
クリップした質問は、後からいつでもマイページで確認できます。
またクリップした質問に回答があった際、通知やメールを受け取ることができます。
クリップを取り消します
-
良い質問の評価を上げる
以下のような質問は評価を上げましょう
- 質問内容が明確
- 自分も答えを知りたい
- 質問者以外のユーザにも役立つ
評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。
質問の評価を上げたことを取り消します
-
評価を下げられる数の上限に達しました
評価を下げることができません
- 1日5回まで評価を下げられます
- 1日に1ユーザに対して2回まで評価を下げられます
質問の評価を下げる
teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。
- プログラミングに関係のない質問
- やってほしいことだけを記載した丸投げの質問
- 問題・課題が含まれていない質問
- 意図的に内容が抹消された質問
- 過去に投稿した質問と同じ内容の質問
- 広告と受け取られるような投稿
評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。
質問の評価を下げたことを取り消します
この機能は開放されていません
評価を下げる条件を満たしてません
質問の評価を下げる機能の利用条件
この機能を利用するためには、以下の事項を行う必要があります。
- 質問回答など一定の行動
-
メールアドレスの認証
メールアドレスの認証
-
質問評価に関するヘルプページの閲覧
質問評価に関するヘルプページの閲覧
checkベストアンサー
0
まずはご自身の記述に間違いがないのかをしっかり確認する癖をつけたほうが良いですよ。
else (day === 6);{
elseのあとに条件式が入っているのがそもそもおかしいですし、条件式のあとにセミコロンが入っているため、
var sheet1 = ss.getSheetByName("土");
これが必ず実行されているのだと思われます。
また、sheet1をactivateして、ssに対して処理を行うよりも、直接sheet1に処理を行ったほうが、コードの可読性（今どこがActiveSheetなのか考えなくて良い）があがります。
//sheet1をアクティブシートにする
// sheet1.activate(); (不要)
//シートの最終行番号、最終列番号を取得
var startRow = 2;
var startColumn = 1;
var lastRow = sheet1.getLastRow();
var lastColumn = sheet1.getLastColumn();
// シートの縦横すべてを取得
var sheetData = sheet1.getSheetValues(startRow, startColumn, lastRow, 2);あと好みの問題かもしれませんが、ifの中で宣言しているのが気持ち悪いので、外に出したほうがいいと思います。（他の言語だとスコープの問題で中で宣言できないこともありますし）
投稿
-
回答の評価を上げる
以下のような回答は評価を上げましょう
- 正しい回答
- わかりやすい回答
- ためになる回答
評価が高い回答ほどページの上位に表示されます。
-
回答の評価を下げる
下記のような回答は推奨されていません。
- 間違っている回答
- 質問の回答になっていない投稿
- スパムや攻撃的な表現を用いた投稿
評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。
15分調べてもわからないことは、teratailで質問しよう！
- ただいまの回答率 88.36%
- 質問をまとめることで、思考を整理して素早く解決
- テンプレート機能で、簡単に質問をまとめられる
2018/08/22 23:44
ifで条件指定をしていた流れでelseの後にもつけてしまいました。。
まだ理解が甘かったようです。
ご指摘ありがとうございます。
elseの後ろの条件式とセミコロンを削除し、無事に曜日に従ったシートを取得できました！
また、可読性や他言語での応用のご指摘もありがとうございます。
非常に勉強になりました！
今回もベストアンサーとさせていただきます！
本当にありがとうございました。