А
@roxblnfk что скажешь на счет request-model?
пока не знаю. Есть ещё пробелы в понимании работы.
Size: a a a
2020 October 16
RT
AM
Я тут вспомнил об увиденном недавно инструменте phpmetrics и натравил его на yii-queue. Результат по сложности/поддерживаемости неутешительный получился) Провел какое-то время, пытаясь понять, как можно улучшить архитектуру, но ничего путного не придумал. Может, у кого будут мысли?
Psalm натрави. Он может идей дать.
В
мерси, попробую
AM
Плюс тут не ясно, какие классы ему не нравятся.
В
С одной стороны, класс Queue вроде как перегружен. С другой - это вполне себе хороший фасад.
AB
phpunit тоже вроде умеет в такой анализ
А
в скрутинайзере ещё можешь посмотреть разные метрики по классам и методам
В
Плюс тут не ясно, какие классы ему не нравятся.
Да все основные 😅 Сейчас напишу
В
Я это делал в ветке, где стратегии обработки фейлов пишу. Так вот, в порядке уменьшения: Worker, SignalLoop, SynchronousDriver, DispatcherFactory, Queue, дальше - стратегии пошли. Внушний круг перечислил.
T
Я это делал в ветке, где стратегии обработки фейлов пишу. Так вот, в порядке уменьшения: Worker, SignalLoop, SynchronousDriver, DispatcherFactory, Queue, дальше - стратегии пошли. Внушний круг перечислил.
Write all possible tests first, then refactor, works for me.
AM
phpunit тоже вроде умеет в такой анализ
CRAP score.
AM
TradersVE
Write all possible tests first, then refactor, works for me.
Хороший подход 👍
AB
CRAP score.
ага, оно самое
AB
Хороший подход 👍
пока пишешь тесты там уже ясно бывает что надо вводить новые сущности
2020 October 17
Д
Увидел метод ArrayHelper::getValueByPath и у меня вопрос. Будет ли метод ArrayHelper::hasValueByPath.
Сейчас есть keyExists, но по какой стратегии он резолвит ключи?
Сейчас есть keyExists, но по какой стратегии он резолвит ключи?
СП
Увидел метод ArrayHelper::getValueByPath и у меня вопрос. Будет ли метод ArrayHelper::hasValueByPath.
Сейчас есть keyExists, но по какой стратегии он резолвит ключи?
Сейчас есть keyExists, но по какой стратегии он резолвит ключи?
Сейчас вложенность вообще не поддерживается. Там фишка в возможность регистронезависимой проверки.
Надо ли делать keyexists для вложенных массивов. Есть случаи когда это надо?
Надо ли делать keyexists для вложенных массивов. Есть случаи когда это надо?
NO
Да вообще какую-то фигню сделали, с этим разделением.... Неудобно
NO
Ни туда ни сюда
NO
Оставили какие-то костыли в хелпере, ради того чтобы в ActiveRecord избавиться от одного IF