[Réglé] [Inventaire] Optimisation

[nemesis_elite]

Bonjour bonjour,

Voilà, j'ai commencé la mise en "code" d'un inventaire.
Mon code marche, mias je le trouve un peu lourd.
Voici ce que ça donne :


C'est assez basique je sais ^^

Voici mon code

Code PHP :

<?php 

/************** Initialisation des var *****************/

$idPerso = 1; // id du perso

$case = 0; // numero des cases du tableau

$ligne = 0; // numero de la ligne du tableau ( pour la boucle)

$obj = array(); // Array qui contiendra les infos des objets



/**************** Début du code *********************/



$req = mysql_query('SELECT i.id_objet AS id_objet, o.nom AS nom_objet, o.image AS image_objet

                    FROM inventaire i

                    JOIN objets o ON i.id_objet = o.id

                    WHERE i.id_perso = '.$idPerso);

                    

$nbObjet = mysql_num_rows($req); // On compte combien le perso a d'objet



while($res = mysql_fetch_array($req)) // On met toutes les infos des objets dans un tableau bi-dimensionnel

{

    $obj[] = array($res['id_objet'],$res['nom_objet'],$res['image_objet']);

}



echo 'Votre sac :<br /><br /><table border=1>'; // On ouvre la table

 

while($ligne < 5) // Lignes

{

    $i = 0; // On met le numero des cases à 0

    

    echo '<tr>'; // On ouvre une ligne

    

    while($i < 4) // Cases

    {

        echo '<td width="24" height="24">'; // On ouvre une case

        if($case < $nbObjet){ // on verifie si 

        echo '<img src="'.$obj[$case][2].'" />';

        }else{ // sinon on met une image vide ^^

        echo '<img src="images/vide.png" />';

        }

        echo '</td>'; // On referme la case

        

        // On incrémente 

        $i++; 

        $case++;

    

    }

    echo '</tr>'; // On ferme la ligne

    $ligne++; // On incrémente 

}

echo '</table>'; // On ferme la table 

[Dradge]

Code PHP :

<?php

while($res = mysql_fetch_array($req)) // On met toute les infos des objets dans un tableau bi-dimensionnel

{

    $obj[] = array($res['id_objet'],$res['nom_objet'],$res['image_objet']);

}

?>

[Sephi-Chan]

Le code me paraît correct.
Recopier le résultat de la requête dans ton tableau permet une manipulation plus simple.
Tu peux au besoin ajouter une dimension à ton tableau, ou bien un autre élément à la deuxième dimension.

C'est donc une solution flexible est soft.


Sephi-Chan

[nemesis_elite]

Merci pour vos réponse très rapide !

@ Dradge je ne vois pas trop comment...

Il n'y a donc rien à alleger ? Sephi ?

[Sephi-Chan]

Moi je trouve que le tableau temporaire est une bonne chose, car le traitement est bien plus simple, mais ce n'est que mon avis.
Après en effet ça peut mettre 27µs de plus que si tu ne t'en servais pas. iffle:

[nemesis_elite]

Justement c'est a ce niveau

Code PHP :

<?php 

while($i < 4) // Cases

    {

        echo '<td width="24" height="24">'; // On ouvre une case

        if($case < $nbObjet){ // on verifie si

        echo '<img src="'.$obj[$case][2].'" />';

        }else{ // sinon on met une image vide ^^

        echo '<img src="images/vide.png" />';

        }

        echo '</td>'; // On referme la case

        

        // On incrémente

        $i++;

        $case++;

    

    } 

[Sephi-Chan]

Je viens de remarquer un truc. J'édit dans peu de temps.
Peux tu me copier ici le print_r() de ton tableau temporaire $obj, s'il te plaît ?

Je pense qu'il y a en effet une optimisation à faire, en utilisant l'opérateur modulo (%), de manière à n'avoir qu'une seule boucle.


Sephi-Chan

[Dradge]

27µs de plus et le double de mémoire utilisée !!!

Code PHP :

<?php 

/************** Initialisation des var *****************/

$idPerso = 1; // id du perso

$case = 0; // numero des cases du tableau

$ligne = 0; // numero de la ligne du tableau ( pour la boucle)

$obj = array(); // Array qui contiendra les infos des objets



/**************** Début du code *********************/



$req = mysql_query('SELECT i.id_objet AS id_objet, o.nom AS nom_objet, o.image AS image_objet

                    FROM inventaire i

                    JOIN objets o ON i.id_objet = o.id

                    WHERE i.id_perso = '.$idPerso);

                    

$nbObjet = mysql_num_rows($req); // On compte combien le perso a d'objet



$cpt = 0;

echo 'Votre sac :<br /><br /><table border=1>'; // On ouvre la table



//Une première boucle qui affiche les objets

while($res = mysql_fetch_array($req)) // On met toutes les infos des objets dans un tableau bi-dimensionnel

{

    if($cpt%5==0){

        echo'<tr>';

    }

    echo '<td width="24" height="24"><img src="',$res['image_objet'],'" /></td>'; // On ouvre une case

    if($cpt%5==4){

        echo'</tr>';

    }

    $cpt++;

}



//Et une seconde pour complèter avec les cases vides

while($cpt < 20){

    if($cpt%5==0){

        echo'<tr>';

    }

    echo '<td width="24" height="24"><img src="images/vide.png" /></td>'; // On ouvre une case

    if($cpt%5==4){

        echo'</tr>';

    }

    $cpt++;

}

echo'</table>'; 

[nemesis_elite]

Merci beaucoup a vous deux, en effet ce donne a peu près le même resultat.

J'aurais eu du mal a trouvé, vu que je ne connais mais pas l'opérateur modulo.
Pourrait tu juste mexpliquer cette ligne :

Code PHP :

<?php 

if($cpt%5==0){

        echo'<tr>';

    } 

[Sephi-Chan]

Je propose cette solution, que je trouve plus légère d'un point de vue traitement, plus condensée aussi.
Je peux commenter le code au besoin.

Code PHP :

<?php 

$obj = array(

array('id' => 1, 'nom' => 'Epée', 'image' => 'images/Armes/Epee-1.png'),

array('id' => 2, 'nom' => 'Casque', 'image' => 'images/Defence/Casque-1.png')

);



define('CASES', 20);

define('ELEMENTBYLINE', 4);



$tbl = '<table><caption>Inventaire</caption>';

for($i = 0; $i < CASES; $i++){

    if($i%ELEMENTBYLINE == 0){ $tbl .= "\n<tr>"; }

    (isset($obj[$i])) 

        ? $tbl .= "\n\t".'<td><img src="'.$obj[$i]['image'].'" alt="'.$obj[$i]['nom'].'"></td>'

        : $tbl .= "\n\t".'<td><img src="images/vide.png" alt="Vide"></td>'

    ;

    if($i%ELEMENTBYLINE == ELEMENTBYLINE-1){ $tbl .= "\n</tr>"; }

}

$tbl .= "\n</table>\n";



echo $tbl;