forumKonu aç

Kod incelemesi kültürü nasıl kurulur? On yıllık gözlemlerim

RRıdvan Y***18 gün önce·48 mesaj·17,1 B görüntülenme#ekip#kalite#süreç
RRıdvan Y***Uzman
Görev
Yazılım takım lideri
Katılım
Eyl 2023
Mesaj
196

Doki · Arayüz tasarımı · 2023

#1

Uzun süredir takım liderliği yapıyorum, acele etmeden yazayım çünkü bu konuda en çok yapılan hata aceleye getirmek.

Kod incelemesi denetim değil, eğitim aracıdır. Bu cümleyi kabul etmeyen ekiplerde süreç her zaman aynı yere gidiyor: kıdemli kişi hata avlar, genç kişi savunmaya geçer, incelemeler yavaşlar, sonunda kimse bakmadan onaylar.

İşe yarayan birkaç kural biriktirdim, paylaşayım.

Küçük parçalar halinde gönderin. Beş yüz satırlık bir değişikliği kimse gerçekten okuyamaz, herkes "iyi görünüyor" yazar. İki yüz satırın altında kalan değişikliklerde bulunan sorun sayısı belirgin şekilde artıyor.

Yorumu kişiye değil koda yazın. "Bunu neden böyle yaptın" yerine "burada şu durumda ne olur" diye sorun. Aynı bilgi, bambaşka bir konuşma.

Tarz tartışmasını otomatikleştirin. Girinti, tırnak, isimlendirme gibi konular araçla halledilmeli. İnsan zamanı bunlara harcanırsa, gerçek sorunlar gözden kaçıyor.

CCerenKatılımcı
Görev
Test uzmanı
Katılım
Mar 2024
Mesaj
178
En çok yardımcı olan#2

Test tarafından katılıyorum ama bir şey soracağım: incelemede ne aradığınız yazılı mı?

Sorumun sebebi şu: çoğu ekipte kod incelemesi var, ama neye bakılacağı kişiye göre değişiyor. Biri isimlendirmeye bakıyor, biri hata yönetimine, biri hiçbir şeye. Sonra "inceledik" deniyor ve aynı hatalar üretime çıkıyor.

Bizde işe yarayan şey kısa bir kontrol listesi oldu. Uzun değil, beş madde: hata durumları ele alınmış mı, sınır değerler düşünülmüş mü, geriye dönük uyumluluk bozuluyor mu, test yazılmış mı, kayıt ve izleme yeterli mi.

Ölçtüm, tahmin etmiyorum: bu liste konduktan sonraki çeyrekte üretime çıkan hata sayımız belirgin düştü. Listenin sihri yok, sadece herkesin aynı şeye bakmasını sağlıyor.

KKayahanKatılımcı
Görev
Full-stack
Katılım
Haz 2024
Mesaj
118
#3

küçük parça kuralına katılıyorum ama pratikte zor. bazen değişiklik doğası gereği büyük oluyor.

bizde çözüm şu oldu: büyük değişikliği göndermeden önce yaklaşımı tek sayfada yazıp paylaşıyoruz. beş dakikalık okuma, bir saatlik inceleme tartışmasını önlüyor.

yanlış yaklaşımı beş yüz satır yazdıktan sonra duymak en kötüsü. o noktada kimse geri dönmek istemiyor ve kötü karar üretime çıkıyor.

AAhmetYeni üye
Görev
Öğrenci · yazılım
Yapı
aile şirketi
Katılım
Oca 2025
Mesaj
48
#4

Yeni başlayan biri olarak bir şey söyleyebilir miyim?

Benim için en zor kısım, yorumu kişisel algılamamak değil. Anlamadığım bir yorum geldiğinde tekrar sormaya çekiniyorum, aptal görüneceğimi düşünüyorum. Sonra tahmin ederek düzeltiyorum ve yanlış düzeltiyorum.

Bilmiyorum bu sadece bende mi ama belki başkalarında da vardır.

RRıdvan Y***Uzman
Görev
Yazılım takım lideri
Katılım
Eyl 2023
Mesaj
196

Doki · Arayüz tasarımı · 2023

#5

Sadece sizde değil, çok yaygın ve düzeltmesi bizim işimiz, sizin değil.

Bizde iki şey işe yaradı. Birincisi, yorum yazarken gerekçeyi de yazmak. "Burayı değiştir" yerine "şu durumda hata verir, bu yüzden şöyle olması daha iyi" demek. Gerekçe varsa soru sorma ihtiyacı azalıyor.

İkincisi, iki turdan fazla yazışma olduğunda konuşmayı sohbete taşımak. On beş dakikalık bir görüşme, on beş yorumdan hızlı ve çok daha az yıpratıcı.

Bir de şunu söyleyeyim: aptalca soru yok. Sizin sorduğunuz soruyu ekipteki üç kişi daha merak ediyor ama sormuyor. Sorduğunuzda dördüne birden iyilik yapmış oluyorsunuz.

SSerhatKatılımcı
Görev
Go geliştirici
Katılım
May 2024
Mesaj
88
#6

Tarz tartışmasını araca bırakmak tek başına en büyük kazanç. Bir haftada kurulur, yıllarca zaman kazandırır.

SSinemUzman
Görev
Proje yöneticisi
Katılım
Eki 2023
Mesaj
176
#7

Proje yönetimi tarafından bir uyarı: inceleme süresini plana yazın.

Bizde şöyle oldu, uzun süre incelemeyi işin bir parçası saymadık, planda sadece geliştirme süresi vardı. Sonuç: her inceleme "işi geciktiren şey" olarak görüldü ve göstermelik yapıldı.

Plana yazdığımızdan beri kimse incelemeyi baltalamıyor, çünkü artık geciktiren bir şey değil, planın kendisi.

GGürkanKatılımcı
Görev
Enerji sektörü
Yapı
butik ajans
Katılım
Kas 2023
Mesaj
94
#8

Aynısını yaşadım.

AAyşegülKatılımcı
Görev
Butik otel
Yapı
imalathane
Katılım
Ağu 2024
Mesaj
86
#9

tam olarak böyle... işte bunu baştan yazılı hâle getirmezsneiz sonra tartışma çıkıyor.

not olarak bırakayım, lazım olur.

TTolga Y***Uzman
Görev
İhracat sorumlusu
Sektör
Matbaa
Yapı
zincir mağaza
Katılım
Ağu 2023
Mesaj
3
#10

Sonuna kadar katılıyorum. Herkesin yaptığı şey, doğru olduğu anlamına gelmiyor.

Sorunuz olursa yazın, elimden geldiğince cevaplarım.

TTülay K***KatılımcıTopluluk üyesi
Katılım
Mar 2023
Mesaj
216
#11

Bende tam tersi oldu, o yüzden yazıyorum. açıkçası asıl mesele rakam deil, o rakamın neye göre çıktığı.

Sonucu buraya yazarsanız başkalarına da faydası olur.

ŞŞerife K***Veteran
Görev
Klinik yöneticisi
Sektör
Elektrik-elektronik
Yapı
yeni kurulmuş girişim
Katılım
Ara 2023
Mesaj
128
#12

Bir uyarı bırakayım. işte kararı verirken elinizde hangi veri var, önce ona bakın.

Kayıt tutmayan hiçbi süreç düzelmiyor çünkü neyi düzelteceğinizi bilmiyorsunuz. Başka türlü yapan varsa onu da merak ederim.

KKaan G***UzmanTopluluk üyesi
Katılım
Haz 2023
Mesaj
94
#13

takipteyim.

EElif B***Katılımcı
Görev
Kurye koordinatörü
Sektör
Sağlık hizmetleri
Yapı
holding bünyesinde bir şirket
Katılım
Ara 2023
Mesaj
228
#14

Bizde de aynı.

AAlper E***KatılımcıTopluluk üyesi
Katılım
Ara 2024
Mesaj
184
#15

Kaydettim.

JJülide A***Katılımcı
Görev
Muhasebe müdürü
Sektör
Mücevherat
Yapı
20 kişilik firma
Katılım
May 2024
Mesaj
103

Doki · Güvenlik açığı taraması · 2026

#16

konuyu toparlayayım çünkü birkaç farklı cevpa verildi... küçük bi deneme ile başlayın tmmını bi anda bağlamayın.

ben olssam bu yoldan giderdim.

GGökhan A***Katılımcı
Görev
Üretici · mobilya
Katılım
Eki 2023
Mesaj
74
#17

Kusura bakmayın da bu her durumda geçerli dgl. Küçük ölçekte işe yarayan çözüm büyüyünce çöküyor, bunu geç öğrendim.

Yanlışım varsa düzeltin.

YYasemin S***KatılımcıTopluluk üyesi
Katılım
Şub 2024
Mesaj
42
#18

Bu kadar açık anlatan yazı az bulunuyor... yani sormaktan çekinmeyin sormayan hep daha pahalıya ödüyor.

Kapsam büyüyorsa ya süre ya bütçe büyümeli. yani üçüncü seçenek yok. Başka türlü yapan varsa onu da merak ederim.

RRecepYeni üye
Görev
Tesisatçı
Yapı
20 kişilik firma
Katılım
Ara 2024
Mesaj
22
#19

tam zamaninda bir konu

AAlper P***Katılımcı
Görev
Sistem yöneticisi
Sektör
Gayrimenkul
Yapı
20 kişilik firma
Katılım
Ağu 2024
Mesaj
85
#20

Bunu not aldım, teşekkürler.

Yanıt yaz