Demande optimisation d'un script de démarrage

Version imprimable

Bonjour à tous,
Alors voila, il m'a été demandé de faire un script de démarrage pour tester la présence d'application, et envoyer des rapports (par mail) puis créer des fichiers sur un espace défini.
Mon script est aujourd'hui fonctionnel depuis quelques semaines.

Ma demande aujourd'hui, et une assistance pour optimiser mon script.

Je ne suis pas d'une formation de développeur, mais avec la toile internet, j'arrive à me dépatouiller. Par contre, concernant l'optimisation c'est une autre histoire...

Voici mon script (qui s’exécute par GPO). Il est commenté pour plus de visibilité, et j'ai changé le champ de quelques variables du début. A adapter à son système.

Par avance merci de vos futures retours.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
<#-------------------------------------------
Version : 0.2
Information du Script
Propriétaire du script : Moi
Nom du Script : Ordinateurs.ps1
Description : Au démarrage de la session, récupère différentes informations et les écrit dans un fichier Utilisateur sur le réseau
 
#>
 
#---------------------------
#Déclaration des variables
#---------------------------
#Variable de récupération du nom d'utilisateur
$UserName = (Get-Item env:\UserName).value
#Variable de récupération du Nom de l'ordinateur
$NomOrdi = (Get-Item env:\COMPUTERNAME).value 
#Variable du répertoire de localisation user
$DossierOrdi = "\\SRVXYZ\inventaire$\Ordinateurs\"
#$Date = [System.DateTime]::Now
$Date = get-date -uformat "%A - %d/%m/%Y / GMT %Z"
#Variable des Applicatifs à tester
$PathFusion = "${Env:ProgramFiles}\FusionInventory-Agent"
$PathAgentKas = "${Env:ProgramFiles(x86)}\Kaspersky Lab\NetworkAgent"
$PathClientKas = "${Env:ProgramFiles(x86)}\Kaspersky Lab\Kaspersky Endpoint Security for Windows"
$Doc = "C:\temp"
$VersionScript = "V0.2"
$DateScript = "15 fev 2019"
 
 
#------------------------------------
#==========================================================================
# Paramètre ENVOI DU MAIL
#==========================================================================
$expediteur = "script@temporaire.fr"
$destinataire = "Info@temporaire.Fr"
$serveur = "@serveur"
#$objet = "Absence d'application du " + "$Date" + "--sur $NomOrdi"# + "--$UserName"
#$texte = "En fichier joint, les applications absentes du poste " + $NomOrdi + " avec l'adresse IP : " + $ip
 
#Send-MailMessage -From "<$expediteur>" -To "<$destinataire>" -Subject "$objet" -Body "$texte" -SmtpServer "$serveur" -Attachments "$NewFile" 
 
#-----------------------------
#Début des actions
#----------------------------
 
#Variable concaténé
$FileOrdi = $DossierOrdi + $NomOrdi + ".txt"
 
#Test si le fichier existe
$FichierExist = Test-Path $FileOrdi
$PathFusionExist = Test-Path $PathFusion
$PathAgentKasExist = Test-Path $PathAgentKas
$PathClientKasExist = Test-Path $PathClientKas
 
$ip = (Get-WmiObject -Class win32_networkadapterconfiguration | where {($_.ipaddress -ne $null) -and ($_.defaultipgateway -ne $null) -and ($_.DNSDomain -ne $null)} | Select-Object IPAddress -First 1).IPAddress[0]
 
#Permet vérifier que l’ordinateur est en ligne avant de lancer une action sur lui via un script.
$ping = new-object System.Net.NetworkInformation.Ping
$reply = $ping.send($NomOrdi)
if ($reply.status –eq “Success”)
{
#write-host "$ComputerName is online."
 
    If ($FichierExist -eq $False)
    {
    #Création du fichier Si il n'existe pas
    New-item -Path $FileOrdi -itemType file
    ADD-content -path $FileOrdi -value "Le : $Date"
    ADD-content -path $FileOrdi -value "Nom de l'ordinateur : $NomOrdi"
    ADD-content -path $FileOrdi -value "Adresse IP du poste : $ip"
    ADD-content -path $FileOrdi -value "----------------------------------"
 
    #Test de présence des répertoires d'installation
    #Fusion Inventory
        if ($PathFusionExist -eq "True")
        {
        ADD-content -path $FileOrdi -value "Installation Fusion Inventory OK"
        }
        Else
        {
        ADD-content -path $FileOrdi -value "Installation Fusion Inventory NON-OK"
        #-------------------------------
		#Variable pour mail personnalisé à l'application
		$appli = "Fusion Inventory"
		$objet = "Absence de " + $appli + " du " + "$Date" + "--sur $NomOrdi"
		$texte = "En fichier joint, l'application " + $appli + " est absente du poste " + $NomOrdi + " avec l'adresse IP : " + $ip + "`n" + "`n" + "`n"
        $texte += "Emplacement Documentation du script : $Doc" + "`n"
        $texte += "Version du script : $VersionScript" + "`n"
        $texte += "Date du script : $DateScript"
        #Envoie d'un mail
        #Send-MailMessage -From "<$expediteur>" -To "<$destinataire>" -Subject "$objet" -Body "$texte" -SmtpServer "$serveur" -Attachments "$FileOrdi"     
        Send-MailMessage -From "<$expediteur>" -To "<$destinataire>" -Subject "$objet" -Body "$texte" -SmtpServer "$serveur" -Attachments "$FileOrdi"
        }
    #Agent Kaspersky
        if ($PathAgentKasExist -eq "True")
        {
        ADD-content -path $FileOrdi -value "Installation Agent Kaspersky OK"
        }
        Else
        {
        ADD-content -path $FileOrdi -value "Installation Agent Kaspersky NON-OK"
        #-------------------------------
		#Variable pour mail personnalisé à l'application
		$appli = "Agent Kaspersky"
		$objet = "Absence de " + $appli + " du " + "$Date" + "--sur $NomOrdi"
		$texte = "En fichier joint, l'application " + $appli + " est absente du poste " + $NomOrdi + " avec l'adresse IP : " + $ip + "`n" + "`n" + "`n"
        $texte += "Emplacement Documentation du script : $Doc" + "`n"
        $texte += "Version du script : $VersionScript" + "`n"
        $texte += "Date du script : $DateScript"        
        #Envoie d'un mail
        Send-MailMessage -From "<$expediteur>" -To "<$destinataire>" -Subject "$objet" -Body "$texte" -SmtpServer "$serveur" -Attachments "$FileOrdi"      
        }
    #Client Kaspersky
        if ($PathClientKasExist -eq "True")
        {
        ADD-content -path $FileOrdi -value "Installation Client Kaspersky OK"
        }
        Else
        {
        ADD-content -path $FileOrdi -value "Installation Client Kaspersky NON-OK"
        #-------------------------------
		#Variable pour mail personnalisé à l'application
		$appli = "Client Kaspersky"
		$objet = "Absence de " + $appli + " du " + "$Date" + "--sur $NomOrdi"
		$texte = "En fichier joint, l'application " + $appli + " est absente du poste " + $NomOrdi + " avec l'adresse IP : " + $ip + "`n" + "`n" + "`n"
        $texte += "Emplacement Documentation du script : $Doc" + "`n"
        $texte += "Version du script : $VersionScript" + "`n"
        $texte += "Date du script : $DateScript"
        #Envoie d'un mail
        Send-MailMessage -From "<$expediteur>" -To "<$destinataire>" -Subject "$objet" -Body "$texte" -SmtpServer "$serveur" -Attachments "$FileOrdi"      
        }
    }
    else {
    #Supprimer le fichier
    Remove-Item $FileOrdi
    #Créer le nouveau fichier
    New-item $FileOrdi -itemType file
    ADD-content -path $FileOrdi -value "Le : $Date"
    #ADD-content -path $FileOrdi -value "Nom d'utilisateur : $UserName "
    ADD-content -path $FileOrdi -value "Nom de l'ordinateur : $NomOrdi"
    ADD-content -path $FileOrdi -value "Adresse IP du poste : $ip"
    ADD-content -path $FileOrdi -value "----------------------------------"
 
    #Test de présence des répertoires d'installation
    #Fusion Inventory
        if ($PathFusionExist -eq "True")
        {
        ADD-content -path $FileOrdi -value "Installation Fusion Inventory OK"
        }
        Else
        {
        ADD-content -path $FileOrdi -value "Installation Fusion Inventory NON-OK"
        #-------------------------------
		#Variable pour mail personnalisé à l'application
		$appli = "Fusion Inventory"
		$objet = "Absence de " + $appli + " du " + "$Date" + "--sur $NomOrdi"
		$texte = "En fichier joint, l'application " + $appli + " est absente du poste " + $NomOrdi + " avec l'adresse IP : " + $ip + "`n" + "`n" + "`n"
        $texte += "Emplacement Documentation du script : $Doc" + "`n"
        $texte += "Version du script : $VersionScript" + "`n"
        $texte += "Date du script : $DateScript"        
        #Envoie d'un mail
        Send-MailMessage -From "<$expediteur>" -To "<$destinataire>" -Subject "$objet" -Body "$texte" -SmtpServer "$serveur" -Attachments "$FileOrdi" 
        }
    #Agent Kaspersky
        if ($PathAgentKasExist -eq "True")
        {
        ADD-content -path $FileOrdi -value "Installation Agent Kaspersky OK"
        }
        Else
        {
        ADD-content -path $FileOrdi -value "Installation Agent Kaspersky NON-OK"
        #-------------------------------
		#Variable pour mail personnalisé à l'application
		$appli = "Agent Kaspersky"
		$objet = "Absence de " + $appli + " du " + "$Date" + "--sur $NomOrdi"
		$texte = "En fichier joint, l'application " + $appli + " est absente du poste " + $NomOrdi + " avec l'adresse IP : " + $ip + "`n" + "`n" + "`n"
        $texte += "Emplacement Documentation du script : $Doc" + "`n"
        $texte += "Version du script : $VersionScript" + "`n"
        $texte += "Date du script : $DateScript"        
        #Envoie d'un mail
        Send-MailMessage -From "<$expediteur>" -To "<$destinataire>" -Subject "$objet" -Body "$texte" -SmtpServer "$serveur" -Attachments "$FileOrdi" 
        }
    #Client Kaspersky
        if ($PathClientKasExist -eq "True")
        {
        ADD-content -path $FileOrdi -value "Installation Client Kaspersky OK"
        }
        Else
        {
        ADD-content -path $FileOrdi -value "Installation Client Kaspersky NON-OK"
        #-------------------------------
		#Variable pour mail personnalisé à l'application
		$appli = "Client Kaspersky"
		$objet = "Absence de " + $appli + " du " + "$Date" + "--sur $NomOrdi"
		$texte = "En fichier joint, l'application " + $appli + " est absente du poste " + $NomOrdi + " avec l'adresse IP : " + $ip + "`n" + "`n" + "`n"
        $texte += "Emplacement Documentation du script : $Doc" + "`n"
        $texte += "Version du script : $VersionScript" + "`n"
        $texte += "Date du script : $DateScript"        
        #Envoie d'un mail
        Send-MailMessage -From "<$expediteur>" -To "<$destinataire>" -Subject "$objet" -Body "$texte" -SmtpServer "$serveur" -Attachments "$FileOrdi" 
        }
 
    }
 
}
else
{
exit
}

Je ne suis pas un spécialiste et je ne sais pas ce que tu veux optimiser mais voici une proposition.
J'ai fait cela à la volé donc c'est à tester et corriger en cas de faute de frappe/syntaxe mais c'est l'idée.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
<#-------------------------------------------
Version : 0.2
Information du Script
Propriétaire du script : Moi
Nom du Script : Ordinateurs.ps1
Description : Au démarrage de la session, récupére différentes informations et les ecrit dans un fichier Utilisateur sur le réseau
 
#>
 
 
#---------------------------
#Déclaration des variables
#---------------------------
 
#Variable de récupération du nom d'utilisateur
$UserName = (Get-Item env:\UserName).value
#Variable de récupération du Nom de l'ordinateur
$NomOrdi = (Get-Item env:\COMPUTERNAME).value 
#Variable du répertoire de localisation user
$DossierOrdi = "\\SRVXYZ\inventaire$\Ordinateurs\"
#$Date = [System.DateTime]::Now
$Date = get-date -uformat "%A - %d/%m/%Y / GMT %Z"
#Variable des Applicatifs à tester
$PathFusion = "${Env:ProgramFiles}\FusionInventory-Agent"
$PathAgentKas = "${Env:ProgramFiles(x86)}\Kaspersky Lab\NetworkAgent"
$PathClientKas = "${Env:ProgramFiles(x86)}\Kaspersky Lab\Kaspersky Endpoint Security for Windows"
$Doc = "C:\temp"
$VersionScript = "V0.2"
$DateScript = "15 fev 2019"
 
# Variable Mail
$expediteur = "script@temporaire.fr"
$destinataire = "Info@temporaire.Fr"
$serveur = "@serveur"
 
# Autres variables calculés
$FileOrdi = $DossierOrdi + $NomOrdi + ".txt"
$ip = (Get-WmiObject -Class win32_networkadapterconfiguration | where {($_.ipaddress -ne $null) -and ($_.defaultipgateway -ne $null) -and ($_.DNSDomain -ne $null)} | Select-Object IPAddress -First 1).IPAddress[0]
 
 
#---------------------------
#Function
#---------------------------
 
function TestApp {
	Param (
	[string]$PathApp,
	[string]$NomApp,
	[string]$FileOrdi,
	[string]$NomOrdi,
	[string]$expediteur,
	[string]$destinataire,
	[string]$serveur,
	[string]$ip,
	[string]$Doc,
	[string]$VersionScript,
	[string]$DateScript
	)
 
	if (Test-Path $PathApp -eq "True")
	{
		ADD-content -path $FileOrdi -value "Installation " + $NomApp + "OK"
	}
	Else
	{
		ADD-content -path $FileOrdi -value "Installation " + $NomApp + "NON-OK"
 
		#Variable pour mail personnalisé à l'application
		$objet = "Absence de " + $NomApp + " du " + "$Date" + "--sur $NomOrdi"
		$texte = "En fichier joint, l'application " + $NomApp + " est absente du poste " + $NomOrdi + " avec l'adresse IP : " + $ip + "`n" + "`n" + "`n"
		$texte += "Emplacement Documentation du script : $Doc" + "`n"
		$texte += "Version du script : $VersionScript" + "`n"
		$texte += "Date du script : $DateScript"
		Send-MailMessage -From "<$expediteur>" -To "<$destinataire>" -Subject "$objet" -Body "$texte" -SmtpServer "$serveur" -Attachments "$FileOrdi"
	}
}
 
 
#---------------------------
#Début des actions
#---------------------------
 
#Permet de vérifier que l’ordinateur est en ligne avant de lancer une action sur lui via un script.
if ((new-object System.Net.NetworkInformation.Ping).Send($NomOrdi).Status -ne [Systemormation.IPStatus]::Success)
{
	exit
}
 
 
If ((Test-Path $FileOrdi) -eq $True)
{
	#Supprime le fichier
	Remove-Item $FileOrdi
}
 
#Crée le nouveau fichier
New-item $FileOrdi -itemType file
ADD-content -path $FileOrdi -value "Le : $Date"
ADD-content -path $FileOrdi -value "Nom de l'ordinateur : $NomOrdi"
ADD-content -path $FileOrdi -value "Adresse IP du poste : $ip"
ADD-content -path $FileOrdi -value "----------------------------------"
 
#Test de présence des répertoires d'installations
TestApp -PathApp $PathFusion -NomApp "Fusion Inventory" -FileOrdi $FileOrdi -NomOrdi $NomOrdi -expediteur $expediteur -destinataire $destinataire -serveur $serveur -ip $ip -Doc $Doc -VersionScript $VersionScript -DateScript $DateScript
TestApp -PathApp $PathAgentKas -NomApp "Agent Kaspersky" -FileOrdi $FileOrdi -NomOrdi $NomOrdi -expediteur $expediteur -destinataire $destinataire -serveur $serveur -ip $ip -Doc $Doc -VersionScript $VersionScript -DateScript $DateScript
TestApp -PathApp $PathClientKas -NomApp "Client Kaspersky" -FileOrdi $FileOrdi -NomOrdi $NomOrdi -expediteur $expediteur -destinataire $destinataire -serveur $serveur -ip $ip -Doc $Doc -VersionScript $VersionScript -DateScript $DateScript

03/03/2019, 22h06
silennnce

Bonjour EricLM.

Merci de ton retour,
Pour préciser ma demande. Je sais que j'utilise beaucoup de boucle if, else... Et à mon sens c'est pas des plus propre et des plus lisible. Pareil pour certaines variables qui je pense, peuvent être déclarées autrement.
Tu passes de 200 Lignes à 100 Lignes. Ca c'est de l'optimisation. Le temps de lire et comprendre tes modifications.

Je vais faire quelques essaies dès la semaines prochaines. En déplacement actuellement.

Je reviens faire un suivi la semaine prochaine.

Salut,
Pour les appels successifs à Add-Content
Code:

1 2 3 4 5 ADD-content -path $FileOrdi -value "Le : $Date" ADD-content -path $FileOrdi -value "Nom de l'ordinateur : $NomOrdi" ADD-content -path $FileOrdi -value "Adresse IP du poste : $ip" ADD-content -path $FileOrdi -value "----------------------------------"
crée une seule string et effectue un seul appel :
Code:

1 2 3 4 5 6 7 8 $Line=@" Le : $Date Nom de l'ordinateur : $NomOrdi Adresse IP du poste : $ip ---------------------------------- @" ADD-content -path $FileOrdi -value $Line
Étudie la classe StringBuilder qui accélérera tes traitements et clarifiera la lecture du code.

En passant ceci est équivalent:
Code:

1 2 3 4 5 + "`n" + "`n" + "`n" + "`n`n`n" #voir ("`n"*3)

10/03/2019, 22h10
silennnce

Bonjour et merci de vos retours.
Je reprend le travail aujourd'hui, le temps de reprendre le bateau, de comprendre vos retours, et de tester de mon coté.
Et je reviens faire un bilan.

Bonjour,

Voici mon retour, après quelques modifications/corrections et en me basant sur le retour de Eric et Laurent.
Ce script fonctionne comme le précédent, il est beaucoup plus simple à lire et évolutif facilement.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
#-------------------------------------------
 
#---------------------------
#Déclaration des variables
#---------------------------
 
#Variable de récupération du nom d'utilisateur
$UserName = (Get-Item env:\UserName).value
#Variable de récupération du Nom de l'ordinateur
$NomOrdi = (Get-Item env:\COMPUTERNAME).value 
#Variable du répertoire de localisation user
$DossierOrdi = "c:\temp"
#$Date = [System.DateTime]::Now
$Date = get-date -uformat "%A - %d/%m/%Y / GMT %Z"
#Variable des Applicatifs à tester
$PathFusion = "${Env:ProgramFiles}\FusionInventory-Agent"
$PathAgentKas = "${Env:ProgramFiles(x86)}\Kaspersky Lab\NetworkAgent"
$PathClientKas = "${Env:ProgramFiles(x86)}\Kaspersky Lab\Kaspersky Endpoint Security for Windows"
$Doc = "C:\temp"
$VersionScript = "V0.3"
$DateScript = "14 mars 2019"
 
# Variable Mail
$expediteur = "script@temporaire.fr"
$destinataire = "Info@temporaire.Fr"
$serveur = "@serveur"
 
# Autres variables calculés
$FileOrdi = $DossierOrdi + $NomOrdi + ".txt"
$ip = (Get-WmiObject -Class win32_networkadapterconfiguration | where {($_.ipaddress -ne $null) -and ($_.defaultipgateway -ne $null) -and ($_.DNSDomain -ne $null)} | Select-Object IPAddress -First 1).IPAddress[0]
 
 
#---------------------------
#Function
#---------------------------
 
function TestApp {
	Param (
	[string]$PathApp,
	[string]$NomApp,
	[string]$FileOrdi,
	[string]$NomOrdi,
	[string]$expediteur,
	[string]$destinataire,
	[string]$serveur,
	[string]$ip,
	[string]$Doc,
	[string]$VersionScript,
	[string]$DateScript
	)
 
	if ((Test-Path $PathApp) -eq "True")
	{
		ADD-content -path "$FileOrdi" -value "Installation $NomApp OK"
	}
	Else
	{
		ADD-content -path $FileOrdi -value "Installation $NomApp NON-OK"
 
		#Variable pour mail personnalisé à l'application
		$objet = "Absence de " + $NomApp + " du " + "$Date" + "--sur $NomOrdi"
		$texte = "En fichier joint, l'application " + $NomApp + " est absente du poste " + $NomOrdi + " avec l'adresse IP : " + $ip + "`n" + "`n" + "`n"
		$texte += "Emplacement Documentation du script : $Doc" + "`n"
		$texte += "Version du script : $VersionScript" + "`n"
		$texte += "Date du script : $DateScript"
		Send-MailMessage -From "<$expediteur>" -To "<$destinataire>" -Subject "$objet" -Body "$texte" -SmtpServer "$serveur" -Attachments "$FileOrdi"
	}
}
 
 
#---------------------------
#Début des actions
#---------------------------
 
#Permet de vérifier que l’ordinateur est en ligne avant de lancer une action sur lui via un script.
if ((new-object System.Net.NetworkInformation.Ping).Send($NomOrdi).Status -ne "Success")
{
	exit
}
 
 
If ((Test-Path $FileOrdi) -eq $True)
{
	#Supprime le fichier
	Remove-Item $FileOrdi
}
 
#Crée le nouveau fichier
New-item $FileOrdi -itemType file
#Le début de contenu du fichier
$ContenuFile=@"
Le : $Date
Nom de l'ordinateur : $NomOrdi
Adresse IP du poste : $ip
----------------------------------
"@
 
ADD-content -path $FileOrdi -value $ContenuFile
 
#Test de présence des répertoires d'installations
TestApp -PathApp $PathFusion -NomApp "Fusion Inventory" -FileOrdi $FileOrdi -NomOrdi $NomOrdi -expediteur $expediteur -destinataire $destinataire -serveur $serveur -ip $ip -Doc $Doc -VersionScript $VersionScript -DateScript $DateScript
TestApp -PathApp $PathAgentKas -NomApp "Agent Kaspersky" -FileOrdi $FileOrdi -NomOrdi $NomOrdi -expediteur $expediteur -destinataire $destinataire -serveur $serveur -ip $ip -Doc $Doc -VersionScript $VersionScript -DateScript $DateScript
TestApp -PathApp $PathClientKas -NomApp "Client Kaspersky" -FileOrdi $FileOrdi -NomOrdi $NomOrdi -expediteur $expediteur -destinataire $destinataire -serveur $serveur -ip $ip -Doc $Doc -VersionScript $VersionScript -DateScript $DateScript

Maintenant je tente de regarder le fonctionnement des StringBuilder. Pour l'instant je pêche un peu, c'est plus poussé et technique. mais je tente de comprendre pour m'améliorer.

Merci déjà pour ce 1er jet très concluant.

14/03/2019, 11h16
ericlm128

C'est tout une théorie les concaténations de string vs l'objet StringBuilder.

Techniquement si tu fait beaucoup (le tout étant de savoir ce que beaucoup veux dire) de concaténation de string (+=, +) tu devrait utiliser un objet StringBuilder.
Lorsqu'un développeur de renom comme Laurent Dardenne voit successivement plusieurs += ça le titille :lol:

Si tu a peux de concaténation, l'utilisation d'un string comme tu le fait peux s'avérer plus rapide.

Techniquement, un objet string est immuable, si tu "ajoute" du texte il va recréer un nouveau objet string alors que l'objet StringBuilder ne sera créé qu'une seule fois. Il utilise une gestion d'allocation mémoire au seins de l'objet pour pouvoir s'étendre.

Voici un code te permettant de benchmarker les deux façon en faisant varier $nbr pour le nombre d'itération

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
$nbr = 100
 
(Measure-Command {
    $string = ''
    foreach($i in 1..$nbr)
    {
        $string += "J'ajoute du texte"
    }
    [void]$string
}).TotalMilliseconds
 
 
(Measure-Command {
    $sb = New-Object -TypeName "System.Text.StringBuilder"
    foreach($i in 1..$nbr)
    {
        [void]$sb.Append("J'ajoute du texte")
    }
    [void]$sb.ToString()
}).TotalMilliseconds

15/03/2019, 00h29
silennnce

Merci Eric pour tes explications, cela m'aide grandement à mieux comprendre cette partie.
C'est pas encore parfait dans mon cerveau, mais je vais essayer de m'y entrainer à l'utiliser et le comprendre.

Je pense que je peux clore ce sujet. Le script est réutilisable si quelqu'un cherche un équivalent, ou possible de l'adapté.

Merci a tous les deux pour vos retours. Très constructifs.