Hangisi daha sürdürülebilir - if / else veya boolean ifadesi aracılığıyla boole ataması?

Hangisi daha sürdürülebilir kabul edilir?

if (a == b) c = true; else c = false;

veya

 c = (a == b);

Code Complete'e bakmayı denedim, ancak bir cevap bulamıyorum.

İlkinin daha okunabilir olduğunu düşünüyorum (kelimenin tam anlamıyla yüksek sesle okuyabilirsiniz), ki bu da onu daha sürdürülebilir hale getiriyor. İkincisi kesinlikle daha mantıklı ve kodu azaltır, ancak C # geliştiricileri için sürdürülebilir olduğundan emin değilim (bu deyimi daha çok örneğin Python'da görmeyi beklerim).

İkinci seçenek daha iyidir.

Kodun amacını gizleyerek sürdürülebilirliği zedeleyen akıllı programlama kısayollarına karşı dikkatli olmanız için kesin bir neden var. Yani, soruyu sorduğun için seni suçlamıyorum.

Ancak, c = (a == b);akıllı bir hile örneği olarak düşünmüyorum . Bu, basit bir kavramın doğrudan bir temsilidir. Mümkün olduğunca düz ileri.

Düzgün, "sürdürülebilir" (eksik parantez ve tek satırlık bir yapı, olmadan ilk örneğinin biçimlendirme yapmak bu kodu doğuracak akıllı kısayol düşünün):

if (a == b)
{
    c = true; 
}
else 
{
    c = false;
}

Deneyimlerime göre, böyle bir ayrıntılı, hataya eğilimli şekilde basit boole mantığı yazmak "iffy" kodunun bir işaretidir. Bu kod tabanında ne kadar karmaşık mantığın ele alındığını merak etmeme neden olacaktı.

İlk olarak, iki formunuzun eşdeğer olmadığını anlayın.

if (a == b) c = true;

cadeğeri eşitse true olarak ayarlanır bve değilse değeri zaten olduğu gibi kalır.

c = (a == b);

caeşitse true olarak ayarlanır bve değilse false olarak ayarlanır.

İkinci formun eşdeğerini istiyorsanız, ilk formun tarzında, şöyle yazmanız gerekir:

if (a == b) {
  c = true;
} else c = false;

Şimdi, ikisinden hangisinin daha okunabilir, daha sürdürülebilir ve bir şey değiştirilirse hata getirme olasılığı daha düşük. İkinci form ile sopa.

İlk formunuzun daha okunabilir olduğuna katılmıyorum - tek bir satırda iki deyim olması kesinlikle deyimsel C # değildir ve ifparantez kullanmadan bir ifadenin olması önerilmez .

İkincisi, ikinci formun nasıl daha az bakım yapılabilir olduğunu görmüyorum - sürdürecek bir şey yok. Bu arasındaki ilişkinin basit bir ifadedir ave bve bir daha basitçe ifade edilemedi.

İkinci formu tercih etmenin bir başka nedeni, cbildirimi tek bir ifadede atayabilmeniz ve atayabilmenizdir.

bool c = (a == b);

Değişkenleri değiştirmek kolayca hatalara yol açabilir, bu yüzden bundan kaçınırım. İfadenin kullanılması if, değişkenin koşuldan önce bildirilmesini ve sonra değiştirilmesini gerektirir.

"daha sürdürülebilir" çok öznel olabilir.

Genellikle okunabilirliği ve kod azaltma amacını tercih ederim. Azaltılmış formu kullanarak 8 yazılan karakteri kaydettiğinizi düşünüyorum.

Dili ve kültürü dil etrafında ele almak bence 'okunabilirliğin' bir özelliğidir.

Ortaya çıkan bayt kodunu optimize etmek için kodun azaltılmasında performansın neden olabileceği zamanlar vardır, ancak bu bazı profillemeden sonra dikkatle yapılmalıdır.

İkinci. Daha az tekrarını (KURU) vardır ve ne olup bittiğini anlamak daha kolay, yani colsun veya olmasın değerine sahip ave olmayacak beşittir.

IMHO, daha da iyisi

c = a == b

Tam yazdığım gibi

• 1 + 2 + 3 onun yerine ((1 + 2) + 3)
• 5 + 3 * 7 onun yerine (5 + (3 * 7))

Açıkçası ve önemsiz gereksiz kod bir erdem değildir. Karmaşık.

Aşağı seçmenler, lütfen gözden geçirilmiş cevabımda neyin yanlış olduğunu açıklayın.

Evet, c = (a == b);okumak zor olabilir (daha da kötüsü, StyleCop gereksiz parantezden şikayet ediyor), ama yine de basitliğini sevdim a == b. Bu nedenle, burada her iki kullanmak ister ne ave baynıdır:

private static bool NoPeriod
{
    get
    {
        return this.MyWaveLength == this.HerWaveLength;
    }
}

Ve sonra şunları yapabilirsiniz this.c = this.NoPeriod:

this.c = this.MyWavelength == this.HerWaveLength;