Problème d'affichage

[Sephi-Chan]

Je dirais tout d'abord que ton approche est mauvaise : un membre désinscrit devrait conserver son identifiant et simplement changer de nom affiché. Là tu flingues tes références inutilement.

Ensuite, je ne comprends pas l'intérêt d'aliaser des champs qui gardent le même nom, en mettant des majsucules : cette pratique diminue inutilement la lisibilité de qualité de ton code en ajoutant du code superflux (plus de code = plus de bugs, d'autant plus que tu n'utilises pas le mot clé AS pour aliaser). Peux-tu expliquer ça ? De même, je m'interroge sur la casse compliquée que tu utilises pour nommer tes colonnes et tes tables. Pourquoi ne pas se contenter de minuscules ?

Bon choix d'utiliser le sprintf, mais tu n'utilises pas le bon placeholder. Pour placer un identifiant (un entier), mieux vaut utiliser ID_Forum_Sujets = %d.

Préfixer les appels de fonctions par des @ est dangereux : c'est un cache misère qui finira par te revenir dans la gueule.

Enfin, intégrer du HTML dans du PHP n'est pas une bonne idée, mieux vaut faire l'inverse, toujours pour la lisibilité. Il serait également intéressant d'utiliser du CSS externe plutôt qu'en ligne : le gâchis est conséquent en terme de poids des pages et de lisibilité.

Exemple :

<?php if($nombreLecturePosts >= 1): ?>

  <?php while($posts = mysql_fetch_array($requeteLecturePosts)): ?>

    <table>

      <tr>

        <th class="pseudo"><?php echo $posts['PSEUDO'] ?></th>

        <th class="timestamp">

          Le <?php echo date('d/m/Y', $posts['DATE']) ?> 

          à <?php echo date('G\h. i\m\i\n.', $posts['DATE']) ?>

        </th>

      <tr>

      <tr>

        <td colspan="2" class="message">

          <?php echo bbcode($posts['MESSAGE']) ?>

          <?php if($posts['SIGNATURE_ETAT'] == 1): ?>

            <hr />

            <?php echo bbcode($posts['SIGNATURE']) ?>

          <?php endif; ?>

        </td>

      </tr>

    </table>

  <?php endwhile; ?>

<?php endif; ?>

Voilà, voilà ! :p


Sephi-Chan