Информация об изменениях

Сообщение Re[4]: Как решать конфликтные ситуации при code review? от 30.01.2024 20:13

Изменено 30.01.2024 20:17 Aleksei_Lekomtsev

Re[4]: Как решать конфликтные ситуации при code review?
Здравствуйте, _ABC_, Вы писали:

_AB>Здравствуйте, Aleksei_Lekomtsev, Вы писали:


A_L>>Я умею работать в команде и писать комментарии и люблю слушать конструктивную критику по существу

_AB>А что "по существу", а что нет — решаешь в команде только ты, да?
С чего ты это решил?)


_AB>А то, что твой код понять сложно из-за того, что название переменных неудачные и вводят в заблуждение, например — это не существенно, да?


Почему неудачные? Может кто-то читает плохо и вводится в заблуждение? Бесполезно тут что-то тебе писать. Много промышленного кода написано в разных командах,
все кто хотел понять — понимали. Специально не придумывал названия переменных чтобы никто не понял

_AB>И что ты поленился написать комментарий к сложной, неочевидной логике — это тоже на работу приложения не влияет, значит не существенно?

Почему сложная неочевидная логика? Почему поленился? Обычный код, никакого rocket science

_AB>А то, что ты спустя полгода сам не поймёшь, что там к чему без комментария и с хреновыми названиями переменных, а пофиксить надо срочно, клиенты теряют деньги из-за изменения обстоятельств, например, — это ерунда, "не влияет на работу приложения". Я всё правильно понял?

Почему спустя полгода я не пойму? Почему хреновые названия?

Извини, ты странный и очень аггресивный. Навыдумывал кучу всего, что к реальности не имеет никакого отношения. Дальше обвинил в этом абсолютно незнакомого человека и навешал на некого кучу ярлыков.
Ты как будто живешь в своем собственном иллюзорном мире, где события происходят по одному тебе известному сценарию
Re[4]: Как решать конфликтные ситуации при code review?
Здравствуйте, _ABC_, Вы писали:

_AB>Здравствуйте, Aleksei_Lekomtsev, Вы писали:


A_L>>Я умею работать в команде и писать комментарии и люблю слушать конструктивную критику по существу

_AB>А что "по существу", а что нет — решаешь в команде только ты, да?
С чего ты это решил?)


_AB>А то, что твой код понять сложно из-за того, что название переменных неудачные и вводят в заблуждение, например — это не существенно, да?


Почему неудачные? Может кто-то читает плохо и вводится в заблуждение? Бесполезно тут что-то тебе писать. Много промышленного кода написано в разных командах,
все кто хотел понять — понимали. Специально не придумывал названия переменных чтобы никто не понял

_AB>И что ты поленился написать комментарий к сложной, неочевидной логике — это тоже на работу приложения не влияет, значит не существенно?

Почему сложная неочевидная логика? Почему поленился? Обычный код, никакого rocket science

_AB>А то, что ты спустя полгода сам не поймёшь, что там к чему без комментария и с хреновыми названиями переменных, а пофиксить надо срочно, клиенты теряют деньги из-за изменения обстоятельств, например, — это ерунда, "не влияет на работу приложения". Я всё правильно понял?

Почему спустя полгода я не пойму? Почему хреновые названия?

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