PowerShell脚本执行后意外移动并重命名文件夹问题排查

你的脚本之所以会意外移动并重命名原文件夹，核心原因有两个：

1. 循环次数与剩余文件数量不匹配 #

你一开始就获取了$FileCount作为总文件数，但在循环前已经移走了第一个文件，剩余文件数实际是$FileCount - 1，但你的循环还是按原总数跑了$FileCount次。当最后一次循环时：

• Get-ChildItem -Path $path -Force -File | Select-Object -First 1会返回$null（所有文件已被移走）
• 字符串拼接后，"C:\Users\User1\Desktop\MergeTest\_First\$FileInWork"直接变成了原文件夹的完整路径C:\Users\User1\Desktop\MergeTest\_First\
• 此时Move-Item会把整个原文件夹移动到目标路径，并将其重命名为$Counter.xlsx

2. 错误的路径拼接方式 #

你直接将FileInfo对象（比如$FirstFile、$FileInWork）嵌入字符串构建源路径，这种方式不可靠：

• 当对象为$null时，会直接指向原文件夹路径
• 如果文件名包含空格或特殊字符，还可能引发路径解析错误
  正确的做法是使用对象的FullName属性获取完整文件路径

这里是修正后的脚本，解决了上述问题：

$path = "C:\Users\User1\Desktop\MergeTest\_First\"
# 一次性获取所有文件集合，避免重复调用Get-ChildItem
$allFiles = Get-ChildItem -Path $path -Force -File

if ($allFiles.Count -eq 0) {
    Write-Host "目标文件夹中没有文件，退出脚本"
    exit
}

$FinalReport = "C:\Users\User1\Desktop\MergeTest\___Final\TestOutput.xlsx"
# 移动第一个文件，使用FullName确保路径正确
Move-Item -Path $allFiles[0].FullName -Destination $FinalReport

# 循环处理剩余文件，次数为总文件数-1
for($Counter = 0; $Counter -lt ($allFiles.Count - 1); $Counter++) {
    # 直接使用集合中的后续文件，无需重复查询
    $FileInWork = $allFiles[$Counter + 1]
    # 使用Join-Path构建目标路径，避免手动拼接错误
    $destinationPath = Join-Path -Path "C:\Users\User1\Desktop\MergeTest\__Second\" -ChildPath "$Counter.xlsx"
    Move-Item -Path $FileInWork.FullName -Destination $destinationPath
    Write-Host "已移动文件: $($FileInWork.Name)"
}

• 一次性获取所有文件集合，提升脚本运行效率
• 添加空文件夹判断，避免无文件时触发错误逻辑
• 使用Join-Path构建路径，彻底规避手动拼接的潜在问题
• 输出更清晰的日志信息，方便调试排查

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