Discussion :

[Dachetache]

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 :

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

[ebastien]

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.

[François DORIN]

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 :

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

[Dachetache]

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.
J'ai fait ça

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

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

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

Mais rien ne s'affiche dans mon combobox.

[François DORIN]

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

[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()

[popo]

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" ?