Reprise du message précédent :
j'avais pensé à switch mais j'ai lu que c'est apparement moins rapide. Je sais à quoi correspond isset et avec empty, j'ai le même résultat. Par contre, je veux bien que tu détailles un peu plus la deuxième solution

(et oui, je débute, c'est pour ça que je dis que y'auras du taf' )

EDIT : http://www.vulgarisation-informati [...] er-php.php
4 ms de plus que  if/else

 
4ms de "gagnés" (enfin 'gagnés', optimisons) pour 100000 pages affichées, réflechis un peu à ce que ça représente et occupe toi plus souvent de la lisibilité de ton code que de son 'optimisation'...

Sans compter que les 4ms pour 100000 pages, c'est avec un if/else, pas un if/elseif/elseif/elseif

Un switch bien implémenté (je sais, je rêve, c'est PHP) doit pouvoir faire son dispatch en O(1), et quand bien même il serait mal implémenté même dans le pire des cas il aura des performances équivalentes à une suite de elseif (O(n)), donc la différence sur une monstruosité pareille est très probablement inférieure à 4ms

En bonus, ici on tape 2 conditionels nestés alors qu'avec un switch on a qu'un niveau de conditionnelles

Disons que je me soucie de ça pour prendre l'habitude, dès le départ, l'habitude (ou du moins essayer) de faire en sorte que ça aille aussi rapidement que possible et ne pas y revenir 36.000 fois. Mais ce que je souhaite avant tout, c'est que ça marche quand même

J'ai bien aimé le tableau associatif mai sj ene vois pas trop comment faire sachant qu'il s'agit de ça :

Autrement en passant par un switch, c'est envisageable de faire un premier switch pour déterminer s'il s'agit de la modif du profil ou de la lecture et si c'est modifier, à l'intérieur de refaire un switch pour selectionner la cat' ?

C'est complètement con et c'est une mauvaise habitude.

Le souci de performance maximale et le souci de correction du code sont antinomique, la recherche de performance menant 90% du temps à un code plus complexe et moins lisible.

Tu mets les chaînes à afficher dans ta hashmap  

J'ai pas compris la question, mais modéliser ton machin if/else/elseif/trucbidule avec un switch ça se fait en un seul lookup

Quoi qu'on en dise, on est quasiment tous passé par là un jour ou l'autre même si pour la pluspart des gens, ca a été avec les apostrophes et les guillemets. Mais il faut redescendre sur terre et réaliser qu'il y a des optimisations qui ne servent à rien du tout : qu'est ce que t'en as à faire que ton serveur soit capable de fournir une page de plus ou de moins par semaine?
 
En plus de ça dans le cas présent tu te bases sur un test qui ne correspond pas à ta situation vu que le test n'utilise qu'un seul "if" et non pas une série de "if/elseif".
 
Au fait : une base de donnée mal optimisé entraine des baisses de vitesse qui peuvent dépassé la disaine de seconde dans certain cas. Qu'est ce que 40 millionémes de seconde face à ça? Il vaut mieux s'occuper d'abord des problémes qui risquent de faire perdre 250 000 fois plus de temps tu ne crois pas?

 
Donc en fait faut d'abord faire un truc qui marche, qu'est pas forcément rapide    
 

 
 
Tu pourrais détailler parce que les les seules choses que j'ai trouvé avec hashmap et array, c'est en Java    
 

 
Grosso modo (c'est crade mais bon stun exemple )
 

 
Ou bien une fois le switch pour le mode démarré pour les cat', j'utilise un if/else à la place du deuxième switch ?

On fait le truc le plus simple possible et qui marche à coup sûr, et quand on a fini si et seulement si les performances sont insuffisantes, on sort son profiler, on regarde où sont le bottlenecks et on les optimise

http://en.wikipedia.org/wiki/Hashmap
 

On peut le faire comme ça oui, néamoins je factoriserais probablement une partie du traitement dans des fonctions à part histoire de rendre l'ensemble plus lisible.
 
Donc pour commencer tu n'as pas besoin des variables intermédiaires ($mode et $cat) et peut faire un truc du style

switch ($_GET['mode'])
 {
   case 'edition' :
     switch ($_GET['cat'])
       {
         case 'compte' :
         // le code pour modifier les informations de connexion du compte
         break;
         case 'perso' :
         // le code pour modifier les informations personnelles
         break;
         case 'config' :
         // Le code pour modifier sa config du forum
         break;
         case 'images' :
         // Le code pour modifier les images (avatar, smileys etc.)
         break;
         default :
         // Le code par défaut
        break;
   case 'voir'
   // le code
  default :
  // le code par défaut
  break;
  }

 
Et ensuite personnellement je créerais une fonction ou une méthode pour e.g. la gestion de l'édition, contenant elle même un switch dispatchant sur la catégorie (et en fait je ferais probablement l'inverse, d'abord dispatcher sur la catégorie puis dispatcher sur l'action)

Ben non, ça c'est uniquement si tu balances le contenu d'un $_GET ou d'un $_POST dans une page, dans une DB ou dans un eval (rock & roll ), ici le contenu de $_GET ne dépasse pas l'entrée du switch et on le teste explicitement contre une série de valeurs admissibles (la meilleur manière de ne pas avoir de problème, accessoirement), donc tout va très bien (madame la marquise), il n'y a aucune faille de sécurité dans le code posté ici.

Je pige rien, je suis nul en anglais

switch ($_GET['mode'])
 {
   case 'edition' :
     switch ($_GET['cat'])
       {
         case 'compte' :
         // le code pour modifier les informations de connexion du compte
         break;
         case 'perso' :
         // le code pour modifier les informations personnelles
         break;
         case 'config' :
         // Le code pour modifier sa config du forum
         break;
         case 'images' :
         // Le code pour modifier les images (avatar, smileys etc.)
         break;
         default :
         // Le code par défaut
        break;
   case 'voir'
   // le code
  default :
  // le code par défaut
  break;
  }

Tu veux dire mettre le contenu des 'case' dans des fonctions, à part ? Dans ce cas, par souci de lisibilité, cela sous-entends de faire pareil pour le reste du code pour que ça soit lisible non ?

Pour sécuriser $_GET['mode'] et $_GET['cat'], je me dis que si ce n'est pas un des cas, ça renverra le contenu par défaut mais je ne sais pas si ça suffit et si mon raisonnement est juste.

EDIT : toasted

Oui et probablement.

Ca suffit largement, tant que tu n'utilises pas directement le contenu de $_GET['mode'] et $_GET['cat'] dans une interpolation tu n'as aucune raison de t'en faire à ce niveau là.

http://fr.wikipedia.org/wiki/Table_de_hachage ?

 
Je comprends déjà mieux le contenu mais ça m'a l'air un peu trop compliqué à mon niveau pour l'instant  

 
Clair que les astuces que j'ai postées ici sont juste à connaître et pas forcément à utiliser tout le temps. Perso j'utilise des switch dans mon code et je n'en suis pas mort, c'est juste pour comparer la rapidité d'exécution de quelques structures pouvant faire la même chose. Après, en fonction de la rapidité, chacun choisira ce qu'il veut en connaissance de cause, bien que certaines fonctions ou formulations soient nettement plus rapides que d'autres.

Le gros problème que j'ai avec tes schémas est qu'ils ne partent pas tous 0 et ne sont pas à la même échelle, genre le graphe entre echo et print ou celui entre file_exists et is_file qui donnent l'impression d'une différence énorme alors qu'en réalité elle est insignifiante

En effet l'échelle n'est pas la même, à vrai dire je n'y ai pas vraiment pensé ! Cependant certaines valeurs sont proches de la cinquantaine de millisecondes tandis que d'autres benchs durent nettement plus longtemps, il est donc difficile de les mettre à l'échelle en gardant un maximum de précision. Mais j'essaierai de refaire tout ça plus proprement

D'un autre coté, je vois pas en quoi un switch est plus lisible perso.
une bonne série de If bien indenté c'est au moins aussi lisible.

Parce qu'on sait qu'on comparera tout à la même valeur, et qu'on a pas de problèmes d'évaluation multiples d'une fonction, donc on peut regarder la valeur initiale (celle du switch) puis passer rapidement sur tous les case, alors que dans le cas d'une suite de if/elseif on doit parser toutes les conditions

 
Faudrait tester si la différence existe toujours une fois Zend optimiser ou eAccelerator activé

Bon, j'ai un souci avec la confirmation (pour le moment, c'est juste l'inscription et la connexion), quand je met un mauvais argument dans valid=xx, ça me met que les champs sont vides  (alors qu'il y a un message prévu au cas où c'est un tout autre arrgument qui est rentré) et que deux variables postées venant de $_POST['pseudo'] ne sont pas déclarées. Si j'utilise un switch, ça pourrait me sortir d'affaire ou bien ce n'est pas du à la structure conditionnelle if/else ?

EDIT : ça me le fais sur l'ancienne version du fichier, y'a donc bien un souci.

EDIT2 : ok, corrigé, c'est bon

je viens juste de percuter qu'h.fr n'a pas d'url rewriting, en fait ça sert à rien? Tout les gourous du SEO disent qu'il le faut absolument...

http://forum.hardware.fr/hfr/Overc [...] ujet-1.htm
 
spa de l'url rewriting ça ?

 
Y'a du rewriting que dans les parties accessibles par les bots, une fois connecté et pour les features des connectés, y'a pas.

Non. L'URL rewriting est "branché" dès que tu utilises la combo en bas de page et que tu parcours les topics  Il tombe si tu repasses sur des liens spécifiques à un utilisateur connecté (liste de drapeaux, édition de profils).

Je comprends pas trop, les deux cas prévus pour $_GET['mode'] sont séparés, hors les deux contenus des echo apparaissent l'un après l'autre . Qu'est-ce qui cloche ?

switch ($_GET['mode'])
     {
      case 'edition' :       
         echo '<div class="corps_forum">
        <img src="./template/images/bullet_arrow.png"/> <a href="index.php" title="Index du forum"><strong>Forum de test</strong></a> 
        <img src="./template/images/bullet_arrow.png"/> Profil <img src="./template/images/bullet_arrow.png"/> Edition du profil';
       switch ($_GET['cat'])
        {
         case 'compte' :
         // le code pour modifier les informations de connexion du compte
         break;
         case 'perso' :
         // le code pour modifier les informations personnelles
         break;
         case 'config' :
         // Le code pour modifier sa config du forum
         break;
         case 'images' :
         // Le code pour modifier les images (avatar, smileys etc.)
         break;
         default :
         // Le code par défaut
         break;         
       }
         break;    
     case 'voir':
     echo '<div class="corps_forum">
        <img src="./template/images/bullet_arrow.png"/> <a href="index.php" title="Index du forum"><strong>Forum de test</strong></a> 
        <img src="./template/images/bullet_arrow.png"/> Profil <img src="./template/images/bullet_arrow.png"/> Voir un profil';
    break;
    default :
    echo PAGE_ERREUR;
    break;
   }

Merci d'avance  

L'URL rewriting est partout là où c'est utile, cétout y'a pas à chercher plus loin.  

Ok, problème corrigé, un break mal placé. Par contrre, à moins de mettre un isset devant $_GET['mode'] et $_GET['cat'] (ce qui a pour effet de mettre la première valeur par défault quelquesoit le cas), j'ai un message d'erreur m'informant que les variables 'mode' et 'cat' ne sont pas définies. Comment je peux éviter ça ?

switch (isset($_GET['mode']))
     {
      case 'edition' :       
         echo '<div class="corps_forum">
        <img src="./template/images/bullet_arrow.png"/> <a href="index.php" title="Index du forum"><strong>Forum de test</strong></a> 
        <img src="./template/images/bullet_arrow.png"/> Profil <img src="./template/images/bullet_arrow.png"/> Edition du profil';
       switch (isset($_GET['cat']))
        {
         case 'compte' :
         // le code pour modifier les informations de connexion du compte
         break;
         case 'perso' :
         // le code pour modifier les informations personnelles
         break;
         case 'config' :
         // Le code pour modifier sa config du forum
         break;
         case 'images' :
         // Le code pour modifier les images (avatar, smileys etc.)
         break;
         default :
         // Le code par défaut
         break;         
       }
         break;    
     case 'voir':
     echo '<div class="corps_forum">
        <img src="./template/images/bullet_arrow.png"/> <a href="index.php" title="Index du forum"><strong>Forum de test</strong></a> 
        <img src="./template/images/bullet_arrow.png"/> Profil <img src="./template/images/bullet_arrow.png"/> Voir un profil';
    break;
    default :
    echo PAGE_ERREUR;
    break;
   }

hum, si je n'm'abuse, isset retourne un booléen, comment ça peut passer dans tes cases là ?

c'est sans doute ça qui fait merdouiller (mais ça avait "l'avantage" de masquer les erreurs qui apparaissent). Là c'est enlever par contre ça me met ça : Notice: Undefined index: cat (idem pour mode).

(j'édite le code ci-dessus)

C'est ce que j'avais percuté après coup. Pas la peine de rewriter quand ça sert à rien  

 
Bah tu fais un if avec ton isset. Si la variable est declarée, tu rentres dans le switch. Sinon c'est le else du if et puis voilà.

Ben tu peux faire de plusieurs manières. Exemple rapido :

if (isset($_GET['mode'])
    $mode = $_GET['mode'];
else
    $mode = null;
 
switch ($mode) {
[...]

 
Ça te permet d'appliquer éventuellement un filtrage sur les données envoyées par le client.

En fait, j'ai fait comme ça :
 

if(!empty($_GET['mode']))
    {
      switch ($_GET['mode']) // pour choisir le mode (ou editer ou voir le profil)
      {
        case 'edition' :       
          echo '<div class="corps_forum">
          <img src="./template/images/bullet_arrow.png"/> <a href="index.php" title="Index du forum"><strong>Forum de test</strong></a> 
          <img src="./template/images/bullet_arrow.png"/> Profil <img src="./template/images/bullet_arrow.png"/> Edition du profil';
         if(!empty($_GET['cat'])){
          switch ($_GET['cat'])
           {
            case 'compte' :
              echo '<p>compte</p>';
            break;
            case 'perso' :
              echo '<p>perso</p>';
           break;
           case 'config' :
             echo '<p>config</p>';
           break;
           case 'images' :
             echo '<p>image</p>';
           break;
           default :
             echo '<p>code par defaut</p>';
           break;             
          }
         }
         else
         {
          echo '<p>code par defaut</p>';
         }
           break;       
        case 'voir':
          echo '<div class="corps_forum">
          <img src="./template/images/bullet_arrow.png"/> <a href="index.php" title="Index du forum"><strong>Forum de test</strong></a> 
          <img src="./template/images/bullet_arrow.png"/> Profil <img src="./template/images/bullet_arrow.png"/> Voir un profil';
        break;
        default :
          echo PAGE_ERREUR; // autre cas pour 'mode'
        break;
       }
    }
else {
echo PAGE_ERREUR;
}

 
Le !empty permet de vérifier ce qui est rempli (si ça l'est, déjà et ce qui va être affiché selon ce qui est rentré )

dans cette page:
 
http://forum.hardware.fr/forum1f.p [...] w=0&nojs=0
 
en haut à droite, a coté de date dernier message, ya trois petit tiret, ca sert à quoi joce?

ben cliques dessus... tu verra

ca fait rien chez moi

ca coche toutes les cases à cocher de la colonne.

ha ok, j'avais pas de drapeau quand j'ai cliqué

 
 
Je ne comprends pas trop où ça merdoie :
dans le code ci-dessous, pour uploader un smiley, je propose un formulaire mais auparavant, je vérifie s'il n'y a pas déjà un smiley d'uploadé dans la BDD (pour ne pas changer de smiley et n'en uploader qu'un seul). Donc requête, si c'est égal à NULL (donc le champ est vide), on propose le formulaire sinon messages d'erreur. Hors, que le champ soit vide ou rempli j'ai un message d'erreur
 

$requete_smiley = mysql_query('SELECT membre_smiley FROM forum_membres') or die (mysql_error());
          $affichage = mysql_fetch_assoc($requete_smiley);
          
          if($affichage['membre_smiley'] == NULL)
            {
             echo '<p>
                 <form method="post" action="confirmation.php?valid=upload">
                 <table id="profil">
                  <thead>
                   <tr>
                     <th colspan="3">Uploader un smiley</th>
                     </tr>
                 </thead>
                 <tbody>
                  <tr>
                   <td class="intitule"><label for="upload_smiley">&Agrave; partir d\'un fichier :</label></td>
                   <td class="champ_profil"><input name="upload_smiley" type ="file" id="upload_smiley" class="fichier"/></td>
                   <td class="indication"><img src="./template/images/help.png" title="L\'astuce"/> <em>Votre smiley doit faire 20 x 20 pixels et peser 10 Ko au maximum</em></td>
                  </tr>
                 </tbody>                 
                </table>
                <input type="submit" value="Valider" class="validation"/></p>';           
            }
         else 
            {
            echo INTERDIT_UPLOAD_SMILEY; 
             
            }

mmm... test tout bêtement un if (empty($affichage['membre_smiley']))
En mysql, vide ou NULL, ce n'est pas la même chose, après faut voir comment il intéragit avec PHP, mais dans le doute, le empty résoudra ton soucis.

Ouaip, c'est ce que j'ai fait et effectivement, en allant ajouter dans la BDD le nom du fichier (le script d'upload n'est pas fini), il m'affiche bien le message d'erreur car le champ est rempli. Seulement quand je supprimes l'entrée directement dans la base, le smiley n'aparait plus dans le profil (normal) mais j'ai toujours le message d'erreur me disant que j'ai déjà un smiley