Code pas totalement fonctionnel

Code pas totalement fonctionnel - PHP - Programmation

Marsh Posté le 27-04-2014 à 12:22:40    

Bonjour,
 
Je tente de faire apparaitre les plantes mellifères, fruits et légumes des mois en cours, mais j'ai un petit souci au niveau des plantes mellifères, autant les 2 premiers mois tout s'affiche nickel, autant par la suite ils se répètent une puis à plusieurs reprises, pour mieux visualiser le problème, voici la page en question : http://co-nexion.com/page/allsaisons
 
Le code  
 

Code :
  1. <?php
  3. $db = Zend_Registry::get('database');
  5. echo "<center><h1>Plantes méllifères, aromates, fruits, légumes, salades et champignons de saison</h1></center><br /><br />";
  8. $mois = array("", "Janvier", "Février", "Mars", "Avril", "Mai", "Juin", "Juillet", "Août", "Septembre", "Octobre", "Novembre", "Décembre" );
  10. for ($i = 1; $i <= 12; $i++)
  11. {
  12.     $select1 = "SELECT * FROM saisonapicole WHERE Mois='$mois[$i]'";
  13.     $toElect1 = $db->fetchAll($select1, array(), Zend_Db::FETCH_ASSOC);
  16.     echo '<center><h3>' . $mois[$i] . '</h3></center>';
  19.     $OldCat1 = '';
  20.     foreach ($toElect1 as $elect1)
  21.     {
  22.  $Fleurs=$elect1['Fleurs'];
  23.  $Cat1 = $elect1['Categorie'];
  25.         if (($OldCat1 == '') || ($Cat1 != $OldCat1))
  26.         {
  27.             // si on est en début d'itération (OldCat vide)
  28.             // OU qu'on a changé de catégorie (OldCat et Cat différents)
  29.             // afficher le nom de catégorie ET l'aliment
  30.             echo '<br /><strong>Plantes méllifères</strong> : ' . $Fleurs;
  31.         } else
  32.         {
  33.             // sinon afficher juste l'aliment précédé d'une virgule
  34.             echo ', ' . $Fleurs;
  35.         }
  36.         $OldCat1 = $Cat1; // mettre a jour l'ancienne catégorie avec la courante pour le test a la prochaine itération
  37.     }
  39. echo "<br />";
  41. $select = "SELECT * FROM saison WHERE Mois='$mois[$i]' ORDER BY Categorie";
  42.     $toElect = $db->fetchAll($select, array(), Zend_Db::FETCH_ASSOC);
  44. $OldCat = '';
  45. foreach ($toElect as $elect)
  46.     {
  47.         $Cat = $elect['Categorie'];
  48.         $Aliment = $elect['Nom'];
  49.         if (($OldCat == '') || ($Cat != $OldCat))
  50.         {
  51.             // si on est en début d'itération (OldCat vide)
  52.             // OU qu'on a changé de catégorie (OldCat et Cat différents)
  53.             // afficher le nom de catégorie ET l'aliment
  54.             echo '<br /><strong>' . $Cat . 's</strong> : ' . $Aliment;
  55.         } else
  56.         {
  57.             // sinon afficher juste l'aliment précédé d'une virgule
  58.             echo ', ' . $Aliment;
  59.         }
  60.         $OldCat = $Cat; // mettre a jour l'ancienne catégorie avec la courante pour le test a la prochaine itération
  61.     }
  62.     echo "<br /><br />";
  63. }
  64. ?>


 
 
Les tables :  
saison : Id, Nom, Categorie, Mois
saisonapicole : Id, Fleurs, Mois, Special, Categorie
 
 
Pourriez vous m'aider à résoudre ce problème ? D'avance merci.

Reply

Marsh Posté le 27-04-2014 à 12:22:40   

Reply

Marsh Posté le 28-04-2014 à 09:26:10    

C'est sûr que c'est pas ta base qui est pourrie ?

Reply

Marsh Posté le 28-04-2014 à 16:19:21    

Comment ça ? Du genre mal construite ?


---------------
Comment assembler & optimiser soi même son pc
Reply

Marsh Posté le 28-04-2014 à 16:21:57    

Du genre il y a des doublons je pense ?

Reply

Marsh Posté le 28-04-2014 à 17:26:49    

Aurais-je du faire une seule table ? J'avais tenté d'abord une requête SQL impliquant les 2 tables mais ayant du mal à avoir des résultats j'ai préféré faire comme cela..


---------------
Comment assembler & optimiser soi même son pc
Reply

Marsh Posté le 29-04-2014 à 09:31:46    

Fais voir la structure de ta bdd ;)

Reply

Marsh Posté le 29-04-2014 à 17:19:13    

saison : Id, Nom, Categorie, Mois
saisonapicole : Id, Fleurs, Mois, Special, Categorie  
 
L'id est en mediumint auto_increment, les autres sont en varchar 100.


---------------
Comment assembler & optimiser soi même son pc
Reply

Marsh Posté le 30-04-2014 à 07:28:58    

Sers-toi de jointures, les requêtes dans les boucles c'est le mal  :o  
 
Là rien d’étonnant tes doublons.
C'quoi la différence entre les catégories des 2 tables ? C'est leur relation ?


Message édité par bistouille le 30-04-2014 à 07:29:49
Reply

Marsh Posté le 30-04-2014 à 17:34:21    

Il n y a pas de différence pour la catégorie des 2 tables, enfin, celle de saisonapicole n'a même pas vraiment d'utilité en fait... Et niveau jointure, comme dit, j'avais tenté mais ne suis pas doué..


---------------
Comment assembler & optimiser soi même son pc
Reply

Sujets relatifs:

Leave a Replay

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