回答編集履歴

2021/12/19 03:19

投稿

スコア0

answer CHANGED Viewed

@@ -278,4 +278,7 @@
 ２．コードエディタ画面に戻ると、「appsscriptjson」というファイルが表示されているはずなので、そこをクリックし、timeZoneが「Asia/Tokyo」になっているか確認します。
 （「Asia/Tokyo」になっていない場合は、「Asia/Tokyo」に直して保存）
-![イメージ説明](4d65f0709bc8d82373da3e929823dac3.png)
+![イメージ説明](4d65f0709bc8d82373da3e929823dac3.png)
+---
+参考に、配列から要素を除外する方法（splice関数）について書こうとしましたが、書くと文字数制限を超えるためここでは割愛します。必要があればコメントに記載します。

2021/12/19 03:19

投稿

退会済みユーザー

スコア0

answer CHANGED Viewed

@@ -1,4 +1,4 @@
-まずは、回答から。質問文コードとの差異を下記に記載しました。（色付きの行のうち、先頭が「+」となっている行は追加、「-」となっている行は削除）
+直すとすれば下記のようになるでしょうか
 ```diff
 function createEvent() {
   //▼予定を追記するGoogleカレンダーIDを取得する　<<セルC9(行9,列3)
@@ -182,7 +182,6 @@
 +   mysheet1.getRange(16 + i, 8, 1, 1).setValue("登録済");　//スプシH列に「登録済」と追記
 ```
 # 改善案
 元々のコードですが、同じ処理について列を変えて繰り返すコードを追加する過程で
 列の指定を誤って修正してしまっていると思われる部分がところどころにあります。

2021/12/19 03:16

投稿

退会済みユーザー

スコア0

answer CHANGED Viewed

@@ -159,7 +159,7 @@
 「||」は難しい言葉で「論理和演算子」と呼ばれるものです。
 ごく簡単に言えば、if文で「A || B」という形で使うと、AまたはBのどちらかが正の場合、文が実行されます。
-参照：[||(論理和)](https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Operators/Logical_OR)、
+参照：[MDN：||(論理和)](https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Operators/Logical_OR)
 （２）IF文とfOR文どちらを先に書けばよいのか
@@ -172,7 +172,7 @@
 （３）「登録済」フラグの書き込み
 元の質問文では、カレンダーに登録した行についてH列～K列に「登録済」という値を設定しようとされていますが、
-実際は常に16行目～最後の行まで込むようになってしまっています。
+実際は常に16行目～最後の行まで書き込むようになってしまっています。
 ここは、カレンダーを登録した行だけに書き込むようにする必要があります（下記）
 ```diff

追記

2021/12/19 02:52

投稿

退会済みユーザー

スコア0

answer CHANGED Viewed

@@ -182,6 +182,92 @@
 +   mysheet1.getRange(16 + i, 8, 1, 1).setValue("登録済");　//スプシH列に「登録済」と追記
 ```
+# 改善案
+元々のコードですが、同じ処理について列を変えて繰り返すコードを追加する過程で
+列の指定を誤って修正してしまっていると思われる部分がところどころにあります。
+注意深くデバッグして正しく動けば問題ないのですが、現状のアプローチでは下記のような問題があります。
+・仮に登録処理の列が増えたり減ったりした場合に、４つのブロックそれぞれを変更しなければならなくなる。
+・その際にチェックが不足していると、同じような列指定ミスが発生する可能性が高くなる。
+２つ前の御質問と同様に、規則性を見出してまとめられるところはまとめた方が、変更に強く、バグを出しにくいコードになります。
+たとえば今回で言えば、「各日付の４つの処理件名列と、登録済フラグを書き込む列はそれぞれ4列ずつ離れている」という規則性があります。
+![イメージ説明](2f15bc29bae1d4534e8f5568db68e4a3.png)
+そこで、「件名の取得とカレンダーへの登録、及び登録済フラグの書き込み」を1つの処理としてまとめ、
+列を1つずつ移動させればすっきりしたコードになります。
+下記では、もともとのforループ（targetRowを動かすループ）の中に、もう1つforループを加えています。
+この追加したforループの中で、column という変数を0～3までループさせています。
+各処理対象に対して column 変数が示す値を加算することで、１列ずつ移動して処理を行っています。
+```js
+// 4列分の処理をまとめたコード
+function createEvent2() {
+  //▼予定を追記するGoogleカレンダーIDを取得する　<<セルC9(行9,列3)
+  const ss = SpreadsheetApp.getActiveSpreadsheet();
+  const mysheet1 = ss.getSheetByName("カレンダー転記用");
+  const calId = mysheet1.getRange(9, 3).getValue();
+  const cal = CalendarApp.getCalendarById(calId);
+  const targetRows = mysheet1.getRange('C13').getValue();
+  console.log("targetRows " + targetRows);
+  //予定①～予定④に予定なし分(土、日、祝日)はL列にフラグ立て
+  const endRow = targetRows + 15
+  console.log("endRow " + endRow);
+  for (let i = 16; i < endRow + 1; i++) {
+    const myRange = mysheet1.getRange(i, 4, 1, 4);
+    if (myRange.isBlank()) {
+      mysheet1.getRange(i, 12, 1, 1).setValue("対象外");
+      //getRange(行目,列目,●行分,●列分)
+    }
+  }
+  //▼データ取得範囲の指定
+  //各種予定のデータ取得範囲の指定
+  //getRange(行目,列目,●行分,●列分)
+  //予定配列>>セルB16(行16,列2)～L列まで
+  const eventRange = mysheet1.getRange(16, 2, targetRows, 11);
+  console.log("eventRange " + eventRange.getA1Notation());
+  //▼各予定データを配列(myEvent)として取得する
+  const myEvent = eventRange.getValues();
+  Logger.log(myEvent);
+  //▼各日付ごとに処理する
+  for (let i = 0; i < targetRows; i++) {
+    let myDate = myEvent[i][0]; //「日付」は起点B16から下にi行、右に0
+    let ngFlag = myEvent[i][10];  //「対象外フラグ」は起点B16から下にi行、右に10
+    // 「対象外」であればスキップして次の行へ。
+    if (ngFlag === "対象外") {
+      continue;
+    }
+    //▼「①~④予定」を一列ずつ処理（column：0～3までループ）
+    for (let column = 0; column < 4; column++) {
+      let flag = myEvent[i][6 + column];  //「①～④登録済フラグ」は起点B16から下にi行、右に6+column
+      // 「登録済」であればスキップして隣の列へ。
+      if (flag === "登録済") {
+        continue;
+      }
+      //「予定名(title)」を取得
+      let title = myEvent[i][2 + column];  //「①～④予定名」は起点0(B16)から下にi行、右に2+column
+      //▼カレンダーへの予定登録
+      cal.createAllDayEvent(title, new Date(myDate));
+      //▼スプレッドシートへの「登録済」フラグ記載
+      mysheet1.getRange(16 + i, 8 + column, 1, 1).setValue("登録済");　//スプシH列に「登録済」と追記
+    }
+  }
+  //▼ダイアログMsgの表示
+  Browser.msgBox("Googleカレンダーへの転記作業が完了しました。\n登録内容はカレンダー転記シートで確認してください。", Browser.Buttons.OK);
+}
+```
 # その他
 本件の質問とは直接関係ありませんが、スクリプトのタイムゾーン設定が日本時間になっていない場合、日時がずれてGoogleカレンダーに登録されることがあります。

修正

2021/12/19 02:45

投稿

退会済みユーザー

スコア0

answer CHANGED Viewed

@@ -102,7 +102,7 @@
     //▼スプレッドシートへの「登録済」フラグ記載
     //getRange(行目,列目,●行分,●列分)　>>セルJ16以降に転記
 -   mysheet1.getRange(16, 10, targetRows, 1).setValue("登録済");　//スプシJ列に「登録済」と追記
-+    mysheet1.getRange(16 + i, 10, targetRows, 1).setValue("登録済");　//スプシJ列に「登録済」と追記
++    mysheet1.getRange(16 + i, 10, 1, 1).setValue("登録済");　//スプシJ列に「登録済」と追記
   }
   //▼「④予定」の取得
@@ -172,14 +172,16 @@
 （３）「登録済」フラグの書き込み
 元の質問文では、カレンダーに登録した行についてH列～K列に「登録済」という値を設定しようとされていますが、
-実際は常に16行目に書き込むようになってしまっています。
+実際は常に16行目～最後の行まで込むようになってしまっています。
-ここは、カレンダーを登録した行に書き込むようにする必要があります（下記）
+ここは、カレンダーを登録した行だけに書き込むようにする必要があります（下記）
-```
+```diff
     //▼スプレッドシートへの「登録済」フラグ記載
     //getRange(行目,列目,●行分,●列分)  >>セルH16以降に転記
 -   //mysheet1.getRange(16, 8, targetRows, 1).setValue("登録済");　//スプシH列に「登録済」と追記
 +   mysheet1.getRange(16 + i, 8, 1, 1).setValue("登録済");　//スプシH列に「登録済」と追記
 ```
 # その他
 本件の質問とは直接関係ありませんが、スクリプトのタイムゾーン設定が日本時間になっていない場合、日時がずれてGoogleカレンダーに登録されることがあります。

修正

2021/12/19 02:01

投稿

退会済みユーザー

スコア0

answer CHANGED Viewed

@@ -102,7 +102,7 @@
     //▼スプレッドシートへの「登録済」フラグ記載
     //getRange(行目,列目,●行分,●列分)　>>セルJ16以降に転記
 -   mysheet1.getRange(16, 10, targetRows, 1).setValue("登録済");　//スプシJ列に「登録済」と追記
-+    mysheet1.getRange(16, 10, targetRows, 1).setValue("登録済");　//スプシJ列に「登録済」と追記
++    mysheet1.getRange(16 + i, 10, targetRows, 1).setValue("登録済");　//スプシJ列に「登録済」と追記
   }
   //▼「④予定」の取得
@@ -170,7 +170,16 @@
 今回の場合はforループの中で１行ずつ条件判定しているので、for文が先で正解です。
+（３）「登録済」フラグの書き込み
+元の質問文では、カレンダーに登録した行についてH列～K列に「登録済」という値を設定しようとされていますが、
+実際は常に16行目に書き込むようになってしまっています。
+ここは、カレンダーを登録した行に書き込むようにする必要があります（下記）
+```
+    //▼スプレッドシートへの「登録済」フラグ記載
+    //getRange(行目,列目,●行分,●列分)  >>セルH16以降に転記
+-   //mysheet1.getRange(16, 8, targetRows, 1).setValue("登録済");　//スプシH列に「登録済」と追記
++   mysheet1.getRange(16 + i, 8, 1, 1).setValue("登録済");　//スプシH列に「登録済」と追記
+```
 # その他
 本件の質問とは直接関係ありませんが、スクリプトのタイムゾーン設定が日本時間になっていない場合、日時がずれてGoogleカレンダーに登録されることがあります。