Problème de variable dans les fonctions. (windows forms)

Version imprimable

Bonsoir, voila mon souci,

voici un code qui fonctionne tres bien

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
[void] [System.Reflection.Assembly]::LoadWithPartialName("System.Drawing") 
[void] [System.Reflection.Assembly]::LoadWithPartialName("System.Windows.Forms") 
$objForm = New-Object System.Windows.Forms.Form 
$objForm.Text = "Data Entry Form"
$objForm.Size = New-Object System.Drawing.Size(300,200) 
$objForm.StartPosition = "CenterScreen"

$OKButton = New-Object System.Windows.Forms.Button
$OKButton.Location = New-Object System.Drawing.Size(75,120)
$OKButton.Size = New-Object System.Drawing.Size(75,23)
$OKButton.Text = "OK"
$OKButton.Add_Click({$x=$objTextBox.Text;$objForm.Close()})
$objForm.Controls.Add($OKButton)

$objForm.Topmost = $True
$objForm.Add_Shown({$objForm.Activate()})
[void] $objForm.ShowDialog()

j'ai besoin de le faire marcher sous cette forme :

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
[void] [System.Reflection.Assembly]::LoadWithPartialName("System.Drawing") 
[void] [System.Reflection.Assembly]::LoadWithPartialName("System.Windows.Forms") 

function add-form(){
$objForm = New-Object System.Windows.Forms.Form 
$objForm.Text = "Data Entry Form"
$objForm.Size = New-Object System.Drawing.Size(300,200) 
$objForm.StartPosition = "CenterScreen"
}
function add-button(){
$OKButton = New-Object System.Windows.Forms.Button
$OKButton.Location = New-Object System.Drawing.Size(75,120)
$OKButton.Size = New-Object System.Drawing.Size(75,23)
$OKButton.Text = "OK"
$OKButton.Add_Click({$x=$objTextBox.Text;$objForm.Close()})
$objForm.Controls.Add($OKButton)
}

function turnon {
$objForm.Topmost = $True
$objForm.Add_Shown({$objForm.Activate()})
[void] $objForm.ShowDialog()
}

add-form
add-button
turnon

vaut il mieux utiliser des returns ? (mais je ne sait pas trop comment...) des variables globales ? une autre subtilité ?
voilà le topo, j'ai hate d'avoir une solution je suis sur un script qui me passionne xD

Trouver ! je ne sait pas si c'est la meilleur solution mais cela marche ! je reste ouvert à toutes proposition plus adapté !

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
[void] [System.Reflection.Assembly]::LoadWithPartialName("System.Drawing") 
[void] [System.Reflection.Assembly]::LoadWithPartialName("System.Windows.Forms") 

function add-form(){
$script:objForm = New-Object System.Windows.Forms.Form 
$script:objForm.Text = "Data Entry Form"
$script:objForm.Size = New-Object System.Drawing.Size(300,200) 
$script:objForm.StartPosition = "CenterScreen"
}
function add-button(){
$script:OKButton = New-Object System.Windows.Forms.Button
$script:OKButton.Location = New-Object System.Drawing.Size(75,120)
$script:OKButton.Size = New-Object System.Drawing.Size(75,23)
$script:OKButton.Text = "OK"
$script:OKButton.Add_Click({$x=$objTextBox.Text;$script:objForm.Close()})
$script:objForm.Controls.Add($script:OKButton)
}

function turnon {
$script:objForm.Topmost = $True
$script:objForm.Add_Shown({$script:objForm.Activate()})
[void] $script:objForm.ShowDialog()
}

add-form
add-button
turnon

Salut,

Citation:

Envoyé par gretch

vaut il mieux utiliser des returns ?

C'est une possible conception, dans ton cas la fonction retournera un objet différent, mais paramétré à l'identique.

Citation:

Envoyé par gretch

(mais je ne sait pas trop comment...)

Write-Output, Return et par défaut PS émet tout objet dans le pipeline :
Code:

1 2 3 4 5 $OKButton = New-Object System.Windows.Forms.Button Write-Output $OKButton #précise l'intention $OKButton #comportement par défaut identique à Write-Output Return $OKButton # émet l'objet dans le pipe et quitte le code en cours
Citation:

Envoyé par gretch

des variables globales ? une autre subtilité ?

On retarde autant que possible l'usage de globale, la portée script est une solution. Cela dépend de l'usage de ton code :
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 [void] [System.Reflection.Assembly]::LoadWithPartialName("System.Drawing") [void] [System.Reflection.Assembly]::LoadWithPartialName("System.Windows.Forms") function add-form(){ $script:objForm = New-Object System.Windows.Forms.Form $script:objForm.Text = "Data Entry Form" $script:objForm.Size = New-Object System.Drawing.Size(300,200) $script:objForm.StartPosition = "CenterScreen" } function add-button{ $OKButton = New-Object System.Windows.Forms.Button $OKButton.Location = New-Object System.Drawing.Size(75,120) $OKButton.Size = New-Object System.Drawing.Size(75,23) $OKButton.Text = "OK" $OKButton.Add_Click({$x=$objTextBox.Text;$objForm.Close()}) $script:objForm.Controls.Add($OKButton) } function turnon { $script:objForm.Topmost = $True $script:objForm.Add_Shown({$objForm.Activate()}) [void] $script:objForm.ShowDialog() } add-form add-button turnon test-path variable:objForm test-path variable:script:objForm $script:objForm.Dispose()
Ne pas oublier l'appel à Dispose(), ce n'est pas une option :-)
Dans ton dernier post, les objets ajoutés ainsi objForm.Controls.Add() n'ont pas besoin d'avoir la portée script,.
N'oublie pas le code suivant :
Code:

1 2 $script:OKButton.Add_Click({$x=$objTextBox.Text;$script:objForm.Close()})

Une autre approche pour manipuler tes composants, sous réserve de le nommer :

Code:

1
2
3
4
5
6
7
8
9
  $OKButton.Name = "btnOK"
  $OKButton.Add_Click(
   {
     Write-host "Composant courant :$($this.Name)"
     Write-host "Recherche le texte d'un composant en utilisant son nom :$(($script:objForm.Controls.Find('btnOK',$false)).Text)"
     $script:objForm.Close()
   }
  )

20/08/2014, 14h31
gretch

re :)

bon un petit retour...

les returns : j'ai pour l"instant abandonné cette idée, je passe par la scope script comme tu as pu le constater :)

Citation:

Ne pas oublier l'appel à Dispose(),

mouarf, que fait donc cet appel ~~(il "clos" la form si j'ai bien compris cela ne va t'il pas m'empécher de la modifier ensuite ?)~~je crois que j'ai pas tout compris, vu que tu utilise aussi l'appel .close() :aie:,

D'une manière générale, pour voir ou je veux en venir je te propose de jeter un petit coup d'oeil sur ce post :
http://www.developpez.net/forums/d14...m/#post7935402

en gros le but de la manip est de convertir un GUI KODA en GUI Powershell :) c'est bien partie ! (entre autre grace à toi ;) )
n'hesite pas à venir donner un coups de main xD

Merci encore !
20/08/2014, 15h02
Laurent Dardenne

Citation:

Envoyé par gretch

mouarf, que fait donc cet appel

Il libère explicitement les ressources de l'objet Form. Close() devrait suffire, sauf que :

Citation:

Il existe deux situations dans lesquelles un formulaire n'est pas disposé sur Close : (1) lorsqu'il fait partie d'une application MDI et que le formulaire n'est pas visible ; (2) lorsque vous avez affiché le formulaire en utilisant ShowDialog. Dans ces circonstances, vous devez appeler manuellement Dispose pour marquer tous les contrôles du formulaire pour le garbage collection.

Citation:

Envoyé par gretch

n'hesite pas à venir donner un coups de main xD

J'ai du temps en ce moment, mais j'ai d'autres priorités. Désolé.