スプレッドシートから別のスプレッドシートへの転記

Question

前提・実現したいこと

スプレッドシートA(転記元)に書き込まれたらスプレッドシートB(転記先)へ転記するgasを組んでいます。
リンクをキーにして、今までにすでに転記したものは飛ばし、まだ転記してないものを最終行に転記する、というふうにしたく
繰り返し処理を使っています。

ここに質問の内容を詳しく書いてください。
（例）PHP(CakePHP)で●●なシステムを作っています。
■■な機能を実装中に以下のエラーメッセージが発生しました。

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

取得まではうまくいっているのですが、最終行に転記するというところがどうしてもうまくいきません。
例えば転記するものが2つあったとしたら、当初の最終行に2つともが上書きされて転記されてしまいます。(結果的に1つのみが転記された状態)
1つが転記されたら、またその次の行に転記したいのですが、
Sheet_CopyTo.getRange(Lastrow,1).setValue([●●●]);
Lastrowではだめなのでしょうか。

該当のソースコード

function myFunction() {
  //元データ取得
  var SS_CopyFrom =SpreadsheetApp.openById("1yCj10xjobcbh9yewktYLEh2fmqAK3oirOgP8UoYAetQ");
  var Sheet_CopyFrom = SS_CopyFrom.getSheetByName('転記元');
  
  //元データの最終行を取得
  var LastRow = Sheet_CopyFrom.getLastRow(); //最終行を取得

  //貼り付け先のスプレッドシートのIDを指定してシート名を指定する <転記先テスト>
    var SS_CopyTo = SpreadsheetApp.openById("11-7GvhoLJ4GI4ifhh-kwYBi3HYKA_039oSMqbx77HxY");
    var Sheet_CopyTo = SS_CopyTo.getSheetByName('転記先');

  //転記先データの最終行を取得
  var Lastrow = Sheet_CopyTo.getLastRow()+ 1; //最終行を取得


  //繰り返し処理
  for(let i = 1; i <= LastRow; i++) {

    //リンクを取得する。
    var Link = Sheet_CopyFrom.getRange(i,1).getValue();
    //リンクが今までにあったか検索する。
    const values = Sheet_CopyTo.getRange(1, 5, Sheet_CopyTo.getLastRow()).getValues().flat();
    console.log(values.indexOf(Link));
    
    if (values.indexOf(Link)== -1) {　//存在しない場合に実施したい処理

      //日付を取得する。
      var Day = Sheet_CopyFrom.getRange(i,2).getValue();
      
      //タイトルを取得する。
      var Title = Sheet_CopyFrom.getRange(i,5).getValue();
      
      //企業名を取得する。
      var Company = Sheet_CopyFrom.getRange(i,4).getValue();

      //コピーした日付を貼り付ける。
      Sheet_CopyTo.getRange(Lastrow,1).setValue([Day]); 

      //コピーしたタイトルを貼り付ける。
      Sheet_CopyTo.getRange(Lastrow,2).setValue([Title]); 

　　//コピーした企業名を貼り付ける。
      Sheet_CopyTo.getRange(Lastrow,3).setValue([Company]); 
      
    }


    }
 

}

試したこと

ここに問題に対して試したことを記載してください。

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

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

Answer 1

転記するものが2つあったとしたら、当初の最終行に2つともが上書きされて転記されてしまいます。(結果的に1つのみが転記された状態)

これは、書き込み先が、転記先の最終行 「Lastrow」として、ループ中で固定されているためではないでしょうか。

      //コピーした日付を貼り付ける。
      Sheet_CopyTo.getRange(Lastrow,1).setValue([Day]); 

      //コピーしたタイトルを貼り付ける。
      Sheet_CopyTo.getRange(Lastrow,2).setValue([Title]); 

　　//コピーした企業名を貼り付ける。
      Sheet_CopyTo.getRange(Lastrow,3).setValue([Company]);

あと「自分は気を付けているし、よく気づく方なので絶対に間違えることはない」というのならば特に問題はないのですが、
通常、
転記元：LastRow
転記先：Lastrow
のように、大文字小文字だけで内容を区別すると間違いやすいので、たとえば
転記元：fromLastRow
転記先：toLastRow
のように意味が分かりやすい変数名とした方が、間違いにくくなると思います。

---

Lastrowを他の場所で使用しないのであれば、
forループごとにLastrowを+1すればよいでしょう。

(略)
　　//コピーした企業名を貼り付ける。
      Sheet_CopyTo.getRange(Lastrow,3).setValue([Company]); 
      // 書き込み先の行を1プラス
      Lastrow = Lastrow + 1;
    }

またはappendRow()という関数を使って一番下の行に追加こともできます。

（略)
      //日付を取得する。
      var Day = Sheet_CopyFrom.getRange(i,2).getValue();
      
      //タイトルを取得する。
      var Title = Sheet_CopyFrom.getRange(i,5).getValue();
      
      //企業名を取得する。
      var Company = Sheet_CopyFrom.getRange(i,4).getValue();

      //最下行にコピーした日付を貼り付ける。
      Sheet_CopyTo.appendRow([Day,Title,Company]); 
    }

Comments

[0131]

ご回答いただきありがとうございます。
固定されているから同じ行に重ねて転記してしまうのですね、、！
「Lastrow」を固定せずに下へと転記していくには、どのように修正すれば良いのでしょうか。

また、転記先と転記元のシートについて、アドバイスありがとうございます！
気をつけていても間違ってしまうことがあるので、おっしゃる通り名前をわかりやすく変更しようと思います。

[退会済みユーザー]

追記しました