回答編集履歴

3

誤字を修正

2020/09/10 07:14

投稿

Daregada
Daregada

スコア11990

test CHANGED
@@ -10,7 +10,7 @@
10
10
 
11
11
  - forループでは、「日付」の値から月・日の値を読み取り、今日の月・日と一致する場合のみメール送信の処理を行なう。そういえば、どこにも「生年月日」の値は使っていないが何のためにあるのかな?
12
12
 
13
- - メール送信処理では、名前、メールアドレス、(個別の)件名をリストから読み取り、メール本文を(名前を置換する処理なども含めて)生成してから、メールを送信している。元のコードでは名前をシートから読み取っていたが、リスト全体を読み込んだ`value`に名前も含まれるので、そちらを使うほうが高速だ。
13
+ - メール送信処理では、名前、メールアドレス、(個別の)件名をリストから読み取り、メール本文を(名前を置換する処理なども含めて)生成してから、メールを送信している。元のコードでは名前をシートから読み取っていたが、リスト全体を読み込んだ`values`に名前も含まれるので、そちらを使うほうが高速だ。
14
14
 
15
15
 
16
16
 

2

誤字を修正

2020/09/10 07:14

投稿

Daregada
Daregada

スコア11990

test CHANGED
@@ -10,7 +10,7 @@
10
10
 
11
11
  - forループでは、「日付」の値から月・日の値を読み取り、今日の月・日と一致する場合のみメール送信の処理を行なう。そういえば、どこにも「生年月日」の値は使っていないが何のためにあるのかな?
12
12
 
13
- - メール送信処理では、名前、メールアドレス、(個別の)件名をリストから読み取り、メール本文を(名前を置換する処理なども含めて)生成してから、メールを送信している。元のコードでは名前をシートから読み取っていた、リスト全体を読み込んだ`value`に含まれるので、そちらを使うほうが高速だ。
13
+ - メール送信処理では、名前、メールアドレス、(個別の)件名をリストから読み取り、メール本文を(名前を置換する処理なども含めて)生成してから、メールを送信している。元のコードでは名前をシートから読み取っていた、リスト全体を読み込んだ`value`に名前も含まれるので、そちらを使うほうが高速だ。
14
14
 
15
15
 
16
16
 

1

表現を修正

2020/09/10 06:30

投稿

Daregada
Daregada

スコア11990

test CHANGED
@@ -6,7 +6,7 @@
6
6
 
7
7
  - 以下のコードでは、名前・誕生日・メールアドレスなどのリストが書かれたシートを(便宜的に)「シート1」としている。もっとふさわしいシート名を付けているなら、`getSheetByName`の引数を変更すること。このシートのオブジェクトを`sheet1`として取得し、リスト全体を`values`に読み込んでいる。
8
8
 
9
- - 「本文」シートのオブジェクトは`sheet2`として取得し、本文を`text`に読み込んでいる。件名のデフォルト値(リストのE2セルの値を利用している)も、こっちのB1とかA2とかに書いた方がいいのでは?
9
+ - 「本文」シートのオブジェクトは`sheet2`として取得し、A1セルに書かれた本文を`text`に読み込んでいる。そういえば、件名のデフォルト値(リストの書かれたシートのE2セルの値を利用している)も、こっちのB1とかA2とかに書いた方がいいのでは?
10
10
 
11
11
  - forループでは、「日付」の値から月・日の値を読み取り、今日の月・日と一致する場合のみメール送信の処理を行なう。そういえば、どこにも「生年月日」の値は使っていないが何のためにあるのかな?
12
12