루프에 사용할 논리. 가장 쉬운 가장 쉬운 대

Question

필자는 전문가 프로그래머가 아니기 때문에 어떤 논리를 이해하기 쉽지만 효율적인 스크립트로 사용해야합니까? 모든 AD 사용자를 검색하고 AD 속성 (예 : 사무실)을 기반으로하는 스크립트를 작성 중이므로 몇 가지 다른 속성 (도시, 우편 번호 등)에 문자열을 쓸 것입니다. 나는 Foreach를 가지고 놀았고 그 루프 내에서 replace 또는 where 절을 가진 set-ADUser를 사용했다. 또는 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 : 당신은 후 새 줄을 시작할 수 있습니다 |, (, {, ,, 미완성 두 피연산자 + 같은 사업자 또는 매개 변수의 긴 목록을 포장 라인의 끝에 하나의 역음을`사용합니다.
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 
    }