我正在尝试为每个群组获取用户。然后获得有关用户的更多详细信息。
有人能看出为什么以下不起作用?
do
{
$Group=@('"Group1"'
'"Group2"',
'"Group3"',
'"Group4"',
'"Group5"')
try{
$Group | ForEach{
ForEach ($member in (Get-ADGroupMember $_ ))
{
$user = Get-ADUser $member -Properties *
$useracc = $user.samaccountname
$userid = $user.EmployeeID
$userout = $useracc + " - " + $userid
write-output $userout | out-file -filepath c:\temp\New.txt -Append -NoClobber
}
}
}
Catch
{
write-output $userout | out-file -filepath c:\temp\error.txt }
}
until ($memeber -eq $Null)
答案 0 :(得分:1)
好的,让我们开始吧,你的Do{}Until()
循环毫无意义,至少在你提供的示例代码中是这样。接下来,您的Try/Catch
应该不那么全球化了。我避免在ForEach
循环中放置ForEach-Object
循环,因为它确实难以跟踪事物,但这就是我。
那么,你有什么功能。我更换了"' Group1'"使用我所属的域中的组的可分辨名称,按原样运行代码,并输出预期的文件。没有理由它不适用于多个组。它有点慢,但我将其归因于脚本的编写方式。我重写了一下,这应该做同样的事情,但更快,并且稍微好一点的错误处理:
$Groups=@('"Group1"'
'"Group2"',
'"Group3"',
'"Group4"',
'"Group5"')
ForEach($Group in $Groups){
Try{
Get-ADGroupMember $Group | Where {$_.objectClass -eq 'user' } | Select -expand distinguishedName | Get-ADUser -Properties EmployeeID | ForEach {
"{0}`t-`t{1}" -f $_.samaccountname, $_.employeeid | Add-Content C:\Temp\New.txt
}
}
Catch{
"Error getting members for $Group" | Add-Content C:\Temp\Errors.txt
}
}