Requete SQL sous C#

Version imprimable

Requete SQL sous C#

Bonjour à tous,

Je sollicite votre aide concernant un problème avec une requête SQL en C#.

Ma requête fonctionne bien (testé sous Access). Elle fonctionne également bien en C# lorsqu'elle est de la forme suivante :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
 
	Local_reader = Globals.Bdd.dbSendSqlQuery("SELECT SUM(TOTAL_KWH_JOUR) AS COMPTEUR_PERMANANENT "+
				"FROM "+
				"( "+
				"SELECT JOUR, SUM(KWH) AS TOTAL_KWH_JOUR "+
				"FROM "+
				"( "+
				"SELECT datevalue(ecoEnergie.DATE_HEURE) AS JOUR, Hour(ecoEnergie.DATE_HEURE) AS HEURE, Avg(ecoEnergie.KW_DOM) AS KWH "+
				"FROM ecoEnergie "+	
				"WHERE datevalue(ecoEnergie.DATE_HEURE) = '02/11/2011' "+
				"GROUP BY datevalue(ecoEnergie.DATE_HEURE), Hour(ecoEnergie.DATE_HEURE) "+
				") "+
				"GROUP BY JOUR "+
				");");
			while (Local_reader.Read())
			{
				Globals.Tags.COMPTEUR_JOUR_DOM.Value = Convert.ToDouble(Local_reader[0].ToString());
				//ListBox1.Items.Add(Local_reader[0].ToString());
			}
			Local_reader.Close();

Mon soucis se situe au niveau de la date. Dans mon cas ci-dessus, je rentre ma date directement '02/11/2011'.

Je souhaite pouvoir rentrer la date en récupérant l'heure du PC :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
 
	Local_reader = Globals.Bdd.dbSendSqlQuery("SELECT SUM(TOTAL_KWH_JOUR) AS COMPTEUR_PERMANANENT "+
				"FROM "+
				"( "+
				"SELECT JOUR, SUM(KWH) AS TOTAL_KWH_JOUR "+
				"FROM "+
				"( "+
				"SELECT datevalue(ecoEnergie.DATE_HEURE) AS JOUR, Hour(ecoEnergie.DATE_HEURE) AS HEURE, Avg(ecoEnergie.KW_DOM) AS KWH "+
				"FROM ecoEnergie "+	
				//"WHERE datevalue(ecoEnergie.DATE_HEURE) = datevalue("+DateTime.Now.ToString()+") "+
				"GROUP BY datevalue(ecoEnergie.DATE_HEURE), Hour(ecoEnergie.DATE_HEURE) "+
				") "+
				"GROUP BY JOUR "+
				");");
			while (Local_reader.Read())
			{
				Globals.Tags.COMPTEUR_JOUR_DOM.Value = Convert.ToDouble(Local_reader[0].ToString());
				//ListBox1.Items.Add(Local_reader[0].ToString());
			}
			Local_reader.Close();

Cela ne fonctionne pas, j'ai une exception.

Auriez-vous des pistes pour me mettre sur la voie? Car je n'ai trouvé de méthode C# pour récupérer juste la date de "DateTime.now"

Merci

03/11/2011, 13h31
asmduty

Bonjour,

met un point d'arrêt et un espion sur Datetime.now.tostring() et tu veras que tu récupères probablement : "02/11/2011 00:00:00"
Tu peux donc créer ta chaine en concatenant "now.day", "now.month", "now.year" et les "/" qui vont bien
03/11/2011, 13h37
Nathanael Marchand

Non! Il faut utiliser les requêtes parametrées. Faire des bidouilles de concatenation avec les dates c'est très mauvais!
http://webman.developpez.com/article...ameter/csharp/
03/11/2011, 13h39
bizet

Salut

Comme le dit Nathanael, il est très fortement conseillé d'utiliser les requêtes paramétrées.

Sinon ton erreur est que la date du jour n'est pas entourée de simple quote.

Code:

//"WHERE datevalue(ecoEnergie.DATE_HEURE) = datevalue('"+DateTime.Now.ToString()+"') "+
03/11/2011, 15h12
fabpeden

Citation:

Envoyé par bizet

Salut

Comme le dit Nathanael, il est très fortement conseillé d'utiliser les requêtes paramétrées.

Sinon ton erreur est que la date du jour n'est pas entourée de simple quote.

Code:

//"WHERE datevalue(ecoEnergie.DATE_HEURE) = datevalue('"+DateTime.Now.ToString()+"') "+

Merci pour votre aide, j'ai essayé les 2 méthodes, celle ci-dessus + la concaténation avec now.day..., et dans les 2 cas cela ne fonctionne pas.

Je suis en train de regarder avec les requêtes paramétrées.

Merci

Je viens d'essayer avec DateTime.Now.ToString().Remove(11) mais pas mieux.
04/11/2011, 10h24
asmduty

Effectivement les requêtes paramétrées sont bien sur l'idéal, mais ça n'explique aucunement pourquoi ça ne fonctionne pas (curiosité :) )
04/11/2011, 10h39
bizet
En regardant ca de plus pres, je pense que tu ne devrais pas utiliser le datevalue.
Code:

1 2 //"WHERE ecoEnergie.DATE_HEURE = ' "+DateTime.Now.ToString()+" ') "+
Tu peux aussi mettre un point d’arrêt sur la requête et l'exécuter directement sur la base pour voir l'erreur retournée.
04/11/2011, 11h16
kheironn

D'accord avec Bizet. Tu récupères la requête réellement envoyée, et tu regardes ce qui est écrit. Au cas où tu l'exécutes dans le requêteur du SGBD...
S'il y a une erreur de syntaxe, tu la verras directement.

TOUJOURS tester ses requêtes avant de les utiliser
07/11/2011, 14h13
fabpeden
Bonjour à tous, bon j'ai réussi à trouver mon erreur.

Il fallait écrire :
Code:

1 2 "WHERE datevalue(ecoEnergie.DATE_HEURE) = '"+DateTime.Today.ToString("d")+"' "+
Merci encore
07/11/2011, 14h41
Nathanael Marchand
Citation:
Envoyé par fabpeden

Bonjour à tous, bon j'ai réussi à trouver mon erreur.

Il fallait écrire :

Code:

1 2 "WHERE datevalue(ecoEnergie.DATE_HEURE) = '"+DateTime.Today.ToString("d")+"' "+

Merci encore
Non! Non! Non! Non et encore non! Je répète encore une fois qu'il faut passer par un paramètre. Faire ce genre de magouilles avec les dates n'apporte que des problèmes. Il faut BANNIR les concaténations pour faire des requêtes. En plus d'être DANGEREUX (injection SQL), cela empêche la réutilisation des plans d'executions ce qui POURRI les perfs!
07/11/2011, 15h30
fabpeden

Oui, merci.

Mais en fait dans un premier temps, je voulais comprendre pourquoi en direct cela ne fonctionnait pas.

Maintenant je vais pouvoir m'attaquer au passage par paramètres.