PowerShell脚本逻辑问题求助：所有设备误报离线

核心问题分析 #

脚本出现所有设备被标记为离线的情况，主要存在两个关键逻辑漏洞：

1. WMI调用未捕获异常，导致在线设备误判为离线 #

当设备能Ping通（Test-Connection返回$true）但WMI服务无法访问时（比如权限不足、防火墙拦截WMI端口、远程WMI未启用），Get-WmiObject会抛出终止性错误，直接跳出当前if(Test-Connection)代码块，进入else分支，错误标记设备为离线。

2. 在线设备的登录状态判断逻辑错误 #

原脚本中，判断无用户登录的if块执行后，没有对应的else分支，而是直接执行后续的Write-Output 'Computer in Use.'，导致无论是否有用户登录，都会输出"Computer in Use."，完全不符合预期逻辑。

修复后的脚本代码 #

Function Get-RemoteLogonStatus{
    ForEach ($line in Get-Content C:\ADComputers.csv) {
        $ComputerName = $line.Trim()  # 去除可能存在的空格/换行符
        if(Test-Connection -ComputerName $ComputerName -Count 2 -Quiet -ErrorAction SilentlyContinue){
            try {
                $computerSystem = Get-WmiObject -Class Win32_ComputerSystem -ComputerName $ComputerName -ErrorAction Stop
                if($computerSystem.username -eq $null) {
                    Write-Output "No user logged in. RESTART COMPUTER - $ComputerName"
                    # Shutdown /r /t 0 /M \\$ComputerName
                }
                else {
                    Write-Output "Computer in Use. - $ComputerName"
                }
            }
            catch {
                Write-Output "Online but WMI access failed. - $ComputerName"
            }
        }
        else {
            Write-Output "Computer OFFLINE. - $ComputerName"
        }
    }
}

修复说明 #

• 添加异常捕获：用try/catch包裹Get-WmiObject调用，当WMI访问失败时，输出明确的错误信息，避免误判为离线。
• 修正登录状态判断逻辑：给无用户登录的if块添加对应的else分支，确保两种状态分别输出正确内容。
• 添加字符串清理：用.Trim()去除CSV行中可能存在的空格、换行符，避免因无效的计算机名导致连接失败。
• 统一变量名：确保$ComputerName变量名大小写一致（虽然PowerShell大小写不敏感，但统一命名可避免潜在问题）。

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