Conversion impossible !

Version imprimable

24/05/2017, 18h52
vic1997

2 pièce(s) jointe(s)

Conversion impossible !

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 :
Pièce jointe 278918

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) :

Pièce jointe 278921

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

Merci à vous !
25/05/2017, 03h09
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]?
25/05/2017, 15h56
vic1997

2 pièce(s) jointe(s)

Citation:

Envoyé par 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]?

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 :
Pièce jointe 279272

cependant ça me donne cette erreur :

Pièce jointe 279257
25/05/2017, 16h06
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/
25/05/2017, 16h44
vic1997

Citation:

Envoyé par 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/

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 !

En fait, voici le comportement :

Code:

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.

Citation:
Envoyé par jcdentons
En fait, voici le comportement :
Code:

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 ?

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

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
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);
  }
}

Citation:

Envoyé par jcdentons

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

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
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 !

29/05/2017, 01h04
vic1997

3 pièce(s) jointe(s)

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 !

Pièce jointe 280258

EDIT :

J'ai finalement peut-être trouvé, j'ai juste rajouté ça :
Pièce jointe 280275

Cependant j'obtiens désormais cette erreur :

Pièce jointe 280276

Sauriez vous d'où ça peut venir ?
29/05/2017, 06h24
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.