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

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

ただいまの
回答率

89.99%

GAS:処理が順番どおりに行かない

解決済

回答 2

投稿 編集

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

CoAT.Yuki

score 7

前提・実現したいこと

Google Script と Google Sheetを使って、簡単な受注書発行システムを、下記の様な処理で作っています。

  1. 顧客リストから「新規見積, 受注済」のステータスになっている顧客を検出
  2. その顧客群から、「受注書済」にではない顧客を検出
  3. その顧客の「商談ID (sid)」と格納先の「フォルダID (fid)」を取得
  4. 同じGsheetファイル上にある他のシートに作成した受注書に「商談ID」を自動転記
  5. 同じスクリプト上にある、書類を作成する別のファンクションに「商談ID」と「フォルダID」を渡す
  6. 書類作成

発生している問題

今回問題になっているのが、上記のプロセスの4番に当たる、「商談IDの転記」の部分です。
なぜか、この4番が、6番の「書類を作成」した後に実行されます。
よって、連続してこのスクリプトを起動すると、受注書に転記される商談IDが一つづつ、ずれてしまいます。

例)
商談ID:0001 / 書類へ転記:空白、又は以前に入力されていた値
商談ID:0002 / 書類へ転記:0001
商談ID:0003 / 書類へ転記:0002

該当のソースコード

01 検出
function testdetect (){
  var ssid = "1ti3KmfS6xY3Dk-3492D6no1FczdU90KSE_DIsqVu6zU";// スプレッドシートのID
  var ss = SpreadsheetApp.openById(ssid);// 指定したスプレッドシートを開く

  var formsheet   = ss.getSheetByName('フォームの回答 1');//フォームデータがたまるシート
  var fsDataRange = formsheet.getDataRange();
  var values      = fsDataRange.getValues();
  var lastRow     = fsDataRange.getLastRow();  

  for(var i = lastRow-1; i>=1; i--){
    var status    = values[i][20];//顧客のステータス
    var docStatus = values[i][80];//書類作成ステータス:検出用
    var docS = formsheet.getRange(i+1,81);//書類作成ステータス:自動入力用
    var sid = values[i][2];//商談id
    var fid = values[i][79];//フォルダid

    switch(status){

      //受注済の顧客を検出
      case "新規見積, 受注済":
        if(docStatus != "受注書済"){

          //受注書作成済と記入 
          docS.setValue("受注書済");

          //受注書作成プロセス・スタート
          J_export(sid,fid); 

        }else if(docStatus == "受注書済"){
          docS.setValue("受注書済");
        }
        break;
        default:
        break;
    }
  }
}

02 書類セットアップ
function J_export(sid,fid)
{        
  //受注書用
  var ssid = "1ti3KmfS6xY3Dk-3492D6no1FczdU90KSE_DIsqVu6zU";// スプレッドシートのID
  var ss   = SpreadsheetApp.openById(ssid);// 指定したスプレッドシートを開く

  var JSheetid = "1576291853";  // PDF化したいシートID(gid)を指定
  var Fid = fid;  // PDFの保存先となるフォルダID

  var docJ         = ss.getSheetByName('受注書');//受注書
  var docDataRange = docJ.getDataRange();
  var docSid       = docJ.getRange(4,22);//受注書:商談ID入力セル

  /////////////////!!!!ここが書類出力前に受注書にIDを付与するところです!!!!///////////////
  //受注書にIDを付与
  docSid.setValue(sid);
  ////////////////////////////////////////////////////////////////////////////////

  // ファイル名に使用する名前を取得
  var filename = "J-"+sid+"_受注書";

 //ここで書類作成するファンクションに値を渡し、書類作成します。
  J_PDF_FIN(Fid, ssid, JSheetid, filename);
}

03 書類作成
function J_PDF_FIN(Fid, ssid, JSheetid, filename)
{
  var ss     = SpreadsheetApp.getActiveSpreadsheet(); 
  var folder = DriveApp.getFolderById(Fid);// PDFファイルの保存先となるフォルダをフォルダIDで指定 

// スプレッドシートをPDFにエクスポートするためのURL。このURLに色々なオプションを付けてPDFを作成
  var url = "https://docs.google.com/spreadsheets/d/SSID/export?".replace("SSID", ssid);

  // PDF作成のオプションを指定
  var opts = {
    exportFormat: "pdf",    // ファイル形式の指定 pdf / csv / xls / xlsx
    format:       "pdf",    // ファイル形式の指定 pdf / csv / xls / xlsx
    size:         "A4",     // 用紙サイズの指定 legal / letter / A4
    portrait:     "true",   // true → 縦向き、false → 横向き
    fitw:         "true",   // 幅を用紙に合わせるか
    sheetnames:   "false",  // シート名をPDF上部に表示するか
    printtitle:   "false",  // スプレッドシート名をPDF上部に表示するか
    pagenumbers:  "false",  // ページ番号の有無
    gridlines:    "false",  // グリッドラインの表示有無
    fzr:          "false",  // 固定行の表示有無
    gid:          JSheetid   // シートIDを指定 sheetidは引数で取得
  };

  var url_ext = [];

  // 上記のoptsのオプション名と値を「=」で繋げて配列url_extに格納
  for( optName in opts ){
    url_ext.push( optName + "=" + opts[optName] );
  }

  // url_extの各要素を「&」で繋げる
  var options = url_ext.join("&");

  // API使用のためのOAuth認証
  var token = ScriptApp.getOAuthToken();

  // PDF作成
  var response = UrlFetchApp.fetch(url + options, {
    headers: {
      'Authorization': 'Bearer ' +  token
      }
      });

  var blob = response.getBlob().setName(filename + '.pdf');

  // PDFを指定したフォルダに保存
  folder.createFile(blob);
}

試したこと

転記のタイミングが変わるかな?と思い、色々と書く場所を変えて見たのですが、
結局、書類が先に出力されてしまい、転記が後回しになってしまいます。。。

GASへ順番をしっかり伝えられる方法を、どなたかご存知の方がいらっしゃいましたら、
是非ご教授お願いいたします!

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 2

checkベストアンサー

+1

スクリプト全体が不明なため、この提案が直接の解決方法かどうかは分かりませんが、問題の原因は、スプレッドシートへ値を書き込んだ後に保存されていないのではないかと推測しました。そこで、J_export()の関数内にあるスクリプトを次のように変更してみてください。

From

docSid.setValue(sid);

To:

docSid.setValue(sid);
SpreadsheetApp.flush(); // <--- これを追加

注意

  • 関数J_PDF_FIN()の中身が不明なため、すでにその関数内でSpreadsheetApp.flush()を使用されている場合は、他に問題がある可能性を考えます。

参考

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/11/01 18:48 編集

    ご回答ありがとうございます。
    flush(); を試したところ、うまく処理ができました!
    処理が早すぎるのは良いことなのですが、こんな落とし穴があるとは。。。
    ありがとうございました!

    キャンセル

  • 2019/11/03 08:32

    問題が解決したとのことで安心しました。ご連絡ありがとうございました。

    キャンセル

+1

何となくで申し訳ありませんが、setValue後の書類作成が速すぎるかもしれません。

Utilities.sleep(5000);
を挟んで数秒待機するのも一つの手かもしれません。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/11/01 18:47

    ご回答ありがとうございました。
    お教わりしたことを試してみます!

    キャンセル

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

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