Amélioration de script ?

Version imprimable

Bonjour !

Tout d'abord, sachez que je suis un débutant en scripting, un vrai de vrai qui souhaite pourtant mettre les deux pieds dans la cour des grands !
le script suivant permet d’afficher plusieurs information sur tous les drives d'un serveur
Code:

1 2 3 4 5 6 7 8 param ($computer = ".") get-wmiobject -computer $computer win32_logicaldisk | where {$_.drivetype -eq 3} | tee-object -variable disques | select-object @{e={$_.systemname};n="Système " }, @{e={$_.name};n="Disque"}, @{e={[math]::round($_.size/1GB,2)};n="Capacité (Go)"}, @{e={[math]::round($_.freespace/1GB,1)};n="Disponible (Go)"}, @{e={[math]::round(([int64]$_.freespace/[int64]$_.size*100),0)};n="(%)"}
mon souci c'est que le résultat s'affiche avec retour à la ligne

Système : SERVEUR_A
Volume : C:
Capacité (Go): 100
Disponible (Go): 20
(%) : 20

j'aimerais avoir un résultat sur une seul ligne vu que j'ai plusieurs serveur à tester en m^me temps

Système Volume Capacité (Go) Disponible (Go) (%)
------- ------ ------------- --------------- ---
SERVEUR_A C: 100 20 20

Merci

26/03/2015, 19h02
Med_be

Quand j'enlève la virgule à la fin de l'avant dernier ligne @{e={[math]::round($_.freespace/1GB,1)};n="Disponible (Go)"} j'ai le résultat souhaité mais sans l’exécution de la dernière ligne ???? :calim2:
26/03/2015, 19h59
Laurent Dardenne

Ajoute Format-Table et tu peux regrouper avec le paramètre -GroupBy

Merci bcp !!!!!! ça marche sans un group by :)

Code:

1
2
3
4
5
6
7
8
9

param ($computer = ".")
get-wmiobject  -computer $computer win32_logicaldisk | where {$_.drivetype -eq 3} | tee-object -variable disques |
Format-Table  @{e={$_.systemname};n="Système " },
              @{e={$_.name};n="Disque"},
              @{e={[math]::round($_.size/1GB,2)};n="Capacité (Go)"},
              @{e={[math]::round($_.freespace/1GB,1)};n="Disponible (Go)"},
              @{e={[math]::round(([int64]$_.freespace/[int64]$_.size*100),0)};n="(%)"}

j'essaie maintenant d'améliorer le script pour lire d'un fichier texte ou il y aura la liste de tous les serveurs et exporter le résultat dans un fichier csv

Voila mon essaie :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 [cmdletbinding()] param ( [parameter(ValueFromPipeline=$true,ValueFromPipelineByPropertyName=$true)] [string[]]$ComputerName = $env:computername ) begin {} process { foreach ($Computer in $ComputerName) { get-wmiobject -computer $computer win32_logicaldisk | where {$_.drivetype -eq 3} | tee-object -variable disques | Format-Table @{e={$_.systemname};n="Système " }, @{e={$_.name};n="Disque"}, @{e={[math]::round($_.size/1GB,2)};n="Capacité (Go)"}, @{e={[math]::round($_.freespace/1GB,1)};n="Disponible (Go)"}, @{e={[math]::round(([int64]$_.freespace/[int64]$_.size*100),0)};n="(%)"} } } end {}
je l’exécute avec la ligne suivante :

Code:

get-Content computers.txt | .\get-freespace.ps1 | Export-Csv Result.csv -notype

Mais ça ne marche pas

Merci d'avance pour votre aide

Avec PS il faut formater en dernier recours, le Select-Object d'origine était à la bonne place.
Format-Table émet des objets de formatage interprétés par la console, ils n'ont donc plus rien avoir avec ceux d'origine.

Ensuite dans le bloc process :

Code:

foreach ($Computer in $ComputerName)

A moins de vouloir traiter un tableau de tableaux, l'itération sur une string suffit:
Code:

1 2 3 4 5 6 7 param ( [parameter(ValueFromPipeline=$true,ValueFromPipelineByPropertyName=$true)] [string]$ComputerName = $env:computername ) process { get-wmiobject -computer $computer win32_logicaldisk | ...

Bonjour,

Merci pour ton retour !! J'ai essayé d'appliquer ce que j'ai compris mais ça ne marche toujours pas :(

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
[cmdletbinding()]
param (
 [parameter(ValueFromPipeline=$true,ValueFromPipelineByPropertyName=$true)]
    [string]$ComputerName = $env:computername
)            

begin {}
process {
 foreach ($Computer in $ComputerName) {
get-wmiobject  -computer $computer win32_logicaldisk | where {$_.drivetype -eq 3} | tee-object -variable disques |
select-object @{e={$_.systemname};n="Système " },
              @{e={$_.name};n="Disque"},
              @{e={[math]::round($_.size/1GB,2)};n="Capacité (Go)"},
              @{e={[math]::round($_.freespace/1GB,1)};n="Disponible (Go)"},
              @{e={[math]::round(([int64]$_.freespace/[int64]$_.size*100),0)};n="(%)"}
}            
}
end {}

30/03/2015, 09h29
Laurent Dardenne
Essaie ceci :
Code:

1 2 process { get-wmiobject -computer $ComputerName win32_logicaldisk ...
Citation:

Envoyé par Med_be

ça ne marche toujours pas

Si c'est tjr le cas il te faut être plus prècis.

MErci !!!
ça marche très bien , je n'ai pas compris au début qu'il ne fallait pas utiliser foreach ......

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
[cmdletbinding()]
param (
 [parameter(ValueFromPipeline=$true,ValueFromPipelineByPropertyName=$true)]
    [string]$ComputerName = $env:computername
)            


process {

get-wmiobject  -computer $ComputerName win32_logicaldisk | where {$_.drivetype -eq 3} | tee-object -variable disques |
select-object @{e={$_.systemname};n="Système " },
              @{e={$_.name};n="Disque"},
              @{e={[math]::round($_.size/1GB,2)};n="Capacité (Go)"},
              @{e={[math]::round($_.freespace/1GB,1)};n="Disponible (Go)"},
              @{e={[math]::round(([int64]$_.freespace/[int64]$_.size*100),0)};n="(%)"}
}