Size: a a a

2020 November 05

OS

Oleg Shevelev in Go Get A Job
Rostislav Teryaev
Про 1 пункт. Я вижу какие-то разные мнения. Часто встречаю, что "ууу, код не закомментирован - плохой". Но в чистом коде Р. Мартин наоборот грязью поливал комментарии.
Частая проблема это когда присутствуют все пункты из списка... там уже не важно есть ли комментарии...
источник

RT

Rostislav Teryaev in Go Get A Job
Oleg Shevelev
Частая проблема это когда присутствуют все пункты из списка... там уже не важно есть ли комментарии...
а у кого ревью проводилось?
источник

OS

Oleg Shevelev in Go Get A Job
Тоесть?
источник

RT

Rostislav Teryaev in Go Get A Job
уровень разработчика
источник

RT

Rostislav Teryaev in Go Get A Job
Мне просто кажется это при начале программирования на языке или в принципе такое встречается
источник

OS

Oleg Shevelev in Go Get A Job
От джунов до тимлидов.
источник

RT

Rostislav Teryaev in Go Get A Job
Ну как бы про паники везде написано. Их используют наверное те, кто в го только пришел.
Про разные файлы в пакете - это может быть у джунов
источник

RT

Rostislav Teryaev in Go Get A Job
хм, занятно
источник

OS

Oleg Shevelev in Go Get A Job
Всё очень зависит от школы в которой учился... если коллеги научили делать паники всюду - усё...
источник

OS

Oleg Shevelev in Go Get A Job
Разные файлы?
источник

RT

Rostislav Teryaev in Go Get A Job
а, там не было такого пункта? я видимо придумал) Ну разбивать код в одном пакете на разные файлы
источник

RT

Rostislav Teryaev in Go Get A Job
Oleg Shevelev
Всё очень зависит от школы в которой учился... если коллеги научили делать паники всюду - усё...
блин странно всё же. Коллеги коллегами. Но не в вакууме же все существуем. Есть интернет, где любая статья\видео говорит "не паникуй".
источник

OS

Oleg Shevelev in Go Get A Job
Разбивать код - это может быть очень даже не плохо. Пример, есть база данных, есть User, Company, Tower сущьности. Каждая занимает примерно с экран... в одном файле им тесно, плюс ещё парочку функций к ним добавить к каждой... и вуаля - без комментариев что это за User.K8 уже и не разобраться...
источник

OS

Oleg Shevelev in Go Get A Job
Rostislav Teryaev
блин странно всё же. Коллеги коллегами. Но не в вакууме же все существуем. Есть интернет, где любая статья\видео говорит "не паникуй".
Так такие потом тоже других учат. Дескать "пишу комменты только там где надо"... а где надо не знает:)
источник

OS

Oleg Shevelev in Go Get A Job
После того как мне сделали часовые ревью кода в первый месяц в Авито... я пересмотрел то как пишу.
источник

RT

Rostislav Teryaev in Go Get A Job
Oleg Shevelev
Разбивать код - это может быть очень даже не плохо. Пример, есть база данных, есть User, Company, Tower сущьности. Каждая занимает примерно с экран... в одном файле им тесно, плюс ещё парочку функций к ним добавить к каждой... и вуаля - без комментариев что это за User.K8 уже и не разобраться...
это и имел в виду. Не могу читать чужой код, не отрефакторив его, не разнеся разные сущности с методами по разным файлам
источник

OS

Oleg Shevelev in Go Get A Job
Критерий был простой. Вот сидит программист, смотрит в код и не понимает зачем этот кусок нужен. Ему приходится напрягаться.
источник
2020 November 06

G

GW in Go Get A Job
Rostislav Teryaev
блин странно всё же. Коллеги коллегами. Но не в вакууме же все существуем. Есть интернет, где любая статья\видео говорит "не паникуй".
Да ладно? Доступа к базе нет, но ты не паникуй, сеть упала, но ты не паникуй и т. д.
источник

G

GW in Go Get A Job
Rostislav Teryaev
а, там не было такого пункта? я видимо придумал) Ну разбивать код в одном пакете на разные файлы
Это нормально кстати.
источник

G

GW in Go Get A Job
Oleg Shevelev
Критерий был простой. Вот сидит программист, смотрит в код и не понимает зачем этот кусок нужен. Ему приходится напрягаться.
Угу, а я пишу тул на заказ, и хочу что-бы только я его пилил, периодически, за вознаграждение. Зачем мне в коде комментарии.

Как с индусским кодом. Никто ведь не знает почему они  так пишут код.
источник