Discussion :

[Rakken]

Bonjour,

Je croise régulièrement dans du code php, des fichiers .php qui ne comportent pas de balise fermante.
Genre la premiere balise est <?php et nulle part il n'y a de ?>

Après m'être renseigné auprès de certaines personnes ne fermant pas leur balise php, il apparait que l'interet est d'être sur qu'il ne reste pas de ligne vide en fin de fichier (ce que provoquer des problèmes d'envoi de header par exemple).

Personellement, je ne trouve pas ca très propre de ne pas fermer, mais dans le doute je pose la question, qu'est ce qui est le mieux, fermer ou ne pas fermer la balise php ?

[parksto]

c'est inutile de la fermer, et source d'erreur... pourquoi le faire ?



ne pas indenter son code est correct aussi, mais source d'erreur... personne ne le fait. c'est pareil pour moi.


si un jour tu bosse à plusieurs sur un projet, tu sera triste que tout le monde mette cette balise fermante le jours ou tu auras une erreur d'envoi de header et que tu devra vérifier tous les fichiers du projet pour les vérifier

[randimby]

Si tu veux éviter les espaces en fin de script, le mieux est de faire

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
return;
?>

Le return s'assure que tout code après, y compris les codes non PHP, ne seront pas traités.

[randimby]

c'est inutile de la fermer, et source d'erreur... pourquoi le faire ?

Pas d'accord. Il est question de propreté. S'il est inutile de laver la vaisselle après avoir mangé, et qu'en plus, les laver augmente le risque de les casser, tu ne les laverais pas?

[parksto]

je ne trouve justement pas très propre de dire à l'interpreteur php que le code php est terminé et qu'on repasse dans une zone à ne pas interpréter alors que celle-ci n'existe pas.


c'est ça pour moi l'utilité de la balise fermante.


à noter au passsage que zend (oui oui, eux même) déconseille d'utiliser cette balise fermante lorsqu'on peut s'en passer

Pour les fichiers contenant uniquement du code PHP, le tag de fermeture ("?>") n'est jamais permis. Il n'est pas requis par PHP. Ne pas l'inclure permet de prévenir les problèmes liés à l'injection accidentelle d'espaces blancs dans la sortie.

(extrait des conventions de codage du zend framework)

[Rakken]

Après recherche dans la doc, je suis tombé la dessus

Et donc, dans la doc il est écrit que c'est optionnel, mais sans véritable préconisation...
En tout cas, ca n'est pas visiblement pas une "faute", même simplement de norme, de ne pas en mettre, même si je continue a trouver ce fonctionnement un peu limite.

Bref, je crois que je vais continuer a la mettre (ne pas le faire reste "optionnel", ca me donne aussi l'option de faire ce qui me semble propre ^^), mais je vais par contre arreter de raler sur ceux qui "l'oublie", volontairement ou non.

Merci ;-)