Kontrol akışının onu gereksiz kıldığı durumlarda “başka” kullanılmalı mıdır?

Bazen aşağıdaki örneğe benzer kod üzerine rastlamak (bu fonksiyon tam olarak bu sorunun kapsamı dışında ne yapar):

function doSomething(value) {
  if (check1(value)) {
    return -1;
  }
  else if (check2(value)) {
    return value;
  }
  else {
    return false;
  }
}

Gördüğünüz gibi if, else ifve elseifadeleri ifadeyle birlikte kullanılır return. Bu rahat bir gözlemci için oldukça sezgisel görünüyor, ama ben else-s bırakmak ve böyle kodu basitleştirmek için (bir yazılım geliştirici açısından) daha zarif olacağını düşünüyorum :

function doSomething(value) {
  if (check1(value)) {
    return -1;
  }
  if (check2(value)) {
    return value;
  }
  return false;
}

Bir returnifadeyi izleyen her şey (aynı kapsamda) hiçbir zaman yürütülmeyeceğinden bu mantıklıdır; yukarıdaki kod anlamsal olarak ilk örneğe eşittir.

Yukarıdakilerden hangisi iyi kodlama uygulamalarına daha çok uyuyor? Her iki yöntemin de kod okunabilirliği açısından herhangi bir dezavantajı var mı?

Düzenleme: Bu soru referans olarak sağlanan yinelenen bir öneri yapılmıştır . Diğer soruda gösterildiği gibi yinelenen ifadelerden kaçınmayı istemediğim için sorumun farklı bir konuya değindiğine inanıyorum. Her iki soru da biraz farklı şekillerde de olsa tekrarları azaltmaya çalışır.

Onsuz olanı seviyorum elseve işte neden:

function doSomething(value) {
  //if (check1(value)) {
  //  return -1;
  //}
  if (check2(value)) {
    return value;
  }
  return false;
}

Çünkü bunu yapmak gerekmediği hiçbir şeyi kırmadı.

Tüm formlarında nefret bağımlılığı (bir işlevi adlandırmak dahil check2()). İzole edilebilecek her şeyi izole edin. Bazen ihtiyacınız olur elseama bunu burada görmüyorum.

Kodun anlambilimine bağlı olduğunu düşünüyorum. Üç vakanız birbirine bağımlıysa, açıkça belirtin. Bu, kodun okunabilirliğini artırır ve herkes için anlaşılmasını kolaylaştırır. Misal:

if (x < 0) {
    return false;
} else if (x > 0) {
    return true;
} else {
    throw new IllegalArgumentException();
}

Burada xher üç durumun değerine açıkça bağlısınız . Sonuncuyu atlarsanız else, bu daha az açık olacaktır.

Davalarınız doğrudan birbirine bağlı değilse, atlayın:

if (x < 0) {
    return false;
}
if (y < 0) {
    return true;
}
return false;

Bunu bağlamsız basit bir örnekle göstermek zor, ama umarım benim fikrimi anlarsın.

Ben ikinci seçeneği (ayrı tercih ifshayır ile else iferken ve return) ANCAK olmasıdır sürece kod blokları kısa oldukları gibi .

Kod blokları uzunsa kullanmak daha iyidir else if, çünkü aksi takdirde kodun sahip olduğu birkaç çıkış noktasını net bir şekilde anlayamazsınız.

Örneğin:

function doSomething(value) {
  if (check1(value)) {
    // ...
    // imagine 200 lines of code
    // ...
    return -1;
  }
  if (check2(value)) {
    // ...
    // imagine 150 lines of code
    // ...
    return value;
  }
  return false;
}

Bu durumda, daha iyi kullanmak else if, dönüş kodunu bir değişkende saklamak ve sonunda sadece bir dönüş cümlesine sahip olmak.

function doSomething(value) {
  retVal=0;
  if (check1(value)) {
    // ...
    // imagine 200 lines of code
    // ...
    retVal=-1;
  } else if (check2(value)) {
    // ...
    // imagne 150 lines of code
    retVal=value;
  }
  return retVal;
}

Ancak, işlevlerin kısa olması için çaba göstermesi gerektiği gibi, kısa tutun ve ilk kod snippet'inizdeki gibi erken çıkın.

Her ikisini de farklı koşullarda kullanıyorum. Doğrulama kontrolünde, diğerini atlıyorum. Bir kontrol akışında diğerini kullanıyorum.

bool doWork(string name) {
  if(name.empty()) {
    return false;
  }

  //...do work
  return true;
}

vs

bool doWork(int value) {
  if(value % 2 == 0) {
    doWorkWithEvenNumber(value);
  }
  else {
    doWorkWithOddNumber(value);
  }
}

İlk durumda, daha önce tüm önkoşulları kontrol ediyor, bunları ortadan kaldırıyor ve daha sonra çalıştırmak istediğiniz gerçek koda ilerliyormuşsunuz gibi daha fazla bilgi geliyor. Bu nedenle, gerçekten çalıştırmak istediğiniz sulu kod doğrudan işlevin kapsamına aittir; işlevin gerçekte ne hakkında olduğunu.
İkinci durum, her iki yolun da çalışacak geçerli kod olduğu gibi, bazı koşullara dayalı olarak işlevle ilgili olanla aynıdır. Bu itibarla, birbirine benzer kapsam seviyelerine aittirler.

Gerçekten önemli gördüğüm tek durum böyle kod:

List<?> toList() {
    if (left != null && right != null) {
        Arrays.asList(merge(left, right));
    }
    if (left != null) {
        return Arrays.asList(left);
    }
    if (right != null) {
        return Arrays.asList(right);
    }
    return Arrays.asList();
}

Burada yanlış bir şeyler var. Sorun şu ki, returneklenip eklenmeyeceği belli değil Arrays.asList. İlgili yöntemlerin daha ayrıntılı incelenmesi olmadan bunu düzeltemezsiniz. Her zaman tam kapsamlı if-else blokları kullanarak bu belirsizliği önleyebilirsiniz. Bu:

List<?> toList() {
    if (left != null && right != null) {
        Arrays.asList(merge(left, right));
    } else if (left != null) {
        return Arrays.asList(left);
    } else if (right != null) {
        return Arrays.asList(right);
    } else {
        return Arrays.asList();
    }
}

önce açık dönüş eklemezseniz derlemez (statik olarak kontrol edilen dillerde) if.

Bazı diller ilk stile bile izin vermez. Sadece kesinlikle zorunlu bağlamda veya uzun yöntemlerde ön koşullar için kullanmaya çalışıyorum:

List<?> toList() {
    if (left == null || right == null) {
        throw new IllegalStateException()
    }
    // ...
    // long method
    // ...
}

Eğer kodu ve kötü uygulamayı takip etmeye çalışıyorsanız, böyle fonksiyondan üç çıkış olması gerçekten kafa karıştırıcıdır.

İşlev için tek bir çıkış noktanız varsa, elses gereklidir.

var result;
if(check1(value)
{
    result = -1;
}
else if(check2(value))
{
    result = value;
}
else 
{
    result = 0;
}
return result;

Aslında daha da ileri gidelim.

var result = 0;
var c1 = check1(value);
var c2 = check2(value);

if(c1)
{
    result = -1;
}
else if(c2)
{
    result = value;
}

return result;

Giriş doğrulamasında tek bir erken getiri için tartışabileceğiniz kadar adil. Sadece bir if içinde mantığı tüm toplu sarma kaçının.

Gereksiz ifadeyi atlamayı tercih ederim. Ne zaman onu görmek (kod inceleme veya yeniden düzenleme) yazar kontrol akışını anlamak ve kodda bir hata olabilir merak ediyorum.

Yazar, else ifadesinin gerekli olduğuna inanıyorsa ve bu nedenle, iade ifadesinin kontrol akışı sonuçlarını anlamıyorsa, kesinlikle böyle bir anlayış eksikliği, bu işlevde ve genel olarak büyük bir hata kaynağıdır.