[SQL] Problème de requête

**eduardosan** · 06/06/2007, 14h57

Bonjour

j'ai un problème dans ma fonction, lorsque je teste la requête SQL, elle fonctionne correctement, mais dans ma fonction, rien ne s'affiche, pouvez-vous m'aider et me dire ce qui ne va pas ? Merci

Code php :

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
function datesqlcategory($catego, $date1, $date2){
	$sql = mysql_query("SELECT avg( a.delai )
FROM (
 
SELECT id, category, DATEDIFF( closedate, date ) AS delai
FROM glpi_tracking
WHERE ( date
BETWEEN '$date1%'
AND '$date2%' )
AND (
 
STATUS = 'old_done'
AND category = '%$catego'
)
) AS a
GROUP BY category");
	$req = mysql_fetch_row($sql) or die ("Non cloturé ".mysql_error()); 
	echo "<td>".$req[0]."</td>";

**BornBanane** · 06/06/2007, 15h28

Je l'aurais écrit comme ca :
( mettre la requête dans un variable fait un peu plus propre je trouve )

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
 
$query = ' SELECT avg( a.delai ) '.
' FROM ( '.
' SELECT id, category, DATEDIFF( closedate, date ) AS delai '.
' FROM glpi_tracking '.
' WHERE ( date '.
' BETWEEN "'.$date1.'%" '.
' AND "'.$date2'%" ) '.
' AND ('.
' STATUS = "old_done" '.
' AND category = "%'.$catego.'" '.
' ) '.
' ) AS a '.
'GROUP BY category';
//echo $query; // Ca aide toujours à voir ce qui peut cloquer dans la requête
$sql = mysql_query($query);
$req = mysql_fetch_row($sql) or die ("Non cloturé ".mysql_error());
echo "<td>".$req[0]."</td>";

Est-ce mieux ?

**eduardosan** · 06/06/2007, 15h38

Ca n'affiche toujours rien,
j'ai décommenter //echo $query, j'obtiens ça :

SELECT avg( a.delai ) FROM ( SELECT id, category, DATEDIFF( closedate, date ) AS delai FROM glpi_tracking WHERE ( date BETWEEN "2006-12-01%" AND "2006-12-01%" ) AND ( STATUS = "old_done" AND category = "6" ) ) AS a GROUP BY category

**Celira** · 06/06/2007, 15h41

salut
En plaçant un mysql_error sur l'execution de la requête, ça renvoie quelque chose ?

Code php :

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

$sql = mysql_query($query) or die (" erreur : ".mysql_error());

edit : koo

**koopajah** · 06/06/2007, 15h43

Remplaces ca :

Code :

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

1
2
$sql = mysql_query($query);
$req = mysql_fetch_row($sql) or die ("Non cloturé ".mysql_error());

par

Code :

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

1
2
$sql = mysql_query($query)or die ("Non cloturé ".mysql_error());
$req = mysql_fetch_row($sql);

pour vérifier que ton mysql_query() se passe bien.

Sinon, tu es sur que SELECT * FROM ( autre requete) est valable en SQL ? Dans mes lointains souvenirs il fallait utiliser IN pour utiliser des requetes imbriquées (mais ca a peut etre changé depuis, je demande au cas où).

EDIT :

Celira ^^, grillé again!

**eduardosan** · 06/06/2007, 15h47

Envoyé par koopajah

Remplaces ca :

Code :

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

1
2
$sql = mysql_query($query);
$req = mysql_fetch_row($sql) or die ("Non cloturé ".mysql_error());

par

Code :

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

1
2
$sql = mysql_query($query)or die ("Non cloturé ".mysql_error());
$req = mysql_fetch_row($sql);

J'ai fait ta modif, alors là rien ne s'affiche mais le code continue de s'afficher pour le reste (il y a des espaces vides certes, mais le reste s'affiche alors qu'avant, ça bloquait aux espaces vides)

**BornBanane** · 06/06/2007, 15h48

Envoyé par eduardosan

SELECT avg( a.delai ) FROM ( SELECT id, category, DATEDIFF( closedate, date ) AS delai FROM glpi_tracking WHERE ( date BETWEEN "2006-12-01%" AND "2006-12-01%" ) AND ( STATUS = "old_done" AND category = "6" ) ) AS a GROUP BY category

Est-ce la requête que tu voulais, si tu l'avais écrit sur papier ?
Et si tu as la possibilité de lancer la requête sql avec un phpmyadmin ( ou par un autre moyen ) pour voir ce qu'il peut en ressortir.

**koopajah** · 06/06/2007, 15h50

Ca veut dire que ta requete est bien formée du point de vue de SQL. Par contre je viens de m'apercevoir que tu utilises BETWEEN en SQL avec des % dans la valeur. Les % sont utilisés pour le LIKE, il ne faut pas les mettre pour le BETWEEN a mon avis c'est pour ca que tu n'as aucun résultat qui revient de ta requête, et ca me surprend qu'elle marche dans ton phpmyAdmin ou autre frontal SQL.

en plus pourquoi passer par un FROM() au lieu de ca:

Code :

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

SELECT id, category, AVG(DATEDIFF( closedate, date )) AS delai FROM glpi_tracking WHERE ( date BETWEEN "2006-12-01%" AND "2006-12-01%" ) AND ( STATUS = "old_done" AND category = "6" ) ) AS a GROUP BY category

**eduardosan** · 06/06/2007, 15h50

Envoyé par francoisIT

Est-ce la requête que tu voulais, si tu l'avais écrit sur papier ?
Et si tu as la possibilité de lancer la requête sql avec un phpmyadmin ( ou par un autre moyen ) pour voir ce qu'il peut en ressortir.

La requête sous phpmyadmin fonctionne, mais une fois dans ma fonction plus rien, je ne vois pas le problème pourtant

**eduardosan** · 06/06/2007, 15h51

Envoyé par koopajah

Ca veut dire que ta requete est bien formée du point de vue de SQL. Par contre je viens de m'apercevoir que tu utilises BETWEEN en SQL avec des % dans la valeur. Les % sont utilisés pour le LIKE, il ne faut pas les mettre pour le BETWEEN a mon avis c'est pour ca que tu n'as aucun résultat qui revient de ta requête, et ca me surprend qu'elle marche dans ton phpmyAdmin ou autre frontal SQL.

Tu mets donc quoi à la place des % pour le BETWEEN ?

**koopajah** · 06/06/2007, 15h54

Ben rien, tu mets pas de % pour un between!

J'aurai fait ca clairement moi:

Code :

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

SELECT id, category, AVG(DATEDIFF( closedate, date )) AS delai FROM glpi_tracking WHERE date >= "2006-12-01" AND  date <= "2006-12-01" AND STATUS = "old_done" AND category = "6" GROUP BY category

EDIT : autant pour moi, ton jour est le meme pour le début et la fin de la période.
Est-ce toujours le cas?

**eduardosan** · 06/06/2007, 15h58

Envoyé par koopajah

Ben rien, tu mets pas de % pour un between!

J'aurai fait ca clairement moi:

Code :

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

SELECT id, category, AVG(DATEDIFF( closedate, date )) AS delai FROM glpi_tracking WHERE date >= "2006-12-01" AND  date <= "2006-12-01" AND STATUS = "old_done" AND category = "6" GROUP BY category

Oui mais dans ma table, la valeur est 2006-12-01 00:00:00 (l'heure)
et je n'ai pas envi de demander aux gens de taper l'heure

**eduardosan** · 06/06/2007, 15h59

Envoyé par koopajah

Ben rien, tu mets pas de % pour un between!

J'aurai fait ca clairement moi:

Code :

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

SELECT id, category, AVG(DATEDIFF( closedate, date )) AS delai FROM glpi_tracking WHERE date >= "2006-12-01" AND  date <= "2006-12-01" AND STATUS = "old_done" AND category = "6" GROUP BY category

EDIT : autant pour moi, ton jour est le meme pour le début et la fin de la période.
Est-ce toujours le cas?

Non le 1er est 1 et le 2e 31, j'ai fait une erreur de copie

**koopajah** · 06/06/2007, 16h02

Dans ce cas ma requete marcherait, mais dans tous les cas, voila une modif qui fonctionnerait pour le cas où la date de début est la meme que la date de fin :

Code :

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

SELECT id, category, AVG(DATEDIFF( closedate, date )) AS delai FROM glpi_tracking WHERE DATE_FORMAT(date,'%Y-%m-%d') >= "2006-12-01" AND  DATE_FORMAT(date,'%Y-%m-%d') <= "2006-12-01" AND STATUS = "old_done" AND category = "6" GROUP BY category

Le fait d'utiliser >= et <= permet de gérer toutes les heures en fait, mais faut quand meme utiliser la dernière version de ma requête pour que ca marche

**BornBanane** · 06/06/2007, 16h09

Chose très bête, tu te connectes bien à mysql avant de lancer ta fonction ? (Je me permet de te poser la question car parfois ca m'arrive de l'oublier

)

**eduardosan** · 06/06/2007, 16h39

Envoyé par francoisIT

Chose très bête, tu te connectes bien à mysql avant de lancer ta fonction ? (Je me permet de te poser la question car parfois ca m'arrive de l'oublier

)

Oui bien sur, j'ai déjà fait l'erreur mais plus maintenant

**koopajah** · 06/06/2007, 16h50

Est ce que tu as testé la simplification de la requete et l'utilisation du DATE_FORMAT() comme dit dans mon message précédent?

**eduardosan** · 07/06/2007, 09h05

Envoyé par koopajah

Est ce que tu as testé la simplification de la requete et l'utilisation du DATE_FORMAT() comme dit dans mon message précédent?

Voici ma fonction complète

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
 
function datesqlcategory($catego, $date1, $date2){
$query = ' SELECT avg( a.delai ) '.
' FROM ( '.
' SELECT id, category, DATEDIFF( closedate, date ) AS delai '.
' FROM glpi_tracking '.
' WHERE ( date '.
' BETWEEN "'.$date1.'%" '.
' AND "'.$date2.'%" ) '.
' AND ('.
' STATUS = "old_done" '.
' AND category = "'.$catego.'" '.
' ) '.
' ) AS a '.
'GROUP BY category';
//echo $query; // Ca aide toujours à voir ce qui peut cloquer dans la requête
$sql = mysql_query($query)or die ("Non cloturé ".mysql_error());
$req = mysql_fetch_row($sql);
echo "<td>".$req[0]."</td>";

Elle fonctionne correctement, merci à vous

[SQL] Problème de requête

PHP & Base de données

Discussions similaires

Partager

Partager