Bonjour,
 
  Je travail actuellement pour une société pour qui je dois valider / vérifier / proposer  des choix technique autour de développement web. Cela à commencer par définir des règles de développement afin de rendre le code plus homogène et pérenne sur les différents projets.
 
  Et pour un développeur je tombe sur ce code là et je voudrais le convaincre par une bonne argumentation que ce n'est pas bien, mais les mots me manque et j'ai besoin de votre aide. (sachant que dans quelque jours je ferais, je pense, lire ce post au développeur en question) :
 
Voici le code : un seul echo et c'est bien cela qui me gène :

 
Merci

qu'il développe comme une merde et que sa façon de faire du HTML date d'avant 1999 et que s'il se met pas au niveau et utilises pas des méthodes modernes, il prend la porte, ça marche pas comme argumentation
 
Qu'il regardes du côté du design pattern MVC...

Déjà quand on echo des strings contenant des double-quotes, on les englobes avec des simple pour la lisibilité, genre :

echo "<center><form action=\"scripts/gestion_comptes.php\" id=\"form_modif_compt\" method=\"post\">
<table id=\"tab_general\"><caption align=\"bottom\"><br /><br /></caption>";

Est largement moins bien que

echo '<center><form action="scripts/gestion_comptes.php" id="form_modif_compt" method="post">
<table id="tab_general"><caption align="bottom"><br /><br /></caption>';

Ensuite merci le code à la con Balise center ? Le caption vide avec deux <br /><br /> ? Mieux vaut utiliser des CSS pour la maintenance, là si tu décides que tous les formulaires doivent avoir leur padding top modifié, c'est changé en une fois, au lieu de parcourir tous les scripts et virer les <br />

J'aurais rajouter que de faire un echo sur du HTML ça fait travaille PHP pour rien.
Côté lisibilité ça donne pas envie de lire ce code...
 +1 pour rufo et FlorentG

Perso entre double quotes ou meme en fermant les balises php quand on a affaire à qu'un bloc pur HTML

 
C'est ce que j'avais proposé au développeur de fermer la balise php, car il y a beaucoup de html pour peu de php, mais ca réponse à été " oui mais je suis obliger de réouvrir le php pour faire afficher chaque variable "
 
et là ca manquait d'argumentation technique c'était plus un problème de gout et de couleur (ou pour moi de bonne pratique de la programmation)
 

En ce qui me concerne et pour des raisons de lisibilité, je préfères largement coder en HTML et mettre des balises PHP dans mon code HTML.

 
en même temps mettre

est beaucoup plus propre et lisible et ça ne coûte rien

oui mais dans nos règles de développement interne, les balises courtes sont interdites en référence à la doc officiel de php : http://www.php.net/manual/fr/langu [...] hpmode.php  

Certe me j'en suis moyennement sur car certain serveur pourront être configuré pour exécuter le PHP dans des balise <%PHP %> comme IIS
et donc ton code deviens HS !!
donc ça :

est toujours trop long .....    
sinon tu uniformise avec une fonction d'affichage (par exemple) pour gérer tes sorties écran.

Bonjour,
 
Le seul code qui passera sur tous serveurs est celui-ci :

car le fait que "asp_tags" et "short_open_tag" soient "on" ne donne que la possibilité des les utiliser, sans empêcher cette syntaxe, qui est, de surcroît, la plus courante.

 
oui, ainsi que le code initiale écrit par le développeur ....

Autre argumentation possible : la fermeture / ouverture des balises php pour du code HTML, permet à l'IDE de faire de la coloration syntaxique + d'utiliser les outils adéquoites.
Ex: repérages des balises html ouvrantes / fermantes, de proposer la completion pour les proprietes des balises html...
remarque vu ton logo, t'es surement sous linux, et tu utilises peut etre vi ou une truc du genre, donc ne connaissant pas ton environnement de developpement, mon argument ne vaut peut etre pas un clou (héhé référence subtile au post précédent )

+1
mais je ne vois pas le problème que vous avez avec

<input type="text" value="<?php echo $blabla ?>" />

Le fait qu'il faille écrire

<input type="text" value="<?php echo htmlspecialchars($blabla) ?>" />

<input type="text" value="<?php echo htmlspecialchars($blabla) ?>" />

 
heu, et quid d'une ch'tite librairie dans ton projet ou tu definis un ch'tite fonction qui raccourci tout ca ?
 

 
en fait, perso je n'utilise pas de fonction ci dessus, mais plutot :

(et la trop de php à mon gout pour fermer et r'ouvrir les balises php a chaque fois, d'où le echo)
 

J'ai tendance à faire comme ça aussi. Les lib pour manipuler ma BD et des lib pour gérer l'affichage.

+1 pour rufo, je fais pareille

J'ai ça ouais Une pour le cas normal, l'autre pour les attributs (qui ont besoin d'escaper les " en plus).
 

Ca je n'utilise plus, y'a toujours des cas où ça fait chier, j'ai plutôt un truc basé sur DOM qui se greffe sur un formulaire HTML standard, et se charge d'afficher les valeurs.

 
interessant, un petit bout de code pour montrer comment t'as mis ca en place ?

<input type="text" value="<?php echo htmlspecialchars($blabla) ?>" />

 oui mais la différence avec un echo plein de html ou bien la syntaxe raccourcie (<?=?> ) ?

C'est n'est pas un petit bout de code  

Y'en a qu'on essayer de le voir, ils ont eu des problèmes

NazzTazz > arf, je ne savais pas... je ne connais pas trop le loustic moi
 
FlorentG > Je vous demande pardon cher ami, je ne recommencerai plus, quoi que je suis toujours fort interessé... et toujours prenneur
 
 

Bah sinon il regarde du côté de Symfony, ou de Django (mais Django c'est pas du PHP)