PowerShell中continue语句失效问题：过滤禁用AD用户异常

以下是几种可能导致禁用账户未被跳过、邮件仍发送的原因及对应修复方案：

1. continue的循环作用域不匹配 #

如果你的脚本使用ForEach-Object（管道式遍历）而非foreach语句，continue的行为会不符合预期：

• foreach语句中，continue会直接跳过当前迭代的剩余代码；
• ForEach-Object中，continue等价于return，仅跳过当前管道对象的后续处理，但如果代码嵌套错误（比如邮件操作不在脚本块内），会导致跳过失效。

修复示例：
改用foreach语句确保continue行为正确：

$expiredUsers = Get-ADUser -Filter {AccountExpirationDate -lt (Get-Date)} -Properties Enabled, EmailAddress
foreach ($user in $expiredUsers) {
    if (-not $user.Enabled) {
        continue
    }
    Disable-ADAccount -Identity $user
    Revoke-ADUserAllSession -Identity $user
    Send-MailMessage -To $user.EmailAddress -Subject "账户已过期并禁用" -Body "你的AD账户已因过期被禁用，请联系管理员" ...
}

2. Enabled属性判断时机错误（未刷新对象） #

如果脚本先执行Disable-ADAccount，再判断$user.Enabled，此时$user是初始获取的AD对象，属性值未更新（仍为禁用前的$true），导致continue无法触发，邮件依然发送。

修复示例：
调整顺序，先判断状态再执行操作：

foreach ($user in $expiredUsers) {
    if (-not $user.Enabled) {
        continue
    }
    # 先确认是启用状态，再执行禁用、注销
    Disable-ADAccount $user
    Revoke-ADUserAllSession $user
    Send-MailMessage ...
}

若必须先操作再判断，需重新获取用户最新状态：

foreach ($user in $expiredUsers) {
    Disable-ADAccount $user
    Revoke-ADUserAllSession $user
    # 重新拉取用户对象以获取最新Enabled状态
    $updatedUser = Get-ADUser -Identity $user.SamAccountName -Properties Enabled
    if (-not $updatedUser.Enabled) {
        continue
    }
    Send-MailMessage ...
}

3. Enabled属性获取或判断逻辑错误 #

• 若使用Get-ADUser时通过-Properties指定了自定义属性但未包含Enabled，会导致$user.Enabled为$null，此时-not $user.Enabled的判断结果不符合预期（-not $null为$false，不会触发continue）；
• 若判断逻辑写反（比如if ($user.Enabled) { continue }），会错误跳过启用账户，反而处理禁用账户。

修复示例：
显式指定获取Enabled属性，确保判断逻辑正确：

$expiredUsers = Get-ADUser -Filter {AccountExpirationDate -lt (Get-Date)} -Properties Enabled, EmailAddress
foreach ($user in $expiredUsers) {
    # 明确判断账户是否已禁用
    if ($user.Enabled -eq $false) {
        continue
    }
    # 后续操作
}

4. 邮件操作不在循环迭代范围内 #

如果邮件操作写在循环外部（比如遍历完所有用户后统一发送），无论循环内的continue是否触发，邮件都会执行。需确保邮件操作是循环内针对单个用户的逻辑。

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