Discussion :

[Auresky]

Bonjour à tous !

Je suis en train de réaliser un tableau de suivi des ventes sur MySQL, pour l'affichage des vente du jour aucun soucis (j'utilise current_date).

Maintenant, je souhaite afficher le nombre de ventes du mois, depuis mon champs date. Voilà mes instructions :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
$Myear = date("Y/m");
$reponsenbrcommandemois = $gbdd->query('
SELECT COUNT(id_accessoire) AS nbrcommandemois, DATE_FORMAT(date_commande, "%Y/%m")
FROM produit_commande_accessoire 
WHERE commande.date_commande = '.$Myear.'');
$countnbrcommandemois = $reponsenbrcommandemois->fetch();
$nbrcommandemois = $countnbrcommandemois['nbrcommandemois'];
 
Vente du mois <?php echo $nbrcommandemois ?><br />

J'ai bien conscience que l'instruction est mauvaise car çà me retourne 0, mais je ne vois pas trop comment gérer la date par mois de l'année en cours.

Merci de votre aide !

[CinePhil]

Votre requête est fausse car avec une fonction de groupage et une colonne simple (certe formatée mais simple quand même) dans le SELECT, il faudrait un GROUP BY. Mais comme vous avez une restriction sur la date, vous n'avez tout simplement pas besoin de la partie , DATE_FORMAT(date_commande, "%Y/%m").

En plus, vous indiquez dans le WHERE une table qui n'est pas dans la partie FROM (commande) !

Enfin, il y a un problème de syntaxe à la fin de la requête !

Bref, votre requête doit normalement générer des erreurs. Si vous écriviez votre code autrement, vous pourriez vous en rendre compte.

Plus précisément, quel est votre besoin ?
Obtenir le comptage pour un mois donné ou pour chaque mois d'une année donnée ?

[Auresky]

Oui, c'est fort probable qu'il y est des erreurs, simplement car j'ai garder le principal de l'instruction, j'ai enlever le inner join qui appel justement la table commande ; )

Quelle faute de syntaxe vois tu ?

voici le code entier

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
<?php							
$Myear = date("Y/m");
$reponsenbrcommandemois = $gbdd->query('
SELECT COUNT(produit_commande_accessoire.id_accessoire) AS nbrcommandemois, DATE_FORMAT(commande.date_commande, "%Y/%m"), produit_commande_accessoire.id_commande 
FROM produit_commande_accessoire 
INNER JOIN commande 
ON commande.id_commande = produit_commande_accessoire.id_commande
WHERE commande.date_commande = '.$Myear.'');
$countnbrcommandemois = $reponsenbrcommandemois->fetch();
$nbrcommandemois = $countnbrcommandemois['nbrcommandemois'];
?>
Vente du mois <?php echo $nbrcommandemois ?><br />
<?php echo $Myear ?>

Mon objectif est d'obtenir un comptage pour le mois en cours (février ce mois-ci, mars le mois prochain....) depuis un champs DATE

Par la suite j'adapterai le code pour afficher l'historique des commandes mois par mois et année par année.

[CinePhil]

Mon objectif est d'obtenir un comptage pour le mois en cours (février ce mois-ci, mars le mois prochain....) depuis un champs DATE

Comme ça, ce sera mieux :

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
<?php
$Myear = date("Y/m");
 
$sql = "
	SELECT COUNT(produit_commande_accessoire.id_accessoire) AS nbrcommandemois
	FROM produit_commande_accessoire
	INNER JOIN commande ON commande.id_commande = produit_commande_accessoire.id_commande
	WHERE commande.date_commande = '$Myear'
";
 
$reponsenbrcommandemois = $gbdd->query($sql);
$countnbrcommandemois = $reponsenbrcommandemois->fetch();
$nbrcommandemois = $countnbrcommandemois['nbrcommandemois'];
?>
Vente du mois <?php echo $nbrcommandemois ?><br />
<?php echo $Myear ?>

[Auresky]

Oui çà peux paraître un peu plus propre de cette manière, merci.

Pour ce qui est de l'affichage de la réponse, çà bloque toujours, je pense que c'est à cause du champs DATE qui me renvoie un format YYYY-MM-JJ, pour le coup, j'ai essayer de le formater date_commande en YYYY-MM avec DATE_FORMAT mais sans succès...

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
<?php
$Myear = date("Y-m");
 
$sql = "
	SELECT COUNT(produit_commande_accessoire.id_accessoire) AS nbrcommandemois, DATE_FORMAT(commande.date_commande, "%Y-%m")
	FROM produit_commande_accessoire
	INNER JOIN commande ON commande.id_commande = produit_commande_accessoire.id_commande
	WHERE commande.date_commande = '$Myear'
";
 
$reponsenbrcommandemois = $gbdd->query($sql);
$countnbrcommandemois = $reponsenbrcommandemois->fetch();
$nbrcommandemois = $countnbrcommandemois['nbrcommandemois'];
?>
Vente du mois <?php echo $nbrcommandemois ?><br />
<?php echo $Myear ?>

[Exia_]

Bonsoir,

c'est normal tu ouvre les doubles quotes ("") pour indiquer le format de ta date mais vue que pour construire ta requête tu utilise les doubles quotes ta construction de requête est fausse,

replace $sql par :

Code :

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

1
2
3
4
5
$sql = '
	SELECT COUNT(produit_commande_accessoire.id_accessoire) AS nbrcommandemois, DATE_FORMAT(commande.date_commande, "%Y-%m")
	FROM produit_commande_accessoire
	INNER JOIN commande ON commande.id_commande = produit_commande_accessoire.id_commande
	WHERE commande.date_commande = \'' . $Myear . '\' ';

J'ai remplacer les doubles quotes par des simple '', par contre PHP n’interprète pas ce qui se trouve entre simple quote,

par exemple :

Code :

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

1
2
3
4
$toto = 'je suis toto';
echo "$toto"; //Affichera je suis toto qui est la valeur de la variable $toto
 
echo '$toto'; //Affichera $toto en tant que chaine de caractère et non sa valeur

Seulement dans ta requête tu a besoin des caractère '' pour entourer ta chaîne de caractère $Myear sinon ta requête tombe en echec car ta date sera interpréter comme étant une colonne par MySQL,

pour ne pas tomber dans la même erreur qu'avant il faut échapper les '' qui seront utilisé par MySQL avec le caractère \.

[sabotage]

Tu n'as pas besoin d'avoir la date de le SELECT étant donné que tu la connais d'avance puisque c'est ton critère.
Par contre si tu veux comparer ta date à Y-m il faut la formater en Y-m

Code :

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

1
2
3
4
5
$sql = '
	SELECT COUNT(produit_commande_accessoire.id_accessoire) AS nbrcommandemois
	FROM produit_commande_accessoire
	INNER JOIN commande ON commande.id_commande = produit_commande_accessoire.id_commande
	WHERE DATE_FORMAT(commande.date_commande, "%Y-%m") = "' . $Myear . '"';

[CinePhil]

En SQL, les valeurs textuelles s'entourent d'apostrophes, pas de guillemets !

La bonne manière de faire était donc celle que j'avais donnée dans ma requête, au formatage de la colonne de date près :

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
<?php
$Myear = date("Y/m");
 
$sql = "
	SELECT COUNT(produit_commande_accessoire.id_accessoire) AS nbrcommandemois
	FROM produit_commande_accessoire
	INNER JOIN commande ON commande.id_commande = produit_commande_accessoire.id_commande
	WHERE DATE_FORMAT(commande.date_commande, '%Y-%m') = '$Myear'
";
 
$reponsenbrcommandemois = $gbdd->query($sql);
$countnbrcommandemois = $reponsenbrcommandemois->fetch();
$nbrcommandemois = $countnbrcommandemois['nbrcommandemois'];
?>
Vente du mois <?php echo $nbrcommandemois ?><br />
<?php echo $Myear ?>

Si on veut vraiment isoler la variable PHP du code SQL, on peut aussi faire comme ça :

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
<?php
$Myear = date("Y/m");
 
$sql = "
	SELECT COUNT(produit_commande_accessoire.id_accessoire) AS nbrcommandemois
	FROM produit_commande_accessoire
	INNER JOIN commande ON commande.id_commande = produit_commande_accessoire.id_commande
	WHERE DATE_FORMAT(commande.date_commande, '%Y-%m') = '".$Myear."'
";
 
$reponsenbrcommandemois = $gbdd->query($sql);
$countnbrcommandemois = $reponsenbrcommandemois->fetch();
$nbrcommandemois = $countnbrcommandemois['nbrcommandemois'];
?>
Vente du mois <?php echo $nbrcommandemois ?><br />
<?php echo $Myear ?>

[sabotage]

En SQL, les valeurs textuelles s'entourent d'apostrophes, pas de guillemets !

La syntaxe Mysql permet les deux sans préférence.

[Auresky]

Merci énormement à vous trois !
J'ai passé deux semaine à bouffer du code, faut croire que j'arrivais à saturation, ce n'est pas du tout ma compétences de bases l'informatique !

Je m'obstinais à formater le champs date dans le select.... -_-'

L'instruction est beaucoup plus logique comme çà, merci encore !

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
<?php							
$Myear = date("Y-m");
$reponsenbrcommandemois = $gbdd->query('
SELECT COUNT(produit_commande_accessoire.id_accessoire) AS nbrcommandemois
FROM produit_commande_accessoire 
INNER JOIN commande 
ON commande.id_commande = produit_commande_accessoire.id_commande
WHERE DATE_FORMAT(commande.date_commande, "%Y-%m") =  \'' . $Myear . '\' ');
$countnbrcommandemois = $reponsenbrcommandemois->fetch();
$nbrcommandemois = $countnbrcommandemois['nbrcommandemois'];
?>
Vente du mois <?php echo $nbrcommandemois ?><br />

[CinePhil]

Code :

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

WHERE DATE_FORMAT(commande.date_commande, "%Y-%m") =  \'' . $Myear . '\' ');

Têtu hein ?

Franchement, tu trouves ça plus lisible que ça ?

Code :

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

WHERE DATE_FORMAT(commande.date_commande, '%Y-%m') = '".$Myear."'

Ou que ça ?

Code :

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

WHERE DATE_FORMAT(commande.date_commande, '%Y-%m') = '$Myear'

Le jour où tu oublieras un échappement d'apostrophe ou de guillemet et que tu passeras des heures à chercher pourquoi ta requête ne fonctionne pas, tu te souviendras peut-être de mon message !

[manzeki]

Ventes du mois :

Code PHP :

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

1
2
3
$t = 'SELECT COUNT(p.id_accessoire) nbrcommandemois 
      FROM produit_commande_accessoire p, commande c 
      WHERE c.id_commande = p.id_commande AND DATE_FORMAT(c.date_commande, "%Y-%m") = DATE_FORMAT(NOW(), "%Y-%m")';