Size: a a a

Боль Тимлида

2021 October 28

PD

Phil Delgyado in Боль Тимлида
Вернее часть стайл-гайда..
источник

АС

Альберт Степанцев... in Боль Тимлида
я вам могу привести очень простой пример
как-то я добавил к линтеру самодельное правило, запрещавшее присваивание под if

помогло
астрологи сразу объявили неделю отказа от идиотских ошибок
источник

NN

Nick Nickov in Боль Тимлида
Зачем вы тратите свое время на такую ерунду как создание того, что уже есть?
источник

АС

Альберт Степанцев... in Боль Тимлида
вы о чем?
1. Кто тратит время?
2. На что он тратит время?
источник

PD

Phil Delgyado in Боль Тимлида
А в каком линтере это есть? В багфайндерах - да, есть.
источник

ii

iv iv in Боль Тимлида
Посоны! Вы прочтите вторую половину гугловского гайда на сипипи.
источник

NN

Nick Nickov in Боль Тимлида
Ну сейчас вы говорите про code analyzer который тоже стандартизирован практически под каждый язык. Я все это поставил и даже не настраивал - стандарты отлично работают
источник

АС

Альберт Степанцев... in Боль Тимлида
Имхо вы сейчас пишете, слегка не приходя в сознание.

Нет, вы неправы. Присваивание под if - это не статический анализ. Это как раз часть стиля кода, автоматизирующаяся линтером.
источник

NN

Nick Nickov in Боль Тимлида
Ну если следовать моему примеру то phpmd еще докидываю к линтеру в CI и все ок - никаких присваиваний в if(), никакой повышенной сложности в методах и так далее
источник

NN

Nick Nickov in Боль Тимлида
То же самое для GoLang - все уже изобретено
источник

NN

Nick Nickov in Боль Тимлида
Изобретая свой стайлгайд вы увеличиваете время онбоардинга и повергаете вновь пришедших в уныние от того, что не пользуете стандарты
источник

ii

iv iv in Боль Тимлида
Уй. На это уже давно стат анализаторами проехано. Ваш уровень понятен
источник

ii

iv iv in Боль Тимлида
Тут должно быть нецензурное слово
источник

АС

Альберт Степанцев... in Боль Тимлида
а кто это делает-то?
Вы можете конкретного человека назвать из чата, кто заявил, что он "изобретает свой стайлгайд"?
источник

NN

Nick Nickov in Боль Тимлида
А вообще все это не надо, если у вас pull request не более 300 строк кода. Но для этого надо на планирование тратить не 2-3 дня, а 2-3 недели
источник

NN

Nick Nickov in Боль Тимлида
Все проблемы с долгими ревью, проверками стилей кодирования, стайлгады и прочее ушли внебытье как только стали планировать тщательнее и пошли PR 100-300 строк
источник

NN

Nick Nickov in Боль Тимлида
Какова у вас средняя толщина PR в строках за спринт у каждой команды?
источник

ii

iv iv in Боль Тимлида
За планирование реквеста в 300 строк на 3 недели - выгнал бы нах. Нужен продукт, а не мастурбация.
источник

NN

Nick Nickov in Боль Тимлида
А как вы планируете дату сдачи если вы не бьете задачу на атомы? +- пару месяцев?
источник

ii

iv iv in Боль Тимлида
У меня свой метод.
источник