Coucou :D

Mais c'est vrai que nous (je prend les "anciens" de PHPCS), on est assez "puristes" niveau code PHP :)

Donc faut nous excuser aussi si on s'emporte pour 3 fois rien quelques fois ;)

Je vais rester quand même dans le domaine de la gentillesse :p

"et perso je n'aime pas travailler avec les $_POST pour diverse raison:

echo "salut $_POST['nom']"; => j'ai du mal"

C'est pas parce qu'on a "du mal" qu'il ne faut pas savoir s'adapter. Si $_POST[] existe, c'est pas pour rien. Je comprendrais jamais ce manque de suivi de l'évolution d'un langage. C'est pas parce qu'on connait les bases qu'il faut s'arrêter la. Mais bon, nous avons 2 conceptions différentes de la chose...

Maintenant, je le vois bien que c'est du code rapide. Je m'en suis aperçu dès le début. Suffit de voir l'utilisation de variables non déclarées ou autres genre de petits trucs qui m'ont mis la puce à l'oreille dès le début.

Tu fais ce que tu veux de ce que je viens de te dire plus haut. Les quelques conseils disséminés dans mes posts ne sont pas la que pour cracher sur ton code, ils sont la surtout pour te permettre d'évoluer dans le langage qu'est le PHP. Libre à toi d'interpréter mes posts comme tu le sembleras mais je m'en tiens à mon expérience et je ne peux pas revenir sur mes déclarations.


Après, entreprise/open source, y'a limite un petit foutage de gueule :) C'est pas parce qu'on est en entreprise qu'on n'a pas le droit d'être rigoureux sur du code. C'est ce que je te fais remarquer et il ne faut pas prendre cela pour une attaque personnelle.



Et pour finir :
"donc passer trois heure a faire ce que ce code faire un trois seconde, la propreté, je m'en balance." Je te dis merci. Oh si, un grand merci ! Car c'est grâce à des gens comme toi que des gens comme moi trouvent du boulot :)
Imagine si tout le monde sortait du code suffisamment propre... le quart voir la moitié des développeurs seraient au chômage :)
Finalement, je t'aime bien !

xD

Et si tu utilises un formulaire sur ton site, vas tu utiliser ce même type de fonctionnement ??
J'ai de gros doutes :o


Pour moi, ce code ne sert à rien sauf à masquer une éventuelle flemme du codeur. Je m'explique :

Comment peut-on encore trouver quelque chose d'aussi exubérante :
# $filtre_radio = $_POST['filtre_radio'];
# $name_page = $_POST['name_page'];
# $requete_sql = $_POST['requete_sql'];

???

Mais à quoi cela sert-il ? A cacher le fait que les données proviennent d'un POST ??
La vrai question que tu devrais te poser n'est pas "comment faire pour me débarasser de ce POST afin d'avoir juste les noms de variables ?" mais plutot "comment exploiter au mieux POST pour éviter de dupliquer mon code ?"


Pourquoi vouloir extraire à tout prix quand tout est déjà créé dans un tableau et qu'on vous a mâché tout le travail ?  
Tu peux arriver à la même chose avec POST[] ... :s


Pour le coup du "puisque que l'extracteur n'est pas prévu pour être utiliser DANS le site, mais lors de la création du site web", je la connais on la sort quand on veut pas se faire chier. Mais un petit bout de code montre la qualité d'un code en entier et c'est la ou j'ai un peu peur.

Pas de panique, je ne mange personne, je ne fais qu'exposer mon point de vue.



Ex (sans ton extracteur) {
foreach ( $_POST as $key=>$val ) {
   echo $key.' -> '.$val.'<br />';
}

Pas besoin d'extracteur :)
De plus : $num = preg_match_all( "/name=\"(.*)\"/Us" , $texte, $mots);
Avec ça, tu vas récupérer tous les name="" de chaque balise... sachant que 'presque' toutes mes balises en possèdent, y'a pas un petit problème quelque part ? ;)

"le but du code n'est pas de gagner du temps sur les $_POST[], mais plutot de retrouver toutes les variables utilisées dans un formulaires."

Alors ton code ne sert strictement à rien. Sauf à multiplié par 2 (environ) la charge mémoire, ce code ne sert à rien du tout.

Pourquoi vouloir se forcer à utiliser $name plutôt que $_POST['name'] ??
Quelque chose doit m'échapper... :s

Différence de temps entre $nom et $_POST['nom'] pour l'écriture ?

Minime :s


Surtout que maintenant, avec les nouveaux éditeurs, il te suffit d'une combinaison de touche pour écrire $_POST[] tout seul :s


Mais bon :o

Pour faire un arbre de ce type, utilise XML :

<racine>
<element1>Nom</element1>
<element2>xxx...</element2>
<element3>
  <sselement31>...</sselement31>
  <sselement32>...</sselement32>
</element3>
</racine>

Si tu veux le sauvegarder, parse ton document XML pour le foutre en chaine et ou dans une base SQL par la suite.

Pour un arbre, XML y'a pas mieux !

Il est évident que le gain ne se mesure pas sur "Si i == 1" "Si i == 2" ...

C'est absurde :o

Il est beaucoup plus aisé d'écrire :

switch ( $class->ma_methode() ) {
   case TRUC:
     //
   case MACHIN:
     //
   /// etc..
}

que :

$retour = $class->ma_methode();
  if ( $retour == TRUC ) {
        //
  } elseif ( $retour == MACHIN ) {
        //
  } // etc...

:)

Sinon pour les transactions, je sais qu'on peut contourner comme toi et FHX le dites, mais ce qui me gêne c'est que pour une classe de 30 methodes de transactions ça fait redéfinir 30 mèthodes à vide (bon là on est grosso modo à 5 methodes de transactions donc ça peut aller :)).


Dans ce cas :

abstract class db {
}

class Transactiondb extends db {
}

class NonTransactiondb extends db {
}

Et puis roule ma poule :)

Mala ? J'ai pensé à un truc.

Pourquoi ne pas créer une classe dynamique au moment du parsing XML ? Ou alors, remplir des données dans une classe :

<?xml>
<root>
<translate id="nom_de_la_constante">La traduction qui va avec</translate>
<translate id="nom_de_la_constante_no2">L'autre traduction </translate>
</root>

<?php
// Eventuellement y mettre un Singleton au fesses, c'est plus sympa :D
class Language {
  private $tab = array();
  private $init = false;

   public function setInit($bool) {
    $this->init = $bool;
   }

   public function __set($id, $traduc) {
     if ( $this->init) $this->$tab[$id] = $traduc;
   }

   public function __get($id) {
     if ( !$this->init ) return $this->tab[$id];
   }

}

// Tu fais ton parsing XML, et tu introduis ton couple ID/traduc dans la classe
// Puis, l'appel ce fait comme ceci :
$lang = new Language(...);
echo $lang->ROOT;
echo $lang->WELCOME_HOME;
echo $lang->KIKOO;

// ET puis voila :) Eventuellement, la classe de langage peut se charger du parsing XML à l'init :
$lang = new Language('fr');
// Et on charge les paramètres français depuis le fichier XML francais.
Encore mieux, si on doit faire du multilangage sur un site :
$lang_fr = new Factory_Language('fr');
$lang_de = new Factory_Language('de');
// Et quand on est dans une classe, on rappèle la factory qui se charge de faire un singleton :
class x {
public function x() {
  $lang_fr = new Factory_Language('fr');
  // stuf...
}
}

Une solution rapide et assez efficace. Reste à savoir si ca tiens la route sur x00000 de visiteurs à la secondes :)