AK
Что бы его поправить:)
Чтобы что?
Size: a a a
2020 October 13
AK
Есть код. Он соответсвует спецификации, он покрыт тестами, он проходит линтеры. Зачем его отдавать на ревью? Чтобы что?
AK
Чтобы синьор помидор архитект потешил свое самолюбие докопавшись до какой то незначительной мелочи? 🙂
OS
Что бы удостовериться что ты ничего не забыл, что он реально покрыт тестами, что он реально проходит линтеры и что он реально соответствует спецификации. И к этому добавим что задача на самом деле норм, а не деструктивная.
w
Есть код. Он соответсвует спецификации, он покрыт тестами, он проходит линтеры. Зачем его отдавать на ревью? Чтобы что?
Чтобы проверить а не появилось ли какого нибудь узкого места?
AK
Чтобы проверить а не появилось ли какого нибудь узкого места?
ВЫ понятие узкого места из TOC применяете или вы что-то свое вкладываете в него?
DU
ВЫ понятие узкого места из TOC применяете или вы что-то свое вкладываете в него?
тут скорее про бенчмарк, а не про ТОС
A
Покажи идеальный код:)
это обычно код тимлида)))
w
бывает напишет программист такое
грубо говоря...
func existsSomeEntity() bool {
var a []Entity
db.Get(&a, `SELECT * FROM entities WHERE foo = 'bar' ORDER BY id`)
return len(a) > 0
}грубо говоря...
w
а потом думаешь, что за пиздец в базе творится?
DU
это обычно код тимлида)))
обычно лиды пишут хуже инженеров)
АБ
бывает напишет программист такое
грубо говоря...
func existsSomeEntity() bool {
var a []Entity
db.Get(&a, `SELECT * FROM entities WHERE foo = 'bar' ORDER BY id`)
return len(a) > 0
}грубо говоря...
и не лень было пример генерировать?)
A
обычно лиды пишут хуже инженеров)
зато пока не пришёл новый тим лид, код текущего - идеал для подражания)))
OS
а потом думаешь, что за пиздец в базе творится?
По условиям задачи команда "сильная" и такого кода там видимо быть не должно:)
w
и не лень было пример генерировать?)
к чему вопрос?
w
По условиям задачи команда "сильная" и такого кода там видимо быть не должно:)
а на сколько сильная?)
A
По условиям задачи команда "сильная" и такого кода там видимо быть не должно:)
но так как review нет , то никто никогда не узнает что там есть)))
AK
зато пока не пришёл новый тим лид, код текущего - идеал для подражания)))
А новый умудренный опытом тимлид конечно же в первую очередь предложит все выкинуть и переписать с нуля 😂
A
А новый умудренный опытом тимлид конечно же в первую очередь предложит все выкинуть и переписать с нуля 😂
вот именно)))
поработал сам, дай сделать тоже самое другим)))
поработал сам, дай сделать тоже самое другим)))
OS
а на сколько сильная?)
По условиям холивара на столько что бы код ревью добавляла к задачи минимум неделю и что бы код ревью было бы не нужно:):):)