Forum : Architecture MVC

Bonjour,

Voila je rencontre un petit problème avec mon code.

je voudrais afficher mes données avec ma vue.php, elle n'arrive pas a trouvé une variable je vous montre mon architecture, je débute en architecture MVC si vous avez des astuces n'hésiter pas, merci de votre aide

index.php qui joue le rôle de routeur

<?php
require('controlWord.php');
require('vueWord.php');

if (isset($_GET['action'])) {
    if ($_GET['action'] == 'Controlword') {
        new Controlword();
    }
    elseif ($_GET['action'] == 'Controlword') {
        if (isset($_GET['id']) && $_GET['id'] > 0) {
            new Controlword();
        }
        else {
            echo 'pas de mot';
        }
    }
}
else {
    new Controlword();
}

Mon modèle qui fournit l'accès et les données

<?php
class Modelword
{
    // Renvoie la liste des billets du blog
  public function getWord() {
    $bdd = $this->getBdd();
    $words = $bdd->query('SELECT id, mot FROM hello');
    return $words;
  }

  private function getBdd() {
    $bdd = new PDO('mysql:host=localhost;dbname=test;charset=utf8', 'root', '', array(PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION));
    return $bdd;
  }
}

Ma vue qui affiche le tout je sais que c'est elle l'erreur mais est ce que je dois en faire une classe et créer un gabarit ou je la laisse comme ça

<!DOCTYPE html>
<html>
    <head>
        <meta charset="utf-8" />
        <title>Mes mots</title>
        <link href="style.css" rel="stylesheet" /> 
    </head>

    <body>

        <?php
        while ($data = $word->fetch())
        {
        ?>    

                <p><?= htmlspecialchars($data['mot']) ?></p>
        <?php
        }
        $word->closeCursor();
        ?>

    </body>
</html>

Mon controller qui controle et fournis à la vue et là je sais pas trop comment m'y prendre

<?php

require('modelWord.php');
require('vueWord.php');

class Controlword
{
    private $words;

    public function __construct() {
      $this->words = new Modelword();
    }

    public function getWord()
  {
    $word = $this->words->getBillets();
  }
}

Je cherche des piste pour afficher mes données

Ce que j'obtiens une variable indéfinis sur la vue

26 réponses

Lartak

Réponse acceptée

Il te faut stocker dans une variable le retour de la méthode words dans ton fichier index, car quand tu retournes la variable, c'est ce qu'elle contient que tu retourne, donc par exemple :

$word = $controlWord->word();

quenti77

Réponse acceptée

Je te conseil vivement de revoir un peu le code et surtout l'ordre dans lequel il s'exécute. La tu affiche la vue vueWord.php alors que déjà c'est peut-être pas elle que tu voudra et aussi car les variables ne sont transmis au fichier require que si elles se trouvent avant le require.

Il faudrait donc inverser mais le mieux est de revoir un peu l'architecture et voici ce que j'ai fais de mon côté :

Déjà mon architecture c'est :

Un dossier public avec le fichier index.php
Un dossier src avec dedans 3 sous-dossiers (mais il peut y en avoir plus) qui sont Controllers, Models et Views

Dans src/Models/Connection.php on a le code :

<?php

namespace Tuto\Models;

use PDO;
use PDOStatement;

/**
 * @package Tuto\Models
 * Class Connection
 */
class Connection extends PDO
{
    /**
     * Connection constructor
     * 
     * @param string $dsn
     * @param string $user
     * @param string $pass
     * @param array $options
     */
    public function __construct(string $dsn, string $user, string $pass, array $options = [])
    {
        parent::__construct($dsn, $user, $pass, $options);

        parent::setAttribute(parent::ATTR_DEFAULT_FETCH_MODE, parent::FETCH_ASSOC);
        parent::setAttribute(parent::ATTR_ERRMODE, parent::ERRMODE_EXCEPTION);
        parent::setAttribute(parent::ATTR_EMULATE_PREPARES, false);
    }

    /**
     * Cette méthode peut-être plus poussé (demande moi si tu veux voir)
     * 
     * @param string $statement
     * @param array $params
     * @return PDOStatement
     */
    public function request(string $statement, array $params = []): PDOStatement
    {
        $request = $this->prepare($statement);
        $request->execute($params);

        return $request;
    }
}

Dans src/Models/WordModel.php on a le code :

<?php

namespace Tuto\Models;

use PDOStatement;

/**
 * @package Tuto\Models
 * Class WordModel
 */
class WordModel
{
    /** @var Connection $connection */
    private $connection;

    /**
     * WordModel Constructor
     * 
     * @param Connection $connection
     */
    public function __construct(Connection $connection)
    {
        $this->connection = $connection;
    }

    /**
     * @return PDOStatement
     */
    public function all(): PDOStatement
    {
        // J'aurai surement boucler pour mettre dans des entitées
        // Et cette classe serait un repository et non un model
        // Mais pour le moment on fait au plus simple.
        return $this->connection->request('SELECT id, mot FROM hello');
    }
}

Dans src/Controllers/WordController.php on a le code :

<?php

namespace Tuto\Controllers;

use Tuto\Models\WordModel;

/**
 * @package Tuto\Controllers
 * Class WordController
 */
class WordController
{
    /** @var WordModel $wordModel */
    private $wordModel;

    /**
     * WordController Constructor
     * 
     * @param WordModel $wordModel
     */
    public function __construct(WordModel $wordModel)
    {
        $this->wordModel = $wordModel;
    }

    /**
     * @return array
     */
    public function index(): array
    {
        $words = $this->wordModel->all();

        return [
            'views' => __DIR__ . '/../Views/WordView.php',
            'words' => $words
        ];
    }
}

La vue qui se trouve dans le fichier src/Views/WordView.php à le code :

<html lang="fr">
<head>
    <meta charset="UTF-8">
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
    <meta http-equiv="X-UA-Compatible" content="ie=edge">
    <title>Première page</title>
</head>
<body>

    <!-- faire un système de layout -->
    <h1>Liste des mots :</h1>

    <?php foreach ($words as $word): ?>
    <div>
        <?= $word['id'] ?> : <strong><?= $word['mot'] ?></strong>
    </div>
    <?php endforeach; ?>

</body>
</html>

Enfin le fichier public/index.php contient :

<?php

// Autoloader
spl_autoload_register(function (string $className) {
    if (strpos($className, 'Tuto\\') === 0) {
        $className = str_replace('Tuto\\', '', $className);
        $className = str_replace('\\', '/', $className);

        require __DIR__ . "/../src/{$className}.php";
    }
});

// Je vais faire les checks de paramètre mais
// personnelement j appel pas vraiment ça un routing ^^
// et ça devra être fait ailleurs

use Tuto\Models\Connection;
use Tuto\Models\WordModel;
use Tuto\Controllers\WordController;

$connection = new Connection('mysql:host=localhost;dbname=test;charset=utf8', 'root', 'root');

$controller = $_GET['controller'] ?? 'word';
$action = $_GET['action'] ?? 'index';

$controllers = [
    'word' => [
        'instance' => function () use($connection) {
            $wordModel = new WordModel($connection);
            return new WordController($wordModel);
        },
        'actions' => [
            'index'
        ]
    ]
];

$currentController = $controllers[$controller] ?? $controllers['word'];
$currentInstance = $currentController['instance']();

// Pour éviter tous soucis avec extract sur de potentiel variable ecrasé
function render(array $data = [])
{
    $views = $data['views'] ?? '';
    if (file_exists($views)) {
        unset($data['views']);
        extract($data);

        require $views;
    }
}

if (in_array($action, $currentController['actions']) && method_exists($currentInstance, $action)) {
    $data = $currentInstance->$action();
    render($data);
} else {
    // ERROR;
}

J'espère que ça peut te permettre de mieux comprendre (même si la j'ai vraiment simplifier par rapport à ce que l'on peut faire pour avoir un bon code.

quenti77

Réponse acceptée

Alors attention, ce que tu appel manager, j'ai appelé repository mais c'est grosso modo la même chose. La seul vrai entité que tu as pour le moment c'est "Word" (sans le s vu que c'est pour une seul ligne)

En gros dans les repositories (ou les managers comme tu veux ^^) tu va pouvoir transformer le tableau de PDO en Entité.

Si je reprends le code de mon WordModel.php que je vais renommé en WordsRepository (ou WordsManager si tu préfère). J'en profite pour renommer le dossier "Models" en "Repositories" et les namespaces qui vont avec.

Ensuite je créé un dossier "Entities" dans src et un fichier "Word.php" (qui est donc notre entité. On aurait cette classe là :

<?php

namespace Tuto\Entities;

/**
 * @package Tuto\Entities;
 * Class Word
 */
class Word
{
    // Pour simplifier le code et les explications je
    // mets l'hydratation directement dans cette classe
    // mais on pourrait très bien y mettre un trait par
    // exemple HydratableTrait qui aurait le code de la
    // méthode hydrate que l'on va avoir ici

    /** @var int $id */
    private $id;

    /** @var string $mot */
    private $mot;

    /**
     * Word Constructor
     * @param array $cols
     */
    public function __construct(array $cols = [])
    {
        $this->hydrate($cols);
    }

    /**
     * @param array $cols
     */
    public function hydrate(array $cols = [])
    {
        foreach ($cols as $name => $value) {
            // Transforme du snake_case en CamelCase et prefix avec set
            // exemple : 'create_at' devient => 'setCreatedAt'
            $methodName = 'set' . str_replace('_', '', ucwords($name, '_'));

            if (method_exists($this, $methodName)) {
                // Petite particularité ici. On peut appeler dynamiquement
                // une méthode, propriété et même une variable avec $$methodName (par exemple)
                // Donc la si je reprends l'exemple du created_at on va appeler la méthode
                // setCreatedAt sur l'objet courant ($this)
                $this->$methodName($value);
            }
        }
    }

    // Je simplifie les get/set même si c'est pas psr-1
    public function getId(): int { return $this->id; }
    public function getMot(): string { return $this->mot; }

    public function setId(int $value): void { $this->id = $value; }
    public function setMot(string $value): void { $this->mot = $value; }

    // Juste pour comprendre pourquoi le setter est intéressant ici je vais continue avec
    // l'exemple du created_at même si je ne l'ai pas dans ma table
    public function setCreatedAt($value): void
    {
        if (is_string($value)) {
            $value = DateTime::createFromFormat('Y-m-d H:i:s', $value);
        }

        $this->createdAt = $value;
    }
}

Quelque particularité de code que j'ai essayé de commentrer au mieux. Pour la suite on va modifier le fichier WordsRepository pour ressembler à ceci :

<?php

namespace Tuto\Repositories;

use PDOStatement;
use Tuto\Entities\Word;

/**
 * @package Tuto\Repositories
 * Class WordsRepository
 */
class WordsRepository
{
    /** @var Connection $connection */
    private $connection;

    /**
     * WordModel Constructor
     * 
     * @param Connection $connection
     */
    public function __construct(Connection $connection)
    {
        $this->connection = $connection;
    }

    /**
     * @return Word[]
     */
    public function all(): array
    {
        $request = $this->connection->request('SELECT id, mot FROM hello');

        $words = [];
        foreach ($request as $word) {
            $words[] = new Word($word);
        }

        return $words;
    }
}

Ne pas oublier de rename dans tous les fichiers les noms comme Tuto\Models\WordModel en Tuto\Repositories\WordsRepository et tout ce qui va avec.

Dans la vue il faut juste changer la boucle foreach qui devient :

<?php foreach ($words as $word): ?>
  <div>
    <?= $word->getId() ?> : <strong><?= $word->getMot() ?></strong>
  </div>
<?php endforeach; ?>

Dans le controller j'ai renommé les WordModel en WordsRepository

Lartak

Bonjour.
Tu devrais commencer par expliquer pourquoi tes conditions dans le if elseif sont les mêmes, soit $_GET['action'] == 'Controlword' dans le fichier index.php (soit le router d'après ce que tu dis).
Ensuite, dans ce même fichier, tu ne fais qu'initialiser une classe, si tu n'utilises pas une méthode de la classe et que tu ne stockes pas son retour dans une variable, le reste de ton code ne pourra pas avoir accès à cette variable et à ce qu'elle contient, tu n'aura par conséquent rien de plus que l'instance de la classe, que tu dois elle-même stocker dans une variable.