Des images qui font répéter le nombre de publication Résolu/Fermé

Question

Bonjour, 
J'ai mes annonces qui se répètent selon le nombre d'images présentes dans une annonce donnée, et celà depuis que j'ai changé mes tables. Au départ j'avais 2 tables, maintenant j'en ai 4.
Pour éviter également des soucis de requête plus tard, je voudrais ajouter une image par défaut s'il y en a pas une.


Dans le code ça donne ça:
function afficherAnnonce(){
    global $bdd;
    $res = [];
   $sql=' SELECT  A.*
   ,C.*
   ,I.*
FROM annonces A
LEFT JOIN categories C ON C.id = A.id_categorie
INNER JOIN images_annonces I ON A.id= I.id_annonce
ORDER BY created_date DESC';

try{
      $req = $bdd->prepare($sql);
      $req->execute();
      $data = $req->fetchAll(PDO::FETCH_ASSOC);
      //on reconstruit un array mieux organisé pour avoir les photos
      
      foreach($data as $R){
        if(!isset($res[$R['id']])){
         $res[$R['id']] = $R;
         unset($res[$R['id']]['filepath']);
        }
         $res[$R['id']]['images'][] = $R['filepath'];                                               
      }
      
    }catch(Exception $e){
        echo "Erreur dans la requête " . $sql;
    }
 // le temps des tests pour voir le résultat
    return $res;
      
  }

Configuration: Windows / Chrome 96.0.4664.45

jordane45 · Answer

Bonjour,

Le problème ne se trouve à mon avis pas là ...  mais plutôt dans le code que tu utilises pour afficher ta liste.

max30_3775 · Answer

Bonjour, merci pour ta réponse. Le code pour afficher, le voilà: '; $images = !empty($anounce['images']) ? $anounce['images'] : NULL; if($images){ //boucle sur les images echo'

'; echo '

'; foreach( $images as $img ){ echo '

'; } echo "

"; Que faudrait-il changer?

max30_3775 · Answer

Toutes mes div sont fermées, en même temps j'essayais d'ajouter une image par défaut mais ça ne marchait pas non plus: Est-ce notre requête? car les tables sont comme indiquées dans la photo: '; $images = !empty($anounce['images']) ? $anounce['images'] : []; // array if(!empty($images) ) { //boucle sur les images echo'

'; echo '

'; foreach( $images as $img ) { if(empty($img) ){ $img = 'Projetsite/no_image.png'; } echo '

'; } echo "

"; echo '

'; echo '

<'; echo "

"; echo '

>';echo "

"; echo "

"; } echo "

"; echo ($anounce['titre']); echo ($anounce['description']); echo($anounce['prix']); echo($anounce['created_date']); echo "";} ?> function afficherAnnonce(){ global $bdd; $res = []; $sql=' SELECT A.* ,C.* ,I.* FROM annonces A LEFT JOIN categories C ON C.id = A.id_categorie INNER JOIN images_annonces I ON A.id= I.id_annonce ORDER BY created_date DESC'; try{ $req = $bdd->prepare($sql); $req->execute(); $data = $req->fetchAll(PDO::FETCH_ASSOC); //on reconstruit un array mieux organisé pour avoir les photos foreach($data as $R){ if(!isset($res[$R['id']])){ $res[$R['id']] = $R; unset($res[$R['id']]['filepath']); } $res[$R['id']]['images'][] = ( !empty($R['filepath']) )? $R['filepath'] : 'Projetsite/no_image.png'; } }catch(Exception $e){ echo "Erreur dans la requête " . $sql; } // le temps des tests pour voir le résultat return $res; }

max30_3775 · Answer

Voila ce que ça donne var_dump($anouncement):
8 => 
    array (size=14)
      'id' => string '8' (length=1)
      'created_by_member' => string '1' (length=1)
      'id_categorie' => string '6' (length=1)
      'pseudo' => string 'b' (length=1)
      'telephone' => string '7178457' (length=7)
      'email' => string 'b@gmail.com' (length=11)
      'titre' => string 'voiture à louer' (length=16)
      'description' => string '' (length=0)
      'prix' => string '245' (length=3)
      'created_date' => string '2021-12-08' (length=10)
      'nom' => string 'voitures' (length=8)
      'id_annonce' => string '13' (length=2)
      'images' => 
        array (size=1)
          0 => string 'pictures/voiture.jpg' (length=20)
  1 => 
    array (size=14)
      'id' => string '1' (length=1)
      'created_by_member' => string '0' (length=1)
      'id_categorie' => string '11' (length=2)
      'pseudo' => string 'essai1' (length=6)
      'telephone' => string '44228866' (length=8)
      'email' => string 'essai1@gmail.com' (length=16)
      'titre' => string 'Téléphone à vendre' (length=21)
      'description' => string 'Je vends un téléphone. Appelez si vousêtes interessé' (length=56)
      'prix' => string '1000' (length=4)
      'created_date' => string '2021-12-07' (length=10)
      'nom' => string 'telephones' (length=10)
      'id_annonce' => string '2' (length=1)
      'images' => 
        array (size=1)
          0 => string 'pictures/telephone.jpg' (length=22)
  2 => 
    array (size=14)
      'id' => string '2' (length=1)
      'created_by_member' => string '1' (length=1)
      'id_categorie' => string '2' (length=1)
      'pseudo' => string 'a' (length=1)
      'telephone' => string '36377769' (length=8)
      'email' => string 'a@gmail.com' (length=11)
      'titre' => string 'Maison à vendre' (length=16)
      'description' => string 'Je vends une maison' (length=19)
      'prix' => string '2556' (length=4)
      'created_date' => string '2021-12-07' (length=10)
      'nom' => string 'maisons' (length=7)
      'id_annonce' => string '3' (length=1)
      'images' => 
        array (size=1)
          0 => string 'pictures/maison5.jpg' (length=20)
  3 => 
    array (size=14)
      'id' => string '3' (length=1)
      'created_by_member' => string '1' (length=1)
      'id_categorie' => string '5' (length=1)
      'pseudo' => string 'b' (length=1)
      'telephone' => string '7178457' (length=7)
      'email' => string 'b@gmail.com' (length=11)
      'titre' => string 'Location de bureau' (length=18)
      'description' => string 'Notre agence loue des bureau' (length=28)
      'prix' => string '2000' (length=4)
      'created_date' => string '2021-12-07' (length=10)
      'nom' => string 'bureaux' (length=7)
      'id_annonce' => string '4' (length=1)
      'images' => 
        array (size=1)
          0 => string 'pictures/bureau1.jpg' (length=20)
  4 => 
    array (size=14)
      'id' => string '4' (length=1)
      'created_by_member' => string '1' (length=1)
      'id_categorie' => string '5' (length=1)
      'pseudo' => string 'b' (length=1)
      'telephone' => string '7178457' (length=7)
      'email' => string 'b@gmail.com' (length=11)
      'titre' => string 'Location de bureau' (length=18)
      'description' => string 'Notre agence loue des bureau' (length=28)
      'prix' => string '2000' (length=4)
      'created_date' => string '2021-12-07' (length=10)
      'nom' => string 'bureaux' (length=7)
      'id_annonce' => string '4' (length=1)
      'images' => 
        array (size=1)
          0 => string 'pictures/bureau2.jpg' (length=20)
  5 => 
    array (size=14)
      'id' => string '5' (length=1)
      'created_by_member' => string '1' (length=1)
      'id_categorie' => string '5' (length=1)
      'pseudo' => string 'b' (length=1)
      'telephone' => string '7178457' (length=7)
      'email' => string 'b@gmail.com' (length=11)
      'titre' => string 'Location de bureau' (length=18)
      'description' => string 'Notre agence loue des bureau' (length=28)
      'prix' => string '2000' (length=4)
      'created_date' => string '2021-12-07' (length=10)
      'nom' => string 'bureaux' (length=7)
      'id_annonce' => string '4' (length=1)
      'images' => 
        array (size=1)
          0 => string 'pictures/bureau3.jpg' (length=20)
  6 => 
    array (size=14)
      'id' => string '6' (length=1)
      'created_by_member' => string '1' (length=1)
      'id_categorie' => string '5' (length=1)
      'pseudo' => string 'b' (length=1)
      'telephone' => string '7178457' (length=7)
      'email' => string 'b@gmail.com' (length=11)
      'titre' => string 'Location de bureau' (length=18)
      'description' => string 'Notre agence loue des bureau' (length=28)
      'prix' => string '2000' (length=4)
      'created_date' => string '2021-12-07' (length=10)
      'nom' => string 'bureaux' (length=7)
      'id_annonce' => string '4' (length=1)
      'images' => 
        array (size=1)
          0 => string 'pictures/bureau4.jpg' (length=20)
  7 => 
    array (size=14)
      'id' => string '7' (length=1)
      'created_by_member' => string '1' (length=1)
      'id_categorie' => string '5' (length=1)
      'pseudo' => string 'b' (length=1)
      'telephone' => string '7178457' (length=7)
      'email' => string 'b@gmail.com' (length=11)
      'titre' => string 'Location de bureau' (length=18)
      'description' => string 'Notre agence loue des bureau' (length=28)
      'prix' => string '2000' (length=4)
      'created_date' => string '2021-12-07' (length=10)
      'nom' => string 'bureaux' (length=7)
      'id_annonce' => string '4' (length=1)
      'images' => 
        array (size=1)
          0 => string 'pictures/bureau5.jpg' (length=20)

jordane45 · Answer

Tu dis que tes div sont bien fermées .... je ne pense pas... Tu as, dans ton IF, l'ouverture d'une div ... que tu fermes que après ton if .. echo'

'; Tu peux avoir le bon nombre de

.. si ils ne sont pas bien placés dans ton code ça fou le bordel.. En gros, pour corriger ton souci et mettre une image par défaut, ton code devrait ressembler à un truc du genre '; $images = !empty($anounce['images']) ? $anounce['images'] : ['Projetsite/no_image.png']; // array if(!empty($images) ) { //boucle sur les images echo'

'; echo '

'; foreach( $images as $img ){ echo '

'; } echo "

"; // fin div slider . //ce bloc de div est bien fermé;. echo '

'; echo '

<'; echo "

"; echo '

>'; echo "

"; echo "

"; //---------------------------------------- echo "

"; // fin div container } echo $anounce['titre']; echo $anounce['description']; echo $anounce['prix']; echo $anounce['created_date']; echo ""; // fin div annoncepubs } ?>

max30_3775 · Answer

La div étant fermée au bon endroit, les images présentes dans une annonce font toujours répéter cette dernière, et l'image par défaut ne s'affiche toujours pas: Page d'affichage '; $images = !empty($anounce['images']) ? $anounce['images'] : []; // array if(!empty($images) ) { //boucle sur les images echo'

'; echo '

'; foreach( $images as $img ) { if(empty($img) ){ $img = 'Projetsite/no_image.png'; } echo '

'; } echo "

"; echo '

'; echo '

<'; echo "

"; echo '

>';echo "

"; echo "

"; // fin div container } echo ($anounce['titre']); echo ($anounce['description']); echo($anounce['prix']); echo($anounce['created_date']); echo "";} ?> La fonction affichage: function afficherAnnonce(){ global $bdd; $res = []; $sql=' SELECT A.* ,C.* ,I.* FROM annonces A LEFT JOIN categories C ON C.id = A.id_categorie INNER JOIN images_annonces I ON A.id= I.id_annonce ORDER BY created_date DESC'; try{ $req = $bdd->prepare($sql); $req->execute(); $data = $req->fetchAll(PDO::FETCH_ASSOC); //on reconstruit un array mieux organisé pour avoir les photos foreach($data as $R){ if(!isset($res[$R['id']])){ $res[$R['id']] = $R; unset($res[$R['id']]['filepath']); } $res[$R['id']]['images'][] = ( !empty($R['filepath']) )? $R['filepath'] : 'PROJETSITE/no_image.png'; } }catch(Exception $e){ echo "Erreur dans la requête " . $sql; } // le temps des tests pour voir le résultat return $res; }

Pitet · Answer

Bonjour, Avec ton dernier code, l'image par défaut ne pourra effectivement pas être affichée. En effet à la ligne 5, si la variable $anounce['images'] est vide (donc si l'annonce ne possède pas d'image), la variable $image sera initialisée avec un tableau vide ([]). La condition ligne 7 if(!empty($images)) sera donc fausse et le code php à l'intérieur de ce bloc if ne sera donc pas exécuté. Pour afficher l'image par défaut si l'annonce ne possède pas d'image, il faut donc : - soit déplacer la condition if(!empty($images)) juste avant l'affichage de la boucle et ajouter un bloc "else" à après ce bloc "if" pour afficher l'image par défaut - soit simplement initialiser la variable $image avec un tableau contenant l'image par défaut (['Projetsite/no_image.png']) lorsque l'annonce ne possède pas d'image, voir le dernier exemple proposé par jordane45 à la ligne 4. Tu as également une erreur de syntaxe html pour les boutons de navigation left et right du slider : il n'est pas correct d'utiliser les caractères "<" et ">" comme texte de ces boutons puisque ces caractères sont réservés en html (ouverture et fermeture de balise html). Pour utiliser ces caractères comme texte dans un document html, il faut les convertir en entités html : echo '

'; echo '

& lt;

'; echo '

& gt;

'; echo '

'; -> dans cet exemple, il faut normalement bien supprimer les espaces entre & et lt; ou gt; (le forum ccm converti automatiquement les entités si j'écris cet exemple sans l'espace) https://developer.mozilla.org/fr/docs/Glossary/Entity A ce sujet, essaye de prendre le réflexe d'utiliser le validateur w3c pour vérifier que ton résultat html ne contient pas d'erreur de syntaxe.

max30_3775 · Answer

Merci Jordane et Pitet pour vos retours. J'ai bien ajouté cette ligne: $images = !empty($anounce['images']) ? $anounce['images'] : ['PROJETSITE/no_image.png']; D'ailleurs l'image par défaut se trouve dans le même dossier que lespages php, j'ai beau enlevé donc projetsite pour laisser no_image.png, pour remettre projetsite finalement à sa place. L'image par défaut est toujours absente, et les images des annonces font répéter l'annonce elle même. On peut voir dans le code source l'exemple de vélo et bureau. Voici le code:

<

>

VeloVelo à vendre02021-12-09

<

>

VeloVelo à vendre02021-12-09

<

>

voiture à louer2452021-12-08

<

>

Téléphone à vendreJe vends un téléphone. Appelez si vousêtes interessé10002021-12-07

<

>

Maison à vendreJe vends une maison25562021-12-07

<

>