[c#]Verifier qu'un enregistrement est present dans la BDD

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : [c#]Verifier qu'un enregistrement est present dans la BDD

herks

Salut,

j'ai créé un bout de code qui me permet de verifier si un element est present ou pas dans ma base.
Ca ne me semble pas vraiment optimisé. Voici le code

Code :

SqlCommand myCommand;
SqlDataReader myReader;
SqlConnection myConnection = (SqlConnection)Session["myConnection"];
string req_select = "SELECT * FROM matable WHERE id = @id";
myCommand = new SqlCommand(req_select, myConnection);
//creation et declaration des parametres
myCommand.Parameters.Add(new SqlParameter("@id", SqlDbType.Char, 100));
//Attribution des valeurs aux parametres
myCommand.Parameters["@id"].Value = "2";
myReader = myCommand.ExecuteReader();
int i = 0;
while (myReader.Read())
{
i++;
}
myReader.Close();
if (i >= 1)
Response.Write("Element present dans la base" );
else
Response.Write("Element pas present dans la base" );

Je suis preneur de tous conseil pour ameliorer et optimiser mon code.
Merci

Message édité par herks le 30-06-2005 à 11:06:32

Publicité

cesarr89

myReader vaut pas null normalement quand ca retourne rien?

---------------
!== Force et honneur ==!

herks

apparament ca retourne flase ou true
voici la nouvelle version du code:

Code :

SqlCommand myCommand;
SqlDataReader myReader;
SqlConnection myConnection = (SqlConnection)Session["myConnection"];
string req_select = "SELECT * FROM matable WHERE id = @id";
myCommand = new SqlCommand(req_select, myConnection);
myCommand.Parameters.Add(new SqlParameter("@id", SqlDbType.Char, 100));
myCommand.Parameters["@id"].Value = "2";
myReader = myCommand.ExecuteReader();
if (myReader.Read() == true)
Response.Write("Element present dans la base" );
else
Response.Write("Element pas present dans la base" );

si y'a d'autre soluce plus propre ou mieux optimisé je suis toujours preneur

cesarr89

Ca m'a l'air correct....faudrait un 'expert' pour t'en dire plus..genre FlorentG.

---------------
!== Force et honneur ==!

zetranber

Et pourquoi pas faire :

if ( reader.HasRows )

C'est plus simple quand même
Et ca t'evite une méthode de lecture qui sert a rien ^^

Message édité par zetranber le 30-06-2005 à 11:08:20

FlorentG

Unité de Masse

Ca à l'air potable, en effet. J'enlèverais juste la string req_select qui sert à rien, autant la foutre directe dans le constructeur. Aussi vu qu'on ne retourne qu'un truc, autant utiliser ExecuteScalar(), qui renvoi en même temps la première colonne du premier enregistrement, ou null si y'a rien.

Enfin, ça sert à rien de faire un SELECT *, vu qu'on ne fait que tester. Suffit d'un SELECT id, pas besoin de toute la ligne...

Code :

SqlConnection myConnection = (SqlConnection)Session["myConnection"];
SqlCommand myCommand = new SqlCommand("SELECT id FROM matable WHERE id = @id", myConnection);
myCommand.Parameters.Add(new SqlParameter("@id", SqlDbType.Char, 100));
myCommand.Parameters["@id"].Value = "2";
if (myCommand.ExecuteScalar() == null) // Vérifier syntaxe, je connais pas assez le cécharppe
Response.Write("Element pas present dans la base" );
else
Response.Write("Element present dans la base" );

moi23372

si tu veux pas avoir de probleme avec le null qui pose tjs probleme en général

tu changes légèrement la requete

SELECT count(id) FROM matable WHERE id = @id

voila et ça te renvera 0 s'il n'existe pas...

ATTENTION 0 n'est pas égale à NULL

FlorentG

Unité de Masse

Non, le null ne pose pas problème, vu que c'est un des retour possible de ExecuteScalar() [:spamafote]

moi23372

ici oui, mais dans d'autre cas il peut poser problème! de ce fait j'évite impérativement de retourner un null dans mes programmes! c'est une méthode de travail quoi! ici je ne dis pas que ça vient du null, mais moi personnellement je préfère l'éviter

FlorentG

Unité de Masse

*ouch* tu retourne quoi à la place ? :heink:

Publicité

ixemul

Nan mais sans blague ! ⚡

D'ailleur, il me vient une petite idee sur un debat à lancer...

Mieux vaut-il utiliser une instruction d'egalité sous la forme

Code :

if(MonObjet == null)

Ou plutot

Code :

if(MonObjet.Equals(null))

:??:

herks

ixemul a écrit :

D'ailleur, il me vient une petite idee sur un debat à lancer...

Mieux vaut-il utiliser une instruction d'egalité sous la forme

Code :

if(MonObjet == null)

Ou plutot

Code :

if(MonObjet.Equals(null))

:??:

Moi aussi ca m'interresse de savoir le pourquoi du comment.

En tout k merci a tous pour vos reponses

FlorentG

Unité de Masse

ixemul a écrit :

D'ailleur, il me vient une petite idee sur un debat à lancer...

Mieux vaut-il utiliser une instruction d'egalité sous la forme

Code :

if(MonObjet == null)

Ou plutot

Code :

if(MonObjet.Equals(null))

:??:

En VB.NET, le problème ne se pose presque pas, on utilise généralement :

If Object Is Nothing Then

Sachant que Nothing est l'équivalent de null

ixemul

Nan mais sans blague ! ⚡

Ha ben ouais, mais j'ai delaissé le VB 6 que je ne pouvait plus voir en couleur pour passer à DotNet il y a 1 an maintenant et mon choix c'est naturellement porté sur C# qui me faisait plus tripper en me renvoyant à de vieux souvenirs de C++/java d'étudiant :sweat:

Bref, ma question reste en suspends, si qqun avait le pourquoi du comment il vaut mieux utiliser l'un ou l'autre... question de perf, de conso mémoire ou tout autre explication ca me ferait enormement plaisir

Harkonnen

Modérateur
Un modo pour les bannir tous

ixemul a écrit :

D'ailleur, il me vient une petite idee sur un debat à lancer...

Mieux vaut-il utiliser une instruction d'egalité sous la forme

Code :

if(MonObjet == null)

Ou plutot

Code :

if(MonObjet.Equals(null))

:??:

ça dépend de ce que tu compares.
si tu compares 2 types valeur, alors utiliser l'un ou l'autre est exactement la même chose :

Code :

int a = 5;
int b = 5;
Console.WriteLine(a == b);
Console.WriteLine(a.Equals(b));

te renverra

Citation :

true
true

par contre, si tu compares 2 types references, == fera une comparaison d'identité (les references pointent t'elles sur le même objet ?) , tandis que Equals comparera les valeurs.

Code :

Int32 a = new Int32(5);
Int32 b = new Int32(5);
Console.WriteLine (a == b);
Console.WriteLine (a.Equals(b));

te renverra

Citation :

false
true

attention à l'exception par contre : l'operateur == de la classe String compare les valeurs, et non les references (même si c'est un type reference). il en est de même pour System.Object

pour éviter toute ambiguité, le mieux est d'utiliser Equals() si tu veux comparer les valeurs, et ReferenceEquals si tu veux comparer les references.

---------------
J'ai un string dans l'array (Paris Hilton)

ixemul

Nan mais sans blague ! ⚡

:love: LA lumiere !

merci beaucoup, ça a son importance :jap:

FORUM HardWare.fr

Programmation

C#/.NET managed

[c#]Verifier qu'un enregistrement est present dans la BDD

Sujets relatifs
[ASP/Access]Comment je peux consulter et afficher un enregistrement	[MySQL] sélection aléatoire d'enregistrement en fonction de ....
erreur ajout d'enregistrement	probleme d'insertion dans ma BDD
Verifier une date venant d'un formulaire	Comment vérifier plusieurs variables d'un seul coup ?
[VB] passer à l'enregistrement suivant dans 1 data	Access atteindre un enregistrement spécial
probleme appel date du jour dans BDD	Place enregistrement VB6/SQL
Plus de sujets relatifs à : [c#]Verifier qu'un enregistrement est present dans la BDD

Page générée en 0.112 secondes