JavaScript'teki tek satırlık ifadelerde kaşlı ayraçlar gerekli mi?

Bir keresinde kıvırcık parantezleri tek satırlık ifadelerde bırakmanın JavaScript'te zararlı olabileceğini duydum. Artık sebebini hatırlamıyorum ve Google araması pek yardımcı olmadı.

JavaScript'teki kıvırcık parantez içindeki tüm ifadeleri çevrelemeyi iyi bir fikir haline getiren bir şey var mı?

Soruyorum, çünkü herkes öyle görünüyor.

Hayır

Ancak tavsiye edilir. Eğer ifadeyi genişletirseniz, onlara ihtiyacınız olacaktır.

Bu tamamen geçerli

if (cond) 
    alert("Condition met!")
else
    alert("Condition not met!")

Bununla birlikte, siz her zaman parantez kullanmanız önemle tavsiye edilir, çünkü siz (veya bir başkası) ifadeyi genişletirse, gerekli olacaktır.

Aynı uygulama, tüm C sözdizimi stili dillerinde destekle izlenir. C, C ++, Java, hatta PHP bile parantez olmadan bir satır ifadesini destekler. Sadece iki karakter kaydettiğinizi ve bazı insanların canlandırıcı stilleri ile bir satırı bile kaydetmediğinizi fark etmelisiniz. Tam bir ayraç stilini (aşağıdaki gibi) tercih ederim, bu yüzden biraz daha uzun olma eğilimindedir. Ödünç alma, son derece net kod okunabilirliğine sahip olmanızla çok iyi karşılanmaktadır.

if (cond) 
{
    alert("Condition met!")
}
else
{
    alert("Condition not met!")
}

Okunabilirlik yönü var - bileşik ifadeler olduğunda çok kafa karıştırıcı olabilir. Girintileme yardımcı olur, ancak derleyici / yorumlayıcı için hiçbir şey ifade etmez.

var a;
var b;
var c;

//Indenting is clear
if (a===true)
  alert(a); //Only on IF
alert(b); //Always

//Indenting is bad
if (a===true)
  alert(a); //Only on IF
  alert(b); //Always but expected?

//Nested indenting is clear
if (a===true)
  if (b===true)
    alert(a); //Only on if-if
alert (b); //Always

//Nested indenting is misleading
if (a===true)
  if (b===true)
    alert(a); //Only on if-if
  alert (b); //Always but expected as part of first if?

//Compound line is misleading
//b will always alert, but suggests it's part of if
if (a===true) alert(a);alert(b); 
else alert(c); //Error, else isn't attached

Ve sonra bir genişletilebilirlik yönü var:

//Problematic
if (a===true)
  alert(a);
  alert(b); //We're assuming this will happen with the if but it'll happen always
else       //This else is not connected to an if anymore - error
  alert(c);

//Obvious
if (a===true) {
  alert(a); //on if
  alert(b); //on if
} else {
  alert(c); //on !if
} 

Düşünce, her zaman paranteziniz varsa, o bloğun içine başka ifadeler eklemeyi bildiğinizdir.

Soru bir satırdaki ifadeleri soruyor. Bununla birlikte, sağlanan birçok örnek, birden fazla satır ifadesine dayanarak parantezleri bırakmamanın nedenlerini göstermektedir. Tercih ettiğiniz kodlama stili ise, bir satırda parantez kullanmamak tamamen güvenlidir.

Örneğin, soru bunun iyi olup olmadığını sorar:

 if (condition) statement;

Bunun iyi olup olmadığını sormaz:

 if (condition)
   statement;

Ben daha az gereksiz sözdizimi ile kodu daha okunabilir yapar çünkü parantezleri bırakarak tercih edilebilir olduğunu düşünüyorum.

Kodlama tarzım, kod bir blok olmadığı sürece hiçbir zaman parantez kullanmamaktır. Ve asla tek bir satırda (noktalı virgülle ayrılmış) birden fazla ifade kullanmayın. Bunu kolay okunur ve anlaşılır buluyorum ve 'if' ifadelerinde asla kapsam belirleme sorunları yaşamıyorum. Sonuç olarak, tek bir if koşul ifadesinde parantez kullanmak 3 satır gerektirir. Bunun gibi:

 if (condition) {
   statement;
 }

Daha az dikey alan kullandığı ve kod daha kompakt olduğu için bir satır if ifadesi tercih edilir.

Başkalarını bu yöntemi kullanmaya zorlamam, ama benim için çalışıyor ve parantezleri bırakmanın kodlama / kapsam belirleme hatalarına yol açtığı örneklerle daha fazla anlaşamadım.

Teknik olarak hayır ama başka türlü kesinlikle Evet !!!

"Bu kişisel tercihi" unutun, "kod iyi çalışır", "benim için iyi çalışıyor", "daha okunabilir" yada yada BS. Bir hata yaparsanız ve kodlama yaparken bir hata yapmanın çok kolay olduğuna inanıyorsanız bu çok ciddi sorunlara yol açabilir (İnanmayın ?, ünlü Apple'ın hataya gitmesine bakın ).

Argüman: "Kişisel tercih"

Hayır öyle değil. Mars'tan ayrılan tek kişilik bir takım değilseniz, hayır. Çoğu zaman kodunuzu okuyan / değiştiren başka insanlar olacaktır. Herhangi bir ciddi kodlama ekibinde bu tavsiye edilen yöntem olacaktır, bu yüzden 'kişisel tercih' değildir.

Argüman: "kod iyi çalışır"

Spagetti kodu da öyle! Onu oluşturmanın uygun olduğu anlamına mı geliyor?

Tartışma: "benim için iyi çalışıyor"

Kariyerimde bu problem yüzünden çok fazla hata oluştuğunu gördüm. Muhtemelen kaç kez yorum yaptığınızı 'DoSomething()'ve neden 'SomethingElse()'çağrıldığına şaşkın olduğunuzu hatırlamıyorsunuz :

if (condition) 
    DoSomething();
SomethingElse();

Veya 'SomethingMore' ekledi ve çağrılmayacağını fark etmedi (girinti aksini ima etse bile):

if (condition)
  DoSomething();
  SomethingMore();

İşte sahip olduğum gerçek bir hayat örneği. Birisi find & replace "console.log"=> çalıştırmak için tüm günlüğü kapatmak istedi //"console.log":

if (condition) 
   console.log("something");
SomethingElse();

Sorunu görüyor musunuz?

"Bunlar çok önemsiz, bunu asla yapmam" diye düşünseniz bile; her zaman sizden daha düşük programlama becerilerine sahip bir ekip üyesi olacağını unutmayın (umarım takımın en kötüsü değilsiniz!)

Argüman: "daha okunabilir"

Programlama hakkında bir şey öğrendiysem, basit şeyler çok hızlı bir şekilde çok karmaşık hale gelir. Bunun çok yaygın olduğu:

if (condition) 
    DoSomething();

farklı tarayıcılar / ortamlar / kullanım senaryoları veya yeni özellikler eklendikten sonra aşağıdakilere dönüşür:

if (a != null)
   if (condition) 
      DoSomething();
   else
      DoSomethingElse(); 
      DoSomethingMore();
else 
    if (b == null)
         alert("error b");
    else 
         alert("error a");

Ve bununla karşılaştırın:

 if (a != null) {
    if (condition) { 
       DoSomething();
    }
    else {
       DoSomethingElse();
       DoSomethingMore();
    }
 } else if (b == null) {
    alert("error b");
 } else {
    alert("error a");
 }

Not: Bonus puanları yukarıdaki örnekte hatayı fark edenlere gider.

Sürdürülebilirlik sorunu yok!

Hepinizle ilgili sorun, her yerde noktalı virgül koymanızdır. Birden fazla ifade için kıvırcık parantezlere ihtiyacınız yoktur. Bir ifade eklemek istiyorsanız, virgül kullanın.

if (a > 1)
 alert("foo"),
 alert("bar"),
 alert("lorem"),
 alert("ipsum");
else
 alert("blah");

Bu, beklediğiniz gibi çalışacak geçerli bir koddur!

Kıvırcık parantezleri bir satır ifadesinde kullanmak için programlama nedeni yoktur.

Bu sadece kodlayıcıların tercihleri ​​ve okunabilirliği ile ilgilidir.

Kodunuz bundan dolayı kırılmaz.

@Josh K (Java, C vb. İçin de geçerlidir) tarafından belirtilen nedene ek olarak, JavaScript'teki özel bir sorun otomatik noktalı virgül yerleştirmektir . Wikipedia örneğinden:

return
a + b;

// Returns undefined. Treated as:
//   return;
//   a + b;

Bu nedenle, bu şekilde kullanılırsa beklenmedik sonuçlar da verebilir:

if (x)
   return
   a + b;

Yazmak gerçekten daha iyi değil

if (x) {
   return
   a + b;
}

ama belki burada hata tespit etmek biraz daha kolaydır (?)

Bu bir stil meselesidir, ancak kıvırcık parantezler başkalarının sarkmasını önlemek için iyidir .

Çok sayıda iyi yanıt var, bu yüzden parantezler atlanabildiğinde “kuralımı” söylemek dışında tekrar etmeyeceğim: tek ifadeleri olarak 'geri dönen' veya 'fırlatan' (örn.) Koşullar üzerinde . Bu durumda akış kontrolü zaten sona erdiği açıktır:

“Kötü durum” bile, sonlanan akış kontrolü nedeniyle hızlı bir şekilde tanımlanabilir (ve sabitlenebilir). Bu kavram / yapı “kuralı” aynı zamanda birçok dil için de geçerlidir.

if (x)
    return y;
    always();

Tabii ki, bu yüzden de bir linter kullanabilirsiniz ..

İşte bu yüzden tavsiye edilir

Diyelim yazıyorum

if(someVal)
    alert("True");

Sonra bir sonraki geliştirici gelir ve "Oh, başka bir şey yapmam gerek" der, bu yüzden yazarlar

if(someVal)
    alert("True");
    alert("AlsoTrue");

Gördüğünüz gibi "AlsoTrue" her zaman doğru olacaktır, çünkü ilk geliştirici parantez kullanmadı.

Şu anda bir madenci üzerinde çalışıyorum. Şimdi bile iki büyük senaryoda kontrol ediyorum. Deneysel olarak öğrendim: Kıvırcık parantezler ',', 'if', 'else' içermiyorsa, eğer *, eğer fonksiyon * için arkasındaki kıvırcık parantezleri kaldırabilirsiniz, 'iken', 'do', 'fonksiyon'. Sayısız çizgi kesiliyor.

function a(b){if(c){d}else{e}} //ok  
function a(b){if(c)d;else e}   //ok

Tabii ki kapanış ayracı başka bir kapanış ayracı tarafından takip edilmezse noktalı virgülle değiştirmeniz gerekir.

İşlev virgülle bitmemelidir.

var a,b=function()c;  //ok *but not in Chrome
var b=function()c,a;  //error  

Chrome ve FF'de test edildi.

Her zaman buldum

if(valid) return;

gözümde daha kolay

if(valid) {
  return;
}

ayrıca şartlı gibi

(valid) ? ifTrue() : ifFalse();

kişisel görüşümü okumaktan daha kolay

if(valid) {
  ifTrue();
} else {
  ifFalse();
}

ama sanırım kodlama tarzı geliyor

Soruyu doğrudan yanıtlamıyor, ancak aşağıda bir satırdaki koşul hakkında kısa bir sözdizimi bulunmaktadır

Ör:

var i=true;
if(i){
  dosomething();
}

Şu şekilde yazılabilir:

var i=true;
i && dosomething();

Javascript'te birçok sorun var. Bir göz atın bunu konuşuyor JavaScript mimar Douglas Crockford eğer deyim cezası gibi görünüyor ama dönüş ifadesi bir sorun taşıyabilirler.

return
{
    ok:false;
}
//silent error (return undefined)

return{
    ok:true;
}
//works well in javascript

Bu cevabı benzer bir deneyimle ararken buldum, bu yüzden deneyimle cevaplamaya karar verdim.

Parantezsiz ifadeler çoğu tarayıcıda çalışır, ancak parantezsiz yöntemlerin aslında bazı tarayıcılarda çalışmadığını test ettim.

26 Şubat 2018 itibarıyla, bu ifade Pale Moon'da çalışıyor, ancak Google Chrome'da çalışmıyor.

function foo()
   return bar;

Bir ifadenin başlangıç ​​girinti düzeyi, üstündeki açık ayraçların sayısına eşit olmalıdır. (alıntılanmış veya yorumlanmış kaşlı ayraçlar veya önişlemci yönergelerindeki yönergeler hariç)

Aksi takdirde K&R iyi girinti tarzı olacaktır. Stillerini düzeltmek için, bir satıra kısa basit if ifadeleri koymanızı öneririm.

if (foo) bar();    // I like this. It's also consistent with Python FWIW

onun yerine

if (foo)
   bar();   // not so good

Bir editör yazıyordum, otomatik format düğmesinin çubuğu foo ile aynı satıra kadar emmesini sağlarım ve bu şekilde dönmeden basarsanız çubuğun etrafına parantez eklemesini yaparım:

if (foo) {
  bar();    // better
}

Daha sonra, if ifadesinin gövdesine çubuğun üstüne veya altına yeni ifadeler eklemek kolay ve tutarlıdır

if (foo) {
  bar();    // consistent
  baz();    // easy to read and maintain
}

Bazen ihtiyaç duyuluyor gibi görünüyor! Kendime inanamadım, ancak dün bana bir Firebug oturumunda (son Firefox 22.0) gerçekleşti

if (! my.condition.key)
    do something;

idam do şey rağmen my.condition.key oldu doğrudur . Diş telleri ekleme:

if (! my.condition.var) {
    do something;
}

bu sorunu düzeltti. Görünüşe göre parantez olmadan çalıştığı örneklerin sayısız var, ancak bu durumda kesinlikle işe yaramadı.

Çok kesinlikle gereken bir hat üzerinde birden fazla ifadeyi koymak eğilimindedir İnsanlar hep tabii ki, parantez, gibi şeyler yüzünden

if (condition)
    do something; do something else;

bulmak zor.

Sadece kıvırcık parantezleri başkalarından da bırakabileceğinizi belirtmek isterim. Görüldüğü gibi , John Resig en tarafından bu makalede .

if(2 == 1){
    if(1 == 2){
        console.log("We will never get here")
    }
} else 
    console.log("We will get here")

Eğer ifadeler birden fazla satır kıvırcık olmayan parantez elde etmek için bir yolu var .. (Vay ne ingilizce ..) ama tür tedius:

if(true)
   funcName();
else
   return null;


function funcName(){
  //Do Stuff Here...
}