Sert bir code review geri bildirimini savunmaya geçmeden nasıl karşılarım?

Kısa cevap: Bir PR’a kırk yorum, genelde birinin sizi ciddiye aldığının ve gelişiminize yatırım yaptığının işaretidir — saldırının değil.

Kısa cevap

Her birine itiraz etme dürtüsü tamamen normal, ama o dürtüyle hareket ederseniz size pahalıya patlar. Duygusal tepkiyi teknik cevaptan ayırın.

Neden

1. Aynı gün cevap yazmayın. Kırk yorumu okuyun, sekmeyi kapatın, birkaç saat sonra dönün. Savunma refleksi ilk dakikalarda zirve yapar ve sonra söner. İnsanın pişman olduğu yorumlar, o ilk öfkeyle yazılan ilk taslaklardır.
2. Egoyu koddan ayırın. Review kodla ilgilidir, sizinle değil. “Bu yanlış” demek, kodda bir sorun var demektir; kötü bir mühendis olduğunuz anlamına gelmez. Staff engineer’lar tam da işinizi ciddiye aldıkları için didikler.

Ne yapmalı

1. Yorumları kovalara ayırın. Nesnel bug’lar (sadece düzelt), stil/konvansiyon (düzelt, ucuz), gerçek anlaşmazlıklar (tartış) ve sorular (cevapla). Kırkta otuz beşi çoğunlukla “sadece düzelt” kovasına düşer; aslında tartışmanız gereken bir avuç yorum kalır.
2. Katılmadıklarınızda savunmak yerine sorun. “Burada endişelendiğin failure mode ne tam olarak?” cümlesi bir çekişmeyi sohbete çevirir. Çoğu zaman bilmediğiniz bir bağlam öğrenirsiniz; bazen de karşı taraf sizinkini öğrenir. Her hâlükârda yorum savaşından ucuzdur.
3. Savaşlarınızı seçin — ve sesli yapın. Ortada üç-dört gerçek anlaşmazlık varsa, 10 dakikalık bir görüşme, kırk yorumluk yazışmanın çözemediğini çözer. Yazılı metin, iki tarafta da savunmacılığı büyütür.
4. İyi niyet varsayın, ama zamanla doğrulayın. Çoğu dolaysız reviewer kötü niyetli değil, sadece kısa konuşur. Ama geri bildirim gerçekten saygısızsa (teknik değil, alaycıysa), bu ayrı bir konudur; onunla ya da yöneticinizle konuşulur. Dolaysızlık normaldir, aşağılama değil.
5. Teşekkür edin, hem de içtenlikle. Kısa bir “teşekkürler, bu gerçek bir bug’ı yakaladı ve X konusunda bir şey öğrendim” size hiçbir şeye mal olmaz ama ilişkiyi değiştirir. O kişi kodunuza bir saatini bedavaya harcadı.

Sonuç: Ben olsam kolay %90’ı sessizce düzeltir, gerçek anlaşmazlıkları tek bir kısa görüşmede toplar ve o reviewer’ı o hafta sahip olduğum en değerli bedava öğretmen olarak görürdüm. Kariyerimin başında en çok canımı yakan review’lar bana en çok şeyi öğretenlerdi; beni sert eleştirmeyi bırakanlar ise benden umudu kesmiş olanlardı.