Kodunuzda özlülük veya okunabilirliği tercih ediyor musunuz? [kapalı]

Dil kısayolları genellikle kodu daha kısa yapmak için kullanılabilir.

Örneğin, üçlü ve boş birleştirme operatörleri kod miktarını azaltabilir, ancak tartışmalı olarak okunabilirliğin zararına:

C # 'da:

Person newGuy = new Person();
if (boss == null) {
    newGuy.Boss = GetDefaultBoss();
} else {
    newGuy.Boss = boss;
}

işlevsel olarak şuna eşittir:

Person newGuy = new Person();
newGuy.Boss = boss ?? GetDefaultBoss();

ama belli ki çok daha ayrıntılı.

Okunabilirlik ve özlü olma söz konusu olduğunda çizgiyi nerede çizersiniz?

Her ikisi de.

İlk örneğiniz kesinlikle daha ayrıntılı ve tartışmalı olarak daha açık ... ama aynı zamanda bir tane yerine beş satır taramamı gerektiriyor. Daha da kötüsü, amacına vurgu yapar - bir değer vermek newGuy.Boss.

İkinci örnek, boş birleştirme operatörüne yabancıysam, bana bir saniye tutabilir, ancak amacı hakkında hiçbir şüphem yok ve bir değerin kaynağını arayan daha büyük bir rutini tarıyorsam, Bunu seçmem benim için daha kolay.

Şimdi, bunu kontrast:

if (boss == null) {
    newGuy.Boss = GetDefaultBoss();
    newGuy.IsTemp = true;
    newGuy.AddTask("orientation");
} else {
    newGuy.Boss = boss;
    newGuy.IsTemp = false;
}

...ile:

newGuy.Boss = boss ?? GetDefaultBoss();
newGuy.IsTemp = boss == null;
if ( boss == null ) newGuy.AddTask("orientation");

İkinci örnek yine çok daha kısadır, ancak şimdi aynı test tarafından tetiklenen görevlerin farklı görünmesini sağlayarak amacını gizlemektedir. Burada, eski olanın ayrıntılarının haklı olduğunu hissediyorum.

Her ikisi de iyi hedefler olsa da, birini seçmeye zorlandığımda her zaman Okunabilirlikten yana olacağım .

Ben senin örneğin okunabilirliği hem iyileştirdiğini iddia ediyorum ve kısalık. Ancak şunu göz önünde bulundurun:

if( a > b )
{
    foo = bar
}
else
{
    if( c.isThing() ){
        foo = somethingElse;
    }
    else{
        foo = someFurtherThing.GetFoo();
    }
}

aksine

foo = a > b ? bar ?? whatever.something : c.isThing() ? somethingElse : someFurtherThing.GetFoo();

İkincisi özlü, ancak okumak zordur. İlki ayrıntılı, ancak mantık akışı açıktır.

Sonuç olarak, kısalık, ekrana daha fazla sığma kabiliyetinden başka bir amaca hizmet etmiyor. Okunabilirlik hata ayıklamayı kolaylaştırır ve genellikle tercih edilmelidir.

Genel bir kural olarak, netlik nedeniyle okunabilirliği asla feda etmediğini, ancak bu konuda bilgi eksikliği olan başka programcılara dayanarak okunabilirliği asla yargılamadığını söyleyebilirim.

Sünnet ve okunabilirlik zıt değildir. Bu cevap gibi, bazen daha kısa okunabilir.

Okunabilirliği tercih ettiğimi söyleyebilirim, bu bazen kısa kod kullanmak anlamına geliyor. (Yani daha büyük bir koşullu blok içindeki nispeten basit koşullamalar için üçlü).

Temel olarak, gereksiz yere anlamak zorsa, yapmayın.

Okunabilirlik ilk önce özle çelişki halindedir, çünkü kod başlangıçta yazıldığından daha sık değiştirilir. Diğer yandan:

1. Sözdizimsel gürültü ve kazan plakası kodu genellikle niyetleri gizlemekte ve bu nedenle okunabilirliğe zarar vermektedir. Bazen daha özlü kod da daha okunaklıdır. Örneğin, lambda işlevlerini veya delegeler / birinci sınıf işlevleri yerine, tek yöntemli bir arabirim uygulayan tek yöntemli sınıfları düşünün.

2. Okunabilirlik, dili ve benzersiz / gelişmiş özelliklerini oldukça iyi bilen makul bir tecrübeli programcı için kodun okunmasının ne kadar kolay olduğu, ancak en düşük ortak paydayı bilen ancak bazı yetkin kod maymunu kullananlar açısından kolay bir şekilde değerlendirilmelidir.

Ben sanmıyorum bir yönü henüz söz edilmiştir: nelerdir sizin hedefleriniz?

Tek umursadığın iş güvenliği ise, her şeyin özünde ve küçüklüğünde ol. Ayrıca kodunuzu yorumlayarak atlayın.

Yepyeni bir proje üzerinde çalışırken kodunuzu başkasına kolayca teslim edebilmek istiyorsanız, okunabilirlik, netlik ve pek çok sağlam yorum için gidin.

_{^{Not: Yukarıdaki şahsen sizinle ilgili değildir, @Damovisa; İki pozisyon arasında seçim yapan herkes için.}}

Ayrıntılı sürümün bir avantajı olduğu bir şey var.

Daha fazla satır var ve çoğu hata ayıklayıcı satır odaklı ! Bir ifadenin ortasına bir kırılma noktası ayarlamak çok zordur, ancak bunu genellikle bir blok ifadesinin içine koymak oldukça kolaydır.

Başka bir deyişle, hata ayıklayıcınızın ne zaman başlamasını istiyorsanız, editörünüzde hangisini görmek istersiniz boss == null?

(Bu ?? - operatörünü sevdiğimi söyledi)

Okunabilirlik ilk önce gelmeli, uzun vadede çoğu insan çoğu zaman mevcut kodu değiştirmek veya genişletmek için harcar - okunabilirlik, bakımın büyük bir parçasıdır.

Bununla birlikte, özlülük, okunabilirliğe katkıda bulunabilecek bir şeydir. Örneğin, sorunuzda ikinci snippet hem daha okunaklı hem de daha özlüdür.