Здравствуйте, Pauel, Вы писали:
P>Это препятствие для код ревью, но вовсе не повод отказываться от него.
Это профанация. Процесс ради процесса а не результата.
Ибо довольно бессмысленно давать код на review тем, кто ни в зуб ногой в тематике.
... << RSDN@Home 1.3.110 alpha 5 rev. 62>>
Забанили по IP, значит пора закрыть эту страницу.
Всем пока
Здравствуйте, Артём, Вы писали:
S>>>а мой код на Си обрабатывающий видео поток отдают на ревью java программистам CC>>Капец какой бардак! Аё>Да как они смеют!
Артёмка как обычно нихрена не понял.
... << RSDN@Home 1.3.110 alpha 5 rev. 62>>
Забанили по IP, значит пора закрыть эту страницу.
Всем пока
Здравствуйте, CreatorCray, Вы писали:
P>>Это препятствие для код ревью, но вовсе не повод отказываться от него. CC>Это профанация. Процесс ради процесса а не результата. CC>Ибо довольно бессмысленно давать код на review тем, кто ни в зуб ногой в тематике.
Вот чеклист для код ревью. Здесь по большому счету ничего не требует глубокого знания яп, хватит и поверхностных
— размер пр. Если слишком много — пусть автор заводит фича бранч, сплитает фичу на промежуточные пр.
— есть ли валидация параметров публичных методов
— есть ли тесты — юниты, функциональные, e2e
— корректна ли документация на методы
— изобретены ли велосипеды
— именование публичных методов
— есть ли обработка ошибок
— корректно ли оформлены конфиги
— есть ли магические константы итд
— мутный код вида "а херлиты в базу лезешь сразу из контролера"
— секурити типа "распаковывать jwt токен без валидации ой-ой-ой" создан тикет №10123
А вот если хочется продублировать работу автора кода, т.е. ровно тот кейс, для которого код ревью не нужен, тогда нужно глубокое знание.
И на это потребуется куда бОльшее количество времени, и эффект будет вообще говоря ничтожным.
Здравствуйте, Pauel, Вы писали:
P>Вот чеклист для код ревью.
Мда... симптоматичненько.
P>А вот если хочется продублировать работу автора кода, т.е. ровно тот кейс, для которого код ревью не нужен, тогда нужно глубокое знание. P>И на это потребуется куда бОльшее количество времени, и эффект будет вообще говоря ничтожным.
Мы с тобой настолько на разных уровнях отвественности что даже не интересно.
... << RSDN@Home 1.3.110 alpha 5 rev. 62>>
Забанили по IP, значит пора закрыть эту страницу.
Всем пока
Здравствуйте, CreatorCray, Вы писали:
P>>А бюджеты резиновые, да?
CC>Раз больше сишников кроме него нету то это какой то ССЗБ, ибо bus factor никто не отменял.
Сейчас почти вся индустрия так работает. Позволить себе дублирование инженеров могут только самые денежные конторы. У всех остальных висит этот риск в той или иной степени.
Здравствуйте, Pauel, Вы писали:
P>— изобретены ли велосипеды
Вот как питонист оценит это в плюсовом коде? В плюсах количество необходимых велосипедов сильно отличается в зависимости от версии, тут даже плюсовику может быть не просто.
Здравствуйте, CreatorCray, Вы писали:
P>>И на это потребуется куда бОльшее количество времени, и эффект будет вообще говоря ничтожным. CC> CC>Мы с тобой настолько на разных уровнях отвественности что даже не интересно.
Код ревью, как и любой процесс, строится под конкретные условия, а не от балды, "как правильно", итд. Что бы пересматривать всю логику, как делает ТС, должны быть веские основания.
Здравствуйте, Skorodum, Вы писали:
P>>— изобретены ли велосипеды S>Вот как питонист оценит это в плюсовом коде? В плюсах количество необходимых велосипедов сильно отличается в зависимости от версии, тут даже плюсовику может быть не просто.
Смотря что. Мелкую механику "вычислим смещение и засетаем по указателю" — скорее всего никак. Более крупную, типа "парсим xml регекспами в стиле Шеридана" — вполне себе. А вот все остальные пункты — вообще влёт.
Здравствуйте, Pauel, Вы писали:
CC>>Раз больше сишников кроме него нету то это какой то ССЗБ, ибо bus factor никто не отменял. P>Сейчас почти вся индустрия так работает.
С одним единственным инженером который тянет на себе часть проекта?
Чо, правда?
P> Позволить себе дублирование инженеров могут только самые денежные конторы.
Когда у тебя такой только один это не дублирование.
... << RSDN@Home 1.3.110 alpha 5 rev. 62>>
Забанили по IP, значит пора закрыть эту страницу.
Всем пока
Здравствуйте, CreatorCray, Вы писали:
P>>Сейчас почти вся индустрия так работает. CC>С одним единственным инженером который тянет на себе часть проекта? CC>Чо, правда?
Именно. Такое вот положение дел в индустрии. Проектов, которые ведет один человек, пруд пруди. Проектов, где зоны ответсвенности разработчиков слабо пересекаются много больше. Например, один разработчик ведет кучку пакетов-модулей-фич, другой — другую такую кучку. Пересечение разве что в пакетах которые не сильно важны, что бы выделять на них одного человека.
В редких случаях это иначе.
Разделение труда на марше, такое проходили все отрасли во все времена.
P>> Позволить себе дублирование инженеров могут только самые денежные конторы. CC>Когда у тебя такой только один это не дублирование.
Здравствуйте, Артём, Вы писали:
CC>>Раз больше сишников кроме него нету то это какой то ССЗБ, ибо bus factor никто не отменял.
Аё>У меня несколько коллег- жавистов, которые в прошлом писали на C++. No big deal поревьювить.
Самый лучший код ревью, который у меня был, это ревью от менеджера, который еще кодит на джаве время от времени.
Постоянно находит кучу важных проблем в апи, тестах, итд. Естественно, в мелкую моторику он не лезет — это и не нужно.
Здравствуйте, Pauel, Вы писали:
P>Именно. Такое вот положение дел в индустрии. Проектов, которые ведет один человек, пруд пруди.
Мы с тобой работаем в сильно разных индустриях.
... << RSDN@Home 1.3.110 alpha 5 rev. 62>>
Забанили по IP, значит пора закрыть эту страницу.
Всем пока
Здравствуйте, CreatorCray, Вы писали:
P>>Именно. Такое вот положение дел в индустрии. Проектов, которые ведет один человек, пруд пруди. CC>Мы с тобой работаем в сильно разных индустриях.
Это одна индустрия Просто ты работаешь в конторе где денег более чем достаточно для разработки. Из этого не следует, что у всех так же как и тебя.
Дело не в стеке технологий. Основные причины это бюджеты и цена ошибки на проекте.
Подавляющая часть ИТ это мелочевка, на самом деле, денег еле-еле хватает оплачивать команду, и цена ошибки невысокая. Отсюда ясно, что дублирование это роскошь.
Например, если у конторы большой бюджет, цена ошибки на фронтенде высокая — разумеется это отразится и на том, как комплектуются команды.
А если нативный кодинг, по уши где то в кернел моде, но бюджеты небольшие и цена ошибки невысокая, то здесь будет полно разработчиков одиночек.
Потому в норме нехватку людей можно видеть и в нативной разработке, ос-драйвера-итд, и в веб, бакенде, фронтенде, мобайле, мл, джава, дотнет — где угодно.
