Question

我正在学习PowerShell为我的团队编写工具。我今天处于紧要关头，但是我希望通过删除ForEach循环中的IF语句来简化它，因为命令之间的唯一区别是参数 -replace 或 -remove

Write-Host "This script removes or replaces en masse users' msds-SyncServerURL property"

DO {
    $TargetSSU=Read-Host "`nWhat msds-SyncServerURL do you want to replace or remove?"
    $UsersToBeFixed=Get-ADUser -Filter {msds-SyncServerURL -like $TargetSSU} -Properties ('name','samaccountname','msDS-SyncServerURL')
    IF ($UsersToBeFixed -eq $null) {
        Write-Host "`nNo users appear to have $TargetSSU as their msds-SyncServerURL value. Please try again."
    }
} UNTIL ($UsersToBeFixed -ne $null)

Write-Host "`n`nThe following users have $TargetSSU as their msds-SyncServerURL value:"
$UsersToBeFixed |select name,samaccountname,msds-syncserverurl|ft

DO {
    $Action=Read-Host "Do you want to [R]emove or [U]pdate $TargetSSU?"
} Until (($Action -eq "R") -or ($Action -eq "U"))    
    IF ($Action -eq "U") {
        DO {
            $NewSyncServer=Read-Host "`nEnter the new Sync Server's hostname (not the URL)"
            Write-Host "`nChecking to see if $NewSyncServer has a userfile share..."
            $VerifySyncServer=Test-Path \\$NewSyncServer\userfiles
            IF ($VerifySyncServer -eq $false) {
                Write-host "`n$NewSyncServer does not appear to be a valid Sync Server hostname. Please try again."
            }
        } UNTIL ($VerifySyncServer -eq $true)
        $TargetSSU="https://$NewSyncServer.ourdomain.com"
    }

ForEach ($userToBeFixed in $UsersToBeFixed) {
    Write-Host "`nFixing" ($usertobefixed).name
    IF ($Action -eq "R") {
        Set-ADObject -identity $userToBeFixed -remove @{"msDS-SyncServerUrl" = $TargetSSU}
    }
    IF ($Action -eq "U") {
        Set-ADObject -identity $userToBeFixed -replace @{"msDS-SyncServerUrl" = $TargetSSU}
    }
}

Write-Host "`nHere is the result of the operation:"
foreach ($userToBeFixed in $UsersToBeFixed) {
    Get-ADUser -Identity $userToBeFixed -Properties ('name','samaccountname','msDS-SyncServerURL')|select name,samaccountname,msds-syncserverurl
}

我最初有以下开关，尝试各种引号排列，甚至{$ action =＆＃34;替换＆＃34;}：

Switch($action)
{
    R {'remove'}
    U {'replace'}
}

我还在ForEach循环中尝试了Invoke-Expression：

$CMD="Set-ADObject -identity $userToBeFixed -$action @{`"msDS-SyncServerUrl`" = $TargetSSU}"
Invoke-Expression -Command $CMD

Set-ADObject cmdlet总是会失败，通常会抱怨Set-ADObject无法找到接受＆＃39; -remove＆＃39;，＆＃39等参数的位置参数; System.Object的[]＆＃39;或者＆＃39; System.Collections.Hashtable＆＃39;。

我将问题隔离到Set-ADObject，而不是喜欢用作参数的 $ action 变量。如果我将 - $ action 替换为 -replace 或 -remove ，则可以使用（就像在上面的代码段中一样）。

我意识到这是一个小问题，但是让我感到困扰的是看似没有理由的冗余代码。我很想学习如何解决这个问题。

另外，无关紧要，我还没有找到更好的方法：

Until (($Action -eq "R") -or ($Action -eq "U"))

我搜索并尝试了其他解决方案，如：

Until ($Action -eq @{"R" -or "U"})

但似乎无法巩固对多种条件的评估。这让我感到困扰，但不像我的主要问题那么多。

请放轻松我。我对这整件事情都很陌生。如果有人看到任何其他我可以提高让我知道。我想学习这个。

感谢。

Answer 1

您可以使用splatting解决此问题。

例如：

$action = 'Recurse'
$params = @{ $action = $true }
Get-ChildItem @params

该示例在功能上等同于Get-ChildItem -Recurse

Answer 2

Persistent13's helpful answer向您展示如何使用splatting通过哈希表动态传递参数。

至于：

另外，无关，我还没有找到更好的方法来做到这一点：
  Until (($Action -eq "R") -or ($Action -eq "U"))

PowerShell提供数组包含运算符：-contains（PSV1 +，LHS上的数组）和-in（PSV3 +，RHS上的数组）：

# PSv3+
$Action -in 'R', 'U'

# Equivalent, PSv1+
'R', 'U' -contains $Action

两种形式都将标量操作数与数组操作数的每个元素进行比较（使用-eq逻辑），并在第一次匹配（如果有）时立即返回$True找到。

另一种选择是使用-match运算符和正则表达式：

$Action -match '^(R|U)$'

Answer 3

我喜欢将所有不涉及脚本逻辑（即函数）的内容移动到与PowerShell主脚本不同的文件中。我的大多数脚本都遵循这种结构：

Edit-SyncServerUrl.ps1

#requires -Version 5.1

<#
    Company header
#>

<#
.SYNOPSIS
Comment-based help
#>
[CmdletBinding(SupportsShouldProcess)]
Param([String]$Option)

. "$PSScriptRoot\scriptfunctions.ps1"

<# .. variables .. #>

Switch -Regex ($Option)
{
    '^rem' {Edit-SyncServerURL -Remove}
    '^rep' {Edit-SyncServerURL -Replace}
    Default {Write-Host "Invalid option passed: $Option"}
}

在您的具体示例中，我会将提示设置等的所有逻辑带入一个函数，该函数根据传递给它的参数选择执行路径。你可以做到

Param(
    [ValidateScript({$_ -match '^(u|r)'})]
    [String]$Option=(Read-Host -Prompt 'Would you like to (r)emove or (u)pdate a target msds-SyncServerUrl?'
)

Switch -Regex ($Option)
{
    '^r' { <# Logic #> }
    '^u' { <# Logic #> }
}

Answer 4

Persistent13's explanation如何对需要哈希表的参数进行splat处理。

mklement0's solution简化比较有助于清理那些代码。

将PowerShell变量作为cmdlet参数传递

4 个答案: