[PHP-JS] echo ou pas echo ?

**SpaceFrog** · 07/08/2007, 10h11

Bonjour,

Bon je debute le PHP et je suis surpris par certains codes ...

en effet lorseque je code mon PHP j'aurais tendance à code le html normalement et n'ouvrir les balise php que lorsue c'est nécessaire :

Code :

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

1
2
3
4
5
6
7
8
9
 
<html>
<head>
<title><?php  echo $titre; ?></title>
</head>
<body>
<p>un paragraphe</p>
<?php include($page); ?>
</body>

mais je vois très souvent des codes qui intègrent le html dans le php

Code :

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

1
2
3
4
echo ('<html>');
echo ('<head>');
echo('<title>' . $titre .'</title>')</head>
....

Quelle est la bonne façon de faire si toute fois il y en a une ?
Quels sont les avantages et inconvéniants de chaque syntaxe ?

Autant prendre de bonne habitudes dès le départ ...

Merci de vos conseils

**koopajah** · 07/08/2007, 10h15

J'ai eu la sensation, à force de lire ce forum, que c'est plutot une question de gout/lisibilité qu'autre chose en fait. Certains séparent au maximum le html du php, d'autres n'affichent le html qu'à travers des echo etc, je pense qu'il faut essayer et voir ce qu'on prefere.
Personnellement j'ai tendance à mixer les deux : quand je fais une table en html je trouve ca plus clair de n'ouvrir les balises que pour afficher le contenu de la cellule et de pouvoir bien indenter le html. Par contre quand j'affiche une liste déroulante remplie avec des valeurs php, c'est plus simple de faire des echo plutot que d'ouvrir fermer les balises php partout (pour la boucle, pour la valeur de l'option, pour son contenu, pour la fin de boucle).

EDIT: et pour echo tu n'as pas besoin des () en fait, tu peux faire:

Code :

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

echo "Bonjour ". $nom . "comment vas-tu";

**Raideman** · 07/08/2007, 10h19

Personnellement, je pense que comme vous l'avez dit, les deux méthodes se valent sur le plan de la performance.
Ceci dit, moi aussi, je mixe les deux. J'essaie de regrouper le plus de traitements possible dans mes balises php, et si je dois afficher du code html lors de ce traitement, je le mets dans du echo pour garder une lisibilité du bloc traitement sur lequel je travaille.
Sinon, tout le code html qui ne dépend que peu ou pas du tout de mon php, je le garde en clair, sans echo.

Mais comme vous l'avez dit, c'est un peu chacun ses gouts.

**Bisûnûrs** · 07/08/2007, 10h22

Personnellement j'ai tendance à bien séparer le code HTML du code PHP, question de lisibilité, notamment grâce aux éditeurs de texte qui mettent les codes en couleur, donc ne pas avoir le code HTML tout rouge, et aussi question de maintenabilité. Il est plus facile de maintenir un code :

Code :

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

1
2
3
<p id="truc">
   <?php echo $bidule ?>
<p>

que

Code :

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

echo '<p id="truc">'.$bidule.'<p>';

Après, c'est de l'ordre du goût personnel.

**SpaceFrog** · 07/08/2007, 10h27

ha ben voilà qui est rassurant, j'avais peur que les performances n'en soient affectées...
J'avoue qu'étant issu de html javascript, j'aurais plutot tendance à faire comme mon fillot (Bisunurs)...
Et il est vrai que la coloration syntaxique plaide en faveur de la séparation.

Merci de vos conseils

Je ne ferme pas le poste immédiatement si vous avez d'autres commentaires ...

**jbrasselet** · 07/08/2007, 10h30

Personnellement j'essaye de séparer au plus le code HTML et le code PHP pour la lisibilité

Néanmoins si je suis dans une boucle ou un if, j'ai tendance à ne pas fermermon php et utiliser des echos afin de conserver une structure de bloc.

Bref je fais comme raideman

Mes deux centimes

**trotters213** · 07/08/2007, 10h38

En terme de rapidité, c'est plus rapide de tout mettre dans un echo au lieu d'ouvrir et de fermer tes balises chaque fois que tu fais un appel PHP mais c'est sur que niveau lisibilité, c'est pas top.

Sinon plutôt que de tout séparer, autant utiliser un template : http://cyberzoide.developpez.com/php4/template/

PS : Puisqu'on est dans les echos, ne fais jamais ceci :

Code :

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

echo "mon texte : $ma_var";

préfères-y ceci :

Code :

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

echo "mon texte : ".$ma_var;

voir mieux, ceci :

Code :

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

echo 'mon texte : '.$ma_var;

**Yoshio** · 07/08/2007, 13h43

+1 pour ce qu'a dit trotters213.

Si c'est pour séparer le code php du html comme Bisûnûr autant prendre un moteur de template et le faire complètement.

Sinon mettre tout dans des echo est bien plus lisible que d'ouvrir et fermer les balise <?php ?> à chaque fois. J'aime pas du tout mélanger du code php/html alors tout passe par des echo.

**Amnesiak** · 07/08/2007, 14h21

Envoyé par trotters213

voir mieux, ceci :

Code :

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

echo 'mon texte : '.$ma_var;

Voir même encore mieux d'un point de vue performances :

Code :

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

echo 'mon texte : ', $ma_var;

Ceci afin d'épargner une concaténation de chaine

**Yoshio** · 07/08/2007, 14h39

Envoyé par Amnesiak

Voir même encore mieux d'un point de vue performances :

Code :

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

echo 'mon texte : ', $ma_var;

Ceci afin d'épargner une concaténation de chaine

C'est de très très loin recommander de faire ce genre de fantaisie il me semble.

**Amnesiak** · 07/08/2007, 15h10

Absolument ! Comme l'indique son prototype, echo accepte un nombre variable d'arguments. Il est donc intéressant d'en profiter plutôt que forcer une concaténation systématique non obligatoire. print() par contre ne le permet pas...

void echo ( string arg1 [, string ...] )

**Dsphinx** · 07/08/2007, 15h46

Ton code dépend aussi de ton habitude de travail... Si tu reprend le code d'un CMS (ou le code est déjà écrit) il vaut mieux continuer dans la même forme afin d'uniformiser la structure de ton code !