За и против code review

Здравейте, споделете мнение за code review, птактикува ли се във вашата фирма, има ли смисъл, кога? В какви фирми се прилага големи малки? В какви проекти, при какви екипи(големи малки)?

Само може да има полза от код-ревю. Както умните хора са отбелязали - всичкия код вони ужасяващо, само твоя - малко по-добре. Като се разискват нещата се уточняват храни, тоест практики, които миришат приблизително приемливо за всички.

скалируемост и интерактивност благодарение на foo базираният си бекенд и захранен от AJAX фронтенд

че как да няма полза бе.. иначе ако всеки си пише както си знае

Скучно ти е? Ела да се посмееш: [линк]

github + rss. И докато си пиеш сутрешното кафе, наместо да четеш фишки и днес.бг, преглеждаш какви са ги надробили. След това коментираш (има коментари по линия). И останалите теб така, докато се изгради дълбока и чистосърдечна омраза един към друг. Но поне ще можеш да се оправяш в "чуждо" като се наложи...

— У вас, на Земята, как определяте кой пред кого колко пъти да клекне?
— Така, на око…
— Диваци!

Има полза винаги, но голямата полза е само когато човекът, който прави ревютата познава областта и кода.

За приложимост - прилагал съм го в различни компании - става навсякъде и е яко полезно! ... ако се прави като хората. Иначе е загуба на време и грешни очаквания

Дори да няма качествени проверяващи кода, това че изобщо го има включва опцията "Кумова срама" за хора които обичат анонимно да акат :) като някой изрично ще гледа - ще се старае да пази хигиена :)

Revolution, the only solution,
The armed response of an entire nation
System of a Down P.L.U.C.K.

Питах, защото до сега не съм участвал в такова начинание, и не знам дали има голям смисъл в конкретния случай т.е. ние сме 3ма човека(2ма кодери и шефа, който също коди + 1000 др неща). За момента работим по модули, които нямат много общо, като код интеръкшън. Шефа е наясно с всичко, но примерно аз не знам (в детайли) с какво се занимава колегата , нито пък той с какво аз. Иначе да признавам, че от код ревю би трябвало да се научава доста, особено ако ревизиращите са мн опитни, а колкото до намразването, зависи от подхода на ревизиращите.

Аз съм за!
И съм за coding rules, там където пипат повече от 1-2 души. Не, че това решава проблемите при работата в екип, но ... пак е нещо!

Dum loquo hora fugit

Неформалност е важно тука, искаш да знаеш за некой код - питаш. Какво те ебе колко са меки пишките на ревизиращите - ти си цъкаш твойто.

скалируемост и интерактивност благодарение на foo базираният си бекенд и захранен от AJAX фронтенд