GASを使ってスプレッドシートのA列の画像リストの画像をGoogleDriveにダウンロードしたい

Question

■試したこと
ずばりのソースが落ちてたので、試した

https://gist.github.com/kurozumi/f26c6cc75be55e61e9daadfdf6108771

■問題、エラー
Exception: Cell reference out of range
downloadImages @ コード.gs:17

var url = range.getCell(i,1).getValue();
↑ここでエラー

Cell reference out of rangeで検索して調べましたが
海外サイトばかりで行き詰ってしまいました。

/**
 * A列に入力された画像URLリストを元に画像をダウンロードしてGoogle Driveに保存する
 */
function downloadImages() {
  
  // 現在開いているシートを取得
  var sheet = SpreadsheetApp.getActiveSheet();
  
  // 範囲はA列を指定
  range = sheet.getRange("A1:A");
  
  // 画像URLが入力されている最後の行数を取得
  var row = sheet.getLastRow();
  
  for (i = 0; i < row; i++) {
    // シートから1行ずつ画像URLを取得
    var url = range.getCell(i,1).getValue();
    
    // 画像データを取得
    var response = UrlFetchApp.fetch(url);
    var fileBlob = response.getBlob().setName(fileName);
    
    // 取得した画像をGoogle Driveにアップロード
    var file = DriveApp.createFile(fileBlob);

    // 予め作っておいた画像フォルダの情報を取得
    var folders = DriveApp.getFoldersByName("画像フォルダ");
    while(folders.hasNext()) {
      var folder = folders.next();
      if(folder.getName() == "画像フォルダ"){
        break;
      }
    }

    // ルートディレクトリに画像が保存されているので画像フォルダにコピー
    file.makeCopy(file.getName(), folder);

    // ルートディレクトリの画像を削除
    file.setTrashed(true);
    
  }
}

Answer 1

参考にされたURLのGitHubに掲載されてるコードが間違ってるような。

配列は0スタートですが、スプレッドシート範囲は1からなので、
この書き方だとfor文を1から開始にしてあげないといけません。

しょっぱなに range.getCell(0,1) という存在しないセルを取得しようとしてエラーになってます。

同様に最後は、最終行（row）までとしてあげる必要があります。
for文の最初の部分を以下のように変更してみてください。

処理速度向上を考えれば最初にgetValuesで取得した方が良さそうですが、まずは今のコードで動かすことを優先しています。

 for (i = 1; i <= row; i++) {

Comments

[yasuteratail]

ありがとうございます！
おっしゃる通りソースが間違っていました。
ご指摘の通り、ソースを変更するとエラーが解消されました。
あと,

var fileBlob = response.getBlob().setName(fileName);
ここのfilenameも定義されてなかったので

var fileName = 'test' + i + ".jpg";
var fileBlob = response.getBlob().setName(fileName);

連番振るように変えて、無事にやりたいことができました。

解決して嬉しいです！ありがとうございます！