Discussion :

[casimir92400]

Bonjour,

J'ai besoin d'aide sur un script que j'ai écrit (mega débutant)

Code :

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
Import-module ActiveDirectory
$group = 'mon_groupAD'
$computernamelist = Get-Content c:\Temp\mes_computer.txt

    foreach ($compnamelist in $computernamelist)
        {$computerlistDN = (Get-ADComputer $compnamelist).distinguishedname
        
        $addmember = Add-ADGroupMember -Identity $group -Member $computerlistDN
        
        if ($addmember -Contains "Le nom de compte spécifié est déjà membre du groupe")
                {
                Write-ouput "Member" $compnamelist "est deja present dans le group" $group "" | out-file C:\Temp\MemberAjoute.txt -append
                }
                elseif ($addmember -Contains "Objet de l'annuaire non trouvé")
                    {
                    Write-output "Member" $compnamelist "est pas present dans le domaine" | out-file C:\Temp\MemberAjoute.txt -append
                    }
                    else
                        {
                        Write-output "Ajout du  member" $compnamelist "dans le groupe" $group "" | out-file C:\Temp\MemberAjoute.txt -append
                        }
        }

Mon script s’exécute correctement, mais il ne traite pas la deuxième partie a partir du "if ($addmember -Contains "Le nom de compte spécifié est déjà membre du groupe")"

en gros j'ai pas de log, donc j'ai pas l'impression qu'il comprenne le $addmember -Contains "Le nom de compte spécifié est déjà membre du groupe"

Si vous aviez une idée d'amélioration ...

thanks

[Laurent Dardenne]

donc j'ai pas l'impression qu'il comprenne le $addmember -Contains "Le nom de compte spécifié est déjà membre du groupe"

Pourtant il fait ce que tu lui demandes.
A priori le message que tu recherches est dans le flux d'erreur, de plus ce cmdlet d'après la doc, ne renvoi rien pas défaut dans le pipeline et si c'est le cas c'est un objet ADGroup, pas une chaîne de caractères.Enfin l'opérateur -Contains recherche un élément dans une collection, si $addmember était une string -match ou -eq ou -Like serait plus approprié.
Ajouter une gestion d'erreur serait peut être une solution.

[casimir92400]

Je peux essayer -like à la limite, par contre pour une gestion d'erreur je veux bien mais pour l'instant c'est du chinois pour moi.

[Laurent Dardenne]

Je peux essayer -like à la limite, par contre pour une gestion d'erreur je veux bien mais pour l'instant c'est du chinois pour moi.

A défaut d'apprendre le chinois tu devrais apprendre les bases de Powershell, tu trouveras ici des tutoriaux, dont un sur la gestion d'erreur. Ici le manuel d'utilisateur de la version 1.
Je te conseille d'implémenter une gestion d'erreur, ensuite tu choisiras si besoin l'opérateur adéquate.