Bonjour,

Ce n'est jamais simple de savoir "quand/comment séparer mon code".
Il faut pour cela se poser les bonnes questions.
Pour commencer:
- Est-ce que je peux découper mon code en fonctionnalité ?

Exemple:
J'ai une page d'administration qui affiche la liste des utilisateurs.
Sur cette page, j'ai écrit du code pour vérifier si l'utilisateur est connecté (sinon il n'a pas accès), je récupère la liste des utilisateurs depuis la db, et j'affiche enfin le code html avec mon menu, mon tableau avec mes utilisateurs, mon footer, etc.

La page s'affiche convenablement, la logique est bonne. Maintenant, est-ce que je peux découper mon code ?
Pour cela, je vais relire mon code et voir si je peux trouver une certaine logique là dedans.
Le plus simple est encore de prendre une feuille de papier à côté et travailler par point.

Pour ma page d'administration, je dois:
- Vérifier si l'utilisateur est connecté
- Si l'utilisateur n'est pas connecté, le rediriger vers la page d'accueil (ou lui afficher un message d'erreur)
- Si l'utilisateur est connecté, lui afficher la page d'administration

Là, on voit déjà que j'ai 3 étapes importantes.
On peut donc séparer notre code de cette manière:
- une fonction qui s'occupe de vérifier si l'utilisateur est connecté (retourne par exemple un booléen ou le nom de l'utilisateur connecté)
- une fonction qui redirige le visiteur vers la page d'accueil
- une fonction qui affiche le code html de la page d'administration

Dans notre page d'administration, on commencera par appeler la fonction qui vérifie l'utilisateur connecté.
On récupère le retour de cette fonction dans une variable.
On fait une condition avec cette variable.
Si c'est vrai (si l'utilisateur est bien connecté) on call la fonction qui affiche la page d'administration.
Si c'est faux (simple visiteur non connecté) on call la fonction qui redirige vers la page d'accueil.

Dans un premier temps, on écrit nos 3 fonctions dans la page d'administration. Pas besoin de les sortir tout de suite.
On devrait donc avoir quelque chose comme ceci:

<?php
function checkUser() {...}
function redirectIndex() {...}
function getAdminView() {...}

$user = checkUser();
if ($user) { getAdminView(); }
else { redirectIndex(); }
?>

Le code de la page d'administration est déjà plus propre.
On a bien séparé les différents points. C'est plus lisible.
Cependant, la fonction getAdminView est encore assez lourde.
A l'intérieur, on récupère les utilisateurs depuis la db, on construit un tableau avec les utilisateurs, et on affiche notre page d'administration (menu, tableau, footer).

Nous allons donc reprendre notre feuille de papier et voir si on peut découper notre code.
Les points de la fonction getAdminView:
- se connecter à la db
- exécuter une requête SQL pour récupérer les utilisateurs
- construire un tableau html avec les utilisateurs
- afficher la page d'administration avec le tableau (layout + injection du tableau des utilisateurs dans le content)

Tu l'auras compris, on peut donc créer une fonction par point.

- getDB() => retourne l'objet PDO
- getUsers() => dans cette fonction, on call getDB pour récupérer notre PDO, on exécute notre requête SQL et on retourne la liste des utilisateurs
- getUsersTable($users) => on construit le tableau avec la liste des users qu'on aura eu en paramètre. Et on retourne le tableau html (string).
Pourquoi mettre $users en paramètre et pas faire un call de getUsers dans la fonction ? Car de cette façon, on reste générique. On pourra ainsi réutiliser la fonction getUsersTable en lui fournissant en paramètre une autre liste plus ou moins complète. Exemple: sur une autre page, je veux afficher le même tableau mais contenant uniquement les utilisateurs commençant par la lettre B.

La structure de getAdminView() sera donc:

function getAdminView(){
  $users = getUsers();
  $table = getUsersTable($users);
  $title = 'Page d\'administration';
  $content = $table;
  include 'includes/layout.php';
}

Et voilà, notre page d'administration est donc maintenant découpée par point. Un point = une fonction.
C'est plus long à écrire, le fichier est plus gros, mais la lisibilité est meilleure.
Quand il s'agit d'un gros projet, cette étape est nécessaire. Sinon on se retrouve dans un flou absolu !

Maintenant, nous pouvons nous poser une deuxième question:
- Est-ce que le code que j'écris peut s'utiliser ailleurs ?
Si oui, je vais alors déplacer le code dans un autre fichier qui pourra être utilisé ailleurs.
Vu qu'on a déjà découpé notre code en fonction, le travail sera plus simple.
- checkUser => fonction générique, je peux m'en servir ailleurs. Donc je vais le déplacer ailleurs (dans un fichier "tools.php"/"utils.php" par exemple)
- redirectIndex => je peux m'en servir ailleurs. Je déplace.
- getAdminView => propre à ma page d'administration. Logiquement, je ne devrais pas m'en servir en dehors de ma page. Je le laisse donc là.
- getDB => je peux m'en servir ailleurs. Je déplace.
- getUsers => je peux m'en servir ailleurs. Je déplace.
- getUsersTable => propre à ma page d'administration. Pour l'instant, je ne vois pas l'intérêt de le déplacer (toujours possible de le faire plus tard si je dois afficher mes utilisateurs sur une autre page)

Pour profiter des fonctions que j'ai déplacé, il suffit d'inclure mon fichier tools.php dans ma page.
Exemple:
require 'tools.php'; // On peut aussi remplacer 'require' par 'include' qu'on a déjà utilisé durant les cours. include=inclure le fichier mais ne retourne pas d'erreur si le fichier n'existe pas. require=fait la même chose que include mais retourne une erreur si le fichier n'existe pas. Dans notre cas, il est préférable d'utiliser require afin de retourner une erreur dès le début vu qu'on a besoin de tools.php pour faire notre call à getUsers.
$users = getUsers();


C'est terminé pour la partie "découpage".
A noter qu'ici on a découpé au maximum. Mais il n'y a pas règle.
A nous de juger si c'est utile ou non.
Avec la structure MVC, on verra que cette découpe est plus naturelle.


Pour l'édition d'un utilisateur via un bouton, je te propose d'envoyer l'information en GET à une page d'édition.
Structure:
listing.php (contient une table avec les utilisateurs + un bouton à côté de chaque utilisateur)
edit.php (contient un formulaire d'édition. un input text pour le login, un input password pour le mot de passe)

Au moment où je clic sur un bouton, je suis renvoyé à la page edit.php.
l'input text de ma page edit.php sera complété par le login de l'utilisateur que je souhaite éditer.
l'input password de ma page edit.php ne sera pas complété car je suis incapable de connaitre le mot de passe de l'utilisateur (il est hashé).

Au niveau du bouton, 2 façons de faire:
- un formulaire
- un lien hypertext

Pour le formulaire, on aura:
- un input hidden contenant la valeur de l'identifiant de l'utilisateur qu'on souhaite éditer (ne pas oublier d'ajouter le paramètre name dans la balise)
- un input submit pour valider le formulaire (notre fameux bouton)
- une action qui redirige vers edit.php avec method GET
Pour le lien hypertext, on aura:
- un href qui redirige vers edit.php en rajoutant l'identifiant de l'utilisateur en paramètre dans l'url.
On aura donc quelque chose comme ceci: href="edit.php?id=1234"
(Note: il est possible avec bootstrap de transformer le rendu visuel d'une balise <a> en bouton via des classes comme btn. Voir doc)

La récupération de l'id se fera bien sûr en php (Exemple pour l'hypertexte: href="edit.php?id=<?php echo $user['id'] ?>")

De cette manière, on aura envoyé l'identifiant de l'utilisateur à la page edit.php.
Avec cette information, notre page pourra alors lancer une requête SQL afin de récupérer les informations de l'utilisateur qu'on souhaite éditer.
Exemple pour la requête: 'SELECT * FROM user WHERE id ="'.$_GET['id'].'";'


Si tu bloques encore, ou si mes explications ne sont pas claires, on en reparle demain au cours.

Il s'agissait ici de deux questions de logique.
Et en tant que développeur, la logique est très importante !
Pour l'instant, c'est normal de bloquer sur ce genre de chose.
Pas de quoi s'inquiéter. Cela fait partie du cours.

Mon but est justement de vous mettre dans ce genre de situation.
C'est en pratiquant qu'on forge notre logique.


A la fin du module, vous serez capable de créer un site web dynamique.
C'est bien, mais je ne te le cache pas: n'importe quel étudiant sortant d'une école informatique est capable de le faire.

Mon objectif est de vous démarquer des autres.
En plus de développer un site/une application, vous serez capable de le faire proprement (c'est très important pour une entreprise !).
De booster votre logique (qui est en lien direct avec la propreté de votre code).
D'avoir le réflexe d'aller voir de la doc, de trouver des solutions face à un problème, une inconnue.
D'utiliser et maitriser rapidement des nouveaux outils/librairies.

Et quand je vois que des étudiants cherchent à améliorer la structure du code/la logique, c'est qu'on a déjà atteint quelques objectifs.
Je ne m'inquiète donc vraiment pas sur votre parcours professionnel ! ;)


Bonne journée !

François Bruynbroeck


De : FreeDemoCss <guillaume.cornez@gmail.com>
Envoyé : mercredi 14 mars 2018 01:28
À : François Bruynbroeck
Objet : RE: Problème include, et footer

Bonsoir ou bonjour .

Je vous recontacte car j’ai 2 questions . dont une ou je peux pas faire sans vous.
Si jamais vous n’avez pas le temps ce n’est pas grave j’attendrais jeudi .

Quand en classe vous parliez de splitter le code, est ce que dans le code ci-joint c’est bon pour vous ou ce n’est pas comme ça qu’il fallait faire ? J’avoue que j’était pas super concentré sur ce que vous disiez du coup, j’ai regarder un peu sur les codes des personnes de ma classe mais ça me semble un peut hasardeux chez eux .

La deuxième c’est concernant l’édition des membres.
J’ai rajouter un bouton édit (qui ne sert à rien pour l’instant.)
J’aurais aimer qu’on puisse éditer ce membre la en particulier via son bouton édit qui lui est attribué, mais je ne sais pas trop comment m’i prendre ni comment poser la question sur Google, que ce soit en français ou en anglais .
Du coup je me demandais si vous pouviez m’aiguiller de comment faire ma recherche sur Google ou quelque chose du genre.

Merci beaucoup et bonne journée/soirée.
Encore une fois si vous n’avez pas le temps de me répondre ou quoi c’est pas du tout un problème je sais attendre jusqu’à jeudi.

Provenance : Courrier pour Windows 10

De : François Bruynbroeck
Envoyé le :vendredi 9 mars 2018 18:38
À : FreeDemoCss
Objet :RE: Problème include, et footer

Bonsoir,

Pour le footer, le problème est qu'au niveau de ta css custom, tu places le footer en position absolue:
.footer {
  position: absolute;
}

Du coup, il passe au dessus de ton content. Il suffit de retirer la ligne "position: absolute" pour que cela fonctionne (et ton footer sera alors en bas de page).

Concernant l'include (je suppose que tu parles du formulaire d'authentification), le problème est un peu plus complexe.

Ton formulaire est dans le fichier header.php avec comme action "include/secret.php".
Quand tu valides le formulaire, tu es renvoyé vers include/secret.php (jusque là, c'est normal car tu as bien un dossier include avec un fichier secret.php).

Dans le fichier secret.php, tu fais un include de header.php (qui se trouve un niveau en dessous du dossier include).
Du coup, quand tu souhaites valider le formulaire depuis secret.php (donc depuis l'url: http://monprojet.local/include/secret.php) l'action de ton formulaire reste "include/secret.php".
Et donc, tu seras redirigé vers la page include/include/secret.php (url: http://monprojet.local/include/include/secret.php) car ton action est faite depuis monprojet.local/include/secret.php et non depuis monprojet.local/header.php.

La solution serait donc de ne pas appeler le formulaire (include './header.php') depuis un autre niveau (dans ton cas, depuis le dossier Include).

Niveau structure, je te propose quelque chose comme ceci:

MonProjet/
  index.php
  login.php
  logout.php

  Include/
    header.php
    content.php
    footer.php
    layout.php

  css/
    style.css


index.php va contenir la page d'accueil (en initialisant les variables (ex: title/content) + l'include de include/layout.php).
login.php va vérifier si la personne est connectée. Si pas, afficher le formulaire de connexion (formulaire qui sera placé dans la variable content qui sera envoyé dans la layout. Même principe que pour index.php)
logout.php va supprimer la session et rediriger l'utilisateur vers la page index.php (pas besoin d'écrire de l'html ou d'include la layout. On aura que du php).
Pour info:
- session_unset(); // unset la variable $_SESSION
- session_destroy(); // détruit les données de la session
- header('Location: index.php'); // Rediriger l'utilisateur vers la page index.php

Côté include:
header.php va contenir le menu (navbar)
content.php va contenir la structure du content (ex: <div class="container"><?= $content ?></div>). C'est grâce à la variable content que l'on va pouvoir alimenter notre page.
footer.php va contenir le pied de page (logiquement, pas besoin d'écrire de php là dedans. Juste de l'html)
layout.php va récupérer le header, le content et le footer via des includes. Le tout, dans le body. (ex: https://github.com/FBruynbroeck/cours.php.base/blob/master/04_includes/includes/layout.php)

Je te propose de faire un tour dans le dossier 04_includes accessible à cette adresse: https://github.com/FBruynbroeck/cours.php.base/tree/master/04_includes
Cela va te donner une idée.

J'espère avoir pu éclaircir tes zones d'ombre.

Sache cependant qu'il ne s'agit pas ici d'une structure conventionnelle.
Pour cela, nous verrons le MVC d'ici 2 semaines (pas de stress, ça va aller ! Si tu parviens à jongler avec les includes/requires, le MVC en php ne sera pas trop compliqué).

Si tu as d'autres questions n'hésite pas.
Passe un bon week-end.

François Bruynbroeck


De : FreeDemoCss <guillaume.cornez@gmail.com>
Envoyé : vendredi 9 mars 2018 17:40
À : francois.bruynbroeck@hotmail.com
Objet : Problème include, et footer



Provenance : Courrier pour Windows 10