PowerShell foreach命令报错排查及VM报告脚本优化求助

一、错误原因定位 #

报错"Unexpected token 'foreach' in expression or statement"的直接原因是语法格式错误：
脚本第一行中，数组初始化语句$report = @()和foreach循环之间没有空格，PowerShell解析时会将@()foreach识别为一个无效的语法 token，导致无法正确解析语句。

只需在@()后添加空格，就能解决这个语法错误：

$report = @()
foreach ($vm in Get-View -ViewType Virtualmachine){

二、脚本优化建议 #

原脚本存在性能瓶颈（循环内重复调用Get-View）、数据处理不严谨（如多IP/多存储未处理）、低效数组累加等问题，以下是优化后的版本及优化点说明：

优化后的脚本 #

# 提前批量获取所需的Host和Network信息，避免循环内重复调用Get-View
$allHosts = Get-View -ViewType HostSystem -Property Name
$allNetworks = Get-View -ViewType Network -Property Name

# 直接通过管道收集结果，避免低效的数组累加
$report = foreach ($vm in Get-View -ViewType Virtualmachine -Property Name,Guest,Config,Runtime,Summary,ResourceConfig) {
    # 使用[PSCustomObject]直接创建对象，比Select-Object更高效直观
    [PSCustomObject]@{
        VMName              = $vm.Name
        Hostname            = $vm.Guest.Hostname
        IPAddress           = $vm.Guest.IPAddress -join ', '  # 处理多IP场景
        OS                  = $vm.Config.GuestFullName
        Boottime            = $vm.Runtime.BootTime
        VMState             = $vm.Summary.Runtime.PowerState
        TotalCPU            = $vm.Summary.Config.NumCpu
        CPUAffinity         = $vm.Config.CpuAffinity
        CPUHotAdd           = $vm.Config.CpuHotAddEnabled
        CPUShare            = $vm.Config.CpuAllocation.Shares.Level
        CPUReservation      = $vm.ResourceConfig.CpuAllocation.Reservation
        CPULimit            = $vm.ResourceConfig.CpuAllocation.Limit
        OverallCpuUsage     = $vm.Summary.QuickStats.OverallCpuUsage
        TotalMemory         = $vm.Summary.Config.MemorySizeMB
        MemoryHotAdd        = $vm.Config.MemoryHotAddEnabled
        MemoryShare         = $vm.Config.MemoryAllocation.Shares.Level
        MemoryReservation   = $vm.ResourceConfig.MemoryAllocation.Reservation
        MemoryLimit         = $vm.ResourceConfig.MemoryAllocation.Limit
        MemoryUsage         = $vm.Summary.QuickStats.GuestMemoryUsage
        Swapped             = $vm.Summary.QuickStats.SwappedMemory
        Ballooned           = $vm.Summary.QuickStats.BalloonedMemory
        Compressed          = $vm.Summary.QuickStats.CompressedMemory
        TotalNics           = $vm.Summary.Config.NumEthernetCards
        ToolsStatus         = $vm.Guest.ToolsStatus
        ToolsVersion        = $vm.Config.Tools.ToolsVersion
        HardwareVersion     = $vm.Config.Version
        TimeSync            = $vm.Config.Tools.SyncTimeWithHost
        CBT                 = $vm.Config.ChangeTrackingEnabled
        Portgroup           = $allNetworks | Where-Object { $_.MoRef -in $vm.Network } | Select-Object -ExpandProperty Name -Join ', '
        VMHost              = $allHosts | Where-Object { $_.MoRef -eq $vm.Runtime.Host } | Select-Object -ExpandProperty Name
        Datastore           = $vm.Config.DatastoreUrl | Select-Object -ExpandProperty Name -Join ', '  # 处理多存储场景
        Notes               = $vm.Config.Annotation
        FaultTolerance      = $vm.Runtime.FaultToleranceState
    }
} | Sort-Object VMName

# 直接导出，无需判断数组是否为空
$report | Export-Csv -Path vm.csv -NoTypeInformation

核心优化点 #

• 语法修复：修正了初始的空格缺失问题。
• 性能提升：提前批量获取HostSystem和Network的信息，避免在循环内反复调用Get-View，大幅减少API请求次数。
• 数据严谨性：
  • 处理虚拟机多IP场景，用逗号分隔所有IP地址。
  • 处理虚拟机挂载多存储场景，输出所有关联存储名称。
• 效率优化：
  • 使用[PSCustomObject]直接创建对象，比"" | Select-Object更高效。
  • 通过foreach循环直接输出到管道收集结果，替代低效的$Report += $vms数组累加操作（数组累加每次都会创建新数组，数据量大时性能差异明显）。
• 代码可读性：统一变量命名大小写（如CPULimit、CPUReservation），添加注释说明关键逻辑。
• 冗余代码移除：去掉不必要的IF ($Report -ne "")判断，Export-Csv对空数组会生成合法的空CSV文件。

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