Discussion :

[DarkSeiryu]

Bonsoir !

Tout est dans le titre, mais pour détailler j'ai une méthode qui me sert à faire des requêtes SQL, et qui retourne une List<string> remplie par une DataTable, elle même remplit par un DataAdapter.

Le souci est que lorsque j'essaie d'afficher une des lignes de ma List<string> dans un Label, le seul texte présent est :

System.Data.DataRow

Voici le code de ma méthode SQL :

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
public List<string> SQLSelect_nx1(string unChamp, string uneTable)
    {
        this.fctConnectDB();
        OdbcCommand cmdSelect = new OdbcCommand();
        cmdSelect.Connection = connectionDB;
        cmdSelect.CommandText = "SELECT " + unChamp + " FROM " + uneTable + ";";
        OdbcDataAdapter my_DA = new OdbcDataAdapter(cmdSelect);
        DataTable my_DT = new DataTable();
        my_DA.Fill(my_DT);
 
        List<string> myList = new List<string>();
        for (int i = 0; i < my_DT.Rows.Count; i++)
        {
            myList.Add(my_DT.Rows[i].ToString());
        }
        this.fctDisconnectDB();
        return myList;
    }

Le code de ma méthode qui appelle la méthode SQL :

Code :

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

1
2
3
4
5
6
7
public string getDemonsSauvage()
    {
        List<string> listNomDemons = cs_SQL.SQLSelect_nx1("demNom", "demons");
        int i = this.fctRandom(0, listNomDemons.Count);
        string nomDemons = listNomDemons[i];
        return nomDemons;
    }

Le code de la méthode fctRandom() :

Code :

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

1
2
3
4
5
6
private int fctRandom(int nbDepart, int nbFin)
    {
        Random rdm = new Random();
        int i = rdm.Next(nbDepart, nbFin);
        return i;
    }

Et enfin le code qui me permet d'afficher le texte dans mon label :

Code :

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

lb_nom_demon.Text = cs_WD.getDemonsSauvage();

Comment faire pour que ce soit le texte de ma table qui soit retourné ?

Par avance, merci.

[tomlev]

Code :

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

myList.Add(my_DT.Rows[i].ToString());

La classe DataRow ne redéfinit pas la méthode ToString, donc ToString renvoie le nom du type (c'est l'implémentation par défaut, héritée de object).

Il faut récupérer un champ spécifique de la DataTable :

Code :

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

myList.Add(my_DT.Rows[i][unChamp] as string);

[DarkSeiryu]

Merci tomlev, ça marche niquel !

[DarkSeiryu]

Par contre, j'ai un souci sur une autre 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
public List<string> SQLSelect_nxn(string uneTable)
    {
        this.fctConnectDB(); // Appelle de la méthode de connexion.
        OdbcCommand cmdSelect = new OdbcCommand(); // Instancie une nouvelle commande SQL.
        cmdSelect.Connection = connectionDB; // Indique que la commande est connectée à la DB que la connexion définit.
        cmdSelect.CommandText = "SELECT * FROM " + uneTable + ";"; // Commande SQL.
        OdbcDataAdapter my_DA = new OdbcDataAdapter(cmdSelect);
        DataTable my_DT = new DataTable();
        my_DA.Fill(my_DT);
 
        List<string> myList = new List<string>();
        for (int i = 0; i < my_DT.Rows.Count; i++)
        {
            myList.Add(my_DT.Rows[i][unChamp] as string);
        }
        this.fctDisconnectDB();
        return myList;
    }

La seule différence est au niveau de la requête, donc comment faire pour que ma List<string> soit rempli avec le résultat de la requête ?

Merci.

[tomlev]

Ben ça a pas vraiment de sens... tu fais un "SELECT *", donc a priori tu récupères plusieurs champs. Qu'est-ce que tu veux mettre dans ta List<string> exactement ? Si c'est un champ en particulier, met le nom du champ à la place de unChamp

[DarkSeiryu]

Non non, je veux récupérer toutes les données d'une table, et retourner une List<string> qui contient ces données.

[tomlev]

Dans ce cas il faut que tu concatènes les champs de chaque DataRow
Mais ça me parait pas très pratique d'avoir une chaine qui contient toutes les données de la ligne concaténées
Tu veux en faire quoi après de cette List<string> ?

[DarkSeiryu]

Pour l'instant rien de prévu, c'est juste que je prévois de faire une méthode avec ce genre de requête pour plus tard car il est fort possible qu'on ait a en utiliser pour notre site.

Après je pense qu'on s'en servira pour afficher certaines données, telles des images par exemple.

On fait la List<string> dans la méthode de la classe SQL, et après on tri les données dont on a besoin.

'suis pas sur que ce soit vraiment utile mais c'est ce qui m'a été demandé de faire donc voilà.

[tomlev]

Ben je vois pas trop l'intérêt de faire ça, autant utiliser directement la DataTable...
Mais bon, si on t'a demandé de le faire... y a sûrement une bonne raison

[olibara]

J'avoue que c'est franchement bizarre !
Pourquoi ne pas utiliser directement la datatable

Ca m'interpelle d'autant plus qu'aujourd'hui meme je cherche la meilleure methode pour faire le chemin inverse : creer une datatable a partir d'une List<T>
Parce que la datatable dispose des interface facilitant les tri les filtre le binding en dgv etc. avec Une List<T> il faut definir tout ca !

Je ne dis pas qu'une liste est tres utile mais pour certaine manipulation la datatable peut etre tres pratique or dans ton cas elle est livrée toute chaude depuis le producteur

[DarkSeiryu]

Oui vous avez raison, le mieux est de retourner la DataTable directement, et faire le tri une fois que j'ai effectué un appel de la méthode.

Merci !

[olibara]

Code :

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

et après on tri les données dont on a besoin

On trie ou on Filtre ? Ce n'est pas la meme chose !!

Ces concept on l'ai d'etre asser souvent mélangés

Mais de toute facon avec un DataView sur ta datatable, tu trie et tu filtre tout ce que tu veux

[DarkSeiryu]

Oui oui trier et filtrer sont différents on est bien d'accord ^^

Dans mon cas ce sera surement un tri ^^

Bref je verrai bien, merci encore !