RT
в psr такое часто используют
Size: a a a
2020 August 31
RT
например в реквесте
RT
public function getAttribute($name, $default = null);RT
хотя тут геттер, а там нет
NO
Сразу появляется неоднозначность, что будет если не числовой параметр? Исключение? Дефолтное значение? Пока не залезешь внутрь кода, это непонятно. Придётся держать в голове.
А
Сразу появляется неоднозначность, что будет если не числовой параметр? Исключение? Дефолтное значение? Пока не залезешь внутрь кода, это непонятно. Придётся держать в голове.
Как вообще писать код и не держать то, чем пользуешься, в голове?
NO
Код должен быть максимально простым и дубовым, а уж код хелперов, который по всему проекту используется, и подавно.
NO
NO
Если коротко, чем меньше деталей приходится держать в голове, тем быстрее пишется код, и допускаешь меньше ошибок.
NO
То что этот параметр опциональный, кстати, ещё больше усложняет метод. То он есть, то его нет, если его не вписали - это было намеренно или просто забыли? Не-не-не, я через это уже проходил )
А
Есть вариант вторым аргументом bool указывать дроп исключения, по умолчанию возвращать null
Ни к чему не обязывает и можно избежать проверки на нулл, если нужно исключение
Ни к чему не обязывает и можно избежать проверки на нулл, если нужно исключение
NO
Чтобы читалось хорошо и наглядно, можно написать по такому принципу, разделить все эти нюансы на отдельные методы в стиле fluent interface.
$page = $helper->from($request)->get('page')->asInteger()->withDefault(0)
Но тогда это уже в тот самый RequestHelper превращается )
$page = $helper->from($request)->get('page')->asInteger()->withDefault(0)
Но тогда это уже в тот самый RequestHelper превращается )
NO
Я категорически против усложнения хелперов )
NO
1 аргумент в 100 раз проще чем 2 аргумента
AM
В дифф зайди
Одно замечание пофиксил. Остальные норм и так.
AM
Это норм английский же :)
AM
Кстати лучше здесь с подчёркиваниями пример дать, так как это важный нюанс. Подчёркивания это самый распространённый вариант именования таблиц.
Добавил. Спасибо.
AM
да, там скорее camelToSnake и snakeToCamel
Нет потому что ещё и форма конвертируется.
NO
тогда более наглядный пример нужен
AM
В NumericHelper мне не хватает методов toInteger и isDigits.
Первая сконвертирует значение в int либо null, если это возможно, вторая вернёт true или false, в зависимости от того, может ли первая сконвертировать в int.
Первая сконвертирует значение в int либо null, если это возможно, вторая вернёт true или false, в зависимости от того, может ли первая сконвертировать в int.
А кастовать в (int) и юзать is_numeric() не лучше чем метод?