Sürekli gelişen iş mantığıyla dolu ve her geçen gün değiştirilmesi gereken çok fazla kod üzerinde çalışıyorum. Değişikliklere devam etmeme yardımcı olan iki ipucu: Tüm diğer ifadelerden kaçın ve mümkün olan en kısa sürede geri dön/çık. Asla derin iç içe geçme -> alt rutinler/fonksiyonlar yaratma. ifadeleri kodunuzu çok daha kolay alt (condtion yakınlığı ve kod bloğu) için üst okumak için yaparsa
reddedildiği ile her şeyin ifadeleri Değiştirme:
# business logic block
if ($condition) {
# do something
# code code code
} else {
# code code code
return;
}
# refactored:
if (! $contition) {
# code code code
return;
}
if ($condition) {
# code code code
}
İkincisi, en kısa sürede geri dönüş/çıkış . Benim görüşüme göre, ancak alt programın sonucunu zaten belirledikten sonra, özellikle de kodu yukarıdan aşağıya okumak istediğinizde, herhangi bir ekstra koşul/testten geçme noktasını göremiyorum. Tüm belirsizlikleri ortadan kaldırmak işleri kolaylaştırıyor. Sonuç olarak, özellikle uzun BL listelerinde özellikle kullanmaktan kaçınmayı sonlandırmak isterim. Sonucu bildiğiniz anda geri dönün. Yuvalama seviyesi 2'den fazlaysa, alt rutinler/işlevler oluşturun.
+1 Dosya boyutunu birkaç KB azaltabileceğinden bu çok önemlidir! – Xeoncross
+1 Ayrıca devam etmek, kırmak, atmak için de geçerlidir.Sıklıkla döngüsel karmaşıklığı bu şekilde düşük tutabilir ve böylece kodu daha okunabilir hale getirebilirsiniz. – NikiC
Ayrıca kişisel olarak sadece 'return userIsLoggedIn() && is_array ($ param) && $ param ['count']> 0;' i kullanıyorum. Kısa ve özlü. – NikiC