Google Script功能正常但调试报错，寻求代码精简优化方案

错误原因分析 #

1. 调试参数问题：直接运行fileToSheet函数时未传入有效的文件ID，导致DriveApp.getFileById(fileId)因参数无效抛出错误。调试时需传入正确的文件ID，或直接运行convertFiles并传入目标文件夹ID。
2. 无效的长度调用：Logger.log(files.length)是错误写法，folder.getFiles()返回的是FileIterator对象，没有length属性，无法获取文件数量。

优化后的代码 #

function fileToSheet(file) {
  const datePrefix = Utilities.formatDate(new Date(), "IST", "yyyy.mm.dd");
  const fileInfo = {
    title: `(${datePrefix}) ${file.getName()}`,
    mimeType: MimeType.GOOGLE_SHEETS,
    parents: [{ id: file.getParents().next().getId() }]
  };

  const fileMimeType = file.getMimeType();
  const allowedTypes = [
    MimeType.CSV,
    MimeType.MICROSOFT_EXCEL,
    MimeType.MICROSOFT_EXCEL_LEGACY
  ];

  if (allowedTypes.includes(fileMimeType)) {
    try {
      const spreadsheet = Drive.Files.insert(fileInfo, file.getBlob());
      Logger.log(`已转换文件ID: ${spreadsheet.id}`);
      return spreadsheet.id;
    } catch (err) {
      Logger.log(`转换文件失败 ${file.getName()}: ${err.message}`);
    }
  }
}

function convertFiles(folderId) {
  const folder = DriveApp.getFolderById(folderId);
  const files = folder.getFiles();
  let convertedCount = 0;

  while (files.hasNext()) {
    const file = files.next();
    if (fileToSheet(file)) convertedCount++;
  }

  Logger.log(`共完成 ${convertedCount} 个文件转换`);
}

代码优化说明 #

• 减少API调用：直接传递File对象给fileToSheet，避免重复调用DriveApp.getFileById，提升执行效率。
• 简化类型判断：用数组allowedTypes结合includes方法替代多条件判断，代码更易读且便于后续添加支持的文件类型。
• 错误容错处理：添加try-catch捕获单个文件转换的异常，确保一个文件转换失败不会中断整个批量任务。
• 修复计数逻辑：通过convertedCount统计成功转换的文件数量，替代无效的files.length调用，输出准确的转换结果。
• 简化字符串拼接：使用模板字符串简化文件名前缀拼接，代码更简洁直观。
• 增强日志信息：优化日志内容，添加文件名和错误详情，便于排查问题。

内容的提问来源于stack exchange，提问作者user20485973