Comentează codul, nu persoana

Practica „Comentează codul, nu persoana” înseamnă să te concentrezi exclusiv pe calitatea implementării tehnice, ignorând trăsăturile personale ale autorului. Este crucială deoarece creează un mediu sigur psihologic, încurajând colaborarea deschisă și învățarea continuă. Dacă ignori acest principiu, review-ul devine personal, generând defensivitate, resentimente și o cultură toxică unde colegii evită să ceară feedback sau să admită erorile.

Cum o aplici

1. Folosește vocea pasivă sau impersonală: Evită pronumele „tu” sau „eu”. În loc de „Tu ai greșit”, scrie „Această buclă poate fi optimizată”.
2. Pune întrebări, nu comenzi: Transformă criticile directe în curiozitate constructivă. Ex: „De ce am ales această abordare?” în loc de „Aceasta este o idee proastă”.
3. Separă opinia de fapt: Diferențiază clar între erori obiective (bug-uri, violări de securitate) și preferințe de stil. Pentru stil, folosește formulări precum „Conform ghidului de stil...”.
4. Recunoaște efortul: Începe sau încheie cu aprecieri pentru aspectele bune, înainte de a aduce critici constructive.

Exemplu

Iată cum arată diferența dintre o abordare agresivă și una profesională:

Varianta greșită (atac la persoană):

„De ce ai scris codul așa de haotic? Ești neglijent cu gestionarea memoriei.”

Varianta corectă (focus pe cod):

„Alocarea manuală a memoriei aici pare redundantă, având în vedere că std::unique_ptr ar gestiona automat lifecycle-ul obiectului. Te rog să verifici dacă putem simplifica această secțiune pentru a preveni leak-uri potențiale.”

Cum verifici

• Analiza tonului: Citește comentariile ca și cum ai fi un străin care nu cunoaște contextul. Sună respectuos?
• Metrici de retragere: Monitorizează rata de respingere a pull-request-urilor din motive non-tehnice. O rată mare poate indica probleme de comunicare.
• Feedback anonim: Folosește sondaje periodice pentru a întreba echipa dacă se simte atacată personal în timpul review-urilor.

Greșeli frecvente

• Sarcasmul subtil: Glumele sau ironia sunt adesea interpretate greșit online și pot răni colegii, chiar dacă nu intenționezi acest lucru.
• Generalizările: Evită cuvinte precum „mereu”, „niciodată” sau „întotdeauna” când descrii comportamentul sau codul, deoarece sunt rareori adevărate absolut și sună acuzator.
• Ignorarea contextului: Criticarea unei decizii fără a înțelege constrângerile de timp sau cerințele afacerii din spatele ei poate părea o atitudine de superioritate.