Discussion :

[Ashura95]

Bonjour !

Je me permets d'une fois encore demander des conseils ici...
J'ai surchargé mon main, de manière à récupérer une valeur dans ma fenêtre principale, qui se lance après la fermeture d'une autre.

En gros c'est le principe d'un login, l'utilisateur est sur la form1 du login, une fois ses informations vérifiées, son id passe en paramètre dans le main que j'ai surchargé avec main(int id).

Alors pas de soucis, une simple requête linq dans le main et je récupère bien mon user en tant qu'objet. Seul souci, c'est que une fois en dehors du main, mes variables sont à "null".

Exemple :

dans la fenêtre login si le login et mot de passe correspondent dans la base :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
new Main(_login);

dans la form principale :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
 
User _currentUser;
 
public Main()
       {
             Form1 f1 = new Form1();
             f1.ShowDialog();
             InitializeComponent();
       }
 
public Main(int id)
       {
            myDataContext = new maBaseDataContext();
            _currentUser = myDataContext.User.Single(u => u.UserID == id);
       }

Quand je fais un MessageBox.Show(_currentUser.Nom) par exemple dans le Main surchargé, aucun souci, il s'affiche.

Par contre, dans le load, si j'essaye de faire le même messageBox, la valeur est à null...Quelqu'un aurait-il une solution ?

Merci d'avance.

[benji_dv]

euh ... pas compris, peux tu ré expliquer avec d'autres termes...

[Ashura95]

euh ... pas compris, peux tu ré expliquer avec d'autres termes...

Oui bien sûr !

En gros, j'ai deux formulaires, le premier est Form1, le deuxième Main.
Le form1 est un formulaire de login, qui demande donc à l'utilisateur un identifiant et un mot de passe, si ces deux sont corrects, alors le Main se lance en prenant en paramètre l'identifiant de l'utilisateur.

J'ai donc l'identifiant qui passe en paramètre dans le constructeur du Main, de la manière suivante : new Main(int identifiant).

Le seul souci, c'est que même si je créé une variable en dehors du main de type int, comme in _userID, et que dans le Main, je lui donne comme valeur l'identifiant en faisant _userID = identifiant, cette variable se retrouve à null dans le load, alors que quand elle est dans le Main, elle possède bien la valeur de l'identifiant.

C'est plus clair ?

En résumé, je ne peux utiliser la valeur passée de la Form1 vers mon programme principal autre part que dans le constructeur...

[benji_dv]

Ok,
N'étant pas devant ton poste, en débug, on va procéder par élimination car dans ton code, il y a des choses... à ne pas faire.

Un constructeur est une méthode trèès particulière. Cela se remarque d'autant plus dans les écrans, controles utilisateurs etc.

Je m'explique : avant le passage dans le constructeur, l'objet n'existe pas !
Tant quel e constructeur n'a pas fini sa besogne... l'objet n'existe pas !

Donc, dans un constructeur il ne faut mettre QUE ce qui est nécessaire à l'instanciation de l'objet, ni plus, ni moins !

Le constructeur d'objets graphiques inclus l'appel à InitializeComponents...
attention, cette méthode sert justement à instancier tous les composants d'une forme ... donc, tant que la méthode ne s'est pas déroulée, aucun controle n'existe sur l'écran !!!
Dans ton second constructeur ... il n'y a pas l'appel à InitializeComponents... donc, il ne crée pas les composants graphiques !

Ton code (dans la majorité des cas) doit se situer après cette méthode.

Une fois l'objet créé, il est impossible d'appeler le constructeur comme une "simple méthode".

Dans ton code, le messagebox, tu le réalises dans le constructeur : à mon humble avis, ce n'est pas sa place.

Pour ce genre d'opération, tu disposes de l'évènement Form_Loaded.

Donc, je pense que ton code doit être refactorisé de la sorte :

- Suppression du constructeur à un paramètre...

- Ajout d'une méthode à ta forme login, qui retourne l'identifiant
(ou autre chose, exemple : l'objet utilisateur !) de l'utilisateur
qui tente de se loguer.
Dans le code ci dessous, la méthode se nomme GetIdLoggedUser()

- Constructeur sans paramètre + méthode attachée à l'évènement Load

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
 
public Main()
{
 
    InitializeComponent();
    this.Load +=new EventHandler(Form1_Load); 
}
 
private void Main_Load(object sender, EventArgs e)
{
     LoginForm login = new LoginForm();
     login.ShowDialog()
 
     Int32? id = login.GetIdLoggedUser();
 
     if(id == null)
     {
        // traitement si l'utilisateur n'a pas pu se loguer exemple : application.exit();
     }
     else
     {
        myDataContext = new maBaseDataContext();
        _currentUser = myDataContext.User.Single(u => u.UserID == id);
     }
}

-
qu'en penses tu ?

[Ashura95]

Super ça fonctionne !

Merci beaucoup, surtout d'avoir pris le temps de comprendre et d'expliquer clairement, c'est nettement plus adroit de cette manière, j'aurais effectivement dû y penser ! =)

[benji_dv]

Avec plaisir
Lorsque je peux, je prends le temps !
@+ et bon courage pour la suite