Değişken atama if koşul ifadesi, iyi uygulama mı değil mi? [kapalı]

Bir yıl önce Java gibi klasik OO dillerinden JavaScript'e geçtim. Aşağıdaki kod Java'da kesinlikle önerilmez (hatta doğru değildir):

if(dayNumber = getClickedDayNumber(dayInfo))
{
    alert("day number found : " + dayNumber);
}
function getClickedDayNumber(dayInfo)
{
    dayNumber = dayInfo.indexOf("fc-day");
    if(dayNumber != -1) //substring found
    {
        //normally any calendar month consists of "40" days, so this will definitely pick up its day number.
        return parseInt(dayInfo.substring(dayNumber+6, dayNumber+8));
    }
    else return false;
}

Temel olarak, if koşul ifadesindeki bir değere bir değişken atayabileceğimi ve atanan değeri sanki booleanmış gibi hemen kontrol edebileceğimi öğrendim.

Daha güvenli bir bahis için, genellikle bunu iki satır koda ayırırım, önce atarım sonra değişkeni kontrol ederim, ancak şimdi bunu bulduğuma göre, deneyimli JavaScript geliştiricilerinin gözünde bunun iyi bir uygulama olup olmadığını merak ediyorum.

Bunu tavsiye etmem. Sorun şu ki, değerleri karşılaştırmaya çalıştığınız, ancak veya =yerine tek bir tane kullandığınız yaygın bir hata gibi görünüyor . Örneğin, bunu gördüğünüzde:=====

if (value = someFunction()) {
    ...
}

Yapmak istediklerinin bu mu, yoksa bunu yazmak isteyip istemediklerini bilmiyorsunuz:

if (value == someFunction()) {
    ...
}

Görevi gerçekten yerinde yapmak istiyorsanız, aynı zamanda açık bir karşılaştırma yapmanızı da tavsiye ederim:

if ((value = someFunction()) === <whatever truthy value you are expecting>) {
    ...
}

Bunun iyi bir uygulama olmadığına dair bir kanıt göremiyorum. Evet, bir hata gibi görünebilir, ancak bu mantıklı yorumlarla kolayca giderilebilir. Örneğin alın:

if (x = processorIntensiveFunction()) { // declaration inside if intended
    alert(x);
}

Neden bu işlevin aşağıdakilerle 2. kez çalışmasına izin verilsin:

alert(processorIntensiveFunction());

Çünkü ilk sürüm kötü GÖRÜNÜYOR? Bu mantığa katılmıyorum.

Bunu birçok kez yaptım. JavaScript uyarısını atlamak için iki parantez ekliyorum:

if ((result = get_something())) { }

Bundan kaçınmalısınız, gerçekten kullanmak istiyorsanız, üstüne ne yaptığınızı söyleyen bir yorum yazın.

Bunu Java'da da yapabilirsiniz. Ve hayır, bu iyi bir uygulama değil. :)

(Ve ===tiplendirilmiş eşitlik için Javascript'te kullanın . Crockford'un JS'deki The Good Parts kitabını okuyun.)

Bunu while-döngülerle yaptığınızda bir durum var .
Dosyaları okurken, genellikle şunu yaparsınız:

void readFile(String pathToFile) {
    // Create a FileInputStream object
    FileInputStream fileIn = null;
    try {
        // Create the FileInputStream
        fileIn = new FileInputStream(pathToFile);
        // Create a variable to store the current line's text in
        String currentLine;
        // While the file has lines left, read the next line,
        // store it in the variable and do whatever is in the loop
        while((currentLine = in.readLine()) != null) {
            // Print out the current line in the console
            // (you can do whatever you want with the line. this is just an example)
            System.out.println(currentLine);
        }
    } catch(IOException e) {
        // Handle exception
    } finally {
        try {
            // Close the FileInputStream
            fileIn.close();
        } catch(IOException e) {
            // Handle exception
        }
    }
}

while9. satırdaki -loop'a bakın . Burada, yeni bir satır okunur ve bir değişkende saklanır ve ardından döngünün içeriği çalıştırılır. Bunun bir olmadığını biliyorumif ifadesi , ancak sanırım bir while döngüsü de sorunuza dahil edilebilir.

Bunun nedeni, a kullanırken FileInputStream, her çağırdığınızda FileInputStream.readLine(), dosyadaki bir sonraki satırı okur, yani eğer onu sadece fileIn.readLine() != nulldeğişkeni atamak yerine döngüden çağırırsanız , çağırmak yerine(currentLine = fileIn.readLine()) != null ve sonra onu denilen Döngünün içinde de, yalnızca her ikinci satırı alırsınız.

Umarım anlarsın ve iyi şanslar!

Java'da da if ifadeleri içinde atamalar yapabilirsiniz. İyi bir örnek, bir şeyi okuyup yazmak olabilir:

http://www.exampledepot.com/egs/java.io/CopyFile.html?l=new

Kod:

// Copies src file to dst file.
// If the dst file does not exist, it is created
void copy(File src, File dst) throws IOException 
{
    InputStream in = new FileInputStream(src);
    OutputStream out = new FileOutputStream(dst);

    // Transfer bytes from in to out
    byte[] buf = new byte[1024];
    int len;
    while ((len = in.read(buf)) > 0) {
        out.write(buf, 0, len);
    }
    in.close();
    out.close();
}

Martin Fowlers'ın mevcut kodun tasarımını iyileştirmek için Yeniden Düzenleme kitabına başvurursanız ! Daha sonra, örneğin iyi bir uygulama olabileceği birkaç durum vardır. durumunuzu ileri sürmek için bir işlev veya yöntem çağrısı kullanmak için uzun karmaşık koşul ifadeleri:

"Motivasyon

Bir programdaki en yaygın karmaşık alanlardan biri, karmaşık koşullu mantıktır. Koşulları test etmek ve çeşitli koşullara bağlı olarak çeşitli şeyler yapmak için kod yazarken, kısa sürede oldukça uzun bir yöntem elde edersiniz. Bir yöntemin uzunluğu başlı başına okumayı zorlaştıran bir faktördür, ancak koşullar zorluğu arttırır. Sorun genellikle, hem durum kontrollerinde hem de eylemlerdeki kodun size ne olduğunu söylemesi, ancak neden olduğunu kolayca gizleyebilmesi gerçeğinde yatmaktadır.

Herhangi bir büyük kod bloğunda olduğu gibi, onu ayrıştırarak ve kod parçalarını bu kod bloğunun amacına göre adlandırılan bir yöntem çağrısıyla değiştirerek amacınızı daha net hale getirebilirsiniz. > Koşullarla, bunu koşullu kısım ve alternatiflerin her biri için yaparak daha fazla fayda elde edebilirsiniz. Bu şekilde durumu vurgular ve neye daldığınızı net bir şekilde ortaya koyarsınız. Ayrıca dallanmanın nedenini de vurguluyorsunuz. "

Ve evet cevabı Java uygulamaları için de geçerli. Örneklerde olsa da, koşullu işlevi bir değişkene atamaz.

Bu iyi bir uygulama değil. Yakında kafanız karışacak. Genel bir hataya benziyor: kötüye kullanım "=" ve "==" operatörleri.

Bunu 2 satır koda bölmelisiniz. Yalnızca kodu daha net hale getirmekle kalmaz, aynı zamanda gelecekte yeniden düzenlemeyi de kolaylaştırır. IF koşulunu değiştirdiğinizi hayal edin? Yanlışlıkla satırı kaldırabilirsiniz ve değişkeniniz artık ona atanan değeri almayacaktır.

Bunu daha çok eski usul bir C tarzı olarak düşünürdüm; JavaScript'te gerçekten iyi bir uygulama değildir, bu yüzden bundan kaçınmalısınız.

şöyle bir şey yapabilirsin:

if (value = /* sic */ some_function()){
  use_value(value)
}

Buraya golang'dan geldim, burada yaygın olarak

if (err := doSomething(); err != nil) {
    return nil, err
}

Hangi erro kapsama aldığı ifblokta sadece. Bu nedenle, es6'da yaptığım şey şu ki, oldukça çirkin görünüyor, ancak benim oldukça katı eslint kurallarımı sızlatmıyor ve aynı şeyi başarıyor.

{
  const err = doSomething()
  if (err != null) {
    return (null, err)
  }
}

Ekstra kaşlı ayraçlar yeni bir "sözcük kapsamı" mı tanımlar? Bu da kullanabileceğim constve errdış blokta müsait olmadığım anlamına geliyor .