Прегледи на кодове
Автор: Матиас Карлсон
Трябва да правите прегледи на кодове. Защо? Защото те повишават качеството на кода и намаляват степента на дефекти. Но не непременно поради причините, които може да си помислите.
Тъй като те може да са имали лоши преживявания с рецензии, много програмисти са склонни да отхвърлят рецензиите на кодове. Виждал съм организации, които изискват целият код да премине през официален преглед, преди да премине към производство. Често пъти архитектът или ръководителят на разработката правят този преглед, практика, която може да бъде описана като архитект, който преглежда всичко. Това е записано в ръководството за процеса на разработване на софтуер, така че програмистите трябва да спазват. Някои организации може да се нуждаят от такава твърдост и формални процеси, но много не. В повечето организации този подход не е продуктивен. Търсеният може да почувства, че е съден от комисия за условно освобождаване. Рецензентите се нуждаят както от времето за четене на кода, така и от времето, за да бъдат в крак с всички подробности на системата. Рецензентите могат бързо да станат тесни места в този процес и процесът бързо се изражда.
Вместо просто да се коригират грешки в кода, целта на прегледите на кода трябва да бъде споделяне на знания и установяване на общи насоки за кодиране. Споделянето на вашия код с други разработчици дава възможност за колективно притежание на код. Не ограничавайте техния поток, оставете всеки член на екипа да прегледа кода с останалата част от екипа. Вместо да търсите грешки, трябва да прегледате кода, опитвайки се да го научите и разберете.
Бъдете внимателни по време на прегледите на кода. Уверете се, че вашите коментари са конструктивни, а не разяждащи. Въведете различни роли в дъската за преглед, като избягвате по-старите в екипа да влияят на прегледите на кода. Примерите за роли могат да включват рецензент, фокусиран върху документацията, друг върху изключения и трета страна, търсеща функционалност. Този подход помага да се разпредели тежестта от отзиви сред членовете на екипа.
Преглеждайте кода си редовно, един ден в седмицата. Прекарайте няколко часа на дъската за преглед. Завъртайте ежеседмичните прегледи по обикновен модел. Не забравяйте също така да сменяте ролите между членовете на екипа на всяка среща за преглед. Включете начинаещи в прегледите на кодове. Може да са неопитни, но последните им познания в колежа могат да предоставят различна перспектива. Включва експерти за техния опит и знания; те ще идентифицират склонния към грешки код по-бързо и с по-голяма точност. Прегледите на кода ще текат по-лесно, ако екипът има конвенции за кодиране, които се проверяват с инструменти. По този начин форматът на кода никога няма да бъде обсъждан по време на борда за преглед на кода.
Правенето на рецензии с кодове е може би най-важният фактор за успех. Отзивите са за хората, които се разглеждат. Ако срещата за преглед е болезнена или скучна, ще бъде по-трудно да мотивирате някого. Направете го неформален преглед на кода, чиято основна цел е да се споделят знания между членовете на екипа. Оставете коментарите на snide и вместо това донесете обяд за торта или кафява чанта.
Превод: Еспартако Палма
Компилация от Espartaco Palma (@esparta).
Гордо се хоства на github