Discussion :

[maxence64]

Salut à tous,

Je débute un peu en php et de ce fait en programmation.

Sachant qu'il y a X manières d'écrire la même chose, je me pose souvent la question de savoir qu'elle est la meilleure manière d'écrire telle chose.

Si les modérateurs le permettent je laisserai ce thread ouvert pour toutes questions de syntaxe.

Voici ma 1re question.

Vaut-il mieux écrire ceci :

Code :

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

1
2
3
4
5
6
7
 
if (!empty($_POST['nick']) && !empty($_POST['pass'])){
echo 'formulaire ok';
}
else {
header('Location: http://www.monsite.com/subscribe.php?error=form');
}

Ou ceci :

Code :

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

1
2
3
 
if (empty($_POST['nick']) && empty($_POST['pass'])) : header('Location: http://www.monsite.com/subscribe.php?error=form'); die(); endif;
echo 'formulaire ok';

Ou est-ce que c'est exactement pareil ?
le die permet d'arrêter la suite du code n'est-ce pas ?

Sachant que quand il y a pleins de "else", ça devient lourd de savoir à quoi correspond une erreur dans un gros fichier php : il faut descendre tout en bas, voir TOUS les else et faire le rapport avec le bon if qui se trouve tout en haut

Merci d'avance de vos lumières.

[m4riachi]

Code :

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

1
2
3
4
if (empty($_POST['nick'])  && empty($_POST['pass'])) 
          header('Location: http://www.monsite.com/subscribe.php?error=form'); 
 
echo 'formulaire ok';

[Invité]

Bonjour,
à mon avis, l'écriture avec "if () { ... } else { ... }" est la plus lisible et la plus facile à dégoguer, surtout si le code est "propre" (bien implémenté) :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
<?php
if (empty($_POST['nick']) && empty($_POST['pass']))
{
    header('Location: http://www.monsite.com/subscribe.php?error=form');
    exit;
} else {
    echo 'formulaire ok';
    // ....... suite du traitement .....
?>
<!-- ....... suite du traitement ..... -->
<?php
} // fin traitement
?>