OK
Тип, нет гарантий, что имплементация поиска под капотом будет всегда работать одинаково для любых типов входных значений в долгосрочной перспективе.
согласна со строгостью в get, в has ....
а массив искомых ключей ты как типизируешь?
Size: a a a
2021 July 15
KS
надо подумать, но я бы описал доктайпом и преобразовал в строку
✨
в симфони крутая реализация, когда искомый ключ, параметр и прочее не найден - подсказывает, "Возможно вы искали вот это, или это ?"
✨
вооооо, валидаторами еще всякими обвешаемся ))
OK
Тайпкастить в строку - это тоже выстрел в ногу. Вот придет тебе array на вход, и все упадет с "Array to string conversion", хотя сигнатура не запрещает.
KS
ну да, похоже самый оптимальный вариант это: добавить доп метод для разделения and и or, оставить метод has без изменений и в цикле передавать ключ в метод has и как бы строгая типизация сама по себе сработает, а doctype'ом описать что ожидается массив со строками
OK
Самый оптимальный вариант - это забить и не трогать то. что не сломано)
KS
не, не выход))
МФ
А как же правило бойскаута или Божественного касания?
AP
Что ещё за правило такое?
МФ
Делай чище и лучше, чем было до тебя
МФ
Дядюшка Боб писал у себя об этом
KS
К сожалению все-таки мой PR закрыли(
странно, что не дали шанс исправить, посчитав функционал слабонеобходимым, по всей видимости ребята не очень любят сахар
тут кто-нибудь вносит вклад в опен-соурс? есть успешно слитые изменения?
странно, что не дали шанс исправить, посчитав функционал слабонеобходимым, по всей видимости ребята не очень любят сахар
тут кто-нибудь вносит вклад в опен-соурс? есть успешно слитые изменения?
СВ
Последние в изи админ3)
VK
Полно) кидай пр
VG
Сахар и симфони несовместимы. С сахаром к ларке подкатывать надо)
AN
А что за пр?
AN
Ларка по большей части сахар около симфонийских компонентов😁