var Phenix = function () {
Troll du Web depuis 1996

Formidablement illisible...

J’ai trouvé ce bout de code dans le plugin formidable de SPIP, je cherchais un moyen d’exporter les réponses d’un formulaire.

Franchement comment savoir ce que contient $exporter ? Ça pourrait être n’importe quoi...
Je suis pas toujours super chaud pour mettre des fonctions dans des variables. S’il est possible de faire une structure de if ... else, je pense que c’est plus souhaitable. Bien sur, il y a beaucoup de cas ou c’est très pratique, mais ici, j’ai comme un doute.

  1. <?php
  2. if ($id_formulaire > 0
  3. and $type_export
  4. and $types_echange = echanges_formulaire_lister_disponibles()
  5. and $exporter = $types_echange['exporter'][$type_export]
  6. ){
  7. $exporter($id_formulaire);
  8. }

2 Messages

  • Formidablement illisible...

    Le 23 septembre 2016 à 10:29 par nicod_

    C’est ton point de vue.
    Le mien est que, au contraire, je trouve ça très lisible et efficace.
    Dans l’ordre des tests du if, si echanges_formulaire_lister_disponibles() renvoie false ou null, les tests s’arrêtent là et la suite n’est pas évaluée.
    Ça évite donc des if en cascade.

    J’utilise tout le temps cette notation, et pas que dans SPIP.

    Répondre à ce message

Un message, un commentaire ?

modération a priori

Ce forum est modéré a priori : votre contribution n’apparaîtra qu’après avoir été validée par un administrateur du site.

Qui êtes-vous ?
Votre message
  • Pour créer des paragraphes, laissez simplement des lignes vides.