#43 Refactoring Codacy Issue

app/Controller/DataBaseTestController.php

@@ -25,15 +25,15 @@ public function execute(Request $request): string|null
         // Connexion à la base de données
         $getConnectDb = DatabaseService::getConnect();
         // Requête SQL
-        $etudiantStmt = $getConnectDb->prepare("SELECT * FROM `etudiant`");
-        // Exécution de la requête
-        $etudiantStmt->execute();
+        $etudiantStmt = $getConnectDb->query("SELECT * FROM `etudiant`");
         // Récupération des données
         $etudiants = $etudiantStmt->fetchAll();
 
-        return TwigCore::getEnvironment()->render('home/db.html.twig',
+        return TwigCore::getEnvironment()->render
+        (
+            'home/db.html.twig',
             [
-                'titre'   => 'Exemple avec connextion à la base de données',
+                'titre'     => 'Exemple avec connextion à la base de données',
                 'etudiants' => $etudiants
             ]
         );

app/Template/error/http-Default.html.twig

@@ -0,0 +1,8 @@
+{% extends "base.html.twig" %}
+
+{% block title %}Error{% endblock %}
+
+{% block content %}
+    <h1>Error Default Route</h1>
+{% endblock %}
+

src/EduFramework/Core/Controller/Error/HttpErrorDefaultController.php

@@ -0,0 +1,41 @@
+<?php
+/*
+ * Ce fichier fait partie du Studoo
+ *
+ * @author Benoit Foujols
+ *
+ * Pour les informations complètes sur les droits d'auteur et la licence,
+ * veuillez consulter le fichier LICENSE qui a été distribué avec ce code source.
+ */
+
+namespace Studoo\EduFramework\Core\Controller\Error;
+
+use Studoo\EduFramework\Core\Controller\ControllerInterface;
+use Studoo\EduFramework\Core\Controller\Request;
+use Studoo\EduFramework\Core\View\TwigCore;
+use Twig\Error\LoaderError;
+use Twig\Error\RuntimeError;
+use Twig\Error\SyntaxError;
+
+/**
+ * Class HttpErrorDefaultController
+ * Classe Controller pour les erreurs HTTP
+ */
+class HttpErrorDefaultController implements ControllerInterface
+{
+    /**
+     * Si y a pas de route valide alors j'affiche la page 404
+     * @param Request $request Objet de la requête
+     * @return string
+     * @throws SyntaxError
+     * @throws RuntimeError
+     * @throws LoaderError
+     */
+    public function execute(Request $request): string
+    {
+        return TwigCore::getEnvironment()->render(
+            'error/http-Default.html.twig',
+            []
+        );
+    }
+}

src/EduFramework/Core/Controller/FastRouteCore.php

@@ -13,6 +13,7 @@
 use FastRoute\Dispatcher;
 use Studoo\EduFramework\Core\Controller\Error\HttpError404Controller;
 use Studoo\EduFramework\Core\Controller\Error\HttpError405Controller;
+use Studoo\EduFramework\Core\Controller\Error\HttpErrorDefaultController;
 use Symfony\Component\Yaml\Yaml;
 use Twig\Error\LoaderError;
 use Twig\Error\RuntimeError;
@@ -25,6 +26,11 @@
 class FastRouteCore
 {
     use BuildControllerTrait;
+
+    /**
+     * Objet pour la gestion des routes
+     * @var \FastRoute\RouteCollector
+     */
     private \FastRoute\RouteCollector $routeCollector;
 
     /**
@@ -138,8 +144,7 @@ public function getRoute(): string|null
                 $returnView = $this->buildController($request->getHander())->execute($request);
                 break;
             default:
-                // TODO Refactoring des erreurs
-                $returnView = (new HttpError404Controller())->execute($request);
+                $returnView = (new HttpErrorDefaultController())->execute($request);
                 break;
         }
         return $returnView;

src/EduFramework/Core/Controller/Request.php

@@ -61,7 +61,7 @@ public function __construct(string $route, string $httpMethod)
      */
     public function getHearder(): bool|array
     {
-        if (!function_exists('getallheaders')) {
+        if (function_exists('getallheaders') === false) {
             $headers = [];
             foreach ($_SERVER as $name => $value) {
                 if (substr($name, 0, 5) === 'HTTP_') {

src/EduFramework/Core/Exception/NotDataResponseException.php

@@ -13,11 +13,13 @@
 class NotDataResponseException extends \Exception
 {
     /**
+     *
      * @var string
      */
     protected $message = "Pas de donnée dans la réponse";
 
     /**
+     *
      * @var integer
      */
     protected $code = 403;

src/EduFramework/Core/Service/DatabaseMariadb.php

@@ -26,7 +26,9 @@ public function getManager(): PDO
             ';dbname=' . ConfigCore::getEnv("DB_NAME"),
             ConfigCore::getEnv("DB_USER"),
             ConfigCore::getEnv("DB_PASSWORD"),
-            [PDO::ATTR_PERSISTENT => false]
+            [
+                PDO::ATTR_PERSISTENT => false
+            ]
         );
     }
 }

src/EduFramework/Core/Service/DatabaseMysql.php

@@ -26,7 +26,10 @@ public function getManager(): PDO
             ';dbname=' . ConfigCore::getEnv("DB_NAME"),
             ConfigCore::getEnv("DB_USER"),
             ConfigCore::getEnv("DB_PASSWORD"),
-            [PDO::ATTR_PERSISTENT => false, PDO::MYSQL_ATTR_INIT_COMMAND => 'SET NAMES utf8']
+            [
+                PDO::ATTR_PERSISTENT => false,
+                PDO::MYSQL_ATTR_INIT_COMMAND => 'SET NAMES utf8'
+            ]
         );
     }
 }