Portée des variables dans une classe

Bonjour à tous,

Je voudrais savoir quelle est la porté de ma variable, voici le 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

Partial Class SearchByBatch Inherits System.Web.UI.Page   Private _batchValues As BatchValues = New BatchValues   Protected Sub ButtonSearch_Click() Handles ButtonSearch.Click If TextBoxSearch.ToString <> "" Then Try _batchValues.gridViewValues(Me.TextBoxSearch.Text.ToString, Me.ph) Catch ex As Exception MsgBox(ex.Message) End Try End If   For Each element In _batchValues.listCheckBox MsgBox("test") 'J'ai plusieurs box qui s'affichent Next End Sub   Protected Sub ButtonExportExcel_Clik() Handles ButtonExport.Click For Each element In _batchValues.listCheckBox MsgBox("test") 'Et là rien ne s'affiche .. Next End Sub   End Class

---

Pourquoi est-ce que j'ai des "test" qui s'affichent dans ma première méthode alors que dans la deuxième non ?

J'ai pourtant déclarée la variable avant mes méthodes, elle devrait être "globale" au deux méthodes ?

Merci d'avance :ccool:

Parce que _batchValues.listCheckBox est vide (ne contient aucun élément). Donc tu ne rentre pas dans la boucle.

et pour te convaincre de cette réponse pertinente met ta msgbox(test) avant ton for each ... meilleur debug du monde

sinon le pas à pas est ton ami ça te permet de contrôler ce que tu fais

Les Msgbox servaient juste pour démontrer l'exemple.

J'ai compris que ma fonction _batchValues.listCheckBox ne renvoi rien.

Ce que je ne comprends pas, c'est pourquoi vu que ma variable est globale à la classe ?

Citation:

---

Envoyé par Jah73

Ce que je ne comprends pas, c'est pourquoi vu que ma variable est globale à la classe ?

---

Debug pas à pas en mettant un point d'arrêt sur la ligne 7 de ton code (cf. ton premier post), sur le IF. Regarde si tu rentres bien dedans.

Merci pour vos réponses :ccool:

Petite précision, c'est de l'ASP.NET avec du VB.NET

J'ai compris ce qui se passe :

Quand mon programme arrive à la fin de ma méthode ButtonSearch_Click
, il "détruit" la classe. Du coup, après mon END SUB, ma variable _batchValues n'existe plus. :?

Comment je peux faire ?

En gros, voila ce que fait mon programme :

L'utilisateur clique sur le bouton search et la méthode _batchValues.gridViewValues(Me.TextBoxSearch.Text.ToString, Me.ph) est excécutée.

Dans cette méthode, je créé une liste de checkbox "customs" qui me permettent d'avoir des valeurs associées a chacune d'entre elle.

_batchValues.listCheckBox me permet de récupérer cette liste.

Grace à ma méthode ButtonExportExcel_Clik(), j'aimerai récupérer cette liste mais elle est vide comme la classe a été "détruite"

J'espère que vous comprenez mon problème :(

Effectivement si c'est du Web, il faut persister ton objet... Pour ce faire, en supposant que l'objet soit différent pour chacun des utilisateurs, tu peux utiliser la Session.

Première étape, lors du click sur le bouton, tu stockes ton objet dans la session, avec un code du genre :

Code:

---

1 2 3 4

Protected Sub ButtonSearch_Click(sender As Object, e As EventArgs) ' On insère les valeurs en session Session("TonObjet") = _batchValues.gridViewValues(Me.TextBoxSearch.Text.ToString(), Me.ph) End Sub

---

Ensuite, dans ton évènement pour l'export, tu vérifies si l'objet existe en session. Si c'est le cas, tu le prends en compte, avec un truc du genre :

Code:

---

1 2 3 4 5 6 7 8 9

Protected Sub ButtonExportExcel_Clik(sender As Object, e As EventArgs) If Session("TonObjet") IsNot Nothing Then Dim tonObjet = TryCast(Session("TonObjet"), BatchValues)   If tonObjet IsNot Nothing Then ' Le traitement sur "tonObjet"... End If End If End Sub

---

Si ton objet est commun entre tous les utilisateurs, au lieu d'utiliser la Session, utilise plutôt l'Application ou encore le Cache.

C'est exactement ça :D

Merci beaucoup DotNetMatt ! :ccool:

Voici le code :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18

Protected Sub ButtonSearch_Click() Handles ButtonSearch.Click If TextBoxSearch.ToString <> "" Then Try _batchValues.gridViewValues(Me.TextBoxSearch.Text.ToString, Me.ph) Session("list") = _batchValues.listCheckBox Catch ex As Exception MsgBox(ex.Message) End Try End If End Sub   Protected Sub ButtonExportExcel_Clik() Handles ButtonExport.Click If Session("list") IsNot Nothing Then For Each element In Session("list") 'Traitement Next End If End Sub

---

J'ai besoin d'un dernier renseignement :

Je n'arrive pas à recuperer les checkbox qui sont "Checked", même avec ce code :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

Protected Sub ButtonSearch_Click() Handles ButtonSearch.Click If TextBoxSearch.ToString <> "" Then Try _batchValues.gridViewValues(Me.TextBoxSearch.Text.ToString, Me.ph) Session("list") = _batchValues Catch ex As Exception MsgBox(ex.Message) End Try End If End Sub   Protected Sub ButtonExportExcel_Clik() Handles ButtonExport.Click If Session("list") IsNot Nothing Then For Each element In Session("list").listCheckBox If element.checkBox.Checked Then 'Traitement End If Next End If End Sub

---

Tous les checked sont a false :?

Et si tu castes Session("list") ?

Bonjour DotNetMatt,

J'ai effectué un cast :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

Protected Sub ButtonExportExcel_Clik() Handles ButtonExport.Click Dim batchValue = TryCast(Session("list"), BatchValues)   If batchValue IsNot Nothing Then For Each element In batchValue.listCheckBox If element.checkBox.Checked Then 'Traitement End If Next End If End Sub

---

Les valeurs associées aux checkbox sont bonnes sauf le "Checked" qui n'est pas "mis à jour".

element.checkBox.Checked reste à false :?

Est-ce que dans ta méthode _batchValues.gridViewValues() tu mets bien des valeurs à true dans le résultat retourné ?

C'est à dire des valeurs à true ?

Cette méthode fait partie de mon "controller" me crée des gridViews et checkbox dynamiquement que je stocke dans un PlaceHolder via un ph.Controls.Add(..)

Désolé, la question n'était pas claire et en plus je me suis embrouillé les pinceaux dans ton code :aie:

Est-ce que dans la variable _batchValues tu as bien les bonnes valeurs ? Est-ce que tu récupères correctement l'état des controls (checked ou non) ?

Met un point d'arrêt sur la ligne Session("list") = _batchValues de la méthode ButtonSearch_Click et regarde son contenu afin d'en être sûr.

Pas de souci ;)

Dans mon message #11, je disais :

"Les valeurs associées aux checkbox sont bonnes sauf le "Checked" qui n'est pas "mis à jour"."

Voici un screen :

Pièce jointe 126984
Désolé je suis obligé de cacher les valeurs ..

Et quand je coche les cases avant d'appuyer sur mon bouton, ca ne met pas à jour le "chekced" :?

Un conseil si tu utilises ta list seulement sur cette page et que tu en as pas besoin sur d'autre page utilise le Viewstate plutôt que la Session.

@youtpout978 : Merci pour ta réponse,
Si j'utilise un ViewState("list") = _batchValues, ca ne fonctionne pas, ca me met une erreur :

Une exception non gérée s'est produite lors de l'exécution de la requête Web actuelle. Les informations relatives à l'origine et l'emplacement de l'exception peuvent être identifiées en utilisant la trace de la pile d'exception ci-dessous.

Sans l'erreur ça sera difficile de t'aider, normalement si l'objet tu peux le stocker en session tu peux aussi le stocker dans le viewstate à part si tu empêche l'utilisation du viewstate, reste en session si le viewstate ne fonctionne pas.

Ok en fait avec ton autre post je pense que tu as un problème dans la gestion du cycle de vie de ta page.

Comme tu peux le voir sur la page ASP.NET Page Life Cycle Overview, la gestion du postback se fait après le load. Donc il est probable que ton GridView soit réinitialisé à chaque fois, d'où le fait que tu ne récupères jamais les valeurs checked = true.

@youtpout978 : Effectivement, désolé :?

Voici l'erreur :

Citation:

---

Le type 'CheckBoxCustom' dans l'assembly 'App_Code.qzxkpeim, Version=0.0.0.0, Culture=neutral, PublicKeyToken=null' n'est pas marqué comme sérialisable.

Description : Une exception non gérée s'est produite au moment de l'exécution de la requête Web actuelle. Contrôlez la trace de la pile pour plus d'informations sur l'erreur et son origine dans le code.

Détails de l'exception: System.Runtime.Serialization.SerializationException: Le type 'BatchValues' dans l'assembly 'App_Code.qzxkpeim, Version=0.0.0.0, Culture=neutral, PublicKeyToken=null' n'est pas marqué comme sérialisable.

---

@DotNetMatt :

Comment je peux faire pour y remédier ?

Citation:

---

Envoyé par Jah73

@DotNetMatt :

Comment je peux faire pour y remédier ?

---

En vérifiant si tu es en postback ou non dans le Init :

Code:

---

1 2 3 4 5 6 7

Sub Page_Init(...) If (Not Page.IsPostBack) Then ' Ton code de création du GridView, que tu stockeras dans la session Else ' Si tu es en postback, tu récupères le gridview depuis la session End If End Sub

---

Ta customclass doit être sérialisable pour pouvoir être stocké dans le viewstate ou en session (normalement), pour cela il faut rajouter l'attribut Serializable à ta classe et autre classe qu'elle peut contenir.
http://msdn.microsoft.com/en-us/libr...code-snippet-2

@DotNetMatt: Merci pour tes réponses, on va y arriver ! :ccool:

Je pense que je n'ai pas procédé de le bonne manière :

Code:

---

1 2 3 4 5 6 7 8 9

Private _sessionBatch As New BatchValues 'Permet de créer des gridView vides   Protected Sub Page_Init(ByVal sender As Object, ByVal e As EventArgs) Handles Me.Init If Not Page.IsPostBack Then Session("list") = _sessionBatch Else _sessionBatch = Session("list") End If End Sub

---

Actuellement, si j’appuie sur le bouton Rechercher plusieurs fois, les gridviews s'additionnent; si j'appuie sur le bouton Export, ils disparaissent...

Galère :(


@youtpout978:
Merci, je m'occupe de ca dès que j'ai reglé le problème du dessus :ccool:

Citation:

---

Envoyé par youtpout978

Ta customclass doit être sérialisable pour pouvoir être stocké dans le viewstate ou en session (normalement)

---

Pour le ViewState oui il le faut, par contre pour la session il me semble que ce n'est obligatoire que dans le cas où on utilise le mode InProc, car l'objet SessionState est serializé avant d'être stocké dans la base de données. Mais à vérifier ^^

Citation:

---

Envoyé par Jah73

Actuellement, si j’appuie sur le bouton Rechercher plusieurs fois, les gridviews s'additionnent; si j'appuie sur le bouton Export, ils disparaissent...

---

Arf tu dois encore avoir quelque chose de fait au mauvais moment. Il faut que tu te documentes un peu plus sur le cycle de vie d'une page ASP.NET et savoir où doivent être réalisées certaines opérations. Par exemple en général on déclare les controls dans le Init, ensuite on fait le binding dans le Load. Cet ordre a son importance et doit donc être respecté afin d'avoir le comportement attendu, et c'est surtout le cas quand tu crées tes controls dans le code behind.

Il y a beaucoup de resources sur le sujet, en voici quelques unes :
- FAQ - Quel est le cycle de vie d'une page ?
- Tutoriel : Comprendre la création de contrôles ASP.NET et leur cycle de vie en C#

Sinon voici aussi deux articles en Anglais qui sont bien faits :
- ASP.NET Application and Page Life Cycle (rubrique In What Event We Should Do What?)
- ASP.NET Page Life Cycle

Si ça ne te suffit pas, cherche "asp.net page life cycle" sur Google ;)

Le sujet est plus côté ASP.NET, mais je viens le clore si quelqu'un est dans le même cas que moi.

J'ai suivi les conseils de DotNetMatt. J'ai réussi à stocker mes variables. Pour l'état de ma checkbox, j'ai fait un "_checkBox.AutoPostBack = True" afin qu'elle modifie dynamiquement son état

Si vous voulez plus de détail :

http://www.developpez.net/forums/d13...k/#post7519170

Merci à tous !