AM
TradersVE
I explain how the framework shows the errors of the web application.
Size: a a a
2020 September 10
T
NO
getValue - работает как и раньше, кроме случая с точками - он их НЕ разбирает.
getValuePath - преобразует строчки с точками в массив и вызывает getValue.
getValuePath - преобразует строчки с точками в массив и вызывает getValue.
Ну против такого рефакторинга я не возражаю. Он имеет смысл. Жаль конечно что имя метода удлинится, но что ж поделаешь ) Зато наглядно.
T
getDotted, setDotted
It's fine 👍
NO
@predvoditelev зачем тогда в getByPath передавать Closure или массив? Ведь с массивом и Closure у метода getValue будет такой же результат. Почему тогда не сделать ограничение, чтобы принимало только строку?
Кому нужны будут ключи массивами и Closure, будут напрямую всегда к getValue обращаться.
Так будет ещё чище и логичнее.
Кому нужны будут ключи массивами и Closure, будут напрямую всегда к getValue обращаться.
Так будет ещё чище и логичнее.
AM
@predvoditelev зачем тогда в getByPath передавать Closure или массив? Ведь с массивом и Closure у метода getValue будет такой же результат. Почему тогда не сделать ограничение, чтобы принимало только строку?
Кому нужны будут ключи массивами и Closure, будут напрямую всегда к getValue обращаться.
Так будет ещё чище и логичнее.
Кому нужны будут ключи массивами и Closure, будут напрямую всегда к getValue обращаться.
Так будет ещё чище и логичнее.
А вот на это есть есть юзкейсы в AR
AM
Там можно и closure и строкой с точками одновременно.
NO
Там можно и closure и строкой с точками одновременно.
AR может использовать оба метода )
AB
Не планируется какой-нибудь AbstractController, чтоб туда спарять всякие инфраструктурные вещи (UrlGenerator, ViewRenderer, ResponseFactory)?
Д
Есть в yii-rest такой
AB
А для веба?
Д
Но кандидат на удаление
Д
Никто не хочет :)
AB
А аргументы? 🙂
AB
Просто вот метод что-то уж слишком большой: https://github.com/yiisoft/yii-demo/blob/master/src/Blog/Post/PostController.php#L116
Для сравнения в symfony тоже самое: https://github.com/symfony/demo/blob/master/src/Controller/Admin/BlogController.php#L133
Для сравнения в symfony тоже самое: https://github.com/symfony/demo/blob/master/src/Controller/Admin/BlogController.php#L133
AB
Ну тут еще формы нет, она теоретически уменьшит метод
AB
Но редирект на 5 строчек - это странно
RT
тоже смотрел на этот экшен
RT
надо вынести валидацию в форму