Только что на работе

[Gross]

Написал я WCF сервис, и продолжаю имплементировать новые методы - времени не хватает вот мне и дали в помощь одного долгожителя компании, лет 7 что ли.
Заимплементировал он новый метод и добавил его в интерфейс, дал мне на код ревью(через кодколаборатор). Ну я ему и говорю, переименуй, плиз: из(к примеру) MethodDescription в MethodNameDescription, а то это имя может mislead Он в ответ, не будуду переименовывать, если ты хош, ты и переименовывай. Ну я принял его измения и запросил перевести на меня task, что бы я мог зачекинить под нее.

Да, и, он делает код ревью на уже зачекиненом коде. Попросил делать ревью на не зачекиненом коде...

Это вообще нормально, или я в чем то не прав? Потому что смотрю, что мужик не очень довольный, как и я впрочем

[Medium-rare]

Довольно гнусное нарушение процедуры, если чек-инит без ревью. Плохо, что система разрешает это делать.

На игнорирование замечаний на код-ревью можно двояко посмотреть, с одной стороны, тоже вроде как нехорошо, с другой, ведь не запрещено, unless спросить мнение у начальства или группы.

[M. Ridcully]

А кроме переименования метода у вас какие-то замечания были, по существу?

Ситуации разные бывают, но я сильно не люблю, если мне делают замечания _только_ по стилю. То есть, если ревью: "вот тут переменная неинициализированная, и кстати, может её из X в Y переименовать?" - это нормально. А если всю ревью сполошные вопросы стиля и субьективизм, то я потом этим людям на ревью просто не посылаю - толку от них как правило всё равно никакого.

Когда сам пишу ревью, то если высказываю пожелания по стилю, то это именно пожелания - заранее жму "ship it", чтобы человек мог чекинить, независимо от того, согласен с моими пожеланиями по стилю или нет.

Так что в-принципе подход "хочешь преиеновать - переименуй сам" не лишен логики.

[Gross]

К стилю я не цепляюсь - кто как хочет так и назывыет - тут проблема в том что название не соответствует тому, что делает метод.

[АццкоМото]

Gross, на ваш вопрос никто в принципе не сможет ответить не зная политик компании и ее культуры. Как пример, я много лет проработал в компании, где ревью незачекиненного кода нельзя было устроить в принципе. А мнения автора вообще никто не спрашивал (вернее, могли спросить, но он не имел права вякать, пока его не спросят). При этом и мнение ревьюера - чисто совещательное для модератора. В общем, зависит все - от начала и до конца, нет никаких общеиндустриальных практик, на основании которых можно рассудить вашу ситуацию не зная компанию изнутри

[M. Ridcully]

тут проблема в том что название не соответствует тому, что делает метод.

Ну если название совсем не в кассу - типа (перефразируя), метод NukeLiverpool() шлёт ракеты на Манчестер, то тот человек неадекватно себя повёл.

[Fortinbras]

К стилю я не цепляюсь - кто как хочет так и назывыет - тут проблема в том что название не соответствует тому, что делает метод.

Оу.

[7pm]

Написал я WCF сервис, и продолжаю имплементировать новые методы - времени не хватает вот мне и дали в помощь одного долгожителя компании, лет 7 что ли.
Заимплементировал он новый метод и добавил его в интерфейс, дал мне на код ревью(через кодколаборатор). Ну я ему и говорю, переименуй, плиз: из(к примеру) MethodDescription в MethodNameDescription, а то это имя может mislead Он в ответ, не будуду переименовывать, если ты хош, ты и переименовывай. Ну я принял его измения и запросил перевести на меня task, что бы я мог зачекинить под нее.

Да, и, он делает код ревью на уже зачекиненом коде. Попросил делать ревью на не зачекиненом коде...

Это вообще нормально, или я в чем то не прав? Потому что смотрю, что мужик не очень довольный, как и я впрочем

Позиция того девелопера такова: у вас не хватает времени, он вам помог, написал код, и, возможно, ему надо вернуться разгребать собственные проекты.
Вы ему начинаете ....ть моск c какими-то там названиями. Заметьте, это не моя позиция, это скорее всего его позиция.
Вам придется искать компромис. Тем более, тот долгожитель, скорее всего, генерит быстро много некрасивого кода, но "оно" работает, начальство его ценит.

[Gross]

Позиция того девелопера такова: у вас не хватает времени, он вам помог, написал код, и, возможно, ему надо вернуться разгребать собственные проекты.
Вы ему начинаете ....ть моск c какими-то там названиями. Заметьте, это не моя позиция, это скорее всего его позиция.
Вам придется искать компромис. Тем более, тот долгожитель, скорее всего, генерит быстро много некрасивого кода, но "оно" работает, начальство его ценит.

Да мне, собственно, наплевать на позицию того девелопера, для меня важно, что я не перешел какие то не гласные границы. Я на себя эту задачу перевел и когда дойдет до нее очередь я ее до делаю. Будут спрашивать почему еще не в статусе "закончено" расскажу как есть.

Зы а мужик тот сейчас "помогает" всем в группе, он единственный у кого нет своего фронта работ в проекте.

[Poryadok]

А я в таких случаях иду и подробно обьясняю как кривое название метода за годы многих собьет с толку и убьет время, а переименовывать уже будет поздно, потому что разный клиентский код уже будет содержать текущее название.
Ни разу не было такого, чтобы не получилось как-то утрясти вопрос. Правда, у нас коллектив очень вменяемый.

[IContentProvider]

Да мне, собственно, наплевать на позицию того девелопера, для меня важно, что я не перешел какие то не гласные границы. Я на себя эту задачу перевел и когда дойдет до нее очередь я ее до делаю. Будут спрашивать почему еще не в статусе "закончено" расскажу как есть.

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

[Gross]

Да мне, собственно, наплевать на позицию того девелопера, для меня важно, что я не перешел какие то не гласные границы. Я на себя эту задачу перевел и когда дойдет до нее очередь я ее до делаю. Будут спрашивать почему еще не в статусе "закончено" расскажу как есть.

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

вы когда нибудь пытались "потратить 2 мин" когда куча файлов в чекауте, включая и те в которых нужно переименовать метод да еще есть очень настойчевое пожелание начальства чекинить код только относящийся к одному дефекту/задаче в Rally, т.е. в одном чекине не должно быть кода относящимся к нескольким дефектам/задачам? до конца очередной итерации успею сделать. всему свое время.

[Интеррапт]

Да мне, собственно, наплевать на позицию того девелопера, для меня важно, что я не перешел какие то не гласные границы. Я на себя эту задачу перевел и когда дойдет до нее очередь я ее до делаю. Будут спрашивать почему еще не в статусе "закончено" расскажу как есть.

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

Во блин, никогда на слух не попадалось, что тикеты (для баг-тракинга) называются билетами, вот что значит к американизмам в русской речи привык.

[Gross]

А я в таких случаях иду и подробно обьясняю как кривое название метода за годы многих собьет с толку и убьет время, а переименовывать уже будет поздно, потому что разный клиентский код уже будет содержать текущее название.
Ни разу не было такого, чтобы не получилось как-то утрясти вопрос. Правда, у нас коллектив очень вменяемый.

так объяснял, лично подходил, но бестолку - глаза лубяные и : feel free to rename to whatever you want

[Интеррапт]

А я в таких случаях иду и подробно обьясняю как кривое название метода за годы многих собьет с толку и убьет время, а переименовывать уже будет поздно, потому что разный клиентский код уже будет содержать текущее название.
Ни разу не было такого, чтобы не получилось как-то утрясти вопрос. Правда, у нас коллектив очень вменяемый.

так объяснял, лично подходил, но бестолку - глаза лубяные и : feel free to rename to whatever you want

А про себя он подумал - вот еще, всякие русскоговорящие иммигранты меня будут учить, как правильно на английском методы называть

[Zuk]

а какая проблема переименовать этот метод? за время, что топик открывали управились бы..

[Gross]

а какая проблема переименовать этот метод? за время, что топик открывали управились бы..

узнаю "Привет" Оказывается у меня уже проблема переименовать метод

[Zuk]

а какая проблема переименовать этот метод? за время, что топик открывали управились бы..

узнаю "Привет" Оказывается у меня уже проблема переименовать метод

Я спросила я б переименовала, если меня это волнует. Ходить и настаивать, чтоб сосед что то сделал - утомительно.
Но у вас, очевидно, есть резон так не делать - интересно ж какой..

[Gross]

а какая проблема переименовать этот метод? за время, что топик открывали управились бы..

узнаю "Привет" Оказывается у меня уже проблема переименовать метод

Я спросила я б переименовала, если меня это волнует. Ходить и настаивать, чтоб сосед что то сделал - утомительно.
Но у вас, очевидно, есть резон так не делать - интересно ж какой..

просто интересно, вы все собщения в топике прочитали?
на всякий случай - задачу я перевел на себя и она ждет пока я до нее доберусь и переименую метод.

[Gross]

ну что за напасть?... добрался я до кусков кода зачекиненые этим мужиком - вместо пробелов везде табуляции. и что делать? (я конечно же уже все отформатировал как нужно, но он же еще будет чекинить...)

[Интеррапт]

ну что за напасть?... добрался я до кусков кода зачекиненые этим мужиком - вместо пробелов везде табуляции. и что делать? (я конечно же уже все отформатировал как нужно, но он же еще будет чекинить...)

А у вас code style в компании что-ли не практикуется?

[fruit6]

переформатировать чужой код -- верный способ нажить врага без причины.

[Berlaga]

переформатировать чужой код -- верный способ нажить врага без причины.

С другой стороны, табы вместо пробелов везде -- достаточно веская причина, чтобы переформатировать. Можно и без наживания врагов, просто поговорить с менеджером и написать короткий стайл гайдлайн.

[Gross]

ну что за напасть?... добрался я до кусков кода зачекиненые этим мужиком - вместо пробелов везде табуляции. и что делать? (я конечно же уже все отформатировал как нужно, но он же еще будет чекинить...)

А у вас code style в компании что-ли не практикуется?

практикуется, есть стайл гайд, кастомизированый файл настроек для styleCop, но судя по всему он им не пользуется...

[fruit6]

переформатировать чужой код -- верный способ нажить врага без причины.

С другой стороны, табы вместо пробелов везде -- достаточно веская причина, чтобы переформатировать. Можно и без наживания врагов, просто поговорить с менеджером и написать короткий стайл гайдлайн.

выше было упомянуто что автор топика не может найти общего языка с автором кода.

если автор кода сопротивляется, то нужно проинформировать линейного менеджера чтобы сохранить анонимность, но не лезть в код.