Этикет CodeReview

[trud]

Цитата:

Сообщение от mazzy

Можем.
пока предпочитаю потрыднеть на форуме. Сблизить подходы, так сказать.

оффтоп, но статья в тему
https://habrahabr.ru/post/342244/

Цитата:

Я позволил своему эго повлиять на ревью. Я потратил целый год, восстанавливая эту старую систему до работоспособного состояния. Внезапно появляется новый человек, который с ней возится, и она не хочет всерьёз принимать мои замечания? Я воспринял это как оскорбление, но такое отношение стало контрпродуктивным.

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

[mazzy]

они такие же. просто у них свой этикет выработался в полном соответствии с традицией политкорректности.
https://habrahabr.ru/post/340550/
https://habrahabr.ru/post/342244/

[MikeR]

Цитата:

Сообщение от mazzy

они такие же. просто у них свой этикет выработался в полном соответствии с традицией политкорректности.

Позволь с тобой не во всем согласиться, мне думается у меня именно в этой области побольше опыта. На самом деле очень много зависит от того, кто делает code review и кому ты это делаешь,
Если это белый человек. Допущу себе немного фобии ), то все будет как ты сказал, очень обтекаемо, гибко допустимо и без скандала, хотя тоже есть исключения.
Но очень часто и почти что то всегда в силу прошедших этапов разного рода арабских весен, попадаешь на пакистанца\индуса. Вот эти ребята во первых очень часто говорят с акцентом, что хер разберешь, что имеется ввиду, во вторых и них нет что называется вводной про погоду и все такое, начинаю разговор не вежливо и переходят прямо к делу. Опять же в моем опыте этот народ очень четко понимает, что их понаехало достаточно много и конкуренция среди них просто огромная, именно поэтому они чуют за версту что если им высказать про не компетентность, то это разрушит их очень шаткие планы на работу и они что называется из кожи лезут доказать обратное, готовы к любым скандалам и интригам.
Это мое ИМХО.

[belugin]

Цитата:

Сообщение от trud

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

Но по приезде в США бывшая переводчица обнаружила, что ее английский совершенно не годился для местной разговорной речи, и сильно волновалась. Например, стараясь предостеречь мальчика от простуды, она, обращаясь к нему, буквально переводила свои русские повеления и запреты: «Нельзя пить газировку со льдом» выходило у нее не как I don’t think you should drink soda with ice, а как It is not allowed to drink soda with ice; «не надо открывать окно» получалось не как Don’t open the window, а как It is not necessary to open the window. На английском эти фразы звучали категорично, грубо, и однажды американец шутливо спросил свою русскую половину, не служил ли кто-нибудь из ее родителей в полиции.

[mazzy]

про политкорректность
http://coub.com/view/zkvu5

[greench]

Цитата:

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

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

[MikeR]

Когда я изучал язык, то проходил что такое small talk и could you..
А у пакистано\индусов это между прочим второй гос язык.
У славян английский просто отвратительно преподается в школе и институте.

[mazzy]

Цитата:

Сообщение от MikeR

А у пакистано\индусов это между прочим второй гос язык.
У славян английский просто отвратительно преподается в школе и институте.

Потому что у славян - долго разивался немецкий. а раньше французский.
так исторически сложилось. и у славян. и у индусов )))

[MikeR]

В моей истории Индия была более британской колонией, чем французской.
Французы пошли в Африку и северную Америку. Наполеон так далеко тоже не захаживал.
Пока не хочу добавлять шовинизма, но мне думается, что я понял тайную индийскую душу, почему они такие одиозные, зачем написали бхагават гиту и почему увлекаются камасутрой.
Единственной, на мой взгляд культурной народностью я бы выделил сикхов. Они и говорят по-английски получше, и общаются культурнее.
Но они не составляют основной иммиграционный поток на Запад и Америку.

[greench]

Цитата:

А у пакистано\индусов это между прочим второй гос язык.

Как по мне это влияет только на правильность грамматики. Вежливость больше ментальная штука. Если в общество большей своей частью живет правилами "клюнь ближнего, нагадь на нижнего" как в индии, или "вежливый, значит слабый" как у нас, то хоть сколько вторых государственных не вводи, а модель построения фраз останется неизменной.

[mazzy]

Цитата:

Сообщение от greench

Вежливость больше ментальная штука.

Скорее просто внешние условия и плотность населения.

См. эксперимент "Вселенная-25" - мышиный рай.
https://ru.wikipedia.org/wiki/%D0%9A...�_мышами

Ментальность у всех одинаковая. И хорошо приспосабливающаяся.

[Stitch_MS]

По работе с тайцами приходится общаться. Вот где вежливость. Если тайца спросить что-то вроде "Вам надо вот это вот так. Я правильно понял?", то даже если ты что-то неправильно понял, он не сможет сказать "Нет, Вы неправильно поняли, я имел в виду...". Для тайца сказать такое всё-равно что оскорбить, поэтому они часто говорят "Да" и запутывают собеседника-европейца. Но, справедливости ради, со временем они таки привыкают к европейскому стилю общения, хотя и не сразу.

[mazzy]

Цитата:

Сообщение от Stitch_MS

и запутывают собеседника-европейца

Китайцы также. Причем много переспрашивают, вежливо спрашивают твое мнение.
Но по факту выполненных работ и сроков возникает полное ощущение, что троллят.

[Vals]

Мы партнёры этой штуки https://www.checkmarx.com/

Когда Чекмаркс анализирует код, то прямо указывает на ошибку и рисует вектор атаки Программеры первые два, три месяца сопротивляются

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

[ax_mct]

Цитата:

Сообщение от Vals

Мы партнёры этой штуки https://www.checkmarx.com/

Когда Чекмаркс анализирует код, то прямо указывает на ошибку и рисует вектор атаки Программеры первые два, три месяца сопротивляются

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

Вряд ли он AX кушает. Но что интересно SAP указан как иконка.
Не дай бог такой интрумент клиентам. Если тот же Best Practices в AX на полную включить то у них инфаркт будет.

[oip]

Цитата:

Сообщение от ax_mct

Не дай бог такой интрумент клиентам. Если тот же Best Practices в AX на полную включить то у них инфаркт будет.

В AX еще есть LCS Customization analysis, иногда вполне полезные вещи находит, но если клиент не особо вникая в суть увидит "ошибки" вида "нарушение принципов проектирования" или "Целостность данных"...

[Vals]

Цитата:

Сообщение от ax_mct

Вряд ли он AX кушает. Но что интересно SAP указан как иконка.
Не дай бог такой интрумент клиентам. Если тот же Best Practices в AX на полную включить то у них инфаркт будет.

AX нет, а SAP в части Java
Для SAP есть другая замечательная штуковина - Panaya

С точки зрения проверки кода на C# Java, Ruby и проч - такой инструмент проверки кода - единственная возможность закрыть изветные калитки от несанкционированного доступа.
А проекты в 1-10 млн строк глазками ну никак не просмотреть, а если ещё и решизы каждый час ...

[greench]

Цитата:

но если клиент не особо вникая в суть

если клиент хочет за что-то зацепиться, он найдет от чего поднять панику. Это как в анекдоте: "..было бы за что, вообще убил бы".

[Ivanhoe]

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

Объясняли клиенту что с этим делать и как жить

[Vals]

Цитата:

Сообщение от Ivanhoe

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

Объясняли клиенту что с этим делать и как жить

угу, половина замечаний по стандартной функциональности