From 94dafd735ce7fa0c5a2154b12deb98f23fdb3363 Mon Sep 17 00:00:00 2001
From: Thomas Rubini <74205383+ThomasRubini@users.noreply.github.com>
Date: Thu, 12 Jan 2023 14:31:06 +0100
Subject: [PATCH] switch code to english

---
 Controleurs/ControleurHelloworld.php          | 19 -----
 Controllers/HelloworldController.php          | 19 +++++
 Kernel/AutoLoad.php                           | 54 +++++++++++++
 Kernel/Constants.php                          | 42 ++++++++++
 Kernel/Controller.php                         | 71 +++++++++++++++++
 Kernel/Exceptions/ControllerException.php     |  3 +
 {Noyau => Kernel}/Model.php                   |  0
 Kernel/View.php                               | 24 ++++++
 {Modele => Models}/Helloworld.php             |  0
 Noyau/ChargementAuto.php                      | 55 -------------
 Noyau/Constantes.php                          | 46 -----------
 Noyau/Controleur.php                          | 77 -------------------
 Noyau/Exceptions/ControleurException.php      |  3 -
 Noyau/Vue.php                                 | 27 -------
 .../pied.php => Views/common/footer.php       |  0
 .../entete.php => Views/common/header.php     |  4 +-
 {Vues => Views}/helloworld/testform.php       |  0
 Views/helloworld/view.php                     |  3 +
 Vues/gabarit.php => Views/html.php            | 22 +++---
 Vues/helloworld/voir.php                      |  3 -
 index.php                                     | 38 +++------
 21 files changed, 239 insertions(+), 271 deletions(-)
 delete mode 100644 Controleurs/ControleurHelloworld.php
 create mode 100644 Controllers/HelloworldController.php
 create mode 100644 Kernel/AutoLoad.php
 create mode 100644 Kernel/Constants.php
 create mode 100644 Kernel/Controller.php
 create mode 100644 Kernel/Exceptions/ControllerException.php
 rename {Noyau => Kernel}/Model.php (100%)
 create mode 100644 Kernel/View.php
 rename {Modele => Models}/Helloworld.php (100%)
 delete mode 100644 Noyau/ChargementAuto.php
 delete mode 100644 Noyau/Constantes.php
 delete mode 100644 Noyau/Controleur.php
 delete mode 100644 Noyau/Exceptions/ControleurException.php
 delete mode 100644 Noyau/Vue.php
 rename Vues/standard/pied.php => Views/common/footer.php (100%)
 rename Vues/standard/entete.php => Views/common/header.php (96%)
 rename {Vues => Views}/helloworld/testform.php (100%)
 create mode 100644 Views/helloworld/view.php
 rename Vues/gabarit.php => Views/html.php (57%)
 delete mode 100644 Vues/helloworld/voir.php

diff --git a/Controleurs/ControleurHelloworld.php b/Controleurs/ControleurHelloworld.php
deleted file mode 100644
index 68019a1..0000000
--- a/Controleurs/ControleurHelloworld.php
+++ /dev/null
@@ -1,19 +0,0 @@
-<?php
-
-final class ControleurHelloworld
-{
-    public function defautAction()
-    {
-        $O_helloworld =  new Helloworld();
-        Vue::montrer('helloworld/voir', array('helloworld' =>  $O_helloworld->donneMessage()));
-
-    }
-
-    public function testformAction(Array $A_parametres = null, Array $A_postParams = null)
-    {
-
-        Vue::montrer('helloworld/testform', array('formData' =>  $A_postParams));
-
-    }
-
-}
\ No newline at end of file
diff --git a/Controllers/HelloworldController.php b/Controllers/HelloworldController.php
new file mode 100644
index 0000000..448a2c8
--- /dev/null
+++ b/Controllers/HelloworldController.php
@@ -0,0 +1,19 @@
+<?php
+
+final class HelloworldController
+{
+    public function defaultAction()
+    {
+        $O_helloworld =  new Helloworld();
+        View::show('helloworld/view', array('helloworld' =>  $O_helloworld->donneMessage()));
+
+    }
+
+    public function testformAction(Array $A_urlParams = null, Array $A_postParams = null)
+    {
+
+        View::show('helloworld/testform', array('formData' =>  $A_postParams));
+
+    }
+
+}
\ No newline at end of file
diff --git a/Kernel/AutoLoad.php b/Kernel/AutoLoad.php
new file mode 100644
index 0000000..230177e
--- /dev/null
+++ b/Kernel/AutoLoad.php
@@ -0,0 +1,54 @@
+<?php
+
+require 'Kernel/Constants.php';
+
+final class AutoLoad
+{
+    public static function loadKernelClass ($S_className)
+    {
+        $S_file = Constants::kernelDir() . "$S_className.php";
+        return static::_load($S_file);
+    }
+
+    public static function loadExceptionClass ($S_className)
+    {
+        $S_file = Constants::exceptionsDir() . "$S_className.php";
+
+        return static::_load($S_file);
+    }
+
+    public static function loadModelClass ($S_className)
+    {
+        $S_file = Constants::modelsDir() . "$S_className.php";
+
+        return static::_load($S_file);
+    }
+
+
+    public static function loadViewClass ($S_className)
+    {
+        $S_path = Constants::viewsDir() . "$S_className.php";
+
+        return static::_load($S_path);
+    }
+
+    public static function loadControllerClass ($S_className)
+    {
+        $S_path = Constants::controllersDir() . "$S_className.php";
+
+        return static::_load($S_path);
+    }
+    private static function _load($S_path)
+    {
+        if (is_readable($S_path))
+        {
+            require $S_path;
+        }
+    }
+}
+
+spl_autoload_register('AutoLoad::loadKernelClass');
+spl_autoload_register('AutoLoad::loadExceptionClass');
+spl_autoload_register('AutoLoad::loadModelClass');
+spl_autoload_register('AutoLoad::loadViewClass');
+spl_autoload_register('AutoLoad::loadControllerClass');
diff --git a/Kernel/Constants.php b/Kernel/Constants.php
new file mode 100644
index 0000000..ac075eb
--- /dev/null
+++ b/Kernel/Constants.php
@@ -0,0 +1,42 @@
+<?php
+
+
+final class Constants
+{
+
+    const KERNEL_DIR       = '/Kernel/';
+
+    const EXCEPTIONS_DIR  = '/Kernel/Exceptions/';
+
+    const VIEWS_DIR        = '/Views/';
+
+    const MODELS_DIR      = '/Models/';
+
+    const CONTROLLERS_DIR = '/Controllers/';
+
+
+    public static function rootDir() {
+        return realpath(__DIR__ . '/../');
+    }
+
+    public static function kernelDir() {
+        return self::rootDir() . self::KERNEL_DIR;
+    }
+
+    public static function exceptionsDir() {
+        return self::rootDir() . self::EXCEPTIONS_DIR;
+    }
+
+    public static function viewsDir() {
+        return self::rootDir() . self::VIEWS_DIR;
+    }
+
+    public static function modelsDir() {
+        return self::rootDir() . self::MODELS_DIR;
+    }
+
+    public static function controllersDir() {
+        return self::rootDir() . self::CONTROLLERS_DIR;
+    }
+
+}
diff --git a/Kernel/Controller.php b/Kernel/Controller.php
new file mode 100644
index 0000000..e153451
--- /dev/null
+++ b/Kernel/Controller.php
@@ -0,0 +1,71 @@
+<?php
+
+final class controller
+{
+    private $_A_urlParts;
+
+    private $_A_urlParams;
+
+    private $_A_postParams;
+
+    public function __construct ($S_url, $A_postParams)
+    {
+        // Remove the trailing slash
+        if ('/' == substr($S_url, -1, 1)) {
+            $S_url = substr($S_url, 0, strlen($S_url) - 1);
+        }
+
+        // Remove the leading slash
+        if ('/' == substr($S_url, 0, 1)) {
+            $S_url = substr($S_url, 1, strlen($S_url));
+        }
+
+        // split the url
+        $_A_urlParts = explode('/', $S_url);
+
+        if (empty($_A_urlParts[0])) {
+            $_A_urlParts[0] = 'DefaultController';
+        } else {
+            $_A_urlParts[0] = ucfirst($_A_urlParts[0]) . "Controller";
+        }
+
+        if (empty($_A_urlParts[1])) {
+            $_A_urlParts[1] = 'defaultAction';
+        } else {
+            $_A_urlParts[1] = $_A_urlParts[1] . 'Action';
+        }
+
+
+        $this->_A_urlParts['controller'] = array_shift($_A_urlParts);
+        $this->_A_urlParts['action']     = array_shift($_A_urlParts);
+
+        $this->_A_urlParametres = $_A_urlParts;
+
+        $this->_A_postParams = $A_postParams;
+
+
+    }
+
+    // Execute the controller and action deduced
+
+    public function execute()
+    {
+        if (!class_exists($this->_A_urlParts['controller'])) {
+            throw new ControllerException("Controller " . $this->_A_urlParts['controller'] . " is not valid.");
+        }
+
+        if (!method_exists($this->_A_urlParts['controller'], $this->_A_urlParts['action'])) {
+            throw new ControllerException("Action " . $this->_A_urlParts['action'] . " of controller " .
+                $this->_A_urlParts['controller'] . " is not valid.");
+        }
+
+        
+        $B_called = call_user_func_array(array(new $this->_A_urlParts['controller'],
+            $this->_A_urlParts['action']), array($this->_A_urlParams, $this->_A_postParams ));
+
+        if (false === $B_called) {
+            throw new ControllerException("Action " . $this->_A_urlParts['action'] .
+                " of controller " . $this->_A_urlParts['controller'] . " failed.");
+        }
+    }
+}
\ No newline at end of file
diff --git a/Kernel/Exceptions/ControllerException.php b/Kernel/Exceptions/ControllerException.php
new file mode 100644
index 0000000..b529795
--- /dev/null
+++ b/Kernel/Exceptions/ControllerException.php
@@ -0,0 +1,3 @@
+<?php
+
+class ControllerException extends Exception {}
diff --git a/Noyau/Model.php b/Kernel/Model.php
similarity index 100%
rename from Noyau/Model.php
rename to Kernel/Model.php
diff --git a/Kernel/View.php b/Kernel/View.php
new file mode 100644
index 0000000..4aec9da
--- /dev/null
+++ b/Kernel/View.php
@@ -0,0 +1,24 @@
+<?php
+
+final class View
+{
+    public static function openBuffer()
+    {
+        ob_start();
+    }
+
+    public static function closeBuffer()
+    {
+        return ob_get_clean();
+    }
+
+    public static function show ($S_path, $A_params = array())
+    {
+        $S_file = Constants::viewsDir() . $S_path . '.php';
+        $A_view = $A_params;
+
+        ob_start();
+        include $S_file;
+        ob_end_flush();
+    }
+}
\ No newline at end of file
diff --git a/Modele/Helloworld.php b/Models/Helloworld.php
similarity index 100%
rename from Modele/Helloworld.php
rename to Models/Helloworld.php
diff --git a/Noyau/ChargementAuto.php b/Noyau/ChargementAuto.php
deleted file mode 100644
index eaa78f0..0000000
--- a/Noyau/ChargementAuto.php
+++ /dev/null
@@ -1,55 +0,0 @@
-<?php
-
-require 'Noyau/Constantes.php';
-
-final class ChargementAuto
-{
-    public static function chargerClassesNoyau ($S_nomDeClasse)
-    {
-        $S_fichier = Constantes::repertoireNoyau() . "$S_nomDeClasse.php";
-        return static::_charger($S_fichier);
-    }
-
-    public static function chargerClassesException ($S_nomDeClasse)
-    {
-        $S_fichier = Constantes::repertoireExceptions() . "$S_nomDeClasse.php";
-
-        return static::_charger($S_fichier);
-    }
-
-    public static function chargerClassesModele ($S_nomDeClasse)
-    {
-        $S_fichier = Constantes::repertoireModele() . "$S_nomDeClasse.php";
-
-        return static::_charger($S_fichier);
-    }
-
-
-    public static function chargerClassesVue ($S_nomDeClasse)
-    {
-        $S_fichier = Constantes::repertoireVues() . "$S_nomDeClasse.php";
-
-        return static::_charger($S_fichier);
-    }
-
-    public static function chargerClassesControleur ($S_nomDeClasse)
-    {
-        $S_fichier = Constantes::repertoireControleurs() . "$S_nomDeClasse.php";
-
-        return static::_charger($S_fichier);
-    }
-    private static function _charger ($S_fichierACharger)
-    {
-        if (is_readable($S_fichierACharger))
-        {
-            require $S_fichierACharger;
-        }
-    }
-}
-
-// J'empile tout ce beau monde comme j'ai toujours appris à le faire...
-spl_autoload_register('ChargementAuto::chargerClassesNoyau');
-spl_autoload_register('ChargementAuto::chargerClassesException');
-spl_autoload_register('ChargementAuto::chargerClassesModele');
-spl_autoload_register('ChargementAuto::chargerClassesVue');
-spl_autoload_register('ChargementAuto::chargerClassesControleur');
diff --git a/Noyau/Constantes.php b/Noyau/Constantes.php
deleted file mode 100644
index 53f5ff4..0000000
--- a/Noyau/Constantes.php
+++ /dev/null
@@ -1,46 +0,0 @@
-<?php
-
-// Rappel : nous sommes dans le répertoire Core, voilà pourquoi dans realpath je "remonte d'un cran" pour faire référence
-// à la VRAIE racine de mon application
-
-final class Constantes
-{
-    // Les constantes relatives aux chemins
-
-    const REPERTOIRE_VUES        = '/Vues/';
-
-    const REPERTOIRE_MODELE      = '/Modele/';
-
-    const REPERTOIRE_NOYAU       = '/Noyau/';
-
-    const REPERTOIRE_EXCEPTIONS  = '/Noyau/Exceptions/';
-
-    const REPERTOIRE_CONTROLEURS = '/Controleurs/';
-
-
-    public static function repertoireRacine() {
-        return realpath(__DIR__ . '/../');
-    }
-
-    public static function repertoireNoyau() {
-        return self::repertoireRacine() . self::REPERTOIRE_NOYAU;
-    }
-
-    public static function repertoireExceptions() {
-        return self::repertoireRacine() . self::REPERTOIRE_EXCEPTIONS;
-    }
-
-    public static function repertoireVues() {
-        return self::repertoireRacine() . self::REPERTOIRE_VUES;
-    }
-
-    public static function repertoireModele() {
-        return self::repertoireRacine() . self::REPERTOIRE_MODELE;
-    }
-
-    public static function repertoireControleurs() {
-        return self::repertoireRacine() . self::REPERTOIRE_CONTROLEURS;
-    }
-
-
-}
\ No newline at end of file
diff --git a/Noyau/Controleur.php b/Noyau/Controleur.php
deleted file mode 100644
index b35713d..0000000
--- a/Noyau/Controleur.php
+++ /dev/null
@@ -1,77 +0,0 @@
-<?php
-
-final class Controleur
-{
-    private $_A_urlDecortique;
-
-    private $_A_urlParametres;
-
-    private $_A_postParams;
-
-    public function __construct ($S_url, $A_postParams)
-    {
-        // On élimine l'éventuel slash en fin d'URL sinon notre explode renverra une dernière entrée vide
-        if ('/' == substr($S_url, -1, 1)) {
-            $S_url = substr($S_url, 0, strlen($S_url) - 1);
-        }
-
-        // On élimine l'éventuel slash en début d'URL sinon notre explode renverra une première entrée vide
-        if ('/' == substr($S_url, 0, 1)) {
-            $S_url = substr($S_url, 1, strlen($S_url));
-        }
-
-        // On éclate l'URL, elle va prendre place dans un tableau
-        $A_urlDecortique = explode('/', $S_url);
-
-        if (empty($A_urlDecortique[0])) {
-            // Nous avons pris le parti de préfixer tous les controleurs par "Controleur"
-            $A_urlDecortique[0] = 'ControleurDefaut';
-        } else {
-            $A_urlDecortique[0] = 'Controleur' . ucfirst($A_urlDecortique[0]);
-        }
-
-        if (empty($A_urlDecortique[1])) {
-            // L'action est vide ! On la valorise par défaut
-            $A_urlDecortique[1] = 'defautAction';
-        } else {
-            // On part du principe que toutes nos actions sont suffixées par 'Action'...à nous de le rajouter
-            $A_urlDecortique[1] = $A_urlDecortique[1] . 'Action';
-        }
-
-
-        // on dépile 2 fois de suite depuis le début, c'est à dire qu'on enlève de notre tableau le contrôleur et l'action
-        // il ne reste donc que les éventuels parametres (si nous en avons)...
-        $this->_A_urlDecortique['controleur'] = array_shift($A_urlDecortique); // on recupere le contrôleur
-        $this->_A_urlDecortique['action']     = array_shift($A_urlDecortique); // puis l'action
-
-        // ...on stocke ces éventuels parametres dans la variable d'instance qui leur est réservée
-        $this->_A_urlParametres = $A_urlDecortique;
-
-        // On  s'occupe du tableau $A_postParams
-        $this->_A_postParams = $A_postParams;
-
-
-    }
-
-    // On exécute notre triplet
-
-    public function executer()
-    {
-        if (!class_exists($this->_A_urlDecortique['controleur'])) {
-            throw new ControleurException($this->_A_urlDecortique['controleur'] . " n'est pas un controleur valide.");
-        }
-
-        if (!method_exists($this->_A_urlDecortique['controleur'], $this->_A_urlDecortique['action'])) {
-            throw new ControleurException($this->_A_urlDecortique['action'] . " du contrôleur " .
-                $this->_A_urlDecortique['controleur'] . " n'est pas une action valide.");
-        }
-
-        $B_called = call_user_func_array(array(new $this->_A_urlDecortique['controleur'],
-            $this->_A_urlDecortique['action']), array($this->_A_urlParametres, $this->_A_postParams ));
-
-        if (false === $B_called) {
-            throw new ControleurException("L'action " . $this->_A_urlDecortique['action'] .
-                " du contrôleur " . $this->_A_urlDecortique['controleur'] . " a rencontré une erreur.");
-        }
-    }
-}
\ No newline at end of file
diff --git a/Noyau/Exceptions/ControleurException.php b/Noyau/Exceptions/ControleurException.php
deleted file mode 100644
index 045e712..0000000
--- a/Noyau/Exceptions/ControleurException.php
+++ /dev/null
@@ -1,3 +0,0 @@
-<?php
-
-class ControleurException extends Exception {}
\ No newline at end of file
diff --git a/Noyau/Vue.php b/Noyau/Vue.php
deleted file mode 100644
index 4a6df54..0000000
--- a/Noyau/Vue.php
+++ /dev/null
@@ -1,27 +0,0 @@
-<?php
-
-final class Vue
-{
-    public static function ouvrirTampon()
-    {
-        // On démarre le tampon de sortie, on va l'appeler "tampon principal"
-        ob_start();
-    }
-
-    public static function recupererContenuTampon()
-    {
-        // On retourne le contenu du tampon principal
-        return ob_get_clean();
-    }
-
-    public static function montrer ($S_localisation, $A_parametres = array())
-    {
-        $S_fichier = Constantes::repertoireVues() . $S_localisation . '.php';
-
-            $A_vue = $A_parametres;
-            // Démarrage d'un sous-tampon
-            ob_start();
-            include $S_fichier; // c'est dans ce fichier que sera utilisé A_vue, la vue est inclue dans le sous-tampon
-            ob_end_flush();
-    }
-}
\ No newline at end of file
diff --git a/Vues/standard/pied.php b/Views/common/footer.php
similarity index 100%
rename from Vues/standard/pied.php
rename to Views/common/footer.php
diff --git a/Vues/standard/entete.php b/Views/common/header.php
similarity index 96%
rename from Vues/standard/entete.php
rename to Views/common/header.php
index 3dd4940..9432bf4 100644
--- a/Vues/standard/entete.php
+++ b/Views/common/header.php
@@ -1,2 +1,2 @@
-<?php
- echo '<header> <h1>Titre</h1></header>';
+<?php
+ echo '<header> <h1>Titre</h1></header>';
diff --git a/Vues/helloworld/testform.php b/Views/helloworld/testform.php
similarity index 100%
rename from Vues/helloworld/testform.php
rename to Views/helloworld/testform.php
diff --git a/Views/helloworld/view.php b/Views/helloworld/view.php
new file mode 100644
index 0000000..8749713
--- /dev/null
+++ b/Views/helloworld/view.php
@@ -0,0 +1,3 @@
+<?php
+
+echo "<p>" . $A_view['helloworld']  . "</p>";
diff --git a/Vues/gabarit.php b/Views/html.php
similarity index 57%
rename from Vues/gabarit.php
rename to Views/html.php
index fb18d10..6f8891d 100644
--- a/Vues/gabarit.php
+++ b/Views/html.php
@@ -1,12 +1,12 @@
-<!doctype html>
-<html lang="fr">
-    <head>
-        <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
-        <title>My sweet MVC</title>
-    </head>
-    <body>
-        <?php Vue::montrer('standard/entete'); ?>
-        <?php echo $A_vue['body'] ?>
-        <?php Vue::montrer('standard/pied'); ?>
-    </body>
+<!doctype html>
+<html lang="fr">
+    <head>
+        <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
+        <title>My sweet MVC</title>
+    </head>
+    <body>
+        <?php View::show('common/header'); ?>
+        <?php echo $A_view['body'] ?>
+        <?php View::show('common/footer'); ?>
+    </body>
 </html>
\ No newline at end of file
diff --git a/Vues/helloworld/voir.php b/Vues/helloworld/voir.php
deleted file mode 100644
index b8fdb6b..0000000
--- a/Vues/helloworld/voir.php
+++ /dev/null
@@ -1,3 +0,0 @@
-<?php
-
-echo "<p>" . $A_vue['helloworld']  . "</p>";
diff --git a/index.php b/index.php
index 57665cc..f0c3424 100644
--- a/index.php
+++ b/index.php
@@ -1,46 +1,28 @@
 <?php
-// Ce fichier est le point d'entrée de votre application
+    // Application entry point
 
     require 'vendor/autoload.php';
-    require 'Noyau/ChargementAuto.php';
-    /*
-     url pour notre premier test MVC Hello World,
-     nous n'avons pas d'action précisée on visera celle par défaut
-     /index.php?ctrl=helloworld
-     /helloworld
-     /controleur/nom_action/whatever/whatever2/
+    require 'Kernel/AutoLoad.php';
 
-    */
-/*
-    $S_controleur = isset($_GET['ctrl']) ? $_GET['ctrl'] : null;
-    $S_action = isset($_GET['action']) ? $_GET['action'] : null;
-
-    Vue::ouvrirTampon(); //  /Noyau/Vue.php : on ouvre le tampon d'affichage, les contrôleurs qui appellent des vues les mettront dedans
-    $O_controleur = new Controleur($S_controleur, $S_action);
-*/
-    // load .env file in $_ENV
-
-    $dotenv = Dotenv\Dotenv::createImmutable(Constantes::repertoireRacine());
+    $dotenv = Dotenv\Dotenv::createImmutable(Constants::rootDir());
     $dotenv->load();
 
-    $S_urlADecortiquer = isset($_GET['url']) ? $_GET['url'] : null;
+    $S_url = isset($_GET['url']) ? $_GET['url'] : null;
     $A_postParams = isset($_POST) ? $_POST : null;
 
-    Vue::ouvrirTampon(); // on ouvre le tampon d'affichage, les contrôleurs qui appellent des vues les mettront dedans
+    View::openBuffer();
 
     try
     {
-        $O_controleur = new Controleur($S_urlADecortiquer, $A_postParams);
-        $O_controleur->executer();
+        $O_controller = new Controller($S_url, $A_postParams);
+        $O_controller->execute();
     }
     catch (ControleurException $O_exception)
     {
-        echo ('Une erreur s\'est produite : ' . $O_exception->getMessage());
+        echo ('An error occured: ' . $O_exception->getMessage());
     }
 
 
-    // Les différentes sous-vues ont été "crachées" dans le tampon d'affichage, on les récupère
-    $contenuPourAffichage = Vue::recupererContenuTampon();
+    $content = View::closeBuffer();
 
-    // On affiche le contenu dans la partie body du gabarit général
-    Vue::montrer('gabarit', array('body' => $contenuPourAffichage));
\ No newline at end of file
+    View::show('html', array('body' => $content));
\ No newline at end of file