Google Apps Script问题：文件复制至源文件夹而非新建文件夹

嘿，我帮你找到了问题的两个核心原因，咱们一步步搞定它：

1. 文件复制到错误位置的关键原因 #

你现在代码里把newFolder赋值成了新文件夹的ID字符串：

var newFolder = targetFolder.createFolder(values[i]).getId();

然后调用file.makeCopy(newFolder)——这里踩了makeCopy()方法的重载陷阱：如果传入的是字符串，它会把这个字符串当成新文件名，而不是目标文件夹！这就导致所有文件都被复制到了源文件所在的文件夹（也就是sourceFiles），而不是你新建的文件夹里，这就是你看到的错误。

2. 只有第一个文件夹能拿到文件的问题 #

你把var files = sourceFiles.getFiles();放在了for循环外面，这个files是一个一次性的迭代器——第一次循环里的while循环会把所有文件都遍历完，后面的循环里files就没有内容了，所以只有第一个新建的文件夹会有文件，其他的都空着。

修复后的完整代码 #

下面是修正后的代码，我标注了所有修改点：

var ss = SpreadsheetApp.openByUrl('xxxxx').getSheetByName('xxxxx');
var countries = ss.getRange(2, 1, 50, 1);
var values = countries.getValues();
var sourceFiles = DriveApp.getFolderById('xxxxx'); //存放待复制文件的文件夹
var targetFolder = DriveApp.getFolderById('xxxxx'); //每个新文件夹的父文件夹

for (i = 0; i < values.length; i++){
  // 修改点1：保留新建文件夹的对象，而不是只取ID；同时修正二维数组取值
  var newFolder = targetFolder.createFolder(values[i][0]);
  
  // 修改点2：把获取文件迭代器的代码移到循环内部，每次循环重新获取文件列表
  var files = sourceFiles.getFiles();
  
  while(files.hasNext()) {
    var file = files.next();
    // 修改点3：用明确的重载方法，指定文件名和目标文件夹对象
    file.makeCopy(file.getName(), newFolder);
  }
}

关键修改说明 #

• 用文件夹对象代替ID：targetFolder.createFolder()返回的是文件夹对象，直接传给makeCopy就能正确指定目标位置，避免把ID当成文件名。
• 修正二维数组取值：getValues()返回的是二维数组，values[i]其实是一个包含单个元素的数组，所以要用values[i][0]来获取单元格的实际值，不然新建的文件夹名字会变成[xxx]这种奇怪的格式。
• 循环内重新获取文件：把获取文件列表的代码移到for循环内部，这样每次创建新文件夹时，都能重新拿到源文件夹的所有文件，保证每个新文件夹都能复制到完整的文件。
• 明确调用重载方法：用makeCopy(文件名, 目标文件夹)的写法，清晰告诉脚本要做什么，避免歧义。

替换成这段代码后，应该就能实现你想要的功能了——每个新建文件夹里都会复制到指定的文件，不会再错放到源文件夹里。

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