search

LxLeChat / FlowChartCore

PowerShell Module Written in C# to create DOT graphs from PS Scripts
MIT License
19 stars 3 forks source link

Add Parameter Set For CodeAsText #107

Closed LxLeChat closed 3 years ago

LxLeChat commented 3 years ago

Quand on utilise CodeAsText il serait bien de spécifier si on veut, oui ou non, utiliser PSSA. Car l'utilisation de PSSA ralentit la generation du graph.

LxLeChat commented 3 years ago

Enfait ce qui serait plus "intelligent"... ça serait de passer l'entièreté du code avant dans PSSA plutot que de le faire par codeblock ... ce qui prend du temps ... !

LxLeChat commented 3 years ago

Ok donc effectivement c bien plus intelligent de faire le formatage dans Get-FlowChartNode ... j'ai ajouté un param -FormatScript, qui fait appelle à PSSA. et j'ai sorti ce truc de New-FlowCharGraph

J'ai introduit une petite coquille (un décalage d'espace) dans un script fournit par un compère du FRPSUG!

Get-FLowChartNode -Path C:\temp\laurent.ps1 | New-FLowChartGraph -CodeAsText donne: image

avec formattage via PSSA Get-FLowChartNode -Path C:\temp\laurent.ps1 -FormatScript | New-FLowChartGraph -CodeAsText image

qu'en penses tu @LaurentDardenne c mieux non ? :) je n'ai pas encore poussé le code

LxLeChat commented 3 years ago

arf bon enfait ça fait c.... les bloc de code issue des ifs ou des boucles ne sont pas bien formattés comme ça ... retourne à la base départ !

LaurentDardenne commented 3 years ago

M'est avis que tu oublies le principe de base d'un cmdlet : fait une seule chose et le fait bien ;-)

Format-Script C:\temp\laurent.ps1 | Get-FLowChartNode | New-FLowChartGraph -CodeAsText

YAGNI

LxLeChat commented 3 years ago

Ouais sur le principe je suis tout à fait d accord avec ça :) La c juste que formater le script avant ne resoud pas le pb d affichage pour les codeblock de niveau 2 et supérieur ! Bref j ai fait tellement de tests que je me suis un peu embrouillé ^^ Je vais partir sur un validateset pour codeastext, avec valeur possible: classic, format! Et voila ca fera la rue michelle! Le format tentera d utiliser pssa ..!

LxLeChat commented 3 years ago

Bon du coup je clos CodeAsText est un validateSet avec 2 choix: