Problème d'inbricatation avec mes if

Problème d'inbricatation avec mes if - SQL/NoSQL - Programmation

Marsh Posté le 05-05-2009 à 08:16:52    

Bonjours,
 
Voilà j'ai un petit problème avec mon imbrication de if de plus mon code n'est pas des plus logique donc si quelqu'un pourrait me guider ver une solution plus efficace ce serait gentille.
 
Mon problème est le suivant je veu créer un formulaire ou l'on met deux dates on peu aussi en mettre qu'une ( date_du && date_au) si je mets juste date_du sa va m'affiché tou les renseignement après cette date, et si je me date_au sa me donne les resultats d'après. et si je mets les deu dates sa m'affiche ce qu'il y a entre c'est ce que j'ai fai mais j'ai toujours des problème si quelqu'un voit l'erreur ce serai gentille ou si quelqu'un pourrai me guidé ver une autre solution je le remerci  
 
je vous donne mon code qui est :
 

Code :
  1. <?php
  2. if (isset($date_du) && isset($date_au))
  3. {
  5.  $query_vg = mysql_query("SELECT IF(customers_from LIKE '1st%','1rst',customers_from) AS Origin,
  6.           COUNT(*) AS effectif_vg
  7.         FROM customers  inner join relances_gratuites on customers.customers_id=relances_gratuites.customers_id
  8.         WHERE customers_client='0'
  9.         AND customers_emv='0'
  10.         AND customers_seance ='0'
  11.         AND relances_gratuites_inscription_date>='" . $date_du . "' and  relances_gratuites_inscription_date<='" . $date_au . "'
  12.         GROUP BY Origin " );
  14.      $query = mysql_query("SELECT IF(cu.customers_from LIKE '1st%','1rst',cu.customers_from) AS Origin,
  15.          COUNT(DISTINCT cu.customers_id) AS effectif, count(co.commandes_status) AS command,
  16.          AVG(co.commandes_montant) AS Mmc
  17.        FROM customers cu
  18.        INNER JOIN commandes co
  19.        ON cu.customers_id=co.customers_id
  20.        INNER JOIN relances_gratuites rg
  21.        ON cu.customers_id=rg.customers_id 
  22.        WHERE co.commandes_status='1'
  23.        AND rg.relances_gratuites_inscription_date>='" . $date_du . "' AND  rg.relances_gratuites_inscription_date<='" . $date_au . "' 
  24.        GROUP BY Origin ";
  26.  else if (isset($date_du) && $date_du != '')
  27.  {
  28.   $query_vg = mysql_query("SELECT IF(customers_from LIKE '1st%','1rst',customers_from) AS Origin,
  29.            COUNT(*) AS effectif_vg
  30.          FROM customers  inner join relances_gratuites on customers.customers_id=relances_gratuites.customers_id
  31.          WHERE customers_client='0'
  32.          AND customers_emv='0'
  33.          AND customers_seance ='0'
  34.          AND relances_gratuites_inscription_date>='" . $date_du . "'
  35.          GROUP BY Origin " );
  37.   $query = mysql_query("SELECT IF(cu.customers_from LIKE '1st%','1rst',cu.customers_from) AS Origin,
  38.           COUNT(DISTINCT cu.customers_id) AS effectif, count(co.commandes_status) AS command,
  39.           AVG(co.commandes_montant) AS Mmc
  40.         FROM customers cu
  41.         INNER JOIN commandes co
  42.         ON cu.customers_id=co.customers_id
  43.         INNER JOIN relances_gratuites rg
  44.         ON cu.customers_id=rg.customers_id 
  45.         WHERE co.commandes_status='1'
  46.         AND rg.relances_gratuites_inscription_date>='" . $date_du . "' 
  47.         GROUP BY Origin" );
  48.  }
  50.  else (isset($date_au) && $date_au != '')
  51.  {
  52.   $query_vg = mysql_query("SELECT IF(customers_from LIKE '1st%','1rst',customers_from) AS Origin,
  53.            COUNT(*) AS effectif_vg
  54.          FROM customers  inner join relances_gratuites on customers.customers_id=relances_gratuites.customers_id
  55.          WHERE customers_client='0'
  56.          AND customers_emv='0'
  57.          AND customers_seance ='0'
  58.          AND relances_gratuites_inscription_date<='" . $date_au . "'
  59.          GROUP BY Origin " );
  61.   $query = mysql_query("SELECT IF(cu.customers_from LIKE '1st%','1rst',cu.customers_from) AS Origin,
  62.           COUNT(DISTINCT cu.customers_id) AS effectif, count(co.commandes_status) AS command,
  63.           AVG(co.commandes_montant) AS Mmc
  64.         FROM customers cu
  65.         INNER JOIN commandes co
  66.         ON cu.customers_id=co.customers_id
  67.         INNER JOIN relances_gratuites rg
  68.         ON cu.customers_id=rg.customers_id 
  69.         WHERE co.commandes_status='1'
  70.         AND rg.relances_gratuites_inscription_date<='" . $date_au . "' 
  71.         GROUP BY Origin" );
  72.  }
  73. }
  74. ?>

Reply

Marsh Posté le 05-05-2009 à 08:16:52   

Reply

Marsh Posté le 05-05-2009 à 09:35:43    

je vien de remarqué que si j'enlève les else if et else sa m'enlève des messages d'erreur je comprend pas pourquoi??? normalement quant on fai des imbrication on met else if non???

Reply

Marsh Posté le 05-05-2009 à 09:41:53    

Sur le plan de la logique :
 
Il y a deux critères, donc 2 x 2 cas = 4 cas. Mais votre programme n'a que 3 if. Je ne sais pas si c'est normal de laisser tomber le quatrième cas.
 
Les deux critères sont testés en même temps. Il serait peut-être plus simple de les tester séparément :

if (isset($date_du)) {
  if (isset($date_au)) {
  }
  else {
  }
}
else {
  if (isset($date_au)) {
  }
  else {
  }
}

En regardant le code, on s'aperçoit qu'il y a au moins un autre critère, if ($date_au != ''), et je suppose que l'on devrait aussi avoir if ($date_au != ''). Donc, il faudrait en fait avoir ceci :

if (isset($date_du) && $date_du != '') {
  if (isset($date_au)  && $date_au != '') {
  }
  else {
  }
}
else {
  if (isset($date_au)  && $date_au != '') {
  }
  else {
  }
}


Reply

Marsh Posté le 05-05-2009 à 10:16:03    

Merci pour tes éclaircissements, je suis dessus, mais je voulais comprendre pour le quatrième cas vue que mon premier cas est:  
date_du (à) date_au,  
mon deuxième est:
plus grand que date_du,
et mon troisième est:  
plus petit que date_au.
 
personnellement je vois pas la quatrième??

Reply

Marsh Posté le 05-05-2009 à 10:33:57    

voilà j'a retouché mon code comme celà mais c'est la quatrième condition qui me pose problème
 

Code :
  1. if (isset($date_du) && $date_du != '') {
  2.  if (isset($date_au)  && $date_au != '') {
  4.  $query_vg = mysql_query("SELECT IF(customers_from LIKE '1st%','1rst',customers_from) AS Origin,
  5.           COUNT(*) AS effectif_vg
  6.         FROM customers  inner join relances_gratuites on customers.customers_id=relances_gratuites.customers_id
  7.         WHERE customers_client='0'
  8.         AND customers_emv='0'
  9.         AND customers_seance ='0'
  10.         AND relances_gratuites_inscription_date>='" . $date_du . "' and  relances_gratuites_inscription_date<='" . $date_au . "'
  11.         GROUP BY Origin " );
  13.      $query = mysql_query("SELECT IF(cu.customers_from LIKE '1st%','1rst',cu.customers_from) AS Origin,
  14.          COUNT(DISTINCT cu.customers_id) AS effectif, count(co.commandes_status) AS command,
  15.          AVG(co.commandes_montant) AS Mmc
  16.        FROM customers cu
  17.        INNER JOIN commandes co
  18.        ON cu.customers_id=co.customers_id
  19.        INNER JOIN relances_gratuites rg
  20.        ON cu.customers_id=rg.customers_id 
  21.        WHERE co.commandes_status='1'
  22.        AND rg.relances_gratuites_inscription_date>='" . $date_du . "' AND  rg.relances_gratuites_inscription_date<='" . $date_au . "' 
  23.        GROUP BY Origin " );
  24.  }
  25.  else {
  27.   $query_vg = mysql_query("SELECT IF(customers_from LIKE '1st%','1rst',customers_from) AS Origin,
  28.            COUNT(*) AS effectif_vg
  29.          FROM customers  inner join relances_gratuites on customers.customers_id=relances_gratuites.customers_id
  30.          WHERE customers_client='0'
  31.          AND customers_emv='0'
  32.          AND customers_seance ='0'
  33.          AND relances_gratuites_inscription_date>='" . $date_du . "'
  34.          GROUP BY Origin " );
  36.   $query = mysql_query("SELECT IF(cu.customers_from LIKE '1st%','1rst',cu.customers_from) AS Origin,
  37.           COUNT(DISTINCT cu.customers_id) AS effectif, count(co.commandes_status) AS command,
  38.           AVG(co.commandes_montant) AS Mmc
  39.         FROM customers cu
  40.         INNER JOIN commandes co
  41.         ON cu.customers_id=co.customers_id
  42.         INNER JOIN relances_gratuites rg
  43.         ON cu.customers_id=rg.customers_id 
  44.         WHERE co.commandes_status='1'
  45.         AND rg.relances_gratuites_inscription_date>='" . $date_du . "' 
  46.         GROUP BY Origin" );
  47.  }
  48. }
  49.  else {
  50.   if (isset($date_au)  && $date_au != '') {
  51.  
  52.   $query_vg = mysql_query("SELECT IF(customers_from LIKE '1st%','1rst',customers_from) AS Origin,
  53.            COUNT(*) AS effectif_vg
  54.          FROM customers  inner join relances_gratuites on customers.customers_id=relances_gratuites.customers_id
  55.          WHERE customers_client='0'
  56.          AND customers_emv='0'
  57.          AND customers_seance ='0'
  58.          AND relances_gratuites_inscription_date<='" . $date_au . "'
  59.          GROUP BY Origin " );
  61.   $query = mysql_query("SELECT IF(cu.customers_from LIKE '1st%','1rst',cu.customers_from) AS Origin,
  62.           COUNT(DISTINCT cu.customers_id) AS effectif, count(co.commandes_status) AS command,
  63.           AVG(co.commandes_montant) AS Mmc
  64.         FROM customers cu
  65.         INNER JOIN commandes co
  66.         ON cu.customers_id=co.customers_id
  67.         INNER JOIN relances_gratuites rg
  68.         ON cu.customers_id=rg.customers_id 
  69.         WHERE co.commandes_status='1'
  70.         AND rg.relances_gratuites_inscription_date<='" . $date_au . "' 
  71.         GROUP BY Origin" );
  72.            }
  73.      else {
  74.      
  75.      
  76.      
  77.         }
  78. }

Reply

Marsh Posté le 05-05-2009 à 11:07:04    

pour le dernier else j'ai mis : echo "Veuillez insérer une date valide";
 
c'est la seul chose qui ma paru logique, Merci de votre aide mais pourrai-je avoir la confirmation svp

Reply

Marsh Posté le 05-05-2009 à 11:08:07    

Le quatrième cas est celui où aucune date n'est renseignée, donc où il ne faudrait pas faire de restriction sur le dates dans la requête, pas de test sur les dates dans la clause Where. Mais peut-être que cela retournerait trop de lignes et qu'il ne faudrait pas avoir ce cas. Je ne sais pas.
Edit : Oui, le message est très bien.


Message édité par olivthill le 05-05-2009 à 11:08:35
Reply

Marsh Posté le 05-05-2009 à 11:10:00    

Et quelquechose comme ça :

Code :
  1. $condition = "";
  2. if (isset($date_du) && !empty($date_du)) {
  3.     $condition .= " AND rg.relances_gratuites_inscription_date>='" . $date_du . "' ";
  4. }
  5. if (isset($date_au) && !empty($date_au)) {
  6.     $condition .= " AND rg.relances_gratuites_inscription_date<='" . $date_au . "' ";
  7. }
  8. $query_vg = mysql_query("SELECT IF(customers_from LIKE '1st%','1rst',customers_from) AS Origin,
  9.                        COUNT(*) AS effectif_vg
  10.                         FROM customers c inner join relances_gratuites rg on c.customers_id=rg.customers_id
  11.                         WHERE customers_client='0'
  12.                         AND customers_emv='0'
  13.                         AND customers_seance ='0' 
  14.                         " . $condition . "                        
  15.                         GROUP BY Origin " );
  16.  
  17. $query = mysql_query("SELECT IF(cu.customers_from LIKE '1st%','1rst',cu.customers_from) AS Origin,
  18.                        COUNT(DISTINCT cu.customers_id) AS effectif, count(co.commandes_status) AS command,
  19.                        AVG(co.commandes_montant) AS Mmc
  20.                         FROM customers cu
  21.                         INNER JOIN commandes co
  22.                         ON cu.customers_id=co.customers_id
  23.                         INNER JOIN relances_gratuites rg
  24.                         ON cu.customers_id=rg.customers_id
  25.                         WHERE co.commandes_status='1'
  26.                         " . $condition . "
  27.                         GROUP BY Origin " );


 
Ce ne serait pas plus simple !?
(Et encore, il est possible de faire, je suis allé vite :o)


---------------
Kao ..98 - Uplay (R6S) : kao98.7.62x39 - Origin (BF4, BF1) : kntkao98
Reply

Marsh Posté le 05-05-2009 à 11:25:39    

Merci kao98,
 
tu dis qu'il est possible de faire mieu moi perso sa me va j'ai ce que je voulais et grace a toi j'ai découvert une façon très agréable et bien plus pratique qui me permet d'avoir les deux date ou juste une si elle est renseigné merci mille fois

Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed