回答フォームから顧客マスタへメールアドレスを吸い上げたい

Question

前提・実現したいこと

回答フォームから顧客マスタへメールアドレスを吸い上げたいです。
連想配列を使用してみました。

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

処理が全く進みません。

該当のソースコード

function copyemailaddress() {
  var ss = SpreadsheetApp.getActiveSpreadsheet();
  var sheet = ss.getSheetByName('真顧客マスタ');
  var rows = sheet.getLastRow();
  var columns = sheet.getMaxColumns();
  var range = sheet.getRange(2,1,rows,columns);
  var datas = range.getValues();

  var sheet1 = ss.getSheetByName('フォームの回答 1');
  var rows1 = sheet1.getLastRow();
  var columns1 = sheet1.getMaxColumns();
  var range1 = sheet1.getRange(2,1,rows1,columns1);
  var datas1 = range1.getValues();

//回答シート取得
  var object = {};
  
  for(var i = 3; i < datas1.length; i++){
  var codevalue = datas1[i][5];
  var emailvalue = datas1[i][17];
  object[codevalue] = emailvalue;
  
  }
//真顧客マスタ取得
for(var p=2; p<=datas.length; p++){
  var emailplace = sheet.getRange(p,10);
  var code = sheet.getRange(p,1).getValue();



if(emailplace.isBlank()){
emailplace.setValue(object[code]);
  }//if done
 }//for done
 
}

試したこと

object[codevalue]のログが「情報」としか表示されませんがそういうものでしょうか。
よろしくお願いします。

Answer 1

  var code = sheet.getRange(p,1).getValue();


if(emailplace.isBlank()){
emailplace.setValue(object[codevalue]);

  }//if done

真顧客マスタからのコードの取得の部分が「code」になっていて
setValueする時はobject[codevalue]になってますが
object[code]の間違いではないですか？

Comments

[Sou23]

ありがとうございます。その通りだと思ってそのように修正したのですが（このページのコードも修正したものに書き換えました）、依然メールアドレスは入力されませんでした。ログではcode,codevalue,emailvalueどれも動いているので、なぜなのかわかりません。恐れ入りますが引き続きお力をお借りできれば幸いです。

[xail2222]

ぱっと見おかしそうな所は見つけられないです
code,codevalue,emailvalue全てログに出してみて確認出来ますか？
まぁ私がやるなら、デバッグしてステップ実行してるかな
(追記)
気になったのですが
if(emailplace.isBlank()){
の中には入ってきてますか？
もしかして、1度誤ったコードで流した為、空文字がセットされているとかって事ありませんか？

[xail2222]

空文字でもisbrankはtrueになるみたいですね
失礼しました。

[Sou23]

ありがとうございます。ログを取ったところ、iの方のfor文の中に最後のifまで入れてあげないとcodevalue, emailvalueの設定が消えてしまうようでしたので入れてみたところ、動き始めました。
しかし今度は最初に顧客コードを見つけたところを何回もログを取っており、そしてこの顧客マスタは20000件ありますので、最悪これを20000回繰り返すのではないかと思います。
設計自体に無理があったのでしょうか。

[xail2222]

デバッグでステップ実行しながら、変数の中身をチェックしていますか？

とりあえず、注意点としては、getValuesすると0から始まる配列になるため
var codevalue = datas1[i][5];
var emailvalue = datas1[i][17];
これは i 行 5列と17列 ではありません。
i +2行 6列=F列と18列＝R列です。
ここの認識は合ってますか？
(他の質問のコードを見させてもらいましたが分ってますよね。失礼しました)

codevalue, emailvalueの設定が消えてもobject は残っているので
ループは初めのままで問題ないはずです。

データ数が多くて大変というのであればテスト用に
データ量を少なくしたシートでテストをするのもありだと思いますよ。

[Sou23]

恥ずかしながらデバックでステップ実行という作業をやったことがなく、これから調べてやってみたいと思います。
またループにつきましてありがとうございます。
データ量を減らしたシートも作成してみようと思います。