DS
Ну лично я не вижу логики в том чтобы распаковывать массив в список параметров, чтобы потом в методе его опять запаковать в массив.
Size: a a a
2020 February 27
DS
Ты ж там вроде где-то в пхп-сообществе состоишь, задай им вопрос что быстрее работать будет)
AM
Да ё-моё. Есть https://3v4l.org/, сунуть туда достаточно и посмотреть на время отработки. Не очень серьёзный тест, но покажет.
AM
По логике туда-сюда гонять дороже.
AM
В реальности PHP логика иногда не работает.
T🐜
Ну лично я не вижу логики в том чтобы распаковывать массив в список параметров, чтобы потом в методе его опять запаковать в массив.
В данном случае бесполезно, да. Контроль типа в
можно просто
attach же есть, а другими путями никак не попадает листерможно просто
iterable. Наверное.DS
По логике туда-сюда гонять дороже.
Суть еще в том, что array_values там обязательное, я пробовал, не хочет со строчными ключами распаковывать
АН
@samdark https://github.com/yiisoft/validator/pull/54 глянь этот ПР, почему-то там показывает много коммитов )
AM
потому что master
АН
лучше делать новый бренч для каждого ПР-а ?
AM
АН
ок
AM
иначе невозможно одновременно работать над двумя и более
Д
А, так это не @xepozz сделал, если что.
AM
Но это не важно кто сделал. Вопрос норм, но, если можно, решите его без меня и, если норм, зафигачьте PR...
AM
Сходу сказать не могу как надо и занят докой. Не хочу сильно выпадать из контекста.
AM
Так-то у меня вообще были мысли слить два этих провайдера и просто сделать возможность опционально руками указывать класс события.
AM
Но там код получается с наворотами...
DS
Сходу сказать не могу как надо и занят докой. Не хочу сильно выпадать из контекста.
Ок, пометил себе, проверю на скорость, потом если что ПР сделаю. Я это тоже в контексте работы над пркси-контейнером заметил.
AM
Спасибо.