PowerShell函数返回True而非False问题求助

问题描述 #

编写了Test-ValidDatapack函数，用于检查指定文件夹是否存在pack.mcmeta文件和data目录，任一不存在则返回$false。但哪怕把函数改成直接返回$false，调用它的if语句依然会执行后续逻辑，无法正确获取返回值。

调用代码 #

if (Test-Path $datapackPath) {
    if (Test-ValidDatapack($datapackPath)) {
        $files = foreach ($file in Get-ChildItem -Path $datapackPath -Force -Recurse) {
            $file.FullName
        }
        return $files  
    }  
}

待排查函数代码 #

function Test-ValidDatapack {
    #Parameter for the filepath
    [Parameter(Mandatory=$true)]
    [String]$datapackPath

    $valid = $true #Return variable
    $files = Get-ChildItem $datapackPath #Collects filenames included in datapack
    if ($files.Name -notcontains "pack.mcmeta") { #Checks datapack for 'pack.mcmeta'
        Write-Host "No 'pack.mcmeta' file found. Invalid datapack." -ForegroundColor Red
        $valid = $false
    }
    if ($files.Name -notcontains "data") { #Checks datapack for 'data'
        Write-Host "No 'data' directory found. Invalid datapack." -ForegroundColor Red
        $valid = $false
    }    
    return $valid
}

问题原因及解决 #

1. 函数调用语法错误（核心问题） #

PowerShell中调用函数时，不能用括号包裹单个参数，Test-ValidDatapack($datapackPath)这种写法会把$datapackPath当成一个数组传递给函数，导致函数的返回值在if判断中被误判为$true（非空数组在布尔判断中默认视为真）。

正确的调用方式是用空格分隔参数：

if (Test-ValidDatapack $datapackPath) {
    # 后续逻辑
}

2. 函数内部检查逻辑的潜在问题 #

原函数用Get-ChildItem获取目录内容再判断，存在以下隐患：

• 如果目标目录为空，$files会是$null，访问$files.Name会抛出错误；
• Get-ChildItem默认不显示隐藏文件，若pack.mcmeta是隐藏文件则会漏判；
• -notcontains对数组的判断逻辑在部分场景下不符合预期（比如目录中有重名项时）。

更可靠的检查方式是直接用Test-Path分别验证文件/目录类型：

function Test-ValidDatapack {
    [Parameter(Mandatory=$true)]
    [String]$datapackPath

    $valid = $true

    # 检查pack.mcmeta文件
    $mcmetaPath = Join-Path -Path $datapackPath -ChildPath "pack.mcmeta"
    if (-not (Test-Path -Path $mcmetaPath -PathType Leaf)) {
        Write-Host "No 'pack.mcmeta' file found. Invalid datapack." -ForegroundColor Red
        $valid = $false
    }

    # 检查data目录
    $dataDirPath = Join-Path -Path $datapackPath -ChildPath "data"
    if (-not (Test-Path -Path $dataDirPath -PathType Container)) {
        Write-Host "No 'data' directory found. Invalid datapack." -ForegroundColor Red
        $valid = $false
    }

    return $valid
}

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