Feragat: Ben yeni gelen biriyim (bu benim üçüncü iş günüm) ve takım arkadaşlarımın çoğu benden daha deneyimli.
Kodumuza baktığımda, aşağıdaki gibi bazı kod kokuları ve kötü mühendislik uygulamaları görüyorum:
- Biraz tutarsız adlandırma kuralları
- Mümkün olduğunda salt okunur olarak işaretlenmemiş özellikler
- Büyük sınıflar - Yüzlerce uzatma yönteminden oluşan bir yardımcı sınıf olduğunu fark ettim (birçok tür için). 2500 çizgiden uzun oldu!
- Büyük yöntemler - 150 satır uzunluğunda bir yöntemi yeniden denemeye çalışıyorum.
Son iki gerçek bir sorun gibi görünüyor. Takım arkadaşlarımı daha küçük sınıflar ve yöntemler kullanmaya ikna etmek istiyorum. Ama bunu yapmalı mıyım? Eğer evet ise, o zaman nasıl?
Takımım ana takımdan bir mentorluk yaptı (biz bir uydu takımıyız). Önce ona gitmeli miyim?
GÜNCELLEME : Bazı cevaplar proje hakkında sorulduğundan, lütfen bunun çalışan bir proje olduğunu bilin. IMHO, bu büyüklükteki dev sınıflar / yöntemler her zaman kötüdür.
Neyse, asla takımımı kızdırmak istemiyorum. Bu yüzden sordum - Bunu yapmalı mıyım ve evetse, o zaman bunu nasıl nazikçe yapmalıyım?
GÜNCELLEME : Kabul edilen cevaba dayanarak bir şeyler yapmaya karar verdim: çünkü ben yeniyim, bu yüzden "yeni gözler" içindeki her şeyi görüyorum, bulduğum tüm kod kokularını not edeceğim (pozisyon, neden kötü, nasıl yapabiliriz? daha iyi, ...), ama şu anda, sadece ekibimden saygı duymaya çalışıyorum: "daha iyi kod" yaz, insanları tanı, neden bunu yaptığımızı biliyor ... Zaman doğru olduğunda deneyeceğim ekibime bazı yeni kod politikaları (adlandırma kuralları, daha küçük sınıflar, daha küçük yöntemler, ...) ve mümkünse bazı eski kodları yeniden sormak için sormak. İşe yaramalı, IMHO.
Teşekkür ederim.