Discussion :

[laurentSc]

Bonjour,
mon code est basé sur l'existence de $_POST['user_name'] (if (isset($_POST['user_name']))) et je ne comprends pas pourquoi cette condition n'est pas valide.

index.php :

Code php :

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

1
2
3
 <p>Bienvenue sur le mini site consacré aux villes. </p>
            <?php include('user_authent.php');?>
            <?php require('inc_menu.php'); ?>

user_authent.php :

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
  <?php
            if (!isset($_POST['user_name'])) { ?>
            <p>
              Pour commencer, merci de vous identifier :<p/>
 
              <form method="POST" action="">
                <input type="text" name="user_name" />
                <input type="submit" name ="submit_user_name" value="votre nom" />
            </form>
<?php }
else {
    $user_nom = $_POST['user_name'];
  ?>
    <p>Utilisez le menu de navigation pour consulter les pages du site ou rechercher <a href="search.php?name=<?= $user_nom ?>">ici</a>. <br>
    Bonne visite !</p>
 <?php } ?>

dans villes.php :

Code php :

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

<?php echo "include menu<br/>"; var_dump($_POST); include("inc_menu.php"); ?>

inc_menu.php :

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
 <?php
            $result = $mysqli->query('SELECT ville_id, ville_nom FROM villes');
            while ($row = $result->fetch())
            {
                $villes[$row['ville_id']] =$row['ville_nom'];
            }
            ?>
 
<?php if (isset($_POST['user_name'])) { ?>
        <ul>
            <li><a href="index.php">Accueil</a></li>
            <?php foreach($villes as $id => $ville) : ?>
            <li><a href="villes.php?id=<?php echo $id ?>"><?php echo $ville ?></a></li>
            <?php endforeach ?>
        </ul>
 <?php } ?>

Quand je vais dans villes.php, cela affiche ça :

include menu
array(0) { }

Pourtant, m'étant déjà identifié donc passé dans user_authent.php, l'action du formulaire aurait du créer une variable $_POST, non ?

[laurentSc]

En l'absence de solution, j'ai modifié le code pour ne plus avoir le souci. Tant pis pour ce mystère...

[cavo789]

Bonjour

Sauf erreur (j'ai un doute toutefois) tes variables postées sont accessibles uniquement dans la page qui reçoit la soumission du formulaire et donc user_authent.php uniquement.

Ici tu souhaites récupérer la variable dans un autre script... $_POST n'est pas $_SERVER (accessible partout).

[laurentSc]

Moi aussi, j'ai un doute ; j'étais même persuadé d'autre chose. Pour clarifier la chose, j'ai fait une recherche et trouvé un site qui dit autre chose :

Ils sont super globaux, et donc on peut les atteindre de n’importe où dans le programme (même dans une fonction).

vu dans http://www.oujood.com/php/variable-g...ost-de-php.php

[cavo789]

"n'importe où dans le programme", est-ce qu'il faut comprendre programme par le script lui-même ou n'importe quel script, le doute reste pour moi.

[cavo789]

Note : lorsqu'on fait un include d'un fichier, logiquement, il n''est pas attendu qu'un code s'exécute. C'est un concept aussi certainement pas une règle obligatoire.

En clair : include, c'est charger un fichier. Ensuite, par exemple, tu appelles une fonction qui se trouve dans le fichier.

Et cela répondrait à ton problème... Tu appellerais la fonction avec, comme paramètre, ta variable. Ce faisant ton code sera plus "propre", le risque de bugs potentiels étant alors moindre..

[laurentSc]

J'ai du mal à te suivre. Tu pourrais expliciter un peu plus STP ?

[cavo789]

Il s'agit du concept de side effect expliqué ici : https://github.com/jupeter/clean-code-php

Si je fais un include inc_menu.php rien, strictement rien, ne devrait se passer. Zéro affichage zéro modification, rien.

Dans ton cas, il y aura un output car tu fais un var_dump ou tu écris du contenu html. Un code dit "propre" ne devrait pas faire ça.

Ton fichier devrait implémenter une fonction et, après mon include, là j'appelle la fonction. C'est là que je passerai ta variable.

[laurentSc]

Merci pour le lien qui est intéressant.

J'ai un peu compris ce qu'il fallait faire, mais j'y verrai bien plus clair demain matin. Donc je m'y colle demain et reviens vers toi.

[laurentSc]

Bonjour Christophe,

je suis revenu sur la discussion. J'avoue ne pas savoir comment suivre ton conseil.
Tu dis que le fichier includé devrait implémenter une fonction. Mais ce que je voulais mettre dans le fichier includé (car utilisé à plusieurs endroits, donc c'est plus propre que de dupliquer le code), c'est un formulaire html (parfois son affichage est conditionné) :

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
<?php
$result = $mysqli->query('SELECT ville_id, ville_nom FROM villes');
while ($row = $result->fetch())
{
    $villes[$row['ville_id']] =$row['ville_nom'];
}
?>
    <ul>
        <li><a href="index.php">Accueil</a></li>
        <?php foreach($villes as $id => $ville) : ?>
            <li><a href="villes.php?id=<?php echo $id ?>"><?php echo $ville ?></a></li>
        <?php endforeach ?>
    </ul>

Or, je doute qu'on puisse donner à une fonction un rôle d'affichage. Du reste, dans le lien que tu m'as donné, le rôle d'une fonction semble bien plus limité :

A function produces a side effect if it does anything other than take a value in and return another value or values.

Qu'en penses-tu ?

[cavo789]

Bonjour

Imagine le fichier "main.php" avec un "include menu.php".

(j'écris à la grosse louche)

Code :

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

1
2
3
4
 
echo "Avant mon include";
include "menu.php";
echo "Après mon include";

L'idée de clean code et de avoid side effect est que, ce que je dois voir à l'écran est "Avant mon include" suivi de "Après mon include" ==> le fait de charger menu.php ne devrait avoir aucun impact; ni sur mon affichage ni sur quoi que ce soit d'autre.

Code :

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

1
2
3
4
5
 
ma_variable="Toto";
echo "Avant mon include";
include "menu.php";
echo "Après mon include";

Second exemple ci-dessus; "ma_variable" ne peut pas être modifiée par le simple fait d'inclure menu.php. C'est ça, le side effect. Je ne peux pas "toucher" ni à mon contexte (mes variables, ...) ni à l'affichage par le simple fait d'inclure un fichier.

Regardons ce qui serait une mauvaise approche

main.php

Code :

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

1
2
3
4
 
echo "Avant mon include";
include "menu.php";
echo "Après mon include";

menu.php

Code :

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

1
2
 
echo "Je suis dans menu.php";

Tu le vois, à l'écran, j'aurais donc "Avant mon include" suivi de "Je suis dans menu.php" suivi de "Après mon include" et, ça, c'est pas cool du tout parce que je ne veux pas avoir un affichage parasite.

Autre exemple:

main.php

Code :

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

1
2
3
4
5
6
 
firstname="Toto";
echo "Avant mon include";
include "menu.php";
echo "Après mon include";
echo "Salut ".firstname;

menu.php

Code :

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

1
2
 
firstname="Alfred";

Et paf, "Toto" est devenu "Alfred" (Side effect). Va-y pour déboguer cela ! Si tu lis le code de main.php; tu vas t'arracher les cheveux pour comprendre mais misère, pourquoi Toto n'est plus Toto. (ici je n'ai qu'un seul include; imagine des includes par dizaines)

C'est pas "clean code" tout ça.

Meilleur code

Code :

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

1
2
3
4
5
 
echo "Avant mon include";
include "menu.php";
write_menu("Toto");
echo "Après mon include";

Code :

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

1
2
3
4
5
 
function write_menu(string Firstname) {
    echo "Bonjour " . Firstname;
    echo "Voici ton menu";
}

(j'ai écris à l'arrache; pour illustrer le propos; le code n'est syntaxiquement pas correct)

[laurentSc]

Merci pour ton message écrit à l'arrache.

Tu as bien mis un exemple "pas cool" qui fait un affichage dans menu.php mais tu ne proposes pas d'alternative. Cependant, comme cet affichage doit être fait plusieurs fois, je suis tenté de faire un include. Comme c'est pas cool, comment faut s'y prendre ?

Version pas cool :
main.php :

Code php :

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

1
2
3
4
 
echo "Avant mon include";
include "menu.php";
echo "Après mon include";

menu.php :

Code html :

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

1
2
3
4
5
6
 
<ul>
<li><a href="page1.php">item1</a></li>
<li><a href="page2.php">item2</a></li>
<li>etc</li>
</ul>

[cavo789]

mais tu ne proposes pas d'alternative.

Ben si, mon tout dernier exemple. Le fichier inclus contient une fonction write_menu.

[laurentSc]

Exact ; pas fait attention.

Maintenant, ça doit être cool

write_Menu.php :

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
<?php
function write_Menu($mysqli) {
$result = $mysqli->query('SELECT ville_id, ville_nom FROM villes');
while ($row = $result->fetch())
{
    $villes[$row['ville_id']] =$row['ville_nom'];
}
?>
    <ul>
        <li><a href="index.php">Accueil</a></li>
        <?php foreach($villes as $id => $ville) : ?>
            <li><a href="villes.php?id=<?php echo $id ?>"><?php echo $ville ?></a></li>
        <?php endforeach ?>
    </ul>
<?php } ?>

dans index.php :

Code php :

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

1
2
3
4
5
<?php if (isset($_POST['user_name']))
            {
                include('write_Menu.php');
                write_Menu($mysqli);
            }?>

Est-ce bien du clean code ?

[cavo789]

Yep c'est plus beau, non ? 😉

Tu as dissocié le chargement avec zéro side effect puis, proprement, tu appelles la fonction au bon moment.

C'est "plus mieux" 😉