Discussion :

[vic1997]

Bonjour,

J'ai un petit problème, j'ai une classe Joueur et j'aimerais créer une instance de celle-ci, cependant j'ai un paramètre du constructeur qui est lui même une instance d'un autre objet (Poste), et je ne sais pas comment m'y prendre. Je suis débutant et je comprends pas super bien l'objet donc si ma phrase ne veut rien dire c'est peut-être normal !

Voici l'erreur que j'ai :


et voici ce que j'ai besoin pour créer un nouveau Joueur (Joueur qui hérite d'une autre classe mais je crois qu'on a pas besoin de le savoir ici, je le dis quand même) :



Voilà, j'espère que vous pourrez m'aider !

Merci à vous !

[nouanda]

Et bien le constructeur de ton objet Joueur attend en sixième paramètre un objet Poste. Or tu lui passes un objet string.
Quel est le type de dr[5]?

[vic1997]

Et bien le constructeur de ton objet Joueur attend en sixième paramètre un objet Poste. Or tu lui passes un objet string.
Quel est le type de dr[5]?

Justement dans mon code je veux transformer dr[5] en objet Poste. dr[5] comporte un id sous forme de string apparemment.

Pour résoudre ce problème j'ai fais ce que j'ai entouré en rouge dans le screens suivant :


cependant ça me donne cette erreur :

[jcdentons]

Bonjour vic1997,

Il faut fermer la connexion lorsque vous avez passé votre requête, et la rouvrir avant d'en exécuter une autre.

Le mieux est d'utiliser l'instruction using sur la connexion et sur la commande SQL.
https://docs.microsoft.com/en-us/dot...sing-statement

De plus, il est préférable pour vos requêtes SQL d'utiliser des requêtes paramétrées.
http://webman.developpez.com/article...rameter/vbnet/

[vic1997]

Bonjour vic1997,

Il faut fermer la connexion lorsque vous avez passé votre requête, et la rouvrir avant d'en exécuter une autre.

Le mieux est d'utiliser l'instruction using sur la connexion et sur la commande SQL.
https://docs.microsoft.com/en-us/dot...sing-statement

De plus, il est préférable pour vos requêtes SQL d'utiliser des requêtes paramétrées.
http://webman.developpez.com/article...rameter/vbnet/

Merci pour votre réponse.
Cependant je ne comprends pas, je dois fermer ma connexion alors que juste après je dois l'utiliser pour lire dr[5] ? Je comprends surement mal.

La ligne new Joueur a besoin de la connexion, cependant elle demande à une autre classe quelque chose et celle ci me dit qu'il faut d'abord fermer la connexion, comment faire ? C'est pas logique pour moi !

[jcdentons]

En fait, voici le comportement :

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
public Joueur findById(string code)
{
  MySqlCommand md;
  String req = "SELECT ..."...
  cmd = new MySqlCommand(req, this.c);
  MySqlDataReader dr = cmd.ExecuteReader();
 
  PosteDAO pDAO = new PosteDAO();
  Joueur m = null;
  // Ici, la requête 1 est en train d'être lue.
  if (dr.Read()) 
  {
     m = new Joueur(
        Convert.ToInt32(dr[0]), 
        Convert.ToInt32(dr[1]), 
        Convert.ToInt32(dr[2]),
        dr[3].ToString(),  
        // La lecture de la requête n'est toujours pas terminée.
        pDAO.findById(dr[5].ToString())
        ...
  }
}

et dans findById, vous voulez ouvrir une seconde requête en parallèle sur la même connexion SQL alors que la requête 1 n'est pas terminée d'être exécutée.

[vic1997]

En fait, voici le comportement :

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
public Joueur findById(string code)
{
  MySqlCommand md;
  String req = "SELECT ..."...
  cmd = new MySqlCommand(req, this.c);
  MySqlDataReader dr = cmd.ExecuteReader();
 
  PosteDAO pDAO = new PosteDAO();
  Joueur m = null;
  // Ici, la requête 1 est en train d'être lue.
  if (dr.Read()) 
  {
     m = new Joueur(
        Convert.ToInt32(dr[0]), 
        Convert.ToInt32(dr[1]), 
        Convert.ToInt32(dr[2]),
        dr[3].ToString(),  
        // La lecture de la requête n'est toujours pas terminée.
        pDAO.findById(dr[5].ToString())
        ...
  }
}

et dans findById, vous voulez ouvrir une seconde requête en parallèle sur la même connexion SQL alors que la requête 1 n'est pas terminée d'être exécutée.

Merci pour votre réponse,

Oui c'est ce que j'ai compris, mais comment fermer la requête avant findbyid() alors que justement le dr[5] du findbyid() a besoin de la première requête.
Pouvez vous me montrer comment faire ?

[jcdentons]

Ne connaissant pas l'intégralité du code, je ferais quelque chose comme :

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
26
27
28
29
30
public Joueur findById(string code)
{
  MySqlCommand md;
  String req = "SELECT ..."...
  cmd = new MySqlCommand(req, this.c);
  MySqlDataReader dr = cmd.ExecuteReader();
 
  PosteDAO pDAO = new PosteDAO();
  Joueur m = null;
  string idPoste = null;
  // Ici, la requête 1 est en train d'être lue.
  if (dr.Read()) 
  {
     idPoste = dr[5].ToString();
     m = new Joueur(
        Convert.ToInt32(dr[0]), 
        Convert.ToInt32(dr[1]), 
        Convert.ToInt32(dr[2]),
        dr[3].ToString(),  
        // La lecture de la requête n'est toujours pas terminée.
        null
        ...
  }
  dr.Close();
  dr.Dispose();
  if (!String.IsNullOrEmpty(idPoste))
  {
     m.Poste = pDAO.findById(idPoste);
  }
}

[vic1997]

Ne connaissant pas l'intégralité du code, je ferais quelque chose comme :

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
26
27
28
29
30
public Joueur findById(string code)
{
  MySqlCommand md;
  String req = "SELECT ..."...
  cmd = new MySqlCommand(req, this.c);
  MySqlDataReader dr = cmd.ExecuteReader();
 
  PosteDAO pDAO = new PosteDAO();
  Joueur m = null;
  string idPoste = null;
  // Ici, la requête 1 est en train d'être lue.
  if (dr.Read()) 
  {
     idPoste = dr[5].ToString();
     m = new Joueur(
        Convert.ToInt32(dr[0]), 
        Convert.ToInt32(dr[1]), 
        Convert.ToInt32(dr[2]),
        dr[3].ToString(),  
        // La lecture de la requête n'est toujours pas terminée.
        null
        ...
  }
  dr.Close();
  dr.Dispose();
  if (!String.IsNullOrEmpty(idPoste))
  {
     m.Poste = pDAO.findById(idPoste);
  }
}

Punaise, merci beaucoup à toi ! J'ai enfin réussi ! Ca faisait quelques jours que j'étais bloqué dessus et ta solution a marché ! Merci !

[vic1997]

Bon finalement je reviens, j'ai un autre problème lié au même code, cependant ici c'est pour afficher une liste comportant tous les joueurs (alors qu'avant c'était un joueur avec ID précis).
J'ai tenté de faire le même code mais voilà j'ai cette erreur, comment m'en débarrasser ? Merci d'avance !



EDIT :

J'ai finalement peut-être trouvé, j'ai juste rajouté ça :


Cependant j'obtiens désormais cette erreur :



Sauriez vous d'où ça peut venir ?

[jcdentons]

Ça doit être un des Convert.ToInt32 : un des champs de dr doit être null ou n'est pas un entier. Regardez les valeurs dans le debugger.