表单无法将值传递至AD查询命令的问题排查

你当前的代码存在变量命名冲突、Filter写法不够安全的问题，以下是修正后的实现方案：

核心问题说明 #

• 原代码中$PUID既存储文本框输入值，又存储查询结果，易导致逻辑混淆
• 直接在Filter中拼接字符串存在注入风险，若输入包含单引号会导致命令报错

修正后的代码 #

# 模块导入移到函数外，只需加载一次
Import-Module $scriptPath\AD_Module.psm1

Function Query_PUID{
    # 单独存储文本框输入值，避免变量冲突
    $inputPUID = $TxtBoxInput.Text.Trim()
    
    # 使用变量绑定的Filter写法，避免注入问题
    $userResult = Get-ADUser -Filter "SamAccountName -eq `$inputPUID" | Select-Object -ExpandProperty Name
    
    # 处理查询为空的情况
    if ($userResult) {
        $TxtBoxOutput.Text = $userResult -join "`r`n"
    } else {
        $TxtBoxOutput.Text = "未找到匹配的用户"
    }
}

关键优化点 #

• 将模块导入放在函数外部，避免每次调用函数都重复加载模块
• 用$inputPUID单独存储文本框输入，和查询结果变量$userResult区分开，逻辑更清晰
• 使用-ExpandProperty Name替代ForEach-Object，更简洁高效
• 添加空结果判断，给用户明确的反馈
• Filter中用变量绑定写法（注意反引号转义），避免因输入特殊字符导致命令失败

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