Discussion :

[laurentSc]

Bonjour,

dans mon script l'utilisateur doit saisir le nom d'un groupe (pour copier celui-ci). Par contre en cas d'erreur de saisie, plutôt qu'afficher le vilain message d'erreur de Powershell, je préfère détecter une erreur et afficher un message plus sympa.
Voici le traitement que je fais :

Code powershell :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
$samgrsource = ((Get-ADGroup -Filter {name -eq $groupsource} -Server "gad.schneider-electric.com").sAMAccountName)
$samgrdest = ((Get-ADGroup -Filter {name -eq $groupdest} -Server "gad.schneider-electric.com").sAMAccountName)
Write-Host "source=$samgrsource"
Write-Host ""
Write-Host "dest=$samgrdest"
if (($samgrsource -eq '') -OR ($samgrdest -eq '')) { Write-Host "unknown name for source or destinatory file";exit}

et bien que ça m'affiche ça

source=

dest=

ça n'affiche pas le message d'erreur. Pourquoi ?

[Laurent Dardenne]

Salut,
tu supposes récupérer une chaine vide :

Code powershell :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
$null -eq ''
#False

Le test de déclenchement d'erreur doit tenir compte du type de l'objet testé. Vérifie avec $samgrsource.Gettype()

Et utilise Write-Error et pas Write-Host, C'est plus mieux pour une gestion d'erreur.
Enfin attention à l'usage de l'instruction Exit qui peut avoir des effets de bord sympathique

[laurentSc]

Comme je compte bien implémenter une gestion d'erreur agréable pour l'utilisateur, un peu loin dans mon code, il y a une instruction qui génère une erreur si le groupe destinataire n'existe pas.
J'ai donc fait :

Code powershell :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
#To clean the group to be feeded 
Set-ADGroup -Identity $samgrdest -clear member -Server "gad.schneider-electric.com" 
if ($? -eq $false) { Write-Host "unknown name for destinatory group";exit}

Un peu plus loin, j'ai un code qui porte sur le groupe source, donc

Code powershell :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
Get-ADGroupMember -Identity $samgrsource  -Server "gad.schneider-electric.com"
    if ($? -eq $false) { Write-Host "unknown name for source group";exit} else {Write-Host "OK";exit }

Et en tête de mon script, j'ai mis
$ErrorActionPreference='SilentlyContinue' #to not display the error messages.
Ainsi, j'ai ce que je veux

[laurentSc]

Je viens de faire Write-Host "type source=$samgrsource.Gettype()" et ça affiche

type source=.Gettype()

Que veux-tu dire :

l'instruction Exit qui peut avoir des effets de bord sympathique

cela dit, comme tu est toujours de bon conseil, j'ai un peu cherché et ai utilisé try catch....

[Laurent Dardenne]

ça affiche

Pour exécuter du code dans une chaine on utilise la construction $()

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
#"Texte $(code) texte"
"type source=$( $samgrsource.Gettype() )"

Mais dans ce cas, sans test préalable , si $samgrsource est $null tu auras une erreur d'exécution.

Que veux-tu dire

L'instruction Exit termine l'exécution du script et selon le contexte d'utilisation peut clore la session. L'instruction Return est préférable.

[laurentSc]

Comme j'ai mis $ErrorActionPreference='SilentlyContinue' #to not display the error messages en tête de mon script, pas de message d'erreur mais effectivement :

type source=

alors qu'avec un nom de groupe qui existe :

type source=string

Ca n'explique pas (pour moi !) :

Code powershell :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
$samgrsource = ((Get-ADGroup -Filter {name -eq $groupsource} -Server "gad.schneider-electric.com").sAMAccountName)
$samgrdest = ((Get-ADGroup -Filter {name -eq $groupdest} -Server "gad.schneider-electric.com").sAMAccountName)
Write-Host "type source=$($samgrsource.Gettype())"
if (($samgrsource -eq $false) -OR ($samgrdest -eq $false)) { Write-Error "unknown name for source or destinatory file";return}

[Laurent Dardenne]

Ca n'explique pas (pour moi !)

Un coup tu testes une string, un autre un booléen ça finira bien par tomber en marche

Code powershell :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
 
$null -eq ''
#False
'' -eq $null
#False
 
$null -eq $true
#False
$null -eq $false
#False
 
'' -eq $false
#False
'' -eq $true
#False
 
'' -eq ''
#True
#' ' -eq ''
False
#'A' -eq ''
False

Valide déjà le cas où le contenu est $null si c'est null pas la peine d'aller plus loin et un nom de groupe vide ou contenant que des espace me semble improbable avec l'AD, à vérifier.
Un premier test :

Code powershell :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
$DebugPreference='Continue'
Write-Debug "'$($samgrsource)'"
$DebugPreference='SilentlyContinue'