Discussion :

[vass75]

Bonjour a tous,

Je me permet de solliciter votre connaissance sur mon problème.

Donc je m'explique , j'ai un array() de n photo que je récupère en base. Cet array je voudrais l'afficher dans un tableau (table) , avec un maximum de 4 photos par lignes.

Comment puis-je procéder ? , quel boucle utilisé pour une performance maximum ?

Merci d'avance

Vassilis

[bob633]

Moi j'utiliserais une simple boucle for qui parcours tout ton tableau, et soit un compteur pour compter les 4 par ligne, soit un modulo.

A moins d'avoir des millions de lignes, je suis pas certains de gagner du temps entre un while, un for, un foreach, etc Prend au plus simple pour toi.

[Benjamin Delespierre]

Pour faire des vignettes, un tableau c'est pas l'idéal. Fais ça avec une grille toute simple

Exemple:

Code :

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

1
2
3
4
5
6
7
8
9
10
<div class="thumb-grid">
    <?php foreach ($pictures as $picture): ?>
    <div class="thumb">
        <a href="<?php echo $picture['url'] ?>">
            <img src="<?php echo $picture['thumb'] ?>" alt="<?php echo $picture['label'] ?>">
            <span><?php echo $picture['description'] ?></span>
        </a>
    </div>
    <?php endforeach ?>
</div>

Ensuite, il suffit de faire flotter les .thumb et les dimentionner avec CSS et c'est dans la boite.

[vass75]

j'ai pas totalement cerné le truc ,

j'ai mon array que je parcoure avec un foreach :

Code :

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

1
2
3
4
5
$pic = $cnx->select("SELECT chemin_pic from picture ");
 
foreach($pic as $value){
    echo '<td>'. '<img src=' . $value[0] .' />'.'</td>';
}

Au bout de la 4eme valeurs , faut que je fasse un genre de retour a la ligne et que mon tableau continue a se remplir mais toujours avec 4 valeurs par ligne , je vois pas du tout comment procéder...

Merci d'avance

Vassilis

[rawsrc]

Salut,

le retour à la ligne dans un tableau correspond à une nouvelle ligne du tableau.
Tu dois juste insérer un <tr>.[4 IMAGES].</tr> toutes les 4 images affichées.

[Benjamin Delespierre]

modulo

C'est cool t'avoir tenu compte de ma remarque, ça fait plaisir. J'aurais pu insister sur le fait qu'une grille d'images ne caractérise pas des données tabulaires et que donc l'usage d'une table est inapproprié mais je pensais que c'était implicite... Je pensais également que les développeurs web accordaient de l'importance à la sémantique dans les flux HTML, mais visiblement c'est pas le cas.

Par ailleurs, je serai curieux de voir ce qui se cache sous $cnx->select

[vass75]

effectivement je n'avais pas vu ta réponse , je teste vos solution et je vous tien au courant.

Merci infiniment

EDIT : pour ta curiosité =>

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
public function select($SQL) {
        // Execution de la requête
        $res = mysql_query($SQL, $this->_mysql);
        // La requete ne passe pas
        if ($res === FALSE) {
            // Affiche l'erreur
            echo mysql_error();
            return false;
        }
        // Tableau de retour
        $var = array();
        // Insertion dans le tableau de retour
        while ($results = mysql_fetch_array($res))
            $var[] = $results;
        // Retourne le tableau
        return $var;
    }

[Benjamin Delespierre]

Houlà, elle vient d'où cette classe ? Tu sais qu'il existe un risque de dépassement mémoire sur cette méthode ?

ça par contre c'est très bien if ($res === FALSE) (dans les faits if (!$res) suffit amplement), je vois tellement de code sources sur DVP où le développeur ne teste même pas le retour de l'exécution de la requête (ou pire, fait un or die(mysql_error()))

tu devrais sérieusement considérer l'usage de PDO ou de mysqli, quitte à faire de l'objet pour l'accès aux bases, autant le faire bien

[vass75]

Nan je ne savais pas , merci de me l'apprendre ! c'est moi qui l'ais faites
Ok je vais me tourné vers PDO.

Merci

[Benjamin Delespierre]

Pour être plus explicite: il y a un risque de dépassement mémoire si le volume de données (nombre de tuples pour être exact) est important vu que tout est mis dans le tableau $var. Donc si tu as un million d'entrées retournées, y'a un risque sérieux de dépasser les 128mo allouables par PHP (j'ai eu le cas cette semaine sur un de nos composants...)

Pour résoudre le problème, on peut utiliser les PDOStatements qui ne vont chercher (fetcher) les données qu'au moment ou on les utilise, on n'a donc qu'un seul tuple à la fois en mémoire.

[vass75]

Ah ok je ne savais pas que PHP allouais 128Mo maximum.

PDOStatements => sa ma l'air être ce qu'il me faut pour gérer un tres grand nombre d'enregistrement

Pour mon problème j'ai trouvé une solution

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
$i = 0;
 foreach($pic as $value){
 
                        $i++;
                        echo '<td>'. '<img src=' . $value[0] .' />'.'</td>';
 
                        // 4 image par lignes :)
                        if($i == 4){
                            echo '</tr><tr>';
                            $i = 0;
                        }
                    }

Je pense que je vais me tourné vers une grille comme tu me le suggère B.Delespierre.

Merci d'avoir accordé du temps a mes question.

(je met en résolue)

Bonne fin de journée

PEACE !

[Benjamin Delespierre]

Compression:

Code :

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

1
2
foreach ($pic as $i => $value)
    printf('<td><img src="%s" /></td>%s', $value, $i%4 ==0 ? '</tr><tr>' : '');

Merci d'avoir accordé du temps a mes question.

*sarcasme* Merci à toi d'avoir tenu compte des réponses proposées.