ループのために使用するロジック。最も簡単なものと最も簡単なもの

Question

私は専門家のプログラマーではないので、どのロジックを理解しやすくするために使うべきですか？私はすべてのADユーザーを検索し、AD属性（オフィス）に基づいて、いくつかの文字列をいくつかの他の属性（市、郵便番号など）に書き込むスクリプトを作成しています。私はForeachを使って遊んでいました。そして、そのループの中で、置き換えられたset-ADUserかwhere節を使っていました。 IF、IfElseステートメントの束の非常に面倒な方法です。また、私はSwitch（Case）ステートメントを調べ始めました。

提案がありますか？その上に構築するために、小さな始め

は、私はこれを開始しました：

代わり

$Users = Get-ADUser -searchbase "OU=users,OU=testing,DC=blah,DC=company,DC=com" -filter {samaccountname -like "r*"} -properties * | Select samaccountname,streetaddress,l, postalcode,st,physicaldeliveryofficename 
foreach ($user in $users) { 
set-aduser $user -city "NoWhere" -Postalcode "B2B 2B2"| where {$_.physicaldeliveryofficename -eq "Head Office"} 
} 

またはスイッチ：複雑な外部リソースを照会するには、通常の99％がかかるため

$Users = Get-ADUser -searchbase "OU=users,OU=testing,DC=blah,DC=company,DC=com" -filter {samaccountname -like "remot*"} -properties * | Select samaccountname,streetaddress,l, postalcode,st,physicaldeliveryofficename 
foreach ($user in $users) {switch ($users.physicaldeliveryofficename){ 
"Nowhere town" { -city "NoWhere" -postalcode "A1A A1A"} 
"Anywhereville" { -city "Anywhere" -postalcode "B1B B1B"} 
}} 

Answer 1

1. は、クエリの最適化ランタイム。
2. 使用パイプラインその処理は、リスト全体を蓄積することなく、すぐにを開始しますので：代わりにForEachステートメントを使用| ForEach { ......... }の
   を。
3. 停止onelinersを書く：あなたは+よう|、(、{、,、未完成の2オペランド演算子の後に新しい行を開始するか、パラメータの長いリストをラップする行の末尾に、単一のバッククォート `を使用することができます。
4. Switchは、パイプライン処理時に利用可能なリストがないため、各要素を個別に処理する必要があるため、ここに行く方法です。

Get-ADUser -searchbase "OU=users,OU=testing,DC=blah,DC=company,DC=com" ` 
      -filter 'samaccountname -like "r*"' ` 
      -properties * | 
    Select samaccountname, 
      streetaddress, 
      l, 
      postalcode, 
      st, 
      physicaldeliveryofficename | 
    ForEach { 
     switch ($_.physicaldeliveryofficename) { 
      "Nowhere town" { $city = "NoWhere"; $postalcode = "A1A A1A"; break } 
      "Anywhereville" { $city = "Anywhere"; $postalcode = "B1B B1B"; break } 
      default   { $city = "?"; $postalcode = "" } 
     } 
     Set-ADUser $_ -City $city -Postalcode $postalcode 
    }