Комментарии к публикации
ORM - Object Relational Mapping

stdClass - standart Class?

У меня главная работает а на статью перехожу такая ошибка:

Что это может быть?

Включил xdebug и решил проблему так:

// src/MVCExample/Controllers/ArticlesController.php
use \MVCExample\Models\Article;

и еще добавил туда Article::class в

public function view(int $articleId) 
{
        $result = $this->db->query(
            'SELECT * FROM `articles` WHERE id = :id;',
            [':id' => $articleId],
            Article::class
        );
...
}

А так-же конечно поменял метод вывода в шаблоне через геттер

getName();
getText();

// templates/articles/view.php
<?php include __DIR__ . '/../header.php'; ?>
    <h1><?= $article->getName() ?></h1>
    <p><?= $article->getText() ?></p>
<?php include __DIR__ . '/../footer.php'; ?>

Подскажите, а нельзя ли выводить статью таким методом?

$result = $this -> db -> query('SELECT * FROM `articles` WHERE `id` = ' . $articleId . ';', [], Article::class);

Через конкатенацию?

Все работает, только имя автора не выводится

<?php

namespace MyProject\Controllers;

use MyProject\Services\Db;
use MyProject\View\View;
use MyProject\Models\Articles\Article;

class ArticlesController
{
    /** @var View */
    private $view;
    /** @var Db */
    private $db;

    public function __construct()
    {
        $this->view = new View(__DIR__ . '/../../../templates');
        $this->db = new Db();
    }

    public function view(int $articleId)
    {
        $result = $this->db->query(
            'SELECT * FROM `articles` WHERE id = :id;',
            [':id' => $articleId],
            Article::class
        );
        var_dump($result);
        if ($result === []) {
            $this->view->renderHtml('errors/404.php', [], 404);
            return;
        }

        $author = $this->db->query(
            'SELECT nickname FROM `users` WHERE id = :id',
            [':id' => $result[0]['authorId']],
            Article::class
        );
        var_dump($author);
        $this->view->renderHtml('articles/view.php', ['article' => $result[0], 'author' => $author]);
    }

}

Первый screenshot в описании метода _set по-моему не на месте.

По тексту мы только добавили в класс Article метод _set, который просто выводит на экран Свойство и Значение, без назначения переменных, а на screenshot-е: правильно (отсутствуют свойства author_id, created_at - мы их только печатаем) и неправильно (свойствам authorId, createdAt уже присвоены правильные значения).

Почему нельзя просто создать в классе переменный с таким же название как и в бд? Зачем создавать отдельную функцию?

Когда пытаюсь вывести автора получаю Fatal error:Uncaught Error: Cannot use object of type MyProject\Models\Articles\Article as array in C:\OSPanel\domains\myproject.loc\src\MyProject\Controllers\ArticlesController.php on line 31

<?php
namespace MyProject\Controllers;
use MyProject\Models\Articles\Article;
use MyProject\Services\Db;
use MyProject\View\View;

class ArticlesController
{
    private $view;
    private $db;

    public function __construct()
    {
        $this->view = new View(__DIR__ . '/../../../Templates');
        $this->db = new Db();
    }
    public function view(int $articleId)
    {
        $result = $this->db->query(
            'SELECT * FROM `articles` WHERE id = :id;',
            [':id' => $articleId],
            Article::class
        );
        var_dump($result);
        if ($result == false){
            $result = $this->view->renderHtml('errors/404.php', [], 404);
            return;
        }
        $author = $this->db->query(
            'SELECT nickname FROM `users` WHERE id = :id',
            [':id' => $result[0]['authorId']],
            Article::class
        );
        var_dump($author);
        $this->view->renderHtml('articles/view.php', ['article' => $result[0]]);
    }
}

Где то вы уже объясняли, не могу найти - зачем комментарии типа
/* @var string /
Только для PhpStorm - что-то он там при этом правильно подсвечивает?

А почему у нас метод __set вызывается автоматом? underscoreToCamelCase понятно,он используется в другом методе,а __set почему.

    public function __set($name, $value)
    {
        $camelCaseName = $this->underscoreToCamelCase($name);
        $this->$camelCaseName = $value;
    }

Артем, мы(__set) объявляем переменную $camelСaseName и присваеваем ей значение $name после переработки в методе, а после присваеваем этой же переменной новое значение $value, верно?! что _set дальше с этим делает($camelCaseName)?

А почему в начале класса нужно создавать свойства объектов, которые будут соответствовать полям в БД и можно ли их не создавать, а просто использовать геттеры/сеттеры для работы с свойствами(заранее НЕ объявляя их private $id, private $name и т.д.)?

Спасибо за урок!
А почему для свойства id мы обозначили тип int, но для author_id - string?

Также попытался сделать вывод автора в одиночной статье:

public function view(int $articleId): void
    {
        $result = $this->db->query(
            'SELECT * FROM `articles` WHERE id=:id;', [':id' => $articleId], Article::class
        );
        if ($result === []) {
            $this->view->renderHtml('errors/404.php', [], 404);
            return;
        }

        $nickname = $this->db->query(
            'SELECT nickname FROM `users` WHERE id=:id;', [':id' => $result[0]->getAuthorId()], User::class
        );

        $this->view->renderHtml('articles/view.php', ['article' => $result[0], 'author' => $nickname[0]->getNickname()]);
    }

Для этого пришлось редактировать класс User, чтобы не записывать свойство $nickname в класс Article, так как оно из другой таблицы бд, но ощущение, что выглядит костылем

По умолчанию это будут объекты класса stdClass – это такой встроенный класс в PHP, у которого нет никаких свойств и методов.

Этот класс используется чисто, как заглушка? Только для того, чтобы не возникла ошибка? А то я сразу кинулся прописать геттер в stdClass, а некуда прописывать)). Вроде и домашки не было, а код фактически не до конца работает. Но это и к лучшему, а то заметил, что даже просто прочитать и разобраться что как работает - мало - плохо запоминается/откладывается на подкорке, нужно самому что-то доделать, попробовать по другому сделать, в общем пропустить все через себя не только в теории, но и на практике.
Сделал так:
ArticlesController.php

public function view(int $articleId) {
        $result = $this->db->query('SELECT * FROM `articles` WHERE id = :id;', [':id' => $articleId], Article::class);
        if(empty($result)) {
            $this->view->renderHtml('errors/404.php',[], 404);
            return;
        }
        else {
            $articleAuthor = $this->db->query('SELECT nickname FROM users  WHERE id = :id;', [':id' => $result[0]->getAuthorId()], User::class);
            $this->view->renderHtml('articles/view.php', ['article' => $result[0], 'author' => $articleAuthor[0]]);
        }
    }

templates/articles/view.php

<?php include __DIR__ . '/../header.php'; ?>
    <p>Автор:<?= $author->getNickname(); ?></p>
    <h1><?= $article->getName(); ?></h1>
    <p><?= $article->getText(); ?></p>
<?php include __DIR__ . '/../footer.php'; ?>

класс User

<?php

namespace MyProject\Models\Users;

class User
{

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

    public function getNickname (): string
    {
        return $this->nickname;
    }
}

В метод fetchAll() мы передали специальную константу - \PDO::FETCH_CLASS, она говорит о том, что нужно вернуть результат в виде объектов какого-то класса. Второй аргумент – это имя класса, которое мы можем передать в метод query().

Мне кажется, тут было бы лучше для понимания сказать, что второй аргумент - это класс, объект которого будет создаваться и, со свойствами которого будут ассоциироваться данные, получаемые посредством fetchAll(). Во всяком случае, мне, как начинающему/изучающему так понятнее))

Задачей посложнее почему-то оказалось вывод автора в шаблон со всеми статьями, и кажется мне, решение какое-то корявенькое. Автор кочует в объект Article через объект User, да и смущают абсолютно одинаковые запросы в ДБ за никнеймом, присутствующие как в главном контроллере так и в контроллере статей... Костыль короче как мне кажется какой-то получился..

MainController.php

public function main()
    {
        $articles = $this->db->query('SELECT * FROM `articles`;', [], Article::class);
        for ($i=0; $i<count($articles); $i++) {
            $articleAuthor = $this->db->query('SELECT nickname FROM users  WHERE id = :id;', [':id' => $articles[$i]->getAuthorId()], User::class);
            $articles[$i]->setAuthor($articleAuthor[0]->getNickname());
            }
        $this->view->renderHtml('main/main.php', ['articles' => $articles]);
    }

В класс Article добавил сеттер и геттер для автора:

public function setAuthor ($author): void
    {
        $this->author = $author;
    }
    public function getAuthor (): string
    {
        return $this->author;
    }

В шаблоне main.php со всеми статьями просто запросил автора из объекта Article

<?php include __DIR__ . '/../header.php'; ?>

<?php foreach ($articles as $article): ?>
    <p><?= 'Автор статьи: ' .$article->getAuthor(); ?></p>
    <h2><a href="/articles/<?= $article->getId() ?>"><?= $article->getName() ?></a></h2>
    <p><?= $article->getText() ?></p>
    <hr>
<?php endforeach; ?>
<?php include __DIR__ . '/../footer.php'; ?>

Артем, а как у тебя получается выводить var_dump красиво? У меня каша получается, использую тег pre, но это не так наглядно, как у тебя на скриншотах.

Функция strreplace() заменяет в получившейся строке все символы ‘’ на пустую строку (то есть она просто убирает их из строки). После этого мы получаем строку StringWithSmth

Чтобы снова заработал вывод автора на странице одной статьи нужно добавить Article::class в запрос к таблице со статьями, естественно про use класса не забываем.
После этого в запросе к таблице users нужно использовать уже геттер у объекта с индексом [0], вот так $result[0]->getAuthorId()
Для третьего параметра желательно сделать и передать модель User, но так лень. Главное не допиливать модель Article, лишний геттер гетНикнейм там явно ни к чему, вместо этого просто передадим в renderHtml объект $author[0] целиком, а в шаблоне View выведем пока свойство объекта стандартного объекта $author->nickname. Пока так.

Код методов контроллера ArticlesController.php:

    public function view(int $articleId)
    {
        $result = $this->db->query(
            'SELECT * FROM `articles` WHERE id = :id;',
            [':id' => $articleId], 
            Article::class
        );

        if ($result === []) {
            $this->view->renderHtml('errors/404.php', [], 404);
            return;
        }

        $author = $this->db->query(
            'SELECT * FROM `users` WHERE id = :id;',
            [':id' => $result[0]->getAuthorId()]
        );

        var_dump

        $this->view->renderHtml('articles/view.php', ['article' => $result[0], 'title' => $result[0]['name'], 'author' => $author[0]]);
    }

namespace MyProject\View;

class View
{

    //путь шаблона
    private $templatesPath;

    public function __construct(string $templatesPath)
    {
        $this->templatesPath = $templatesPath;
    }

// создание html на входе имя шаблона и массив
    public function renderHtml(string $templatesName, array $vars = [], int $code = 200)
    {
        // масив разбиваем
        http_response_code($code);
        echo'<pre>';
        var_dump($vars);
        extract($vars);

        ob_start();
//включение путь шаблона/ имя шаблона

        include $this->templatesPath . '/' . $templatesName;
        $buffer = ob_get_contents();
        ob_end_clean();
        echo $buffer;
    }
}

Хоть домашки и не было, повозился чтобы при переходе на статью все работало. Оцените код на грамотность пожалуйста.
User.php

<?php

namespace Models\Users;

class User
{
    /** @var string */
    private $nickname;

    /**
     * @return string
     */
    public function getNickname(): string
    {
        return $this->nickname;
    }
}

ArticlesController.php

<?php

namespace Controllers;

use Services\Db;
use View\View;
use Models\Articles\Article;
use Models\Users\User;

class ArticlesController
{
    /** @var View */
    private $view;

    /** @var Db */
    private $db;

    public function __construct()
    {
        $this->view = new View(__DIR__ . '/../templates');
        $this->db = new Db();
    }

    public function view(int $articleId)
    {
        $result = $this->db->query(
            'SELECT * FROM `articles` WHERE id = :id;',
            [':id' => $articleId], Article::class
        );
        $resultAuthor = $this->db->query('SELECT * FROM users WHERE id=:id',
            [':id' => $result[0]->getAuthorId()], User::class
        );

        if ($result === []) {
            $this->view->renderHtml('errors/404.php', [], 404);
            return;
        }

        $this->view->renderHtml('articles/view.php', ['article' => $result[0], 'author' => $resultAuthor[0]]);
    }
}

view.php

<?php include __DIR__ . '/../header.php'; ?>
    <h1><?= $article->getName() ?></h1>
    <p><?= $article->getText() ?></p>
    <p><?= $author->getNickname() ?></p>
<?php include __DIR__ . '/../footer.php'; ?>

Спасибо за урок.
Не совсем понятна практическая ценность ORM по сравнению с обычными запросами и выводом из БД, но думаю потом дойдет.
Скажите, правильно ли я понял, что так под каждый запрос в БД для вывода чего-либо нужно создавать отдельный класс?

у меня в выдачу изначально был добавлен Count для количества статей каждого автора (и inner join + group by в запросе к базе соответственно), поэтому в класс передалось новое открытое свойство count(id_author). Дописал функцию

public function undelScoreToCamelCase(string $sourse): string {
    $demiReplace = lcfirst(str_replace('_', '', ucwords($sourse,'_')));
    $demiReplace1 = lcfirst(str_replace('(', '', ucwords($demiReplace, '(')));
    return str_replace(')', '', $demiReplace1);
}

получилась следующая строка вывода

<td><?=$author->countIdAuthor?></td>

это вообще правильно ? или от inner join надо отказаться и все дополнительные свойства получать дополнительными запросами? но ведь сделать запрос сразу кажется оптимальнее, чем выполнять его потом для каждой строки-объекта отдельно?

а еще напрягает немного, что все переменные в классе определяются как string, это же сколько туда сюда int в string и обратно если что переводить приходится?

и еще не очень понятно, почему все конструкторы мы создаем в одной папке, а здесь для каждого класса своя папка? или правильно все классы для всех таблиц из одной базы в одну папку собрать?

остальное все отлично работает, спасибо!