Василий М.
Новичок
Читаем, обсуждаем - http://www.phpinfo.su/articles/theory/settery_v_php_pravilnoe_ispolzovanie.html
-
Юбилейный DevConfX пройдет 21-22 июня в Москве. Как всегда — Вы решаете, кто попадет в программу секции Backend — голосуйте за интересные доклады
Сеттеры в PHP
- Автор темы Василий М.
- Дата начала
В случае, если объект - не некий ArrayObject (или делегирующий ему), для начала надо подумать, какого хрена мы пишем $Object->setName() или $Object->getName(). Этого быть вообще не должно, должно быть $this->name.
Теме сто лет в обед:
http://demiurg.livejournal.com/60458.html?thread=735786#t735786
там Crazy все правильно говорит, но, кажется, его никто не понимает. (Признаться, тогда, в 2005-м, я тоже не понял).
Теме сто лет в обед:
http://demiurg.livejournal.com/60458.html?thread=735786#t735786
там Crazy все правильно говорит, но, кажется, его никто не понимает. (Признаться, тогда, в 2005-м, я тоже не понял).
Чего только не придумают вместо нормального проектирования.
Публикация свойств с заранее известными именами - это в нормальном объектном дизайне очень редкая вещь: фактически это нарушение инкапсуляции. Во всех уместных случаях будет __set magic и ничего заранее не опишешь.Василий М.
Новичок
а разве не @var?
Redjik
Джедай-мастер
модель в AR ?Чего только не придумают вместо нормального проектирования.
Василий М.
Новичок
Ну так в AR и будет __set magic.
Redjik
Джедай-мастер
Ну да, только чтобы перечислить поля модели генератор кода в yii создает портянку @property - что в принципе уместно и очень удобно для автокомплита...
AmdY
Само собой. Проблема в том, что это приведет (и приводит!) к коду в пейдж контроллере вида
$Model->title = $Request->title;
$Model->name = $Request->name;
/// еще стопицот строк
Например, все начинающие РоРовцы так делают, пока кто-то не стукнет по рукам и не скажет про update_attributes.
Вопрос, нужно ли нам вообще вне модели знать, какие поля у нее есть? Я склоняюсь к тому, что нет:
Само собой. Проблема в том, что это приведет (и приводит!) к коду в пейдж контроллере вида
$Model->title = $Request->title;
$Model->name = $Request->name;
/// еще стопицот строк
Например, все начинающие РоРовцы так делают, пока кто-то не стукнет по рукам и не скажет про update_attributes.
Вопрос, нужно ли нам вообще вне модели знать, какие поля у нее есть? Я склоняюсь к тому, что нет:
PHP:
$Model = (new Model_User)->loadById($this->userId);
$Form = (new Form)->fillFromModel($Model);
if ($this->Request->isPost()) {
$Model->fetchAttributesFrom($Form)->save();
return $this->Response->redirectToPage($this);
}
$this->View->assign('Form', $Form);Василий М.
Новичок
тогда это не модель, а мусорка, типа array
у нас на работе так делают - наполняет звездные классы как из клиентского кода, так и из методов класса.
в итоге хз какие свойства у объекта есть, каких нет и откуда они там появляются - никто не скажет
угу, но в твоём примере самое главное, чтобы форма возвращала только разрешённые поля, а не все что подсунули в пост. те же роровцы на этом обожглись, вроде так ломали даже гитхаб.
Вот только я не понял, а как связаны магические методы и знание об полях. Плюс магических методов как раз в том, что можно засунуть какие угодно поля, а уже на sql запросе вывалится ошибка какого поля нет.
Судя по всему я тебя недопонял.
Вот только я не понял, а как связаны магические методы и знание об полях. Плюс магических методов как раз в том, что можно засунуть какие угодно поля, а уже на sql запросе вывалится ошибка какого поля нет.
Судя по всему я тебя недопонял.