Size: a a a

2021 March 02

Y

Yaroslav in ctodailychat
У sq есть отличная концепция quality gates
источник

Y

Yaroslav in ctodailychat
А дизайн ревью позволяет устранить ошибки на этапе проработки решения по задаче
источник

Y

Yaroslav in ctodailychat
Еще до того, как код написан
источник

АА

Александр Арбузов... in ctodailychat
Yaroslav
При этом сам факт код ревью не несет пользы
можешь, пожалуйста, пояснить почему ты так считаешь.

кмк, код ревью это не только про ошибки и дизайн.
перенос знаний внутри команды как что-то сделать, возможно новые практики и коллективная (без)ответственность за код тоже важны.
источник

И

Илья in ctodailychat
У меня дома провайдер стал резать порты wireguard
источник

Y

Yaroslav in ctodailychat
Александр Арбузов
можешь, пожалуйста, пояснить почему ты так считаешь.

кмк, код ревью это не только про ошибки и дизайн.
перенос знаний внутри команды как что-то сделать, возможно новые практики и коллективная (без)ответственность за код тоже важны.
Если вам нужнн код ревью - делайте, просто постфактум
источник

O

Onlinehead in ctodailychat
Yaroslav
При этом сам факт код ревью не несет пользы
Ну факт определенно пользы не несет. Но у код-ревью в целом есть выход "полезных" вещей, тут конечно много вопросов про фанатизм, но как практика ДО мержа это в приницпе полезная штука.
Ревью не надо превращать в ритуал определенно. Но оно _иногда_ помогает подчерпнуть всякое интересное из чужого кода, который ты бы до этого не увидел и помогает  в целом сделать код качественнее. В общем - качество не равномерное же, плюс отсутствие ревью позволяет писать write only код, который пишется быстро, весело, задорно, но нихрена окружающим не понятен.
источник

Y

Yaroslav in ctodailychat
Фил Дельгядо недавно на эту тему размышлял. Доклад назыввется: хорошие практики как симптомы больших проблеи
источник

O

Onlinehead in ctodailychat
Ценность ревью относительно автоматических систем в том, что можно написать идеальный с точки зрения системы автоматического анализа код, который будет в целом отвратительным для других. В больших компаниях и проектах читабельность кода и общая внятность логики пожалуй даже важнее, чем какие-то детали реализации
источник

Y

Yaroslav in ctodailychat
Onlinehead
Ценность ревью относительно автоматических систем в том, что можно написать идеальный с точки зрения системы автоматического анализа код, который будет в целом отвратительным для других. В больших компаниях и проектах читабельность кода и общая внятность логики пожалуй даже важнее, чем какие-то детали реализации
Шаг выше сделайте, если ваша архитектура говно - то шансов нет
источник

O

Onlinehead in ctodailychat
Yaroslav
Фил Дельгядо недавно на эту тему размышлял. Доклад назыввется: хорошие практики как симптомы больших проблеи
Очень удобно представлять себе идеальный мир, где и без ревью все пишут хорошо. Круто, я бы сказал. Вообще много вещей становятся простыми и очевидными, если реальность отбросить:)
источник

O

Onlinehead in ctodailychat
Yaroslav
Шаг выше сделайте, если ваша архитектура говно - то шансов нет
ну ты же действительно не думаешь, что хорошая архитектура гарантирует хорошую имплементацию?
источник

Y

Yaroslav in ctodailychat
Onlinehead
ну ты же действительно не думаешь, что хорошая архитектура гарантирует хорошую имплементацию?
Нет, но обратное утверждение верное
источник

Y

Yaroslav in ctodailychat
Плохую архитектуру хороший код не спасет
источник

O

Onlinehead in ctodailychat
Обратное - да. Но это разные вещи совершенно. Это как утверждать, что все мы родились смертными, поэтому таблетки и прививки не нужны, все равно же все помрем:)
источник

Y

Yaroslav in ctodailychat
Onlinehead
Обратное - да. Но это разные вещи совершенно. Это как утверждать, что все мы родились смертными, поэтому таблетки и прививки не нужны, все равно же все помрем:)
Плохой код + хорошая архитектура - это не страшно
источник

O

Onlinehead in ctodailychat
Yaroslav
Плохой код + хорошая архитектура - это не страшно
Нет же:)
источник

O

Onlinehead in ctodailychat
У тебя может быть отличная архитектура, где каждый конкретный компонент проще выкинуть, чем модицифировать. А еще, где в целом все работает прекрасно, но кругом O(n^2). И архитектура у тебя будет прекрасная, но по факту все будет отвратительно.
источник

O

Onlinehead in ctodailychat
а еще прекрасная архитектура может быть наполовину имплементирована в отвратительном стиле, когда у тебя вроде все хорошо, но код невозможно поддерживать, потому что кто-то увлекся функциональным программированием и двусимвольными названиями переменных и функций. Короче много всего печального может произойти с твоей чудесной архитектурой.
источник

O

Onlinehead in ctodailychat
Нельзя рассматривать код и архитектуру отдельно и качество одного совершенно не гарантирует качество другого. Итоговый продукт выйдет не особо хорошим и при плохой архитектуре, и при плохой реализации.
источник