PowerShell自定义ScriptMethod调用失败问题求助

一、问题根源分析 #

你遇到的错误核心原因有两个：

1. .Where()方法返回的是集合而非单个对象
   PowerShell的.Where()实例方法默认返回的是System.Collections.ObjectModel.Collection<PSObject>类型的集合——哪怕只匹配到一个元素。这个集合本身并没有你添加的IncrementOrderCount脚本方法，只有集合内的单个PSCustomObject实例才有，所以直接在集合上调用方法会触发“方法未找到”的错误。

2. 属性名不匹配的潜在bug
   你的函数里用了$_.Name -eq $ProductName作为匹配条件，但调用命令里指定的是-ProductName "Citrix CAL"，而从JSON导入的对象属性是Product（看你手动测试的代码）。这会导致要么找不到匹配项，要么如果对象有Name属性的话匹配错误，虽然不是直接引发当前错误的原因，但也是需要修复的问题。

至于手动创建对象时能正常运行，大概率是你手动测试时直接操作了单个对象（比如$PurchaseOrder[0].IncrementOrderCount(5)），而非通过.Where()返回的集合调用方法，所以没触发这个问题。

二、修复方案 #

针对核心问题，我们可以通过两种方式修复函数中的方法调用逻辑：

方案1：遍历匹配到的每个对象调用方法

把集合中的每个对象单独取出来，逐个调用IncrementOrderCount：

function Update-PurchaseOrder{
    [Cmdletbinding()]
    param(
        [ref]$PurchaseOrder,
        [string]$ProductName,
        [int]$Count
    )
    # 修复属性名：把Name改成Product，和调用参数对应
    $PurchaseOrder.Value.Where({$_.Product -eq $ProductName}) | ForEach-Object {
        $_.IncrementOrderCount($Count)
    }
}

方案2：使用.Where()的First模式获取单个对象

如果你确定每个产品名唯一，可以用.Where()的第二个参数指定只返回第一个匹配项（单个对象而非集合）：

function Update-PurchaseOrder{
    [Cmdletbinding()]
    param(
        [ref]$PurchaseOrder,
        [string]$ProductName,
        [int]$Count
    )
    $matchingItem = $PurchaseOrder.Value.Where({$_.Product -eq $ProductName}, 'First')
    if ($matchingItem) {
        $matchingItem.IncrementOrderCount($Count)
    } else {
        Write-Warning "No product found with name: $ProductName"
    }
}

三、代码优化建议 #

除了修复错误，还有几个可以提升代码效率和符合PowerShell风格的优化点：

1. 避免用+=构建数组
   +=会每次创建新数组，数据量大时效率很低，改用foreach循环直接输出收集结果：

# 替换原来的$PurchaseOrder构建逻辑
$PurchaseOrder = foreach($_price in ($Prices.prices | Where-Object { $_."Part Number" })){
    $PurchaseOrderItem = $_price.psobject.copy()
    $PurchaseOrderItem | Add-Member -Name "OrderCount" -MemberType NoteProperty -Value 0
    $PurchaseOrderItem | Add-Member -MemberType ScriptMethod -Name IncrementOrderCount {
        param([int]$amount)
        # 简化赋值逻辑
        $this.OrderCount += $amount
    }
    # 直接输出对象，PowerShell会自动收集到数组中
    $PurchaseOrderItem
}

2. 去掉不必要的[ref]参数
   PowerShell中PSCustomObject是引用类型，修改对象的属性不需要通过[ref]传递数组，直接传入数组即可，代码更简洁：

function Update-PurchaseOrder{
    [Cmdletbinding()]
    param(
        [Parameter(Mandatory=$true)]
        [PSObject[]]$PurchaseOrder,
        [Parameter(Mandatory=$true)]
        [string]$ProductName,
        [Parameter(Mandatory=$true)]
        [int]$Count
    )
    $matchingItems = $PurchaseOrder.Where({$_.Product -eq $ProductName})
    if (-not $matchingItems) {
        Write-Warning "No product found with name: $ProductName"
        return
    }
    foreach ($item in $matchingItems) {
        $item.IncrementOrderCount($Count)
    }
}

# 调用时不需要[ref]
Update-PurchaseOrder -PurchaseOrder $PurchaseOrder -ProductName "Citrix CAL" -Count 5

内容的提问来源于stack exchange，提问作者Johan de Haan