使用PowerShell通过两个TXT文件更新AD用户manager属性遇阻

你的脚本核心问题是嵌套循环效率低下，且未正确实现经理samaccountname的查找逻辑；另外用Contains判断ID匹配容易出现误判（比如ID 111会被ID 1111错误匹配），Substring固定长度截取也不够灵活。下面是优化后的解决方案：

步骤1：将员工信息转为哈希表（实现快速查找） #

先把Staff.txt里的员工ID与对应samaccountname存入哈希表，后续查找时无需反复遍历整个文件，效率大幅提升：

# 读取员工文件，构建ID到samaccountname的映射哈希表
$staffMap = @{}
Get-Content -Path 'Staff.txt' | ForEach-Object {
    $fields = $_.Split("`t", [System.StringSplitOptions]::RemoveEmptyEntries)
    if ($fields.Count -ge 4) {
        $staffID = $fields[0].Trim()
        $samAccountName = $fields[3].Trim()
        $staffMap[$staffID] = $samAccountName
    }
}

步骤2：遍历主管关系文件，批量更新AD用户 #

处理Supervisors.txt，每行拆分出员工ID和经理ID，从哈希表快速获取对应samaccountname后更新AD属性：

# 读取主管关系文件，逐个处理员工-经理配对
Get-Content -Path 'Supervisors.txt' | ForEach-Object {
    $line = $_.Trim()
    if ([string]::IsNullOrEmpty($line)) { return }
    
    # 拆分员工ID和经理ID（自动忽略多余空格）
    $ids = $line.Split(" ", [System.StringSplitOptions]::RemoveEmptyEntries)
    if ($ids.Count -ne 2) { return }
    
    $employeeID = $ids[0].Trim()
    $managerID = $ids[1].Trim()
    
    # 若员工或经理ID不在员工表中，直接跳过
    if (-not $staffMap.ContainsKey($employeeID) -or -not $staffMap.ContainsKey($managerID)) {
        Write-Warning "员工ID $employeeID 或经理ID $managerID 在Staff.txt中不存在，跳过处理"
        return
    }
    
    $empSam = $staffMap[$employeeID]
    $manSam = $staffMap[$managerID]
    
    # 获取AD用户对象，存在则更新manager属性
    $adUser = Get-ADUser -Filter "samaccountname -eq '$empSam'" -ErrorAction SilentlyContinue
    $adManager = Get-ADUser -Filter "samaccountname -eq '$manSam'" -ErrorAction SilentlyContinue
    
    if ($adUser -and $adManager) {
        $adUser | Set-ADUser -Manager $adManager.DistinguishedName
        Write-Host "已更新用户 $empSam 的经理为 $manSam"
    } else {
        Write-Warning "AD中未找到用户 $empSam 或经理 $manSam，跳过处理"
    }
}

关键优化点说明 #

• 哈希表映射：将查找时间复杂度从O(n)降至O(1)，处理大量数据时效率提升明显
• 灵活字段拆分：用Split结合RemoveEmptyEntries兼容空格/制表符分隔的内容，避免固定长度截取的局限性
• 精准ID匹配：通过哈希表ContainsKey实现严格ID匹配，规避Contains方法的误判问题
• 完善错误处理：增加跳过逻辑和警告提示，便于排查不存在的ID或AD用户
• 可靠属性赋值：设置-Manager时直接使用AD对象的DistinguishedName，比传递对象更稳定

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