D
Это тестовое или что?)
Size: a a a
2021 January 17
I
Нет, это код библиотеки уже написанной
D
Аа, ну я могу ответить так, если бы я писал это, думаю я написал +- так. А если бы нужно было добавить новую функцию, то тупо добавил бы, а не фигачил какой нить декоратор, но с точки зрения подхода жто не совсем верно, нарушает принцип открытости закрытости
D
Видимо @mkusher говорил про это, если я правильно его понял
D
Как думаешь, вот тут нарушается ли принцип SRP?
И ответ на мой взгляд нет, интересно услышать еще мнения твое и других)
AK
Можешь это как то словами объяснить? Вот там в коде, этот объект умеет - клонировать себя, вычислять свой окаймляющий Прямоугольник, пересекать с лучиком, считать свою физ. массу.
Srp я воспринимаю скорее в контексте затрагивания разных зон ответственности, если у вас в компании есть люди, которые занимаются кодом для вычисления физ массы и другие люди, которые занимаются пересечениями лучиков, то лучше этот код разделять. Причем разные люди могут быть как на уровне инженеров, так и продукты или вообще на уровне chief разница.
D
Srp я воспринимаю скорее в контексте затрагивания разных зон ответственности, если у вас в компании есть люди, которые занимаются кодом для вычисления физ массы и другие люди, которые занимаются пересечениями лучиков, то лучше этот код разделять. Причем разные люди могут быть как на уровне инженеров, так и продукты или вообще на уровне chief разница.
Ну т е у класса должен быть один заказчик?
I
Мое мнение совпадает с мнением Aleh. Я не знаю, как его имя произнести правильно.
AK
Srp я воспринимаю скорее в контексте затрагивания разных зон ответственности, если у вас в компании есть люди, которые занимаются кодом для вычисления физ массы и другие люди, которые занимаются пересечениями лучиков, то лучше этот код разделять. Причем разные люди могут быть как на уровне инженеров, так и продукты или вообще на уровне chief разница.
Вместо вычисление физ массы и пересечения лучиков подставляйте что угодно)
AK
Мое мнение совпадает с мнением Aleh. Я не знаю, как его имя произнести правильно.
Загадка для восточных соседей об белорусской транслитерации имени Олег
D
Ой, один исполнитель в вашей трактовке
AK
Ну т е у класса должен быть один заказчик?
1. У модуля
2. Не только заказчик, но и желательно исполнитель
2. Не только заказчик, но и желательно исполнитель
AK
За solid сбольшего стоит цель уменьшения каскада изменений при эволюции проекта. Хотим вносить изменения затрагивая как меньше кода и, что не менее важно, вероятно ломая(ну и вообще воздействуя на) как можно меньше кода
D
1. У модуля
2. Не только заказчик, но и желательно исполнитель
2. Не только заказчик, но и желательно исполнитель
Да, круто. Спасибо)
D
Про исполнителя первый раз услышал
AK
Про исполнителя первый раз услышал
Тут опять же надо быть аккуратным, и имеются ввиду роли, а не прям конкретные люди. Бывают ситуации когда все продукты занимаются всеми вопросами или все девы правят весь код, это обычная ситуация на ранних этапах развития проектов, не имеется ввиду, что для каждого разраба надо отдельный файл пилить
D
Тут опять же надо быть аккуратным, и имеются ввиду роли, а не прям конкретные люди. Бывают ситуации когда все продукты занимаются всеми вопросами или все девы правят весь код, это обычная ситуация на ранних этапах развития проектов, не имеется ввиду, что для каждого разраба надо отдельный файл пилить
Ну т.е на перспективу это о том, что у дева будет его зона компетенции в проекте?
AK
Ну т.е на перспективу это о том, что у дева будет его зона компетенции в проекте?
Да
D
Понял) спасибо что влезли) я в своих рассуждениях ушел вообще не в ту сторону)
I
Да, мне точка зрения Олега нравится, так как она сходится с моей.