probleme requete sql select

Version imprimable

Bonjour

Code:

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.

14/03/2011, 17h25
Pol63

utilises un datareader plutot qu'un dataadapter
14/03/2011, 18h21
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 :?
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 ?!

:arf:
14/03/2011, 19h10
Pol63

à mon avis c'est les 3/4 des tutoriels, qui expliquent : un command, un dataadapter, un dataset et un datagridview ...
14/03/2011, 20h47
olibara

Citation:

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

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)
14/03/2011, 21h00
Pol63

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

Citation:

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

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

Citation:

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

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

Citation:

Envoyé par TomLev

Code:

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:

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

14/03/2011, 22h58
tomlev

Ton "moins pire" rend le code plus lisible mais ne règle pas le problème des apostrophes ;)
15/03/2011, 06h45
olibara

Salut Tomlev

Citation:

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 ?

Citation:

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

(il n'y a pas que les apostrophe)
15/03/2011, 09h46
tomlev

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

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

Code:

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:

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

Citation:
Envoyé par yassin123441
que j'ai remplace par mon code
Code:

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();
Argh :aie:
Tu n'as rien lu du débat ci-dessus à propos des DataAdapter si j'ai bien compris :?

15/03/2011, 12h13
yassin123441

j'ai bien lu tout le débat :)
15/03/2011, 12h14
tomlev

Citation:

Envoyé par yassin123441

j'ai bien lu tout le débat :)

Mais tu t'en fous pas mal on dirait :aie:
Enfin bon, tu fais ce que tu veux...