Чат Telegram
Группа ВКонтакте
Новый комментарий


Sparky

ArticleController:

public function edit(int $articleId): void
    {
        $article = Article::getById($articleId);

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

        $article1 = new Article();
        $article1->setName('Новая статья1 name');
        $article1->setText('Новая статья1 text');
        $article1->setAuthorId('1');
        $article1->save();
    }

ActiveRecordEntity

private function insert(array $mappedProperties): void
    {
        $params2values = [];
        $index = 1;

        foreach ($mappedProperties as $column => $value)
        {
            if (in_array(null, $mappedProperties))
            {
                unset($mappedProperties[array_search(null, $mappedProperties)]);
            }
        }
        foreach ($mappedProperties as $column => $value)
        {
            $params[] = ':param' . $index; // :param1
            $columns[] = $column;
            $params2values[':param' . $index] = $value;
            $index++;
        }

        $sql = 'INSERT INTO `' . static::getTableName() . '` (' . implode(', ', $columns) .
            ') VALUES (' . implode(', ', $params) . ');';
        $db = Db::getInstance();
        $db->query($sql, $params2values, static::class);
    }

Скажите, я правильно понял, если мы хотим изменить существующую позицию в БД, мы делаем $article = Article::getById($articleId) в контроллере и работаем с этим объектом, а если мы создаем новую сущность, то $article1 = new Article() ?
И как избежать использование foreach два раза, криво вроде как-то выглядит? :)

ivashkevich

Привет, сначала ответ на твой первый вопрос - да, всё правильно.

По поводу кода и второго вопроса:

  • в контроллере должен быть отдельный экшн для создания новой сущности (например - create), edit - только для уже существующих записей.
  • первый foreach можно грохнуть вместе со всем его содержимым - не повлияет ни на что. Также советую изучить функцию array_filter.
  • вместо :param{$index} можно использовать :{$column} - будет понятнее.

В целом - всё хорошо, есть небольшая путаница в контроллере, но это ничего страшного, со временем всё будет ;)

Kirill.K

ArticlesController:

public function create() {
        $article = new Article;
        $article->setName('Название новой статьи');
        $article->setText('Текст новой статьи');
        $article->setAuthor(1);
        $article->save();
    }

ActiveRecordEntity:

//задаём калбэк функцию
    private function crossOutNull($con) {
        return($con != null);
    }

    private function insert(array $mappedProperties): void {

        //оставляем только заданные параметры
        $mappedPropertiesDeclaredColumns = array_filter($mappedProperties, 'self::crossOutNull');

        //создаём два массива для формирования SQL запроса
        $articlesColumn = [];
        $articlesValues = [];
        foreach ($mappedPropertiesDeclaredColumns as $column => $value) {
            $articlesColumn[] = $column;
            $articlesValues[] = (is_string($value) ? "'" . $value . "'" : $value);
        }

        $sql = 'INSERT INTO ' . static::getTableName() . '(' . implode(', ', $articlesColumn) . ') VALUES (' . implode(', ', $articlesValues) . ')';

        $db = Db::getInstance();
        $db->query($sql, [], static::class);
    }
ivashkevich

Привет, нужно исправить:

  1. нельзя значение, пришедшее извне помещать в запрос без какой-либо фильтрации - всегда используйте параметризованные значения. В самом запросе только подстановки вида :param1, :param2. Сами значения - в параметрах, с которыми нужно выполнить запрос;
  2. исправление предыдущего пункта приведет к отсутствию необходимости в приведении значений к строкам/не строкам
  3. фильтрация null-значений - это лишнее
Kirill.K

Спасибо, всё принял к сведению, ну и подробное описание обнаружил в последующем уроке)

ivashkevich

красава)

ArtemijeKA

"Преобразовываем это значение из camelCase в строку_с_подчеркушками, например, author_id – именно так называется поле в базе данных"

Можно ли в БД имена столбцов писать сразу в camelCase или так не принято?

ivashkevich

Так не принято. Нужно именовать все маленькими буквами, разделяя слова подчеркушками.

ArtemijeKA

Где можно почитать про
'/(?<!^)[A-Z]/', '_$0'
В частности, что такое < ! и _$0

ivashkevich
[A-Z] - берём большие буквы

(?<!^) - а это означает, что при этом самую первую букву в начале строки мы не берем, даже если она большая

_$0 - это знак подчеркивания, за которым следует нулевое совпадение в регулярке (нулевое - это вся строка, попавшая под регулярку. В нашем случае - это одна большая буква). Таким образом, с помощью preg_replace, мы заменяем все большие буквы A - Z на _A - _Z. А затем с помощью strtolower приводим всю строку к нижнему регистру.
demyanovpaul@yandex.ru

Напишите самостоятельно метод insert(), который будет добавлять в базу новую запись. Не торопитесь, разбейте задачу на несколько компонентов и решите, какую последовательность действий нужно сделать.


private function insert(array $mappedProperties): void
{
        $mappedPropertiesNotNull = array_filter($mappedProperties);

        $columns = [];
        $params = [];
        $params2values = [];
        $index = 1;
        foreach ($mappedPropertiesNotNull as $column => $value) {
            $params[] = ':param' . $index; // :params
            $columns[] = $column; // columns
            $params2values[':param' . $index] = $value; // [:param => value]
            $index++;
        }

        $sql = 'INSERT INTO ' . static::getTableName() . '(' . implode(', ', $columns) . ') VALUES (' . implode(', ', $params) . ')';

        $db = Db::getInstance();
        $db->query($sql, $params2values, static::class);
}
ivashkevich

Отлично! Очень рад твоей скорости)

tomsonst

ArticlesController

public function create():void
    {
        $article = new Article();
        $article->setName('Новый статья');
        $article->setText('Новый текста');
        $article->authorId = '1';
        $article->createdAt = date(c);
        $article->save();
    }

ActiveRecordEntity

private function insert(array $mappedProperties): void
    {
        $columns2params = [];
        $params2values = [];
        $index = 1;
        foreach ($mappedProperties as $column => $value) {
            $param = ':param' . $index; // :param1
            $columns2params[] = $column . ' = ' . $param; // column1 = :param1
            $params2values[':param' . $index] = $value;
            $index++;
        }

        $sql = 'INSERT INTO ' . static::getTableName() . ' SET ' . implode (', ', $columns2params);
        $db = Db::getInstance();
        $db->query($sql, $params2values, static::class);
    }

Чувствую что-то тут не так)

ivashkevich

В каком именно месте ты это чувствуешь?)

tomsonst

Стоит ли cteatedAt так задавать? Или есть возможность это автоматизировать?

ivashkevich

Можно на уровне объектов оперировать с DateTime. А при сохранении и чтении из БД уже преобразовывать к строковому представлению. Как определять, что это поле с датой? Можно сделать специальные аннотации PhpDoc-ом к этому свойству в объекте, и определять с помощью рефлексии.

Ilon

Где-то подсмотрел, но думаю, что понимание темы - это успех)

Article Controller

public function create(): void
    {
        $article2 = new Article();
        $article2->setName('Новая статья 2');
        $article2->setText('Новый текст 2');
        $article2->setAuthorId(1);
        $article2->save();
    }

ActiveRecord

private function insert(array $mappedProperties): void
    {
        $mappedPropertiesNotNull = array_filter($mappedProperties);

        $columns = [];
        $params = [];
        $params2values = [];
        $index = 1;
        foreach ($mappedPropertiesNotNull as $column => $value) {
            $params[] = ':param' . $index;
            $columns[] = $column;
            $params2values[':param' . $index] = $value;
            $index++;
        }

        $sql = 'INSERT INTO ' . static::getTableName() . '(' . implode(', ', $columns) . ')' . ' VALUES (' . implode(', ', $params) . ' )';
        $db = Db::getInstance();
        $db->query($sql, $params2values, static::class);

    }
ivashkevich

Отлично!

ilyaOrlov

Где-то подсматривал, потому что пока не всё так легко, как хотелось бы)

ArticleController.php

public function create()
    {
        $article = new Article;

        $article -> setName('Название новой статьи');
        $article -> setText('Текст новой статьи');
        $article -> setAuthorId(1);

        $article -> save();
    }

ActiveRecordEntity.php

private function insert(array $mappedProperties): void
    {
        var_dump($mappedProperties);
        $mappedPropertiesNotNull = array_filter($mappedProperties);

        $columns = [];
        $params = [];
        $params2values = [];
        $index = 1;

        foreach ($mappedPropertiesNotNull as $column => $value) {
            $params[] = ':param' . $index;
            $columns[] = $column; 
            $params2values[':param' . $index] = $value; 
            $index++;
        }

        $sql = 'INSERT INTO ' . static::getTableName() . '(' . implode(', ', $columns) . ') VALUES (' . implode(', ', $params) . ')';

        $db = Db::getInstance();
        $db->query($sql, $params2values, static::class);
    }

Правильно ли я использую роут?
Routes.php

'~^articles/create$~' => [\MyProject\Controllers\ArticlesController::class, 'create']
ivashkevich

Всё хорошо, при обращении к свойствам и методам объектов не нужно ставить пробелы:

$article->save();
ilyaOrlov

Всё понял) Спасибо

alepawka
 private function insert(array $mappedProperties) : void
        {
            $param2values = [];
            $index = 1;
            foreach ($mappedProperties as $column =>$value) {
                $param = ':param' . $index;
                $columns[] = $column;
                $colums2params[] = $param;
                $param2values[':param' . $index] = $value;
                $index++;
            }
            var_dump($columns);
            $sql = 'INSERT INTO `' . static::getTableName() .'` (' . implode(',', $columns) . ') VALUES (' .  implode(',', $colums2params) . ');';
            var_dump($sql);
            $db = Db::getInstance();
            $db -> query($sql, $param2values, static::class);

        }

//добавил в route.php

'~^articles/(\d+)/create$~' => [\MyProject\Controllers\ArticleController::class, 'create'],
ivashkevich

var_dump в рабочем коде быть не должно - удаляйте перед отправкой на проверку. При обращении к свойствам и методам объектов с помощью стрелочки, её не нужно окружать пробелами.

excent63

Добрый день! Домашнее задание разобрал не полностью сам, смотрел подсказки в комментариях, но главное с трудом дошло откуда всё берется, вроде разобрался))

ActiveRecordEntity.php

...
private function insert(array $mappedProperties): void
    {
        $mappedPropertiesNotNull = array_filter($mappedProperties);

        $columns = [];
        $params = [];
        $params2values = [];
        $index = 1;

        foreach ($mappedPropertiesNotNull as $column => $value) {
            $params[] = ':param' . $index;
            $columns[] = $column;
            $params2values[':param' . $index] = $value;
            $index++;
        }

        $sql = 'INSERT INTO `' . static::getTableName() . '` (' . implode(', ', $columns) . ')' . ' VALUES (' . implode(', ', $params) . ')';

        $db = Db::getInstance();
        $db->query($sql, $params2values,static::class);
    }
...

ArticlesController.php

...
public function create()
    {
        $article = new Article;

        $article->setName('Новая Статья');
        $article->setText('Новый техт');
        $article->setAuthorId(1);

        $article->save();
    }
...

Роут

'~^articles/create$~' => [\MyProject\Controllers\ArticlesController::class, 'create'],

Небольшой вопрос: откуда создание статьи получать логичнее, на отсутствующей со статьёй вот так : '~^articles/(\d+)/create$~', или всё таки '~^articles/create$~'?

ivashkevich

Ок. А откуда возьмёте id ещё не созданной статьи?

excent63

Точно, недодумал )))

radinpasha13@gmail.com
private function mapPropertiesToDbFormat(): array
{
    $reflector = new \ReflectionObject($this);
    $properties = $reflector->getProperties();

    $mappedProperties = [];
    foreach ($properties as $property) {
        $propertyName = $property->getName();
        $propertyNameAsUnderscore = $this->camelCaseToUnderscore($propertyName);
        $mappedProperties[$propertyNameAsUnderscore] = $this->$propertyName;
    }

    return $mappedProperties;

разве в результате мы не получаем массив типа 'author_id'=>string 'authorId'

ivashkevich

А запускать пробовал?)

radinpasha13@gmail.com

да я же не об этом)я просто логику не понимаю,как оно работает.

ivashkevich
$this->$propertyName;

Вместо $propertyName (видим $ - значит это переменная) подставляется значение этой переменной. Если там строка "title", то произойдет обращение к $this->title; (уже без $)

Boodoo

Article.php

<?php
    namespace MyProject\Models\Articles;

    use MyProject\Models\ActiveRecordEntity;
    use MyProject\Models\Users\User;

    class Article extends ActiveRecordEntity
    {
        protected $name;
        protected $text;
        protected $authorId;
        protected $createdAt;

        public function setAuthorId(int $num)
        {
            $this->authorId = $num;
        }
        public function setName(string $name)
        {
            $this->name = $name;
        }
        public function setText(string $text)
        {
            $this->text = $text;
        }
        public function setCreatedAt(string $date)
        {
            $this->createdAt = $date;
        }
        public function getName() : string
        {
            return $this->name;
        }
        public function getText() : string
        {
            return $this->text;
        }
        public function getAuthor() : User
        {
            return User::getById($this->authorId);
        }

        protected static function getTableName() : string
        {
            return 'articles';
        }       
    }

ArticlesController.php

....
public function create()
{
    $article = new Article;
    $article->setAuthorId(1);
    $article->setName('Created note.');
    $article->setText('Created many text for created note.');
    $article->setCreatedAt(date('Y-m-d H:i:s'));
    $article->save();
}

ActiveRecordEntity.php

...
private function insert(array $mappedProperties)
{
    $columns2params = [];
    $params2values = [];
    $index = 1;
    foreach($mappedProperties as $column => $value) {
        $param = ':param' . $index;
        $columns2params[] = $column . ' = ' . $param;
        $params2values[':param' . $index] = $value;
        $index++;
    }
    $sql = 'INSERT INTO ' . static::getTableName() . ' SET ' . implode(', ', $columns2params);
    $db = Db::getInstance();
    $db->query($sql, $params2values, static::class);
}
ivashkevich

Отлично! А почему поля protected?

Boodoo

Не помню)) не менял поля. Нужно сделать private.

ivashkevich

Нужно)

Metey

в роде как по логике правильно, но так и не смог добиться , чтоб оно в базу писало....

public function insert(array $mappedProperties): void
    {

        $params = [];
        $params2values = [];
        $columns = [];
        $index = 1;
        foreach ($mappedProperties as $column => $value) {
            $columns[] = $column;
            $params[] = ':param' . $index; // :param1
            $params2values[':param' . $index] = $value; // [:param1 => value1]
            $index++;
        }
        $sql = 'INSERT INTO ' . static::getTableName() . ' ' . implode(', ', $columns) . ' VALUES ( ' . implode(', ', $params).')' ;
        $db = Db::getInstance();
        $db->query($sql, $params2values, static::class);

    }
ivashkevich

Дебажить пробовал?

Metey

с помощью var_dump , а как посмотреть тот момент где НЕ происходит запись в БД так и не понял

ivashkevich

Проверь, правильно ли формируется запрос. Попробуй скопировать его и выполнить в phpmyadmin

Metey

нашел, забыл скобки поставить в запросе и кроме того не отфильтровал этой строкой $filteredProperties = array_filter($mappedProperties); с пустыми параметрами почему то не работало , наверное потому что нуло давало и нулл не принимает база с настройками NOT NULL...

ivashkevich

Отлично. Молодец!

Iliusha99
private function insert(array $mappedProperties): void
    {
        $params2values = [];
        $columns = [];
        $params = [];
        $values = [];
        $index = 1;
        foreach ($mappedProperties as $column => $value){
            $params2values[':param' . $index] = $value;
            $columns[] = $column;
            $values[] = $value;
            $params[] = ':param' . $index;
            $index++;
        }

        $sql = 'INSERT INTO ' . static::getTableName() . ' (' . implode(', ', $columns) . ') VALUES (' .implode(', ', $params). ')';
        $db = Db::getInstance();
        $db->query($sql, $params2values, static::class);
    }
ivashkevich

Супер!

artemship

Добавил роут:

'~^articles/create$~' => [\MyProject\Controllers\ArticlesController::class, 'create']

Добавил метод create в ArticlesController

    public function create():void
    {
        $article = new Article();
        $article->setName('Новая статья');
        $article->setText('Новый текст');
        $article->setAuthorId(2);

        $article->save();
    }

Метод insert в ActiveRecordEntity

    private function insert(array $mappedProperties): void
    {
        $mappedPropertiesNotNull = array_filter($mappedProperties);
        $columns = [];
        $params = [];
        $params2values = [];
        $index = 1;

        foreach ($mappedPropertiesNotNull as $column => $value) {
            $param = ':param' . $index;
            $columns [] = $column;
            $params[] = $param;
            $params2values[$param] = $value;
            $index++;
        }

        $sql = 'INSERT INTO ' . static::getTableName() . ' (' .
            implode(', ', $columns) . ') ' .
            'VALUES ' . '(' . implode(', ', $params) . ');';
        $db = Db::getInstance();
        $db->query($sql, $params2values, static::class);
    }

Подсмотрел функцию array_filter в комментах :)

ivashkevich

Супер!

khuurak

Поглядел некоторую часть с роутом и фильтром в комментариях

ActiveRecordEntity.php

    private function insert(array $mappedProperties): void
    {
    $mappedPropertiesNotNull = array_filter($mappedProperties);
        $params = [];
        $columns = [];
        $params2values = [];
        $index = 1;

        foreach ($mappedPropertiesNotNull as $column => $value) {
            $params[] = ':param' . $index;
            $columns[] = $column;
            $params2values[':param' . $index] = $value;
            $index++;
        }

        $sql = 'INSERT INTO ' . static::getTableName() . '(' . implode(', ', $columns) . ') VALUES (' . implode(', ', $params) . ')';

        $db = Db::getInstance();
        $db->query($sql, $params2values, static::class);
    }

ArticleController.php

    public function create(): void
    {
        $article = new Article();
        $article->setAuthorId(1);
        $article->setName('Заголовок добавленной статьи в БД');
        $article->setText('Текст добавленной статьи в БД .. ');
        $article->save();
    }

routes.php

return [
    '~^article/(\d+)$~' => [\MyProject\Controllers\ArticleController::class, 'view'],
    '~^article/(\d+)/edit$~' => [\MyProject\Controllers\ArticleController::class, 'edit'],
    '~^article/create$~' => [\MyProject\Controllers\ArticleController::class, 'create'],
    '~^$~' => [\MyProject\Controllers\MainController::class, 'main'],
];
ivashkevich

Отлично

alex.evg.ostr

А если я хочу все это дело повесить на кнопку не уходя на страницу edit, наподобие form action , чтобы по кнопке обновлялись данные в БД, а после редиректа обновлялись на странице, то мне вызов метода на нее только повесить?

ivashkevich

После какого редиректа? Ты точно знаешь, что это такое?

alex.evg.ostr

Я ответил на свой вопрос в статье "Редактируем статьи в блоге на PHP" )

prinzplanloser0514@gmail.com

Я как-то нить потерял,откуда к нам значения в $articleId приходят.

ivashkevich

В каком месте? Отладчиком пробовал пользоваться?

Онлайн обучение PHP
Путь с полного нуля до джуниора!
Начать бесплатно
Логические задачи с собеседований