Google Apps ScriptでGoogleスライドを編集したい

Question

前提・実現したいこと

スプレッドシートのデータを元に、Googleスライドのシェイプ内の文字列を置換してくれるようにしたい。

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

for分で繰り返し処理を行って置換をしているのですが、繰り返し処理の最後の部分だけが置換されてしまう。
下の画像のように、スプレッドシートのデータを元に置換を行いたいのですが、繰り返し処理の一番最後の部分（g→Gの置換）だけが実行されてしまう状況です。

該当のソースコード

function replaceShapeString(){

  var url ='スライドのURL';
  var presentation = SlidesApp.openByUrl(url);  

  var sheet = SpreadsheetApp.openById('スプレッドシートのID').getSheetByName('シート名')
  var values= sheet.getRange('範囲').getValues();
  var lastRow = sheet.getLastRow();
  
  
  for(var i = 2; i <= lastRow; i++) {
    var id = (sheet.getRange(i, 1).getValues());
  }

  for(var i = 2; i <= lastRow; i++) {
    var pattern = (sheet.getRange(i, 2).getValues());
  }

  for(var i = 2; i <= lastRow; i++) {
    var replacement = (sheet.getRange(i, 3).getValues());
  }

    replaceShapeString_(presentation, id, pattern, replacement)

  
}

function replaceShapeString_(presentation, id, pattern, replacement){

  var shape = presentation.getPageElementById(id).asShape();
  var str = shape.getText().asString();
  str = replaceInBrace_(str, pattern, replacement);
  shape.getText().setText(str);
  
}

function replaceInBrace_(str, pattern, replacement){
  
  pattern = '{' + pattern + '}';
  str = str.replace(pattern, replacement);
  return str;
  
}

Answer 1

 for(var i = 2; i <= lastRow; i++) {
   var id = (sheet.getRange(i, 1).getValues());
 }

上記でループする度にidの値を書き換えているため、最後のループ(i = 8)の場合の値になるためです。(patternとreplacementも同じ)

こうすればいいかと思います。

  for (let row = 2; row <= lastRow; row++) {
    const id = sheet.getRange(row, 1).getValue();
    const pattern = sheet.getRange(row, 2).getValue();
    const replacement = sheet.getRange(row, 3).getValue();
    replaceShapeString_(presentation, id, pattern, replacement);
  }

本題とは関係ないですが、スプレッドシートはセルごとにgetRange()するより全体を配列で取得してループした方がいいです。

  const values = sheet.getDataRange().getValues().slice(1);
  for (const [id, pattern, replacement] of values) {
    replaceShapeString_(presentation, id, pattern, replacement);
  }

Comments

[Azamaro]

回答してくださり、ありがとうございます。
ご教示いただいたコードで実行した結果、うまく置換されました。

しかし、上で示したスプレッドシート上に追加で「h→H」を置換するように記述したところ、「TypeError: Cannot read property 'asShape' of null」と出てしまいました。

[draq]

presentation.getPageElementById(id) が null を返しているってことなので、スプレッドシートに入力しているidが間違っているのでしょう。

[Azamaro]

すみません、初歩的なミスをしていました。
回答してくださり、ありがとうございました。解決策、アドバイスともに大変助かりました！

Answer 2

  for(var i = 2; i <= lastRow; i++) {
    （処理1）
  }
  for(var i = 2; i <= lastRow; i++) {
    （処理2）
  }
  （処理3）

このコードの意味は（処理1）を繰り返し行い、その後に（処理2）を繰り返し行いその後に（処理3）を一度だけ行う。という意味です。

各行で、ある列の値を三つ取得し変換する、のであれば

  for(var i = 2; i <= lastRow; i++) {
    （一つ目取得）
    （二つ目取得）
    （三つ目取得）
  　（変換処理）
  }

とする必要があります。