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

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

У вас на работе организовывают инспекции кода (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
avatar
Скрыть

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
avatar
Скрыть

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
avatar
Скрыть

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
avatar
Скрыть

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
avatar
Скрыть

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Фильтры
avatar
  • матерные выражения
Скрыть

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
avatar
Скрыть

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
Этот тред читают 2 пользователя:
Анонимных: 2
Зарегистрированных: 0




(c) 2010-2020 LOR-NG Developers Group
Powered by TimeMachine

Valid HTML 4.01 Transitional Правильный CSS!