Tetapkan variabel dalam pernyataan kondisi apakah, praktik yang baik atau tidak? [Tutup]

Saya pindah satu tahun yang lalu dari bahasa OO klasik seperti Java ke JavaScript. Kode berikut jelas tidak direkomendasikan (atau bahkan tidak benar) di Java:

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;
}

Pada dasarnya saya baru tahu bahwa saya dapat menetapkan variabel ke nilai dalam pernyataan kondisi if, dan segera memeriksa nilai yang ditetapkan seolah-olah itu adalah boolean.

Untuk taruhan yang lebih aman, saya biasanya memisahkannya menjadi dua baris kode, tetapkan terlebih dahulu lalu periksa variabelnya, tetapi sekarang saya menemukan ini, saya hanya ingin tahu apakah ini praktik yang baik atau tidak di mata pengembang JavaScript berpengalaman?

Saya tidak akan merekomendasikannya. Masalahnya adalah, ini terlihat seperti kesalahan umum saat Anda mencoba membandingkan nilai, tetapi menggunakan nilai tunggal, =bukan ==atau ===. Misalnya, saat Anda melihat ini:

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

Anda tidak tahu apakah itu yang ingin mereka lakukan, atau apakah mereka bermaksud menulis ini:

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

Jika Anda benar-benar ingin melakukan tugas tersebut, saya akan merekomendasikan untuk melakukan perbandingan eksplisit juga:

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

Saya tidak melihat bukti bahwa ini bukan praktik yang baik. Ya, ini mungkin terlihat seperti kesalahan tetapi itu mudah diatasi dengan komentar yang bijaksana. Ambil contoh:

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

Mengapa fungsi itu diizinkan untuk berjalan untuk kedua kalinya dengan:

alert(processorIntensiveFunction());

Karena versi pertama TERLIHAT buruk? Saya tidak setuju dengan logika itu.

Saya melakukannya berkali-kali. Untuk melewati peringatan JavaScript, saya menambahkan dua tanda kurung:

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

Anda harus menghindarinya, jika Anda benar-benar ingin menggunakannya, tulis komentar di atasnya dengan mengatakan apa yang Anda lakukan.

Anda juga dapat melakukan ini di Java. Dan tidak, ini bukan praktik yang baik. :)

(Dan gunakan ===dalam Javascript untuk kesetaraan yang diketik. Baca buku Crockford's The Good Parts di JS.)

Ada satu kasus saat Anda melakukannya, dengan while-loops.
Saat membaca file, biasanya Anda melakukan seperti ini:

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
        }
    }
}

Lihat while-loop pada baris 9. Di sana, baris baru dibaca dan disimpan dalam variabel, dan kemudian isi loop dijalankan. Saya tahu ini bukan if-statement, tapi saya rasa beberapa saat loop dapat dimasukkan dalam pertanyaan Anda juga.

Alasan untuk ini adalah bahwa ketika menggunakan FileInputStream, setiap kali Anda memanggil FileInputStream.readLine(), itu membaca baris berikutnya dalam file, jadi jika Anda akan memanggilnya dari perulangan dengan hanya fileIn.readLine() != nulltanpa menetapkan variabel, daripada memanggil (currentLine = fileIn.readLine()) != null, dan kemudian memanggilnya dari di dalam loop juga, Anda hanya akan mendapatkan setiap baris kedua.

Semoga Anda mengerti, dan semoga berhasil!

Anda juga dapat melakukan tugas dalam pernyataan if di Java. Contoh yang bagus adalah membaca sesuatu dan menuliskannya:

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

Kode:

// 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();
}

Jika Anda merujuk ke buku Martin Fowlers Refactoring memperbaiki desain kode yang ada ! Lalu ada beberapa kasus di mana itu akan menjadi praktik yang baik misalnya. persyaratan kompleks yang panjang untuk menggunakan panggilan fungsi atau metode untuk menegaskan kasus Anda:

"Motivasi

Salah satu area kompleksitas yang paling umum dalam sebuah program terletak pada logika kondisional yang kompleks. Saat Anda menulis kode untuk menguji kondisi dan melakukan berbagai hal tergantung pada berbagai kondisi, Anda akan segera berakhir dengan metode yang cukup panjang. Panjang suatu metode dengan sendirinya merupakan faktor yang membuatnya lebih sulit untuk dibaca, tetapi kondisi menambah kesulitan. Masalahnya biasanya terletak pada fakta bahwa kode, baik dalam pemeriksaan kondisi dan tindakan, memberi tahu Anda apa yang terjadi tetapi dapat dengan mudah mengaburkan mengapa itu terjadi.

Seperti halnya blok kode besar lainnya, Anda dapat memperjelas maksud Anda dengan mendekomposisi dan mengganti potongan kode dengan pemanggilan metode yang dinamai sesuai maksud blok kode tersebut. > Dengan kondisi Anda dapat menerima manfaat lebih lanjut dengan melakukan ini untuk bagian bersyarat dan setiap alternatif. Dengan cara ini Anda menyorot kondisi dan membuatnya dengan jelas> bercabang. Anda juga menyoroti alasan percabangan tersebut. "

Dan ya jawabannya juga berlaku untuk implementasi Java. Itu tidak menetapkan fungsi kondisional ke variabel meskipun dalam contoh.

Ini bukan praktik yang baik. Anda akan segera bingung tentang itu. Ini terlihat serupa dengan kesalahan umum: operator misuse "=" dan "==".

Anda harus memecahnya menjadi 2 baris kode. Ini tidak hanya membantu membuat kode lebih jelas, tetapi juga mudah untuk difaktor ulang di masa mendatang. Bayangkan Anda mengubah kondisi IF? Anda mungkin tidak sengaja menghapus garis dan variabel Anda tidak lagi mendapatkan nilai yang ditetapkan padanya.

Saya akan menganggap ini lebih dari gaya C sekolah tua; ini bukan praktik yang baik dalam JavaScript jadi Anda harus menghindarinya.

Anda bisa melakukan sesuatu seperti ini:

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

Saya datang ke sini dari golang, di mana melihat sesuatu seperti itu biasa

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

Di mana errhanya dicakup ke ifblok itu. Karena itu, inilah yang saya lakukan di es6, yang tampaknya cukup jelek, tetapi tidak membuat aturan eslint saya yang agak ketat berubah, dan mencapai hal yang sama.

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

Tanda kurung kurawal menentukan, eh, "lingkup leksikal" baru? Artinya saya bisa menggunakan const, dan errtidak tersedia untuk blok luar.