Poweshell私はあなただけの必要があると思うホーム

Question

このスクリプトは良い

Get-ADUser -filter * | % { Set-ADUser $_ -HomeDrive "H:" -HomeDirectory ('\\Server \home$\' + $_.SamAccountName) } 

動作しますが、それ

ForEach($User in (Get-ADUser -filter * | Select -ExpandProperty SamAccountName)){ if(-not (HomeDrive "H:" HomeDirectory ('\\server\home$\' + $_.SamAccountName)) | Set-ADUser $_ -HomeDrive "H:" -HomeDirectory('\\HYDAD1\home$\' + $_.SamAccountName)) } 

Answer 1

[OK]を、私は今それが

(Get-ADUser -Filter * | Where{$_.Home -ne "H:" -or $_.HomeDirectory -ne ('\\server\home$\' + $_.SamAccountName)}) | % { Set-ADUser $_ -HomeDrive "H:" -HomeDirectory ('\\Server \home$\' + $_.SamAccountName) } 

Answer 2

と間違っていただきましたらすぐ、私は変数でない場合は追加として、そのわからない動作していないドライブの接続構文の助けを借りずに、そしておそらく何らかの構造を使っている。上記のコメントで指摘したように、HomeとHomeDirectoryはユーザーオブジェクトのプロパティです。それらにアクセスできるようにするには、事前にSelect -Expand samaccountnameを使用することはできません。これは、ユーザーのエイリアスを使用するパイプの下の文字列として渡すだけで、これらのプロパティは含まれません。だから私たちはそのビットを取り除くことから始めます。次に、変数$userを参照して、それらをプロパティとして正しく参照する必要があります。それは$user.home -ne "H:"と$user.HomeDirectory -ne ('\\server\home$\' + $user.SamAccountName)のようになります。両方ともに-eqを追加したことに気づくでしょう。これは、等しくないための比較演算子です。それはあなたの文法をちょっとはっきりさせます、今度は構造を見てみましょう。

Ifステートメントにパイプするのではなく、Whereステートメントを使用して、注意が必要なユーザーだけを除外してみましょう。このようにしているので、実際にForEachループにはまだ入っていないので、$userの代わりに$_を使用します。だから、次のようになります。あなたはあなたのスクリプトにこれを組み込む場合

(Get-ADUser -Filter * | Where{$_.Home -ne "H:" -or $_.HomeDirectory -ne ('\\server\home$\' + $_.SamAccountName)}) 

今では、次のようになります。

ForEach($User in (Get-ADUser -Filter * | Where{$_.Home -ne "H:" -or $_.HomeDirectory -ne ('\\server\home$\' + $_.SamAccountName)})){ 
    Set-ADUser $User -HomeDrive "H:" -HomeDirectory('\\HYDAD1\home$\' + $_.SamAccountName) 
}