rulinux.net - Все о GNU/Linux и не только - Форум - Talks

Скрыть

программистам

У вас на работе организовывают инспекции кода (code review)? Какое у вас к этому отношение?

AMUR-WOLF(*) (2018-02-23 15:00:29)

Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:58.0) Gecko/20100101 Firefox/58.0

[Ответить на это сообщение]

Ответ на: программистам от AMUR-WOLF 2018-02-23 15:00:29

Скрыть

Re:программистам

Я не погромист, но считаю кодревью обязательным этапом в продакшене.

anonymous(*)(2018-02-23 17:02:11)

Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/64.0.3282.140 Safari/537.36

[Ответить на это сообщение]

Ответ на: Re:программистам от anonymous 2018-02-23 17:02:11

Скрыть

Re:программистам

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

anonymous(*)(2018-02-23 19:10:57)
Отредактировано anonymous по причине "не указана"
Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/64.0.3282.140 Safari/537.36

[Ответить на это сообщение]

Ответ на: Re:программистам от anonymous 2018-02-23 19:10:57

Скрыть

Re:программистам

Но ревьювить код, чувачков на время испытательного срока обязалово)))

anonymous(*)(2018-02-23 19:14:53)

Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/64.0.3282.140 Safari/537.36

[Ответить на это сообщение]

Ответ на: Re:программистам от anonymous 2018-02-23 19:10:57

Скрыть

Re:программистам

Я работаю в разобщенном молодом (ср. в. 25 лет) коллективе, где единственное достижение процесса разработки - использование SVN с одной веткой.

Начальство раздало каждому программисту по личному проекту. Мне дали три небольших проекта, все три уже в продакшене.

Я составил единый корпоративный стандарт оформления кода и показал, как в IDE Eclipse сделать автоматическую корректировку стиля. Сам этим пользуюсь и радуюсь, но больше никто не хочет.

Архитектура приложения - это личные вкусовые предпочтения разработчика. У нас есть один вундеркинд, который обожает паттерны проектирования и лепит их побольше, никто не может быстро и правильно работать с его кодом. Я паттерны не люблю, но у меня другой уклон, я люблю языковые фичи и являюсь единственным, кто активно использует в java дженерики, лямбда-выражения и функциональные интерфейсы, ссылки на методы, потоковый API, перечисления enum, класс Optional, javadoc с html-тэгами. Все остальные используют только древние и обязательные фичи типа if, switch, for, интерфейсов, абстрактных классов, статических членов, наследования с перегрузкой. Поэтому никто не поймет мой код, хотя я его обильно комментирую через javadoc.

Иногда я смотрю чужой код. Кроме отвратительного стиля, я вижу дыры в коде. Самая типичная дыра - отсутствие проверки, что переменная проинициализирована. У меня настроена IDE так, что эти места подчеркиваются красным, как ошибки. Я уже забыл про NullPointerException, но другие удивляются, как в их хорошо отлаженных программах это исключение возникает. Они два года пилят свои проекты, проекты еще не в продакшене. Если они пойдут в продакшн, все охренеют от количества багов. Я уже через это прошел: пользователи всё делают неправильно, инструкции не читают, самовольно химичат.

Ни Agile, ни Waterfall у нас не применяется, хотя Watefall у нас утвержден документально как процесс разработки ПО. Тестирование у нас тоже не организовано. Каждый свое тестирует сам и думает, что этого вполне достаточно. Парного программирования тоже нет и не будет. Я в своих проектах сделал автоматическое тестирование. Мы расширяем Teamcenter плагинами, используем API Teamcenter, работаем через API с базой данных Oracle. Я не знаю, как использовать блочные тесты в таких условиях. Но я написал интеграционные тесты, которые запускаются в моих плагинах пользователями при нажатии на кнопку в уже развернутом и запущенном приложении. Они приносят пользу, два раза в месяц тесты проваливаются. Когда начальник это увидел, мне пришлось объяснять, что это такое и зачем я это сделал.

При code review у нас будут срачи... Но по крайней мере все узнают, кто что делает.

AMUR-WOLF(*)(2018-02-23 20:05:08)

Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:58.0) Gecko/20100101 Firefox/58.0

[Ответить на это сообщение]

Ответ на: Re:программистам от AMUR-WOLF 2018-02-23 20:05:08

Скрыть

Re:программистам

> . Мне дали три небольших проекта, все три уже в продакшене.

И их кроме тебя никто никогда не видел. Начальник, должно быть, глубоко верующий человек.

anonymous(*)(2018-02-24 00:45:38)

Mozilla/5.0 (Android 7.1.2; Mobile; rv:50.0) Gecko/50.0 Firefox/50.0

[Ответить на это сообщение]

Ответ на: Re:программистам от AMUR-WOLF 2018-02-23 20:05:08Фильтры

матерные выражения

Скрыть

Re:программистам

Типичная совкософт контора. Не рекомендую настаивать на коде ревью, тебя только будут посылать на хуй лишний раз. Вместо этого советую попробовать делать самому все правильно и попробовать подмять под себя это все. Сделать нормальный процесс, в том числе норм тестирование как первый шаг, как второй запретить мерж в тестовую ветку без коде ревью. В продакт ветку только после тестовой ветки и прохождения всех тестов. Вот тогда ты сможешь всю эту пиздобратию наставить на путь истинный))

anonymous(*)(2018-02-24 01:16:26)
Отредактировано anonymous по причине "не указана"
Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/64.0.3282.140 Safari/537.36

[Ответить на это сообщение]

Ответ на: Re:программистам от anonymous 2018-02-24 00:45:38

Скрыть

Re:программистам

> Начальник, должно быть, глубоко верующий человек.
Походу у них все, глубоко верующие в похуизм))

anonymous(*)(2018-02-24 01:17:52)

Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/64.0.3282.140 Safari/537.36

[Ответить на это сообщение]

anonymous@RULINUX.NET~#	Last login: 2024-12-25 23:35:37
Регистрация Вход	Новости \| Разметка \| Пользователи \| Галерея \| Форум \| Статьи \| Неподтвержденное \| Трекер \| Правила форума \| F.A.Q. \| Ссылки \| Поиск

Этот тред читают 1 пользователь:
Анонимных: 1 Зарегистрированных: 0