PowerShell脚本读取CSV文件循环两次致输出重复的问题排查

咱们先梳理下你代码里的核心问题，这些大概率就是导致循环重复执行、输出内容重复的根源：

1. WriteOutputFile函数的语法错误 #

你写的WriteOutputFile函数完全不符合PowerShell的函数定义规范，这会让PowerShell无法正确识别它的函数身份，进而引发各种意外行为，包括可能的重复执行。

错误写法：

WriteOutputFile($username, $pass, $destinationPath) {
 $createUserComand="New-LocalUser -Name ""$username"" -Password (ConvertTo-SecureString ""$pass"" -AsPlainText -Force) -Confirm"
 $createUserComand | out-file -filepath $destinationPath\output.ps1 -Append
}

修正后的正确定义：

function WriteOutputFile($username, $pass, $destinationPath) {
    # 顺便修正变量名拼写：createUserComand → createUserCommand
    $createUserCommand = "New-LocalUser -Name ""$username"" -Password (ConvertTo-SecureString ""$pass"" -AsPlainText -Force) -Confirm"
    $createUserCommand | Out-File -FilePath "$destinationPath\output.ps1" -Append
}

这里要注意两个关键点：

• 必须用function关键字开头声明函数
• 参数格式要遵循PowerShell规范：function 函数名($参数1, $参数2...)

2. 验证循环执行次数 #

为了确认是不是真的ForEach循环跑了两次，你可以在ReadCSV函数里加几行日志，直观查看读取的CSV行数和循环执行次数：

修改后的ReadCSV函数：

function ReadCSV($file_path, $delimeter, $UserInputUser, $UserInputPw){
    $data = Import-CSV -Path $file_path -Delimiter $delimeter
    # 打印读取到的CSV行数，确认是否符合预期
    Write-Host "从CSV读取到 $($data.Count) 条数据"
    $destinationPath = Find-Folders
    $loopCounter = 0
    ForEach($dataset in $data) {
        $loopCounter++
        Write-Host "正在处理第 $loopCounter 条数据"
        $username = $dataset.$UserInputUser
        $pass = $dataset.$UserInputPw
        
        # 原判断逻辑不适用于字符串，换成空值检查更准确
        if(-not [string]::IsNullOrEmpty($username) -and -not [string]::IsNullOrEmpty($pass)){
            WriteOutputFile -username $username -pass $pass -destinationPath $destinationPath
        }
    }
    #Run ReadOutputFile function
    ReadOutputFile -destinationPath $destinationPath
}

3. 修正判断逻辑的小细节 #

原代码里的($username -gt 0) -AND ($pass -gt 0)判断逻辑有问题：用户名和密码都是字符串类型，用-gt 0会触发PowerShell的隐式类型转换，可能导致非预期结果（比如空字符串会被当成0，"" -gt 0返回$false，但字符串"0"同样返回$false）。换成[string]::IsNullOrEmpty()来检查空值，更贴合业务需求。

最后验证 #

运行修改后的代码，查看控制台日志：

• 如果“从CSV读取到X条数据”和“正在处理第X条数据”的次数一致，说明循环只执行了一次，之前的重复输出大概率是函数定义错误导致的
• 如果循环次数确实是实际行数的两倍，那就要检查CSV文件是否存在重复行，或者Find-Folders函数是否返回了多个路径（不过看你的代码是把$destinationPath当成单个路径使用，这个可能性不大）

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