Discussion :

[DarkSeiryu]

Bonsoir à tous.

Je suis actuellement en train de développer une classe SQL pour la réalisation d'un projet, et j'ai un problème pour compter le nombre d'enregistrement dans un DataReader dans une méthode. Voici le code de ma méthode :

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
public string[] SQLSelect_nx1(string unChamp, string uneTable)
{
    this.fctConnectDB();
    OdbcCommand cmdSelect = new OdbcCommand();
    cmdSelect.Connection = connectionDB;
    cmdSelect.CommandText = "SELECT " + unChamp + " FROM " + uneTable + "";
    OdbcDataReader DR;
    DR = cmdSelect.ExecuteReader();
 
    DR.Read();
    int i = DR.VisibleFieldCount;
    string[] tabJeuEnregistrement = new string[i];
    for (int j = 0; j < tabJeuEnregistrement.Length; j++)
    {
        for (int k = 0; k < i; k++)
        {
            tabJeuEnregistrement[j] = DR.GetString(k);
        }
    }
    DR.Close();
 
    fctDeconnectDB();
    return tabJeuEnregistrement;
}

Tel que je le fais là, DR.VisibleFieldCount ne me renvoi qu'un seul enregistrement, alors que ma table en contient 9.

Comment faire pour compter le nombre d'enregistrement dans un DataReader ?

Merci d'avance.

[tomlev]

Tel que je le fais là, DR.VisibleFieldCount ne me renvoi qu'un seul enregistrement, alors que ma table en contient 9.

VisibleFieldCount renvoie le nombre de champs (colonnes), pas le nombre d'enregistrements (lignes).

Comment faire pour compter le nombre d'enregistrement dans un DataReader ?

Il faut lire le reader jusqu'au bout... Les enregistrements ne sont pas vraiment "dans" le DataReader, ce dernier pointe sur 1 enregistrement à la fois. La plupart des SGBD ne renvoient pas le nombre total de lignes de résultat, car ils commencent généralement à renvoyer des résultats avant de savoir combien il y en a...

[DarkSeiryu]

Merci de ta réponse.

Il faut lire le reader jusqu'au bout...

D'accord, mais comment je fais ça ?

[tomlev]

D'accord, mais comment je fais ça ?

Ben tu appelles la méthode Read en boucle, jusqu'à ce qu'elle renvoie false...

Code :

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

1
2
3
4
5
n = 0;
while(DR.Read())
{
    n++;
}

[DarkSeiryu]

ok merci j'essaierai ça ce soir là je peux pas.

Par contre juste une question, vu qu'il y a while(DR.Read()), est-ce qu'il est utile de mettre DR.Close() à la fin du while ou une fois que j'ai rempli mon tableau ?
En clair 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
int n;
		while (DR.Read())
		{
			n++
		}
		string[] tabJeuEnregistrement = new string[n];
		for (int j = 0; j < n; j++)
		{
			for (int k = 0; k < n; k++)
			{
				tabJeuEnregistrement[j] = DR.GetString(k);
			}
		}
		DR.Close();
        fctDeconnectDB();
        return tabJeuEnregistrement;

[tomlev]

Par contre juste une question, vu qu'il y a while(DR.Read()), est-ce qu'il est utile de mettre DR.Close() à la fin du while ou une fois que j'ai rempli mon tableau ?

Ben il faut remplir ton tableau dans le while... une fois que tu as lu entièrement le reader, tu ne peux plus revenir en arrière. Ton dernier code ne fonctionne pas à mon avis, vu que tu as déjà atteint la fin du reader...

Pourquoi tu utilises un tableau ? Ce n'est vraiment pas très pratique quand tu ne connais pas à l'avance le nombre de lignes...
Utilise plutôt une List<string>, ou une DataTable...

Sinon, plutôt que de faire explicitement le Close, tu peux aussi utiliser un bloc using :

Code :

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

1
2
3
4
using(OdbcDataReader DR = cmdSelect.ExecuteReader())
{
    ...
}

La méthode Dispose sera automatiquement appelée à la fin du bloc, et cette méthode appelle Close

[Skalp]

Pourquoi pas un DataSet ?

[DarkSeiryu]

Pourquoi tu utilises un tableau ? Ce n'est vraiment pas très pratique quand tu ne connais pas à l'avance le nombre de lignes...
Utilise plutôt une List<string>, ou une DataTable...

Je veux que ma méthode me retourne un tableau car j'ai besoin ensuite dans une autre classe de faire un random sur le tableau retourné et qu'ensuite, grâce au nombre obtenu par le random, j'affiche le string correspondant dans la ligne de mon tableau, comme ceci :

Code :

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

1
2
3
4
5
6
7
8
public string showDemons()
{
    string[] tabJeuEnregistrement = cs_SQL.SQLSelect_nx1("demNom", "demons");
    Random rdm = new Random();
    int i = rdm.Next(0, tabJeuEnregistrement.Length);
    string nomDemons = tabJeuEnregistrement[i];
    return nomDemons;
}

Et comment je fais dans ma classe SQL pour remplir mon tableau dans le while est ensuite le retourner ?
Je gallères à mort...

Merci d'avance !

EDIT : au fait, je fais ça pour un site (ASP.NET donc) et apparemment je ne peux pas utiliser de List<string>, peut-être est-ce parce que je suis sous VS 2008 ? Et je sais qu'il y a un forum spécial ASP mais comme ma question portait sur du C# pur, j'ai préféré venir ici...

[tomlev]

Je veux que ma méthode me retourne un tableau car j'ai besoin ensuite dans une autre classe de faire un random sur le tableau retourné et qu'ensuite, grâce au nombre obtenu par le random, j'affiche le string correspondant dans la ligne de mon tableau, comme ceci :

Ben tu peux très bien faire la même chose avec un List<string>...

EDIT: j'avais pas vu ton EDIT
Si tu es sous VS2008, tu as le framework 3.5, donc tu as forcément la classe List<T>. Ajoute un :

Code :

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

using System.Collections.Generic;

[DarkSeiryu]

Ah ouais moins casse-tête avec une List<string>

Par contre j'ai une erreur lors de la compilation :

Utilisation d'une variable locale non assignée "listNomDemons"

Voici le code de ma méthode (l'erreur se situe à la ligne en gras) :

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
public List<string> SQLSelect_nx1(string unChamp, string uneTable)
{
    this.fctConnectDB();
    OdbcCommand cmdSelect = new OdbcCommand();
    cmdSelect.Connection = connectionDB;
    cmdSelect.CommandText = "SELECT " + unChamp + " FROM " + uneTable + ";";
    OdbcDataReader DR;
    DR = cmdSelect.ExecuteReader();

    List<string> listNomDemons;
    int i = 0;
    while (DR.Read())
    {
        listNomDemons.Add(DR.GetString(i));
        i++;
    }
    DR.Close();
    return listNomDemons;
}

[tomlev]

Ben le message est assez clair je trouve... tu as déclaré listNomDemons, mais tu ne l'as pas initialisé :

Code :

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

List<string> listNomDemons = new List<string>();

[DarkSeiryu]

Ah ben oui effectivement... Comment j'ai pu oublier ça ?

Enfin bref, maintenant ça marche niquel.

Ce qui m'étonne d'ailleurs lol

Code :

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

1
2
3
4
5
6
7
8
List<string> myList = new List<string>();
int i = 0;
while (DR.Read())
{
    myList.Add(DR.GetString(i));
}
DR.Close();
return myList;

Comme on peut le voir, je laisse toujours i à 0 et pourtant myList reçoit bien tous les enregistrements qui sont dans le DataReader, et j'ai du mal à comprendre pourquoi... Parce que ça ne renvoit qu'un enregistrement à la fois comme tu me l'as dit y'a deux jours tomlev ?

[tomlev]

GetString(i) signifie que tu récupères le "i-ième" champ de l'enregistrement courant. C'est l'index de la colonne, pas de la ligne... Et comme tu ne sélectionnes qu'une colonne, tu fais toujours GetString(0), la variable i est donc inutile...

Quelques petits rappels sur les DataReaders :
http://dotnet.developpez.com/articles/ado1/csharp/

[DarkSeiryu]

ok niquel ^^

Merci bien tomlev !