A
Size: a a a
2020 November 09
z
А на что смотрят? Паттерны и clean code?
Все по-разному. Зависит от ценностей, которых придерживается компания. Если компания дает тестовое задание, можно попробовать уточнить, на что конкретно они обращают внимание. Паттерны и чистый код должны быть по умолчанию, я считаю. Из неочевидного на что лично я обращаю внимание на код-ревью это нейминг.
DM
Еще логирование могут смотреть
A
Еще логирование могут смотреть
ага делать мне нечего из тестового задания продакшен реди код писать
A
может еще пайплайн сделать и кубернетис заиспользовать
A
и грейлог подрубить с забиксом
VG
не заставляйте вас убеждать в необходимости документирования кода
Зачем документировать код, когда можно потратить немного лишнего времени и дать очевидные имена методам/интерфейсам/классам
VG
Если код нуждается в документации, то что то с ним не так
B1
Хай, хелп - может у кого есть под рукою актуальное описание Java Memory Model ?
АД
Хай, хелп - может у кого есть под рукою актуальное описание Java Memory Model ?
https://openjdk.java.net/ вот тут есть
DM
Если код нуждается в документации, то что то с ним не так
Ну да, когда видишь в энтити поле с названием OFKD тебе вовсе не нужна документация, чтобы понять что это, главное чтоб код был хорошим, ага
z
а кто там говорил, что нормальный код в комментариях не нуждается, Бугаенко вроде?
Много кто про это писал еще до Егора, например в the pragmatic programmer (1999)
In general, comments should discuss why something is done, its purpose and its goal. The code already shows how it is done, so commenting on this is redundant and is a violation of the DRY principle.
z
Точно не помню, но скорее всего в Чистом коде тоже про это есть
A
Ну да, когда видишь в энтити поле с названием OFKD тебе вовсе не нужна документация, чтобы понять что это, главное чтоб код был хорошим, ага
да документация бы тут явно помогла
АД
да документация бы тут явно помогла
а как же, там было бы написано // OFKD id
VG
Ну да, когда видишь в энтити поле с названием OFKD тебе вовсе не нужна документация, чтобы понять что это, главное чтоб код был хорошим, ага
Зачем такие имена полям давать?
A
Если уже код написан с такими полями, то страшно даже подумать что бы в документации было бы написано
DM
Зачем такие имена полям давать?
Аббревиатура предметной области
AB
Если уже код написан с такими полями, то страшно даже подумать что бы в документации было бы написано
Ну, в документации написано ОФКД 😊 Какой-нибудь там "общефедеральный классификатор диагностический"