Discussion :

[fabpeden]

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 :

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

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

[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

[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/

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

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

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

[fabpeden]

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 :

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

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

[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é )

[bizet]

En regardant ca de plus pres, je pense que tu ne devrais pas utiliser le datevalue.

Code :

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

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.

[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

[fabpeden]

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

Il fallait écrire :

Code :

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

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

Merci encore

[Nathanael Marchand]

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

Il fallait écrire :

Code :

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

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!

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