Где именно формировать сообщение?

Question

[dev400]

Контроллер

if( !empty($_POST) ) {

            try {
                
                $this->model->insertNews($_POST);
                $this->msg = "Новость успешно обновлена";


            } catch(\Exception $e) {

                $this->msg = $e;

            }

            return false;

        }

$this->msg далее передается в представление.
Правильно ли так формировать уведомления?
Или хрень написал?

Comments

[AlikDex]

это контроллер? Почему модель наследуется от контроллера?
Полная херня =)

[dev400]

AlikDex: в свойство model кладется объект модели

[AlikDex]

Андрей: а можно всю модель сюда? под спойлером.

[dev400]

AlikDex: можно

<?php
namespace Backend\Models;

use Common\Base\Model;

class News extends Model {

    protected $table = 'news';

    protected $key;

    public function allNews() {

        while(!$rows = parent::findAll()){

            throw new \Exception('not find news');

        }

        return $rows;
    }

    public function insertNews(Array $array) {

        $params = [
            "id" => $array['news_id'],
            "header" => $array['news_header'],
            "text" => $array['news_text'],
            "url" => $array['news_url']
        ];

        $sql = "UPDATE {$this->table} SET `header` = :header, `text` = :text, `url` = :url WHERE `id` = :id";

        if( !parent::save( $params, $sql ) ) {

            throw new \Exception('Ошибка сохранения новости');

        }

    }


    public function getSingle($key, $param = []) {

        $this->key = $key;
        if( !$rows = parent::findBy($param) ){
            return false;
        }
        return $rows;

    }

}

Базовая модель

<?php
namespace Common\Base;

use Common\Connection;

abstract class Model {

    /**
     * prefix for tables
     */
    const PREFIX =  "";

    /**
     * @var
     */
    protected $table;

    /**
     * @var
     */
    protected $key;

    /**
     * @return mixed
     */
    public function findAll() {

        return $this->fetch("SELECT * FROM " . self::PREFIX . $this->table . " ");

    }


    /**
     * @param array $args
     * @return array|string
     */
    public function findBy(Array $args) {

        $stmt = Connection::con()->prepare("SELECT * FROM " . self::PREFIX . $this->table . " where {$this->key} = ?");
        $data = "";
        if ( $stmt->execute($args) ) {
            while ($row = $stmt->fetch()) {
               $data[] = $row;
            }
        }
        return $data;

    }


    /**
     * @param $query
     * @return mixed
     */
    protected function fetch($query) {

        $stmt = Connection::con()->query($query);
        $stmt->setFetchMode(\PDO::FETCH_ASSOC);
        $data = "";
        while ($row = $stmt->fetch()) {
            $data[] = $row;
        }
        return $data;

    }


    protected function save(Array $array, $sql) {

        $sth = Connection::con()->prepare($sql);
        return $sth->execute($array);

    }


}

[AlikDex]

Андрей: ну так у вас вроде сохраняется там все и ошибки выводятся. Да и вообще все совсем не так как в вопросе =)

Вообще валидатор должен быть какой-то еще. У которого есть массив ошибок для каждого правила. Что-то типа правила [['param1', 'param2'], 'integer', 'msg'=>'Must be int']

[dev400]

AlikDex: Да, это я на будущее отложил. Сделать по каждому полю валидацию. Я просто хочу уточнить по выводу уведомления пользователю, правильно ли это делать именно так

[AlikDex]

Андрей: Да нормально наверное выбрасывать исключение через throw new, по крайней мере так многие делают. Сам же не использую, ибо: https://ru.wikipedia.org/wiki/%D0%9E%D0%B1%D1%80%D...

Answer 1

[DTX]

Нормально

Comments

[trevoga_su]

че там нормального? =)

[DTX]

trevoga_su: работает же)
А в посте выше вообще в БД предлагают сообщения писать...
Если уж редирект нужен, то лучше в кукисы запихать на один вызов. В CI это будет setFlashdata

Answer 2

[trevoga_su]

Правильно ли так формировать уведомления?

нет
после любого post-запроса надо делать header: location

как передать уведомления - www.phpinfo.su/articles/theory/peredacha_dannyx_po...

Comments

[# a r t u r #]

если не делать переадресацию, то при нажатии на F5 действие будет повторяться...