rulinux.net - Форум - Talks - программистам

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

Fri, 23 Feb 2018 21:17:52 +0300

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

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

Fri, 23 Feb 2018 21:16:26 +0300

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

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

Fri, 23 Feb 2018 20:45:38 +0300

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

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

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

Fri, 23 Feb 2018 16:05:08 +0300

Я работаю в разобщенном молодом (ср. в. 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 у нас будут срачи... Но по крайней мере все узнают, кто что делает.

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

Fri, 23 Feb 2018 15:14:53 +0300

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

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

Fri, 23 Feb 2018 15:10:57 +0300

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

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

Fri, 23 Feb 2018 13:02:11 +0300

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

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

Fri, 23 Feb 2018 11:00:29 +0300

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