Question Amélioration de code

Salut,
Wince34 écrit:

...
j'ai fais un petit programme powershell qui fonctionne mais que je ne trouve pas très esthétique et performant.

Je dois être fatigué, mais j'ai du mal à comprendre ton code et la construction autour de la relation \"code-1\"
Donc ici, pour moi, l'esthétisme( qui selon moi n'a rien à faire dans le dev) et la performance est le cadet de mes soucis.
Wince34 écrit:

Il est fait à l'ancienne.

De le savoir me rassure

6ratgus écrit:

ça améliore l'esthétique et par là la lisibilité.

Effectivement, donc avant tout se préoccuper de la lisibilité.

En regardant de plus près, tout en renommant, les champs on comprend mieux les données.
[code:1]
\"Parent\",\"Child\",\"Niveau\",\"Libelle\"
\"3000 \",\"3006\",\"2\",\"POMME\"
[/code:1]
J'ai eu la même approche que toi, mais je ne suis pas allé plus loin, car le code ne se faisait rien ou si peu.
J'ai donc recherché si qq avait déjà eu ce besoin, j'ai trouvé ceci . Ce qui dans un premier temps évite de coder des classes natives du framework.

La fonction est récursive et reste basée sur des boucles. Ce code n'est pas très efficace, mais pour qq centaines de lignes on peut s'en contenter.
De plus il ne peut créer que 100 niveaux, car Ps n'autorise que 100 appels récursifs.

Les données doivent utiliser des délimiteurs, et la suppression des espaces sur les champs Parent et Child ne serait pas de trop.
Après qq tests et adaptation, voici une autre solution :
[code:1]
@'
\"Parent\",\"Child\",\"Niveau\",\"Libelle\"
\"3000\",\"3006\",\"2\",\"POMME\"
\"3000\",\"3005\",\"2\",\"FRUIT\"
\"3005\",\"5000\",\"3\",\"AGRUME\"
\"3005\",\"5031\",\"4\",\"ABRICOT\"
\"5000\",\"5011\",\"4\",\"ORANGE\"
\"5000\",\"5013\",\"4\",\"CLEMENTINE\"
\"5000\",\"5005\",\"4\",\"CITRON\"
\"5000\",\"5012\",\"4\",\"MANDARINE\"
'@ > c:\temp\datas.csv

#'Facilite' la recherche
$DirectoryInfos=Import-csv c:\temp\datas.csv|Sort Parent,Child

function Set-ChildDirectory
{
#from powershell.cz/2012/11/27/generate-tree-structure/
[CmdletBinding()]
param(
[Parameter(Mandatory = $true,Position = 0)]
$InputObject,

[Parameter(Mandatory = $true,Position = 1)]
[string] $Id,

[Parameter()]
[int] $Shift = 0,

[Parameter()]
[string] $RootName
)

if ($PSBoundParameters.ContainsKey('RootName'))
{ $Name=$RootName }

$indent = ' ' * $shift

$Name=$Name+'\'+$ID
MD $Name > $null
$InputObject |
Where {
$_.Parent -eq $id
} |
ForEach {
Write-Debug (\"{0} {1}\" -f $indent, $_.Child)
Set-ChildDirectory -InputObject $InputObject -id $_.Child -shift $($shift+2)
}
}
$DirectoryInfos = Import-csv c:\temp\datas.csv|sort parent,Child
md 'C:\temp\TestDirByCode' -ea SilentlyContinue > $null
Set-Location 'C:\temp\TestDirByCode'
Set-ChildDirectory -InputObject $DirectoryInfos -Id 3000 -Root $Pwd

# C:\temp\TestDirByCode> show-tree #voir PSCX
# C:\temp\TestDirByCode
# \--3000
# |--3005
# | |--5000
# | | |--5005
# | | |--5011
# | | |--5012
# | | \--5013
# | \--5031
# \--3006
#
#ou Dir -rec |Select fullname
# C:\temp\TestDirByCode> dir -rec |select fullname
#
# FullName
#

---

# C:\temp\TestDirByCode\3000
# C:\temp\TestDirByCode\3000\3005
# C:\temp\TestDirByCode\3000\3006
# C:\temp\TestDirByCode\3000\3005\5000
# C:\temp\TestDirByCode\3000\3005\5031
# C:\temp\TestDirByCode\3000\3005\5000\5005
# C:\temp\TestDirByCode\3000\3005\5000\5011
# C:\temp\TestDirByCode\3000\3005\5000\5012
# C:\temp\TestDirByCode\3000\3005\5000\5013
[/code:1]
L'information Niveau n'est pas nécessaire ici.<br><br>Message édité par: Laurent Dardenne, à: 17/03/14 14:57

6ratgus écrit:

il me semble que une simple boucle avec quelque if ou un switch devrais suffire

Le if je veux bien, voir le Where de la fonction Get-Child, mais pour le switch je suis curieux de voir ça

Un switch est statique ce que n'est pas un arbre binaire, à moins que tu veuilles gérer un nombre de niveaux fixe...