probleme requete sql select

**yassin123441** · 14/03/2011, 17h33

Bonjour

Code :

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

1
2
3
4
5
6
7
            string repertoire_a_supprime = "";
 
            SqlDataAdapter adapter = new SqlDataAdapter(" select repertoire from fichier where nom='" + repertoire_a_supprime + "'", Classe_methode_statique.con);
 
// issi comment récupéré la valeur
 
            System.IO.File.Delete(reprtoire;

comment extraire la repertoire du tableaux pour surprime le fichier ?
Merci.

**Pol63** · 14/03/2011, 18h25

utilises un datareader plutot qu'un dataadapter

**tomlev** · 14/03/2011, 19h21

J'aimerais bien comprendre pourquoi 90% des débutants se croient obligés d'utiliser des DataAdapter, et ne savent pas utiliser un DataReader

Ca fait des années que cette question me turlupine : où est ce tutoriel débile qui apprend à utiliser un DataAdapter même pour récupérer une seule valeur ?!

**Pol63** · 14/03/2011, 20h10

à mon avis c'est les 3/4 des tutoriels, qui expliquent : un command, un dataadapter, un dataset et un datagridview ...

**olibara** · 14/03/2011, 21h47

Envoyé par Tomlev

J'aimerais bien comprendre pourquoi 90% des débutants se croient obligés d'utiliser des DataAdapter, et ne savent pas utiliser un DataReader

C'est marrant j'avais juste l'impression inverse

Il faut dire que dans 80% des cas un DataAdapter c'est quand meme plus pratique qu'un DataReader

Et dans 15% du reste quand on veut juste recuperer une donnée on peut faire un executeScalar

Personellement je n'utilise le datareader que quand je dois recuperer un stream de données binaires comme des images ou de la musique (et chaque fois je peste contre ceux qui on jugé utile de mettre de tels fichiers en DB)

**Pol63** · 14/03/2011, 22h00

comme quoi c'est une question d'habitude ...
on a des dizaines de milliers de lignes de code et personne dans l'équipe de sait à quoi sert le dataadapter ^^

**tomlev** · 14/03/2011, 22h04

Envoyé par olibara

Il faut dire que dans 80% des cas un DataAdapter c'est quand meme plus pratique qu'un DataReader

Et dans 15% du reste quand on veut juste recuperer une donnée on peut faire un executeScalar

Je ne pense pas qu'on puisse vraiment donner des proportions chiffrées, ça dépend complètement des projets... selon les cas, l'un est plus pratique que l'autre, c'est tout.

Mais quand c'est pour récupérer une valeur, ou même une ligne de données, utiliser un DataAdapter, c'est vraiment sortir l'artillerie lourde pour écraser une mouche.

D'autre part, le mode déconnecté (DataAdapter + DataSet) suppose de charger toutes les données en mémoire, ce qui n'est pas forcément adapté si tu as beaucoup de données à traiter. Pour un processus de type "batch" qui doit traiter des millions d'enregistrement, tu es pratiquement obligé d'utiliser un DataReader, ou alors tu vas utiliser une quantité monstrueuse de mémoire...

Mais ce qui m'agace le plus, c'est quand je vois quelqu'un qui crée un DataAdapter, lui affecte une SelectCommand, puis utilise la commande en faisant adapter.SelectCommand.ExecuteReader().... ça prouve vraiment qu'il a rien compris à ce qu'est un DataAdapter

Bref, je ne condamne pas les DataAdapter en tant que tels, je critique seulement leur utilisation à mauvais escient, très fréquente chez les débutants...

Envoyé par olibara

(et chaque fois je peste contre ceux qui on jugé utile de mettre de tels fichiers en DB)

Tu n'as pas toujours d'autres options... par exemple si la DB est distante et que tu n'as pas accès au système de fichier distant, mettre les images dans la DB est une bonne solution.

Bref, tout ça ne fait pas beaucoup avancer le problème de yassin123441

D'ailleurs, dans ce cas il n'y a même pas besoin de DataReader, il suffit de faire un ExecuteScalar sur la commande puisqu'il n'y a qu'une valeur à récupérer... voilà le code :

Code :

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

1
2
3
            SqlCommand command = new SqlCommand("select repertoire from fichier where nom='" + repertoire_a_supprime + "'", Classe_methode_statique.con);
            string repertoire = command.ExecuteScalar() as string;
            System.IO.File.Delete(repertoire);

Soit dit en passant, évite de construire des chaines par concaténation... Avec ce code, si repertoire_a_supprimer contient une apostrophe, la commande générée sera incorrecte et causera une erreur. Utilise plutôt des requêtes paramétrées comme expliqué dans ce tuto

**olibara** · 14/03/2011, 22h59

Envoyé par TomLev

Code :

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

1
2
3
            SqlCommand command = new SqlCommand("select repertoire from fichier where nom='" + repertoire_a_supprime + "'", Classe_methode_statique.con);
            string repertoire = command.ExecuteScalar() as string;
            System.IO.File.Delete(repertoire);

Soit dit en passant, évite de construire des chaines par concaténation...

Ca donne

Code :

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

1
2
3
4
5
6
7
8
9
10
11
// Au Moins pire :
string Select = String.Format("select repertoire from fichier where nom='{0}'", repertoire_a_supprime);
SqlCommand MyCmd = new SqlCommand(Select, Classe_methode_statique.con);
// etc ..
// (mais tu risque toujours une injection sql et des tas d'autres problèmes)
 
// Sinon ceci c'est Mieux 
 
string Select = String.Format("select repertoire from fichier where nom=@RepSup";
SqlCommand MyCmd = new SqlCommand(Select, Classe_methode_statique.con);
      MyCmd.Parameters.AddWithValue("@RepSup", repertoire_a_supprime);

**tomlev** · 14/03/2011, 23h58

Ton "moins pire" rend le code plus lisible mais ne règle pas le problème des apostrophes

**olibara** · 15/03/2011, 07h45

Salut Tomlev

Ton "moins pire" rend le code plus lisible mais ne règle pas le problème des apostrophes

C'est ce que j'avais dis ... non ?

// etc ..
// (mais tu risque toujours une injection sql et des tas d'autres problèmes)

(il n'y a pas que les apostrophe)

**tomlev** · 15/03/2011, 10h46

ah oui, j'avais pas fait gaffe au commentaire du dessous...

**yassin123441** · 15/03/2011, 11h06

merci
se code de tomlev ma bien aide pour récupéré une seule valeur dans mes prochaine requete ,

Code :

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

1
2
3
            SqlCommand command = new SqlCommand("select repertoire from fichier where nom='" + repertoire_a_supprime + "'", Classe_methode_statique.con);
            string repertoire = command.ExecuteScalar() as string;
            System.IO.File.Delete(repertoire);

que j'ai remplace par mon code

Code :

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

1
2
3
4
SqlDataAdapter adapter = new SqlDataAdapter(" select repertoire from fichier where nom='" + nom_a_supprime + "'", Classe_methode_statique.con);
            DataTable dtt = new DataTable();
            adapter.Fill(dtt);
            repertoire_a_supprime = dtt.Rows[0][0].ToString();

**tomlev** · 15/03/2011, 13h08

Envoyé par yassin123441

que j'ai remplace par mon code

Code :

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

1
2
3
4
SqlDataAdapter adapter = new SqlDataAdapter(" select repertoire from fichier where nom='" + nom_a_supprime + "'", Classe_methode_statique.con);
            DataTable dtt = new DataTable();
            adapter.Fill(dtt);
            repertoire_a_supprime = dtt.Rows[0][0].ToString();