if (!$Profile->validate()) {
$Form->Errors->fetchFrom($Profile->getErrors())
} else {
$Profile->save();
}Не, ну смысл это делать в контроллере?
class My_Form extends Form {
public function build()
{
// инициализация элементов, валидаторов и т.д.
$this->addValidate(чего-нибудь);
}
public function isValid($data)
{
$actualData = array_intersect_key($data, $this->elements);
$errors = array();
foreach($this->elements as $id => $element) {
$value = (isset($actualData[$id]) ? $actualData[$id] : null);
$errors += $element->validate($value)->getErrors();
}
$errors += $this->model->validate($actualData)->getErrors();
$this->_errors = $errors;
return ($errors ? false: true);
}
}В смысле?
Если говорить конкретно об этом куске кода http://paste.org/pastebin/view/18126 то
Ведь если я перенесу конкретно данную валидацию, что представлена по ссылке в коде, то она, эта логика, не будет использоваться НИГДЕ БОЛЬШЕ, КРОМЕ ЭТОГО КОНТРОЛЛЕРА! Я не вижу смысла делать ещё один слой.
а чем должен заниматься тогда сервисный слой?
Всё равно, я не понимаю.
+1
О !!! вот это подход. Нет нормальной модели - выносим кучу логики в контроллер. И вместо доработки модели хотим эту логику вынести в ещё один слой. Круто.
Ну ты же не пишешь весь код лапшой.
Еще раз. Если логики немного - то она может находиться в контроллерах.
руки сами потянутся убрать логику из контроллера в модели.
для повторного использования прежде всего.
мне казалось, смысл слоя сервисов - повторно использовать логику, не? Ибо "читаемость кода" - вещь сугубо абстрактная. Для одних код в 50 строк уже не читабелен, другие вот так пишут и ничего.
хехе, я в самом начале угу ещё сказал, что всё в жизни сложнее.
Вот. Моделей можно сделать больше. Но при чём тут сервисный слой?
маппер, если я правильно понял.
нижний уровень DBAL
Фиг его знает, что это из себя представляет. Я с C# и LINQ не очень. Возможно ближе к транспортной структуре.
тоже транспорт
Использовать повторно маленький кусочек, который делает какую-то вполне конкретную операцию - много проще, чем использовать большой кусок, который делает кучу разных вещей.
А он часто-густо и не нужен. Я лишь в 2х задачах приблизился к нему и обе были ERP-класса.
это ты так привык.