Soruyla ilgili dikkat çekici iki konu var, incecik bir şekilde ve son teslim tarihi . Bunlar farklı konular - birincisi iletişim ve takım dinamikleri, ikincisi ise planlama ve önceliklendirme.
Taktiksel olarak . Fırçalanmış ego ve incelemelere karşı olumsuz itirazlardan kaçınmak istediğinizi varsayıyorum. Bazı öneriler:
- Kodlama standartları ve tasarım ilkeleri konusunda ortak bir anlayışa sahip.
- Asla geliştiriciyi eleştirmeyin ya da gözden geçirmeyin , yalnızca kodu . “Siz” veya “kodunuz” kelimesinden kaçının, inceleme altındaki kod hakkında herhangi bir geliştiriciden bağımsız olarak konuşun.
- Gururunuzu, tamamlanmış kodun kalitesine ekleyin , böylece son sonucun iyileştirilmesine yardımcı olacak yorumları gözden geçirin .
- Talep yerine iyileştirmeler önerin . Anlaşmazsanız her zaman bir diyalog var. Kabul etmiyorsanız diğer bakış açısını anlamaya çalışın.
- İncelemelerin her iki yöne de gitmesini sağlayın. Yani, gözden geçirdiğiniz kişinin kodunuzu daha sonra incelemesini isteyin. "Tek yönlü" incelemeleriniz yok.
İkinci kısım önceliklendirmedir . İyileştirmeler için birçok öneriniz var, ancak son başvuru tarihi yaklaşmakta olduğundan, yalnızca birkaç başvurusu yapılacak zaman var.
İlk önce bunun olmasını önlemek istiyorsun! Bunu sürekli, artan incelemeler yaparak gerçekleştirirsiniz. Bir geliştiricinin bir özellik üzerinde haftalarca çalışmasına izin verme ve son anda hepsini gözden geçirme. İkincisi, kod incelemeleri ve inceleme önerilerini uygulama süresi, düzenli planlama ve herhangi bir görev için yapılan tahminlerin bir parçası olmalıdır. Düzgün bir şekilde incelemek için yeterli zaman yoksa, planlamada bir şeyler ters gitti.
Ancak, süreçte bir şeylerin yanlış gittiğini varsayalım ve şimdi birkaç inceleme yorumu ile karşı karşıya kaldınız ve hepsini uygulamak için zamanınız yok. Öncelik vermek zorundasın. Sonra ertelerseniz daha sonra değişmesi en zor ve en riskli olanı seçin.
Tanımlayıcıların kaynak kodda adlandırılması, okunabilirlik ve bakım kolaylığı için inanılmaz derecede önemlidir, ancak gelecekte değişmesi de oldukça kolaydır ve düşük risklidir. Kod biçimlendirme ile aynı. Bu yüzden o şeye odaklanma. Öte yandan, kamuya açık arabirimlerin akıl sağlığı, gelecekteki değişimi gerçekten zor oldukları için, en yüksek önceliğe sahip olmalıdır. Kalıcı verilerin değiştirilmesi zordur - ilk önce tutarsız veya eksik verileri bir veritabanında depolamaya başlarsanız, gelecekte düzeltilmesi gerçekten zordur.
Birim testlerinin kapsadığı alanlar düşük risklidir. Bunları daha sonra her zaman düzeltebilirsin. Değil, ama Alanlar olabilir birim test edilmiş alanlara göre daha düşük bir risk vardır olmak olamaz birim test edilmiş olmalı.
Diyelim ki ünite testi olmayan ve harici bir hizmete kodlanmış bir bağımlılık dahil olmak üzere her türlü kod kalitesi sorunu olan büyük bir kod yığınınız var. Bunun yerine bu bağımlılığı enjekte ederek kod öbeğini test edilebilir hale getirirsiniz. Bu, gelecekte testler ekleyebileceğiniz ve daha sonra sorunların çözülmesi için çalışabileceğiniz anlamına gelir . Kodlanmış bağımlılıkla test bile ekleyemezsiniz. Öyleyse önce bu düzeltmeye gidin.
Ama lütfen bu senaryoda ilk sırada yer almaktan kaçının!