Google Apps Scriptでサブフォルダ内のファイルもアップデート検知したい

Question

前提・実現したいこと

Google Apps sctiptでファイルが追加されたことをメールで通知が来るように
したいと考えています。

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

指定フォルダ内へのファイル追加については問題なく検知できるのですが、
サブフォルダの検知ができません。
サブフォルダまで検索してアップデート日、ファイル名を取得するにはどうすればよいでしょうか？

該当のソースコード

var TARGET_FOLDER_ID = "xxx";
var UPDATE_SHEET_ID = "xxxx";
var UPDATE_SHEET_NAME = "sheet";
var SEND_MAIL_ADDRESS = ["xxx@xxx"]

function updateCheck() {

 
  // 対象フォルダ配下のファイル最終更新日時を取得。
var lastUpdateMap = {};

function filesearch(TARGET_FOLDER_ID) {  
  var targetFolder = DriveApp.getFolderById(TARGET_FOLDER_ID);
  var files = targetFolder.getFiles();
  while (files.hasNext()) {
    var file = files.next();
    lastUpdateMap[file.getName()] = file.getDateCreated();
  }
  var folders = DriveApp.searchFolders("'" + TARGET_FOLDER_ID + "' in parents");
  while (folders.hasNext()) {
    var folder = folders.next();
　　 lastUpdateMap[folder.getName()] = folder.getDateCreated();
    filesearch(folder.getId());
  }  
}
 
filesearch();


  // スプレッドシートに記載されているファイル名と更新日時を取得。
  var spreadsheet = SpreadsheetApp.openById(UPDATE_SHEET_ID);
  var sheet = spreadsheet.getSheetByName(UPDATE_SHEET_NAME);
  var data = sheet.getDataRange().getValues();

  // 取得したデータをMapに変換。
  var sheetData = {};
  // 説明とヘッダを記載しているため2からstart。
  for (var i = 2; i < data.length; i++) {
    sheetData[data[i][0]] = {name : data[i][0], lastUpdate : data[i][1], rowNo : i + 1};
  }

  // 実際のファイルとスプレッドシート情報を比較。
  var updateFolderList = [];
  for (key in lastUpdateMap) {
    if(key in sheetData) {
      // ファイル名がシートに存在する場合。
      if(lastUpdateMap[key] > sheetData[key].lastUpdate) {
        // フォルダが更新されている場合。
        sheet.getRange(sheetData[key].rowNo, 3).setValue(lastUpdateMap[key]);
        updateFolderList.push(key);
      }
    } else {
      // ファイル名がシートに存在しない場合。
      sheet.getRange(sheet.getLastRow() + 1, 1).setValue(key);
      sheet.getRange(sheet.getLastRow(), 3).setValue(lastUpdateMap[key]);
      updateFolderList.push(key);
    }
  }

// 新規及び更新された情報をメール送信。
  if (updateFolderList.length != 0) {
    SEND_MAIL_ADDRESS.forEach(function(o,i) {
      MailApp.sendEmail(SEND_MAIL_ADDRESS[i],"追加通知",
                        "写真が追加されました。" + "\n" +
                        "追加されたファイルは以下の通りです。" + "\n" +
                        updateFolderList.join("\n") + "\n\n"
                        );
    });
  }
}

試したこと

https://ctrlq.org/code/20034-search-drive-files
https://stackoverflow.com/questions/36577071/search-files-inside-sub-folders-in-google-drive
などを見たのですが、私の技量では理解できませんでした。

Comments

[hiroshi0240]

コードには他にも気になることはいくつかありますが、要は「 var files = targetFolder.getFiles();
」のタイミングでそのフォルダ配下の全フォルダ及びファイルを取得する方法を知りたいという理解で良いですか？

[papinianus]

後学のために伺いたいのですが、どこでどういうトリガを仕掛けて実行しているのでしょうか?

[efuji]

>要は「 var files = targetFolder.getFiles();」のタイミングでそのフォルダ配下の全フォルダ及び
>ファイルを取得する方法を知りたいという理解で良いですか？
フォルダ配下のファイルが取得できれば、取得のタイミングはいつでも構わないと考えています。

>どこでどういうトリガを仕掛けて実行しているのでしょうか?
プロジェクトのトリガーの設定から、時間主導>分タイマー　で実行しています。

Answer 1

filesearch(TARGET_FOLDER_ID); // // // // //
ここじゃないですかね。

DriveApp.getFolderById(TARGET_FOLDER_ID);を関数の外にだすと、グローバルのvar TARGET_FOLDER_IDで解決されて正常に動作します。
これを関数の中に入れるとfunction filesearch(TARGET_FOLDER_ID) {の仮引数でマスクされて、グローバルを指さなくなります。
最初に書いたところは質問者様は、実引数なし()にしているので関数内部でTARGET_FOLDER_IDはundefined。そのようなidのフォルダがないのでエラー。

var TARGET_FOLDER_ID = "xxx";
var UPDATE_SHEET_ID = "xxxx";
var UPDATE_SHEET_NAME = "sheet";
var SEND_MAIL_ADDRESS = ["xxx@xxx"]

function updateCheck() {
    // 対象フォルダ配下のファイル最終更新日時を取得。
    var lastUpdateMap = {};

    function filesearch(TARGET_FOLDER_ID) {
        var targetFolder = DriveApp.getFolderById(TARGET_FOLDER_ID);
        var files = targetFolder.getFiles();
        while (files.hasNext()) {
            var file = files.next();
            lastUpdateMap[file.getName()] = file.getDateCreated();
        }
        var folders = DriveApp.searchFolders("'" + TARGET_FOLDER_ID + "' in parents");
        while (folders.hasNext()) {
            var folder = folders.next();
            lastUpdateMap[folder.getName()] = folder.getDateCreated();
            filesearch(folder.getId());
        }
    }

    filesearch(TARGET_FOLDER_ID); // // // // //

    // スプレッドシートに記載されているファイル名と更新日時を取得。
    var spreadsheet = SpreadsheetApp.openById(UPDATE_SHEET_ID);
    var sheet = spreadsheet.getSheetByName(UPDATE_SHEET_NAME);
    var data = sheet.getDataRange().getValues();

    // 取得したデータをMapに変換。
    var sheetData = {};
    // 説明とヘッダを記載しているため2からstart。
    for (var i = 2; i < data.length; i++) {
        sheetData[data[i][0]] = {name : data[i][0], lastUpdate : data[i][1], rowNo : i + 1};
    }

    // 実際のファイルとスプレッドシート情報を比較。
    var updateFolderList = [];
    for (key in lastUpdateMap) {
        if(key in sheetData) {
            // ファイル名がシートに存在する場合。
            if(lastUpdateMap[key] > sheetData[key].lastUpdate) {
                // フォルダが更新されている場合。
                sheet.getRange(sheetData[key].rowNo, 3).setValue(lastUpdateMap[key]);
                updateFolderList.push(key);
            }
        } else {
            // ファイル名がシートに存在しない場合。
            sheet.getRange(sheet.getLastRow() + 1, 1).setValue(key);
            sheet.getRange(sheet.getLastRow(), 3).setValue(lastUpdateMap[key]);
            updateFolderList.push(key);
        }
    }

// 新規及び更新された情報をメール送信。
    if (updateFolderList.length != 0) {
        SEND_MAIL_ADDRESS.forEach(function(o,i) {
            MailApp.sendEmail(SEND_MAIL_ADDRESS[i],"追加通知",
                "写真が追加されました。" + "\n" +
                "追加されたファイルは以下の通りです。" + "\n" +
                updateFolderList.join("\n") + "\n\n"
            );
        });
    }
}

Comments

[efuji]

ありがとうございます！この場合、実行時に引数渡さなきゃですね。無事動作しました。

Answer 2

lastUpdateMapに配下のすべてのフォルダとファイルの情報が入るように再帰的に書いて見ました。

var lastUpdateMap = {};

function filesearch(TARGET_FOLDER_ID) {  
  var files = DriveApp.searchFiles("'" + TARGET_FOLDER_ID + "' in parents");
  while (files.hasNext()) {
    var file = files.next();
    lastUpdateMap[file.getName()] = file.getDateCreated();
  }
  var folders = DriveApp.searchFolders("'" + TARGET_FOLDER_ID + "' in parents");
  while (folders.hasNext()) {
    var folder = folders.next();
　　 lastUpdateMap[folder.getName()] = folder.getDateCreated();
    filesearch(folder.getId());
  }  
}

Comments

[efuji]

ありがとうございます！
しかし、filesearch() 関数を走らせると
"while (files.hasNext()) {"に対して、「指定された ID のアイテムは見つからなかったか、アクセスする権限がありません。」というエラーが出てしまいました。

[hiroshi0240]

フォルダの中にファイルが存在しないものがあるとそうなっちゃいますね。その場合は、filesイテレータにデータがない時のエラー処理を追記すれば良いと思います。

[efuji]

なるほど。確かにそうですね。
しかし、空のフォルダがない状態でもやはり同じエラーとなってしまいました。

[hiroshi0240]

一応確認です。filesearch() 関数の()内のパラメータに使用するフォルダのID
var TARGET_FOLDER_ID = "xxxxx";
は正しく（自分の権限で閲覧可能なフォルダのIDが）入っていますよね。

[efuji]

はい、IDに間違いはありませんでした。
再度、
while (files.hasNext()) {
var file = files.next();
lastUpdateMap[file.getName()] = file.getDateCreated();
}
で動作させると、エラーなく、指定フォルダ直下のファイル名・作成日は取得できました。
教えていただいた関数にするとエラーが出てしまいます。

[hiroshi0240]

では、
var files = DriveApp.searchFiles("'" + TARGET_FOLDER_ID + "' in parents");の部分を
var targetFolder = DriveApp.getFolderById(TARGET_FOLDER_ID);
var files = targetFolder.getFiles();に置き換えて見てください。

[efuji]

たびたび、ありがとうございます。
DriveApp.をfilesearch(){}内で行うとエラーが出るようです。なぜでしょうね。。

filesearch()の外で、var targetFolder = DriveApp.getFolderById(TARGET_FOLDER_ID);、
filesearch()内で
var files =targetFolder.getFiles();
while...
.....
var folders = targetFolder.getFolders();
...
としたところ、エラーは出なかったのですが処理が完了せずタイムアウトとなりました。
今のところ、指定フォルダ内にはフォルダ数は3つ、それぞれの内部に3~5のファイルがあるだけです。

var files = DriveApp.searchFiles("'" + TARGET_FOLDER_ID + "' in parents");などもfilesearch()の外で宣言するとエラーなく実行できましたが、これでは指定フォルダ直下の内容しか取得できませんよね（念のため.logも見てみましたが直下の内容のみでした）。

[hiroshi0240]

私のお渡ししたものは、再帰的にFolderIDから対象ファイルを読み込むので、そとでフォルダーを定義すると同じ処理がループしてタイムアウトになるはずです。fileserch()関数のカッコ内のパラメータでフォルダのIDを都度受け取って対象フォルダが可変できる形で直してください。

[efuji]

あぁ、そうですね。わたしの理解不足で申し訳ありません。
function filesearch(TARGET_FOLDER_ID) {
var targetFolder = DriveApp.getFolderById(TARGET_FOLDER_ID);
var files = targetFolder.getFiles();
でもやはりtargetFolderの行でエラーとなってしまいます。

なぜfilesearch()内ではエラーとなってしまうのでしょう。

[hiroshi0240]

他の回答者さんからも回答もらえるように、一旦ここまで書かれたコード全体をアップしてみて下さい。現状コメントからでは問題箇所が見えません

[efuji]

アドバイス、本当にありがとうございます。
現在のコードに更新しました。

[hiroshi0240]

わかりました。すみません。自分で書いておきながら引数を渡す必要が有ることを失念していました。
ファンクション名で引数を2つもらうように「function filesearch(lastUpdateMap,TARGET_FOLDER_ID)」として実行呼び出しの「filesearch(folder.getId())」部分も、「filesearch(lastUpdateMap,folder.getId())」としてください。同様にご自身で記載されている。「filesearch()」部分も「filesearch(lastUpdateMap,TARGET_FOLDER_ID)」と記載してください。

[efuji]

ありがとうございます！実行呼び出し側から引数を渡さなきゃですね。無事動作しました。papinianus様の回答も非常に助かりましたが、hiroshi0240様には長い間、丁寧にご対応いただきましたので、ベストアンサーに選ばせていただきました。