[Débutant] Conversion de string en int

Conversion de string en int

Bonjour,

j'ai eu un problème dans mon code ci-dessous l'erreur est de type "Argument 1: conversion impossible de string en int.

Voila le code
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 Connexion con = new Connexion(); con.dataGet("select * From Doc where Satus = 'Actif'"); SqlDataReader MyReader; try { MyReader = con.cmd.ExecuteReader(); while (MyReader.Read()) { string SNom = MyReader.GetString("Nom"); comboBox2.Items.Add(SNom); } } catch(Exception ex) { }
l'erreur est au niveau de string SNom précisément c'est le Nom qui est souligné.
Pourriez vous m'aider?

Bonjour,

Le message est normal. La méthode SqlDataReader.GetString attend un integer en paramètre. Il s'agit du numéro de la colonne (base zéro) : Msdn. Il n'y a pas d'overload de cette méthode permettant de passer une string en paramètre.

Bonjour,

Comme dit par ebastien, il n'y a pas de surcharge qui prenne le nom de la colonne. Il faut donc le faire en deux étapes : récupérer le numéro de la colonne correspondante, et ensuite récupérer la valeur.

En rouge, mes modifications :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 using(Connexion con = new Connexion()) { con.dataGet("select * From Doc where Satus = 'Actif'"); SqlDataReader MyReader; try { using(MyReader = con.cmd.ExecuteReader()) { int colNom = MyReader.GetOrdinal("Nom"); while (MyReader.Read()) { string SNom = MyReader.GetString(colNom); comboBox2.Items.Add(SNom); } } } catch(Exception ex) { } }
A noter que j'ai aussi libéré les ressources qui ont besoin d'être libérées (la connexion et le reader), via l'usage du mot clé using

1 pièce(s) jointe(s)

Merci pour vos message.
dans mon cas je souhaite afficher toutes les informations de mon table ci dessous dans le combobox nom. En sachant que la colonne Nom_Famille est la troisième colonnes. En comptant normalement colonne 0 correspond à la Civilité, colonne 1 correspond au Matricule et 2 correspond au Nom_Famille voir image. Pièce jointe 265575
J'ai fait ça

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
void Combobox()
        {
            Connexion con = new Connexion();
            con.dataGet("select * From Doc where Satus = 'Actif'");            
            SqlDataReader MyReader;
            try
            {
                MyReader = con.cmd.ExecuteReader();
                int colNom = MyReader.GetOrdinal("Nom_Famille");
                while (MyReader.Read())
                {
                    string SNom = MyReader.GetString(2);
                    comboBox2.Items.Add(SNom);
                }
            }
            catch(Exception ex)
            {
 
            }
        }

J'ai ajouté dans mon

Code:

1
2
3
4
5
public Accès()
        {
            InitializeComponent();
            Combobox();
        }

Mais rien ne s'affiche dans mon combobox.

Je serais toi, je supprimerai le try/catch dans un premier temps. Car pour l'instant, s'il y a une exception, comme le catch attrape tout mais ne fait rien, tu ne vois rien ! Et je suis prêt à parier qu'il y a une exception qui se produit à un moment ou à un autre...

en élevant le try et catch il y a une erreur de type "Informations supplémentaires : La référence d'objet n'est pas définie à une instance d'un objet." l'erreur est au niveau de MyReader = con.cmd.ExecuteReader()

Citation:

Envoyé par Dachetache

en élevant le try et catch il y a une erreur de type "Informations supplémentaires : La référence d'objet n'est pas définie à une instance d'un objet." l'erreur est au niveau de MyReader = con.cmd.ExecuteReader()

Cela est donc normal que ça ne fonctionne pas.
Tu as une violation d'accès et plutôt que de chercher à comprendre où çà plante, tu te contente de masquer l'exception.
C'est un peu comme te cogner contre un poteau et de continuer à foncer dedans après avoir pris un cachet d'aspirine.

A priori je dirai que "cmd" est à null.

Pour éviter le problème des colonnes j'ai pour habitude de ne jamais mettre select *.
Je renseigne toujours les colonnes qui m'intéressent.
Cela évite de se poser des questions quand à l'ordre des champs (ils sont dans l'ordre où on les a noté)
Cela évite de passer par un GetOrdinal
Cela évite d'avoir un champ de type Guid en plus en cas de réplication.

Au passage, ça compile avec un accent sur "Accès" ?

Bonjour,

J'ai trouvé la solution de mon problème. Merci pour vos conseils