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

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

ただいまの
回答率

89.99%

ファイルが2つできてしまう

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 773

Kodanosuke

score 18

前提・実現したいこと

以下のコードをトリガーを用いてフォームが送信されるごとに実行しているのですが、最終的にドキュメントが2つできてしまいます。この原因として何が考えられますでしょうか。

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

エラーメッセージは特にありません。
ドキュメントが重複して作成されてしまうことが問題です。
ドキュメントは2つとも指定の同じフォルダに作成されています。

  //①
  var ssheet = SpreadsheetApp.getActiveSpreadsheet();
  var sheet = ssheet.getSheetByName('改修後');

  //変数を設定する

  var alldata = sheet.getRange(sheet.getLastRow(), 3 ,1 ,sheet.getLastColumn()).getValues();
  var pre_timestamp = sheet.getRange(sheet.getLastRow(), 1).getValue()
  var timestamp = Utilities.formatDate(pre_timestamp, "JST", "yyyy/MM/dd")
  var name_group = alldata[0][0];
  var name_rep = alldata[0][1];
  var phone_rep = alldata[0][2] 

  var name_bill = alldata[0][3];
  var name_place = alldata[0][4];
  var date = Utilities.formatDate(alldata[0][5] , "JST" ,"yyyy/MM/dd");

  if(alldata[0][6] != ""){

    var rehe_start = Utilities.formatDate(alldata[0][6],"JST" ,"HH:mm");
  }else{

    var rehe_start = [];
  }
  if(alldata[0][7] != ""){

    var rehe_end = Utilities.formatDate(alldata[0][7],"JST" ,"HH:mm");
  }else{

    var rehe_end = [];
  }
  if(alldata[0][8] != ""){

    var per_start = Utilities.formatDate(alldata[0][8],"JST" ,"HH:mm");
  }else{

    var per_start = [];
  }
  if(alldata[0][9] != ""){

    var per_end = Utilities.formatDate(alldata[0][9],"JST" ,"HH:mm");
  }else{

    var per_end = [];
  }


  var medi_add = alldata[0][10];

  //マスター発送日、プラン、オプション選択を設定する
  var target = sheet.getRange(sheet.getLastRow(), sheet.getLastColumn()-6, 1, 7)
  var master = sheet.getRange(sheet.getLastRow(), sheet.getLastColumn()-7)

  var rule = SpreadsheetApp.newDataValidation()
  .requireValueInList(['on','off'])
  .build()
  target.setDataValidation(rule)

  var rule2 = SpreadsheetApp.newDataValidation()
  .requireDate()
  .build()
  master.setDataValidation(rule2)



  var doc = DriveApp.getFileById('XXXXXXXXXX').makeCopy(name_group + '御中 ご依頼内容確認書').getId();
  var doc_body =   DocumentApp.openById(doc).getBody();
  var yobi_list = ["日","月","火","水","木","金","土"];
  var cells = [["ご依頼日"," "+timestamp],[" 団体名(奏者名)","  "+name_group],[" 収録担当者様氏名","  "+name_rep + " 様"],[" 電話番号","  "+phone_rep],[" 収録場所","  "+name_place],[" 収録日時","  "+date + "("+ yobi_list[new Date(date).getDay()] + ")"],[" ゲネプロ時間","  "+rehe_start + "-" + rehe_end],[" 本番時間","  "+per_start + "-" + per_end],[" マスター発送予定日","  "],[" お届け先","  "+medi_add + " 様"]];
  var table1 = doc_body.appendTable(cells).setColumnWidth(0, 150).setColumnWidth(1, 300);
  table1.editAsText().setFontSize(9);
  var cell2 = [["依頼内容・備考等","\n\n"+name_bill+"\n上記のプラン内容にて承りました。"]]
  var table2 = doc_body.appendTable(cell2).setColumnWidth(0, 150).setColumnWidth(1, 300);             
  table2.editAsText().setFontSize(9)
  var doc_table = doc_body.getTables()

  //③

  var folder_to = DriveApp.getFolderById('XXXXXXXXXXXXXXXXXXXXXX');
  folder_to.addFile(DriveApp.getFileById(doc).setName(name_group + " 御中 ご依頼内容確認書 "));
  var test = DriveApp.getFileById(doc)
  DriveApp.getRootFolder().removeFile(test);

}

Google App Script
ソースコード
```

試したこと

・同じ内容のトリガーが設定されていないか確認しました。

補足情報(FW/ツールのバージョンなど)

ここにより詳細な情報を記載してください。

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

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

checkベストアンサー

+1

このところ、スプレッドシート側で、フォーム送信時にトリガー指定した関数が多重実行されるという、質問が重なっています。
(ただし2回目以降はイベントパラメータに入るはずのフォームの値が不完全になる模様)

これまでのところ有効な対応策は、フォームのスクリプトとしてフォーム送信時のトリガーをかけると一回に抑制されるとのことでした。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/03/11 09:39

    回答いただきありがとうございます.
    フォームのスクリプトとしてトリガーをかけるというのは,スプレッドシートからスクリプトを書く際に,関数名をonFormSubmit(e)にするということでしょうか.
    よろしくおねがいします.

    キャンセル

  • 2019/03/11 11:11 編集

    フォームの編集画面の右上にある「…」を縦にならべたアイコンから、スクリプトエディタを開き、そこでコードをかいて、作ったコードを「フォームから」「送信時」のトリガをかける、という意味です。
    ```
    var ssheet = SpreadsheetApp.getActiveSpreadsheet();
    ```
    ここをSpreadsheetApp.OpenById("シートid");にするという変更が必要です。もしかしたらその他にも変更しないといけないポイントがあるかもしれません。

    キャンセル

  • 2019/03/11 14:16

    ありがとうございます.
    やってみます!

    キャンセル

0

これが直接の解決策になりえるかどうかは分かりませんが、例えばLockServiceを使うのは如何でしょうか。一つの使用例として、ユーザが送信ボタンをダブルクリックしてしまうと2度並行してスクリプトが実行されてしまいますのでこれを回避するためにLockServiceを用いています。

サンプルスクリプト

質問にあるスクリプトの上下に、下記のようにLockServiceを使用するためのスクリプトを挿入してみてください。

function myFunction(e) { // 質問から関数名が分からなかったためサンプルとしての関数名です。
  var lock = LockService.getScriptLock();
  lock.waitLock(30000); // スクリプト自体は数秒で終わると思いますが、念のため30秒待つ設定です。

  // ここに実行するスクリプトを入れて下さい。

  lock.releaseLock();
}

参考

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

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

  • ただいまの回答率 89.99%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる