PowerShell导入CSV执行AD操作脚本仅对最后一项生效求排查

问题根因 #

脚本存在两个核心逻辑错误导致仅最后一个用户生效：

• 第一个foreach循环中，每次迭代都直接覆盖了$user变量，且没有将查询到的AD用户属性存储到集合中，循环结束后$user仅保留最后一次迭代的用户数据
• 第一个循环中的Select-Object没有绑定输入源也没有赋值给变量，属于无效执行，查询到的用户属性没有被留存

修复方案 #

不需要拆成两个独立循环，直接在一个循环里完成查询和删除操作即可，也可以先把所有查询到的用户存到独立的集合变量里再统一处理。

修复后代码示例 #

Import-Module ActiveDirectory

# 读取CSV用户列表
$userlist = Import-Csv "c:\delete-edu-accounts-test.csv" -Header samaccountname

# 遍历每个用户执行操作
ForEach ($userItem in $userlist){
    # 查询用户AD属性
    $adUser = Get-ADUser -Identity $userItem.samaccountname -Properties samaccountname, HomeDirectory, profilepath
    
    # 删除家目录
    if ($adUser.HomeDirectory -and (Test-Path $adUser.HomeDirectory)) {
        Remove-Item $adUser.HomeDirectory -Recurse -Force -ErrorAction SilentlyContinue
        Write-Host "已删除家目录: $($adUser.HomeDirectory)"
    }

    # 删除配置文件目录（需要时取消注释）
    <#
    if ($adUser.ProfilePath -and (Test-Path $adUser.ProfilePath)) {
        Remove-Item $adUser.ProfilePath -Recurse -Force -ErrorAction SilentlyContinue
        Write-Host "已删除配置文件目录: $($adUser.ProfilePath)`n"
    }
    #>

    # 删除AD用户
    Remove-ADUser -Identity $adUser.SamAccountName -Confirm:$false
    Write-Host "已删除AD用户: $($adUser.SamAccountName)"
}

额外优化建议 #

• 新增Test-Path判断路径存在性，避免路径不存在时报错终止执行
• 变量命名做了区分，避免循环变量覆盖导致的逻辑混乱
• 增加了错误忽略参数-ErrorAction SilentlyContinue，可以根据需要调整
• 用块注释包裹可选的配置文件删除逻辑，可读性更高

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