Bakım açısından akıllıca, 'parantez içine alınmadan güvenli' sayılır mı?

Giriş else whileparantezleri olmadan bakımın "güvenli" olduğu düşünülüyor mu?

if-elseAşağıdaki gibi parantez olmadan kod yazma ...

if (blah)
    foo();
else
    bar();

... risk taşır çünkü parantez eksikliği, kodun anlamını istemeden değiştirmeyi çok kolaylaştırır.

Ancak, altında da riskli mi?

if (blah)
{
    ...
}
else while (!bloop())
{
    bar();
}

Yoksa else whilearaya giren parantezler “güvenli” sayılıyor mu?

Bu bana bu kodu hatırlatıyor:

if ( ... ) try {
..
} catch (Exception e) {
..
} else {
...
}

Her iki bloğu birleştirdiğinizde, diş tellerini unutarak ve girintiyi artırmazsanız, anlaşılması ve bakımı çok zor bir kod yaratırsınız.

Muhtemelen bu, Jackson Entity Structure Diagram yöntemini kullanarak işlemimi ( zamanın geri dönüşü) öğrendiğim içindi , ancak bir veya birinden sonra gelen tek doğru doğru olmayan terimin müteakip (yani bir merdivene izin vermek) olduğu görüşüne abone oldum.ifelseifelse if

Başka bir şey (amaçlanan herhangi bir amaç yoktur) yanlış anlama ve / veya bakım sorunları için potansiyel bırakır. OP’lerin fikrinin “güvensiz” olmasının temel yönü budur.

Ben de while()aynı çizgiye dahil olmak konusunda çok cagey olurdum else- kaşlı olsun ya da olmasın. Bana doğru okumuyor ... herhangi bir fazladan girinti maskesinin olmaması, bunun elsefıkra olduğunu gösteriyor. Netlik eksikliği, yanlış anlaşılmalara neden olur (yukarıya bakın).

Bu örnekte, şiddetle tavsiye ediyorum / öneriyorum (ve ekibimde ısrar ediyorum):

if ( blah )
{
    ...
}
else
{
    while ( !bloop() )
    {
        bar();
    }
}

Tabii ki, ben de uygun yorumlar görmeyi beklerdim.

-- Düzenle --

Son zamanlarda Apple, temassız bir tek satıra ikinci bir satır ekleyen kötü bakım onarımının neden olduğu bir SSL güvenlik açığından muzdaripti. Öyleyse, sıralanmamış tek çizgilerin iyi olduğu fikrini yatalım.

Her zaman her şeyi parantez içinde tutmam, öğrettiğim ve yorum verdiğim öğretildi. Hataları okumak ve tespit etmek çok daha kolay. Şahsen ben modülerliğin anahtar olduğunu düşünüyorum, bu yüzden daima böyle bir kod yazardım:

    if(blah)
    {
     ....
    }
    else
    {
       while(!bloop()
       {
        bar;
       }
    }

Ben metodu çıkarır ve yapardım .

if ( blah )
{
    ...
}
else
{
   newMethod();
}

Refactoring Catalogue sitesinde açıklanan "özü yöntemi" yaklaşımına bakın :

Birlikte gruplanabilen bir kod parçanız var.

Parçayı, adı yöntemin amacını açıklayan bir yönteme dönüştürün.

void printOwing() {
    printBanner();

    //print details
    System.out.println ("name:    " + _name);
    System.out.println ("amount    " + getOutstanding());
}

                                                                                                        

void printOwing() {
    printBanner();
    printDetails(getOutstanding());
}

void printDetails (double outstanding) {
    System.out.println ("name:    " + _name);
    System.out.println ("amount    " + outstanding);
}

Kötü bir kodlama alışkanlığı olarak düşünürdüm. Bazen mükemmel çalışıyor ve kodun derlenmesinde ve çalıştırılmasında sorun yaşamayacaksınız. Diğer zamanlarda ciddi hatalara neden olabilir ve bu hatayı düzeltmek için saatlerce harcamanız gerekecektir.

Kodunuzu her zaman modüler hale getirmeniz önerilir. Bir süre döngü başka bir yere koymak zorunda kalırsanız, bir bloğa koyun diğerleri böylece, kod üzerinde çalışırken kolayca mantığı anlayabilir.

if ( blah )
{
    ...
}
else
{
    while ( !bloop() )
    {
        bar();
    }
}

Kodları bloklara koymak iyi bir uygulamadır. Aynı zamanda anlaşılması ve hata ayıklaması da daha basit ve kolaydır.

Bu kadar basit kalırsa kişisel olarak hoşuma gitmese de tercihim iyi olur; tercihim basit if / else kod blokları için bile parantez kullanmaktır. Bu sadece benim için daha düzenli.

Bununla birlikte, bunun dağınık olduğu yer, eğer iç içe geçmişseniz / başkaları bazılarını bazılarında olmayanlar ile döngüler. Bütün o spagetti ortasında bir döngü! O kadar kötü kodla çalıştım ve hata ayıklamak ve anlamak kabus gibi. Bu, ilk adımdan itibaren basit kaldığında ve onunla mutlu olduğunuzda sorun değil, ancak daha sonra diğer programcılar gelip bir şeyler ekliyorlar, muhtemelen bir süre içinde başka bir if varsa. İlk başta temiz yazılmışsa, bunun gerçekleşmesi için daha az şans vardır.

Önemli olan şeyleri açıklığa kavuşturmak ki diğer programcılar bir bakışta neyin doğru neyin yanlış olduğunu görebilsinler. Desen doğru görünüyor ve jarse olmaz kod yaz.

Bunun tersi, belki de bazı durumlarda bunun iyi okuduğunu iddia eden bazı insanları görebilmemdir. Kaynağa sahipsem, bu işlemi başka bir şey yapmayı beklerken yapmam gerekiyor. O zaman bile bana göre, while döngüsünü başka bir bloğun içine yerleştirmenin hiçbir farkı yoktur.

Eh, herkes diş telleri kullanmayı tartışsa da onları seviyor gibi görünse de, ben tam tersini tercih ederim. Sadece gerektiğim zaman diş telleri kullanıyorum çünkü onu daha okunaklı ve özlü buluyorum.

if (...)
   foo();
else while(...)
   bar();

... Aslında bunu " else while(...)" okunaklı buluyorum ! Hatta düz İngilizce gibi okur! Ama sanırım insanlar bunu garip bulacaklar çünkü en azını söylemek sıra dışı.

Sonunda, hepimiz onu diş telleriyle aptal geçirmez hale getirme eğilimindeyiz ... çünkü, bilirsin.

Ben her zaman diş telleri koyarım, çünkü acele ederken başka bir çizgi ekleyebilirsin, girintili, diş tellerini unut ve kafanda ne olduğunu çizebilirsin. Açma braketini aynı hatta tutuyorum ama önemli değil. Her iki durumda da, onlara sahip olmak daha iyidir.

if(blah) {
    foo();
}
else {
    bar();
}

Diş tellerinde bu kadar yanlış olan, bu kadar çok insanın yazmaktan kaçınmaya çalıştığı şey nedir?

Hangi sorunu tam olarak else whileçözer?

Parantezler ucuz ve iyidir; zekice ve esprinin aksine kod niyetini açık ve net gösterirler.

Unix Felsefesinden Alıntı:

Berraklık Kuralı: Berraklık zeki olmaktan iyidir.

Bakım çok önemli ve çok pahalı olduğu için, programları en önemli iletişim sanki bunları yürüten bilgisayara değil, gelecekte kaynak kodunu okuyacak ve sürdürecek olan insanlara (kendiniz de dahil olmak üzere) yazıyor.

Yanlış bir şey yok

if (blah)
    foo();
else
    bar();

Tıpkı yanlış bir şey olmadığı gibi

if (blah) foo(); else bar();

doğru koşullarda (koşullarınız değişebilir, ancak belirli stiller çok fazla tekrarlayan kodunuz olduğunda en iyi şekilde kullanılır - o zaman her çizginin görünümü stilistik girintiden daha önemlidir)

Ancak, bir yol seçerseniz, buna devam edin - tutarlılık kraldır. Bu yüzden, ikinci örneğiniz çok kötü, eğer if ifadesi ifadesi için parantez içerdiğinden, while ifadesi dışında değil, diğer cümlecik için parantez içinde olmalıdır.

Ayrıca bu kodu daha önce görmüştüm:

if (x) foo()
{
  bar();
}

ve kodlama standardı nazi tarafından yazılmıştır (her şey için parantezde ısrar eden).

Modern IDE'ler kolayca yeniden biçimlendirmek (gereksiz parantezleri kaldırmak veya eklemek de dahil) ve / veya bir dosyayı kaydetme kodunu yeniden göndermek için kolayca yapılandırılabilir. Böylece örneğiniz otomatik olarak mesela benzeyecek

if (blah) {
  blub();
} else
  while (!bloop()) {
    bar();
  }

Girinti, gereksiz kümeler olmadan bile yuvalamayı her zaman kolayca görülebilir hale getirir. Dolayısıyla, bu IDE ayarlarını zorlamayı başarırsanız, risk görmem.

Şahsen, parantezleri ve çizgi şeritlerini yok sayan kod karmaşasını önlediği için çok daha okunaklı kod buluyorum:

if (blah) blub();
else while (!bloop()) bar();

Fakat elbette, birçok geliştirici sonsuza dek bir gün böyle şeyleri tartışmaktan çok mutludur.