ReferenceErrorになってしまう…

Question

実現したいこと

ここに実現したいことを箇条書きで書いてください。
-Googleフォームから申し込みされたイベント予約をスプレッドシートに集計し、スプレッドシートのGASから予約可否等のメール送信をしたい。

よろしくお願いいたします。

前提

/ここに質問の内容を詳しく書いてください。
イベントの予約をするためのツールを作成しています。Googleフォームで予約申し込み→スプレッドシートで集計し、スプレッドシートのGASから予約可否・QRコードの発行とメール送信をしたいと考えています。

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

エラーメッセージ
	ReferenceError: target is not defined

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

function form(e) {//メイン関数
  var values = e.values;//フォームの回答を取得
  //var stamp = values[0];//タイムスタンプを取得
  var mail = values[1];//アドレス
  var date = values[2];//日付
  var name = values[3];//氏名
  var child = values[4];//お連れ様の人数
  var way = values[5];//イベント知ったツール
  var up = 'FALSE';
  var brank = '';
  var ss = SpreadsheetApp.getActiveSpreadsheet();//スプレッドシートを有効にする
  var sht = ss.getSheetByName('来場者');
  sht.appendRow([brank, mail, name, way, child, up, date]);

  var today = Utilities.formatDate(new Date(), "JST", "yyyy/MM/dd");
  var ss = SpreadsheetApp.getActiveSpreadsheet();
  var sht = ss.getSheetByName("来場者");
  var lastrow = sht.getLastRow(); // 最終行を取得
  var lastcol = sht.getLastColumn(); // 最終列を取得
  var range = sht.getRange(1, 1, lastrow, lastcol);
  var values = range.getValues(); // 情報をオンメモリに保持
  for (var i = 1; i < lastrow; i++) {
    data = values[i][0];
    if (data == "") {
      values[i][0] = getRndStr();
    }
    values[i][8] = false;
    var qrc1 = '=image(\"https://chart.apis.google.com/chart?chs=250x250&cht=qr&chl=\"&A' + (i + 1) + ')';
    values[i][7] = qrc1;
  }
  range.setValues(values); //スプレッドシートに書き戻し

  for (let row = 1; row <= lastrow; row++) {
    if (sht.getRange(row,3).getValue() == name) {
      var id = sht.getRange(row,1).getValue();
    }
  }

//------------------------------------------
// 日程シートから該当日のデータを取得
 // フォームの送信内容を取得

  const items = e.response.getItemResponses();
  const reserveDate = items[2].getResponse();

  // 予約処理
  const result = updateDatesSheet(reserveDate);
  if (result === "success"){
    reserveSheet.appendRow([email, reserveDate]);   // 予約シートに追記
    updateForm();                                   // フォームの「空き状況」を更新
  }


function updateDatesSheet(reserveDate){
  const ss1 = SpreadsheetApp.openById("シートID入れてます");
  const datesSheet = ss1.getSheetByName("日程");
  const table = datesSheet.getDataRange().getValues();
  const index = table.findIndex(row => row[0] === reserveDate);
  const target = {
    rowNum: index + 1,
    date: table[index][0],
    cap: table[index][1],//日程シートB列
    reserved: table[index][3],//日程シートのD列（人数シートから集計した人数）
  }

}
  // 定員超過しなければ「予約済」に加算する
  if (target.reserved < target.cap) {
    datesSheet.getRange(target.rowNum, 3);
    return "success"
  } else {
    return "error"
  }
}


function sendEmail(mail, preferredDate, result){

  const mailTitle = "予約結果";
    const imageurl = 'https://chart.apis.google.com/chart?chs=250x250&cht=qr&chl=' + id;
    const response = UrlFetchApp.fetch(imageurl, option);
    const blob = response.getBlob().getAs(MimeType.JPEG);

  const mailBody = (result === "success")
    ? "予約が完了しました。\n 添付されたQRコードを当日の受付でご提示ください \n"
     +` 【予約日】：${preferredDate}` 
     + '【氏 名】' + name + '\n'
     + '【お申込み人数】' + child + '名\n'
     + '【来場者ID】' + id
    : "定員超過のため予約できませんでした。\n" +
      "下記のフォームから再度申請してください\n" +
      form.getPublishedUrl();
  const option = {
      method: "get",
     "attachments": blob,
     'name' : 'イベント',
  }
  GmailApp.sendEmail(mail, mailTitle, mailBody, option);
 
 }

function getRndStr() {
  var str = "abcdefghijklmnopqrstuvwxyz0123456789";
  var len = 8;
  var result = "";
  for (var i = 0; i < len; i++) {
    result += str.charAt(Math.floor(Math.random() * str.length));
  }
  return result;
  }



### 試したこと

targer前後の「｛　」の場所を変えてみましたが、そうするとindexやtableが白文字になってしまいました。配置する場所はここしかないような気がするので大きな変更はしていません。

Answer 1

質問文のエラーについては、updateDatesSheetの中括弧の位置 がおかしいことが原因です。
このエラー自体は、updateDatesSheet関数の続きのif文をupdateDatesSheet関数に含めるとともに
修正後のupdateDatesSheet関数をform関数の外に出すことで解消されます。

しかし他にも大量のバグが残っていてエラーが出るため、下記に原因と対応を記載します。

（注意：以下は、「2023/09/02 14:01のコメント内で共有されたスプレッドシート」の中に記載された、その時点でのコードに対する言及です。以降にコードが編集されていても関知しません。（関知できません））

１．「var id = sht.getRange(row, 1).getValue(); //ここのid部分が反応しなくなってしまった。」の原因

→ スコープ内で、変数idがどこからも参照されていないからです。
変数id は、sendEmail関数内で使われているため、sendEmail関数を呼ぶときに、idも一緒に引数として渡してやればよいです。
（idだけでなく、 name, childも sendEmail関数内で使われているため、idと一緒に引数として渡してやる必要があります）

２．「TypeError: Cannot read properties of undefined (reading 'getItemResponses')　at form(コード:44:28)」　の原因

→ フォーム入力内容を取得する書き方として、getItemResponses関数は、フォームのコンテナバインドスクリプトとして書いたときだけ有効です。

質問者さんによれば、スプレッドシートのコンテナバインドスクリプトとして書かれているとのことですので、getItemResponsesは使えません。
代わりに、form関数の冒頭の「var values = e.values」のように、valuesプロパティを使いましょう。

ただ、reserveDateに格納しようとしているのは、フォームで入力された予約日ですが、予約日は、form関数の最初の方の

  var date = values[2];//日付

ですでにdate変数として取得済みです。したがって、改めて

var reserveDate = values[2];

とする必要はありません。date変数をそのまま使いまわせばいいです。

その他

３：reserveSheetの未定義エラー

reserveSheet がどこにも定義されていないため、このままだとエラーになります。

４：予約数更新処理のバグ

日程シートの予約数を更新する下記の部分

datesSheet.getRange(target.rowNum, 3) 

ですが、入力範囲を取得しているだけで、値が書き込まれていません。
値を書き込むには、「setValue(＜値＞)」を追加する必要があります。

５. 日付が見つからなかった時のエラー処理

フォームの回答で日付を選択していますが、万が一、選択肢の日付がスプレッドシート上になかったり、フォームの質問とスプレッドシートのデータの間でコピぺミス等で文字不一致になっている場合、現状のコードだと　updateDatesSheet の

  const index = table.findIndex(row => row[0] === reserveDate);

で indexが-1になり、後続の処理でエラーになります。
したがって、きちんとエラートラップを置くべきです。
具体的には、上記の行の下に

  if (index === -1) return "error";

を追加します。

６. 定員超過判定のバグ

現状、updateDatesSheet関数の「 定員超過しなければ「予約済」に加算する」の中で、定員超過の判定処理として

if (target.reserved < target.cap) { 

としていますが、これだと、更新前の予約人数と定員を比べていることになるため、常にtrueになり、実際は超過する場合でも予約できてしまいます。
ここは、［更新前予約人数＋フォームで回答された予約人数］と定員とを比較すべきです。
このためには、呼び出し元のform関数でupdateDatesSheet関数を呼び出す際、updateDatesSheet関数に、フォームから回答された予約人数の情報を渡してやる必要があります。

７. form.getPublishedUrl()がエラーになる。

sendEmail関数の中で、

  const mailBody = (result === "success")
    ? "予約が完了しました。\n 添付されたQRコードを当日の受付でご提示ください \n"
    + ` 【予約日】：${preferredDate}`
    + '【氏 名】' + name + '\n'
    + '【お申込み人数】' + child + '名\n'
    + '【来場者ID】' + id
    : "定員超過のため予約できませんでした。\n" +
    "下記のフォームから再度申請してください\n" +
    form.getPublishedUrl();

としていますが

質問中のコードの「form」は関数でありフォームオブジェクトではないため、getPublishedUrl関数を持っていません。
定員超過時に form.getPublishedUrl() が呼び出されるとエラーになります。
ここはフォームのURLを直接記述する必要があります。

---

修正後の全体コード

上記を踏まえて修正したものが下記です。（一部の要件が不明確のため、期待する動作になっていない部分があるかもしれませんがあしからず。）
注意：updateForm関数は、エラー回避のためダミーで作っていますが、実際は具体的な処理に置き換えてください。

function form(e) {//メイン関数
  var values = e.values;//フォームの回答を取得
  //var stamp = values[0];//タイムスタンプを取得
  var mail = values[1];//アドレス
  var date = values[2];//日付
  var name = values[3];//氏名
  var child = values[4];//お連れ様の人数
  var way = values[5];//イベント知ったツール
  var up = 'FALSE';
  var brank = '';
  var ss = SpreadsheetApp.getActiveSpreadsheet();//スプレッドシートを有効にする
  var sht = ss.getSheetByName('来場者');
  sht.appendRow([brank, mail, name, way, child, up, date]);

  var today = Utilities.formatDate(new Date(), "JST", "yyyy/MM/dd");
  var ss = SpreadsheetApp.getActiveSpreadsheet();
  var sht = ss.getSheetByName("来場者");
  var lastrow = sht.getLastRow(); // 最終行を取得
  var lastcol = sht.getLastColumn(); // 最終列を取得
  var range = sht.getRange(1, 1, lastrow, lastcol);
  var values = range.getValues(); // 情報をオンメモリに保持

  /*来場者シートのデータを全部取得し、1行ずつチェック。
    A列が空欄の場合はランダムな文字列を記入。
    I列にQRコードの数式を設定
  */
  for (var i = 1; i < lastrow; i++) {
    data = values[i][0];
    if (data == "") {
      values[i][0] = getRndStr();
    }
    values[i][8] = false;
    var qrc1 = '=image(\"https://chart.apis.google.com/chart?chs=250x250&cht=qr&chl=\"&A' + (i + 1) + ')';
    values[i][7] = qrc1;
  }
  range.setValues(values); //スプレッドシートに書き戻し

  /*回答の名前と一致する行の１列目をidとして取得する*/
  for (let row = 1; row <= lastrow; row++) {
    if (sht.getRange(row, 3).getValue() == name) {
      var id = sht.getRange(row, 1).getValue();
    }
  }

  //------------------------------------------
  // 日程シートから該当日のデータを取得
  // フォームの送信内容を取得

  // const items = e.response.getItemResponses(); // 削除
  // const reserveDate = items[2].getResponse();  // 下のように修正
  const reserveDate = date;  // dateをそのまま使う

  // 予約処理
  const reservationCount  =  parseInt(child) + 1; // 追加 予約人数（連れの数+1 人）  
  //const result = updateDatesSheet(reserveDate);  // 下のように修正。
  const result = updateDatesSheet(reserveDate, reservationCount );  // 予約人数も渡す。

  // 追加：reserveSheet変数の定義
  const reserveSheet = ss.getSheetByName("予約");

  if (result === "success") {
    reserveSheet.appendRow([mail, reserveDate]); // 予約シートに追記
    updateForm();                                 // フォームの「空き状況」を更新
  }

  // 追加。メール送信時にid, name, childを渡す。
  sendEmail(mail, date, result, id, name, child);
}

/**
 * 引数に指定した予約日の予約人数が、定員オーバーでないか確認する。
 * オーバーしていなければ、日程シートの現在の予約数に予約人数を加算し、文字列"success"を返す。
 * オーバーしている場合は、文字列"error"を返す。
 * 引数：reserveDate フォームに入力された予約日
 *      reservationCount フォームの回答から計算した予約人数
*/
// function updateDatesSheet(reserveDate) {
function updateDatesSheet(reserveDate, reservationCount ) { // 修正：予約人数を受け取る。
  const ss1 = SpreadsheetApp.openById("シートID入れてます");
  const datesSheet = ss1.getSheetByName("日程");
  const table = datesSheet.getDataRange().getValues();
  const index = table.findIndex(row => row[0] === reserveDate);

  // 追加：フォームで回答された日付がスプレッドシート中になかった場合の処理。
  if (index === -1) return "error";

  const target = {
    rowNum: index + 1,
    date: table[index][0],
    cap: table[index][1],
    reserved: table[index][3], // 予約済人数：日程シートのD列
  }

  // 定員超過しなければ「予約済」に加算する
  //if (target.reserved < target.cap) {   // 下記のように修正
  // [更新前予約人数＋フォームで回答された予約人数]と定員とを比較する。
  if (target.reserved + reservationCount  < target.cap) {
    // datesSheet.getRange(target.rowNum, 3); // 下記のように修正
    // 更新後予約済人数を日程シートの予約済（C列？）に書き込む。
    datesSheet.getRange(target.rowNum, 3).setValue(target.reserved + reservationCount);
    return "success"
  } else {
    return "error"
  }
}


//function sendEmail(mail, preferredDate, result) {   
function sendEmail(mail, preferredDate, result, id, name, child) { // id,name,childを受け取るように修正
  const mailTitle = "予約結果";
  const imageurl = 'https://chart.apis.google.com/chart?chs=250x250&cht=qr&chl=' + id;
  const response = UrlFetchApp.fetch(imageurl);  // option削除
  const blob = response.getBlob().getAs(MimeType.JPEG);

  const mailBody = (result === "success")
    ? "予約が完了しました。\n 添付されたQRコードを当日の受付でご提示ください \n"
    + ` 【予約日】：${preferredDate}`
    + '【氏 名】' + name + '\n'
    + '【お申込み人数】' + child + '名\n'
    + '【来場者ID】' + id
    : "定員超過のため予約できませんでした。\n" +
    "下記のフォームから再度申請してください\n" +
    //form.getPublishedUrl();  //修正。エラーになるのでURLを直接記述する必要あり。
    "フォームのURL";

  const option = {
    method: "get",
    "attachments": blob,
    'name': 'イベント',
  }

  GmailApp.sendEmail(mail, mailTitle, mailBody, option);
}

function getRndStr() {
  var str = "abcdefghijklmnopqrstuvwxyz0123456789";
  var len = 8;
  var result = "";
  for (var i = 0; i < len; i++) {
    result += str.charAt(Math.floor(Math.random() * str.length));
  }
  return result;
}


function updateForm() {
  console.log("ダミーのupdateForm()が呼ばれました。")
}

Comments

[10-mo]

SaYGOさん

コメントいただきありがとうございます！target部分、色がつくようになりました。
しかし、今度は

for (let row = 1; row <= lastrow; row++) {
if (sht.getRange(row, 3).getValue() == name) {
var id = sht.getRange(row, 1).getValue(); //ここのid部分が反応しなくなってしまった。
}
}
//------------------------------------------
// 日程シートから該当日のデータを取得

下記のエラーも発生してしまいました。。。
TypeError: Cannot read properties of undefined (reading 'getItemResponses')
at form(コード:44:28)

[退会済みユーザー]

updateDatesSheet 関数内の「 // 定員超過しなければ「予約済」に加算する」の
「datesSheet.getRange(target.rowNum, 3)」
の部分は、
入力する位置を指定しているだけで、シートに何も値が入力されていませんが、これは意図する動作なのでしょうか。
意図する動作でない場合、日程シートのC列に、どの変数の値を入力するのでしょうか？（予約人数？）

コードも全部ではなく一部しか記載されていないようです。もし記載されていない部分にバグが潜んでいた場合、こちらでは手が出ません。
またソースコードだけ見せられても、こちらでコードから想像しながら手元でスプレッドシートやフォームをゼロから作って動作確認しなければいけないという手間があります。

可能であればフォームとスプレッドシートを全部公開状態にして共有していただけると解析が楽なので助かります。

[10-mo]

質問記載のコードは、スプレッドシートのメニューの拡張機能→「Apps Script」から起動したエディタ経由で書いているという理解でいいですか？
→はい、そのようにしています。スプレッドシートとフォームでしか使えないコードがあるんですね、

コードはこれで全てです(-_-;)
これで大丈夫かと思っておりました、、、、

こちらにスプレッドシートとフォームを共有させていただきます。よろしくお願いいたします。

https://docs.google.com/spreadsheets/d/1nrCJsjW9FvMsdA8rnCKyl7_2W9rUm2chs1_HoAPslHk/edit?usp=sharing

https://docs.google.com/forms/d/e/1FAIpQLScTN900Fnwr6JV-keKM2gFSzwkebjC0DNbBIqc19kK2wlNlEw/viewform?usp=sharing

[退会済みユーザー]

共有ありがとうございます。
> コードはこれで全てです
とのことですが、「upDateForm」関数がどこにもありません。この部分は未作成ということでよろしいでしょうか？

[10-mo]

const result = updateDatesSheet(reserveDate);
if (result === "success"){
reserveSheet.appendRow([email, reserveDate]); // 予約シートに追記
updateForm(); // フォームの「空き状況」を更新
}

↑これは違いますでしょうか？

[退会済みユーザー]

そこは単に、updateForm関数を"呼び出している部分"ですよね？

先のコメントは「（呼び出される）updateForm関数の"本体"」が、どこにも書いてないように見えます、という意味でした。

呼び出される本体がないと、呼び出し時にエラーになるはずであるため、質問文に記載し忘れていらっしゃったのではないかと思ったのです。

ただ、実際には呼び出す処理に到達される前に別のエラーが発生していて、発覚していなかっただけのようですね。

おそらくupdateForm関数を呼び出さなくても正常に動くような気がしますので、気になさらなくて結構です。

（完全に憶測ですが、updateForm関数の本体は質問者さんが他のサイト等からコピペしてきたときには存在していたが、ガチャガチャとコードをいじる過程で、別の関数の中に組み換えられてしまい、残った呼び出し部分だけ消すのを忘れられているだけではないか、と推察しています。）

[10-mo]

丁寧にご教示いただきありがとうございます！

一つ一つ、勉強して理解を深めたいとおもいます！