กำหนดตัวแปรในคำสั่งเงื่อนไข if วิธีปฏิบัติที่ดีหรือไม่? [ปิด]

ฉันย้ายจากภาษา OO คลาสสิกเช่น Java ไปเป็น JavaScript เมื่อหนึ่งปีก่อน ไม่แนะนำให้ใช้โค้ดต่อไปนี้ (หรือไม่ถูกต้อง) ใน 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;
}

โดยทั่วไปฉันเพิ่งพบว่าฉันสามารถกำหนดตัวแปรให้กับค่าในคำสั่งเงื่อนไข if และตรวจสอบค่าที่กำหนดทันทีราวกับว่าเป็นบูลีน

เพื่อการเดิมพันที่ปลอดภัยยิ่งขึ้นฉันมักจะแยกโค้ดออกเป็นสองบรรทัดกำหนดก่อนแล้วจึงตรวจสอบตัวแปร แต่ตอนนี้ฉันพบสิ่งนี้ฉันแค่สงสัยว่ามันเป็นแนวทางปฏิบัติที่ดีหรือไม่ในสายตาของนักพัฒนา JavaScript ที่มีประสบการณ์?

ฉันจะไม่แนะนำที่นี่ ปัญหาคือดูเหมือนว่าข้อผิดพลาดทั่วไปที่คุณพยายามที่จะเปรียบเทียบค่า แต่ใช้เพียงครั้งเดียว=แทนหรือ== ===ตัวอย่างเช่นเมื่อคุณเห็นสิ่งนี้:

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

คุณไม่รู้ว่านั่นคือสิ่งที่พวกเขาตั้งใจจะทำหรือตั้งใจจะเขียนสิ่งนี้:

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

หากคุณต้องการทำงานที่ได้รับมอบหมายจริงๆฉันขอแนะนำให้ทำการเปรียบเทียบอย่างชัดเจนเช่นกัน:

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

ฉันไม่เห็นข้อพิสูจน์ว่าเป็นการปฏิบัติที่ไม่ดี ใช่อาจดูเหมือนเป็นความผิดพลาด แต่สามารถแก้ไขได้อย่างง่ายดายโดยการแสดงความคิดเห็นอย่างมีวิจารณญาณ ยกตัวอย่างเช่น:

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

เหตุใดจึงควรอนุญาตให้ฟังก์ชันนั้นทำงานเป็นครั้งที่ 2 ด้วย:

alert(processorIntensiveFunction());

เพราะรุ่นแรก LOOKS แย่? ฉันไม่เห็นด้วยกับตรรกะนั้น

ฉันทำมันหลายครั้ง ในการข้ามคำเตือน JavaScript ฉันเพิ่มสอง parens:

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

คุณควรหลีกเลี่ยงถ้าคุณต้องการใช้มันจริงๆให้เขียนความคิดเห็นไว้ข้างบนว่าคุณกำลังทำอะไรอยู่

คุณสามารถทำได้ใน Java ด้วย และไม่มันไม่ใช่แนวทางปฏิบัติที่ดี :)

(และใช้===ใน Javascript เพื่อความเท่าเทียมกันของการพิมพ์อ่านหนังสือ The Good Parts ของ Crockford ใน JS)

มีอยู่กรณีหนึ่งเมื่อคุณทำเช่นนี้กับwhile-loops
เมื่ออ่านไฟล์คุณมักจะทำเช่นนี้:

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

ดูwhile-loop ที่บรรทัด 9 ที่นั่นบรรทัดใหม่จะถูกอ่านและเก็บไว้ในตัวแปรจากนั้นเนื้อหาของลูปจะถูกรัน ฉันรู้ว่านี่ไม่ใช่if-statement แต่ฉันเดาว่า while loop อาจรวมอยู่ในคำถามของคุณด้วย

เหตุผลก็คือเมื่อใช้ a FileInputStreamทุกครั้งที่คุณเรียกFileInputStream.readLine()มันจะอ่านบรรทัดถัดไปในไฟล์ดังนั้นหากคุณจะเรียกมันจากลูปfileIn.readLine() != nullโดยไม่ต้องกำหนดตัวแปรแทนที่จะเรียก(currentLine = fileIn.readLine()) != nullแล้วเรียกมันจาก ภายในลูปด้วยคุณจะได้ทุกบรรทัดที่สองเท่านั้น

หวังว่าคุณจะเข้าใจและขอให้โชคดี!

คุณสามารถกำหนดภายในคำสั่ง if ใน Java ได้เช่นกัน ตัวอย่างที่ดีคือการอ่านบางอย่างและเขียนออกมา:

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

รหัส:

// 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 Refactoring ปรับปรุงการออกแบบโค้ดที่มีอยู่ ! จากนั้นมีหลายกรณีที่ควรปฏิบัติเช่น เงื่อนไขที่ซับซ้อนยาวเพื่อใช้การเรียกใช้ฟังก์ชันหรือเมธอดเพื่อยืนยันกรณีของคุณ:

"แรงจูงใจ

ส่วนที่พบบ่อยที่สุดของความซับซ้อนในโปรแกรมคือตรรกะเงื่อนไขที่ซับซ้อน ในขณะที่คุณเขียนโค้ดเพื่อทดสอบเงื่อนไขและทำสิ่งต่างๆโดยขึ้นอยู่กับเงื่อนไขต่างๆคุณจะจบลงด้วยวิธีการที่ค่อนข้างยาว ความยาวของวิธีการเป็นปัจจัยที่ทำให้อ่านยากขึ้น แต่เงื่อนไขจะเพิ่มความยาก ปัญหามักจะอยู่ที่รหัสทั้งในการตรวจสอบเงื่อนไขและในการดำเนินการบอกคุณว่าเกิดอะไรขึ้น แต่สามารถปิดบังสาเหตุที่เกิดขึ้นได้อย่างง่ายดาย

เช่นเดียวกับบล็อกโค้ดขนาดใหญ่คุณสามารถทำให้ความตั้งใจของคุณชัดเจนขึ้นได้โดยการแยกย่อยและแทนที่โค้ดส่วนหนึ่งด้วยการเรียกเมธอดที่ตั้งชื่อตามจุดประสงค์ของโค้ดบล็อกนั้น > ด้วยเงื่อนไขคุณจะได้รับประโยชน์เพิ่มเติมโดยการทำเช่นนี้สำหรับส่วนที่มีเงื่อนไขและแต่ละทางเลือก ด้วยวิธีนี้คุณจะเน้นเงื่อนไขและทำให้ชัดเจนว่าคุณกำลังแตกกิ่งก้านสาขาอะไร คุณยังเน้นเหตุผลของการแตกแขนง "

และใช่คำตอบของเขาก็ใช้ได้กับการใช้งาน Java มันไม่ได้กำหนดฟังก์ชันเงื่อนไขให้กับตัวแปรแม้ว่าในตัวอย่าง

มันไม่ใช่การปฏิบัติที่ดี คุณจะสับสนในไม่ช้า ดูเหมือนว่าจะคล้ายกับข้อผิดพลาดทั่วไป: ใช้ตัวดำเนินการ "=" และ "==" ในทางที่ผิด

คุณควรแบ่งรหัสออกเป็น 2 บรรทัด ไม่เพียง แต่ช่วยให้โค้ดชัดเจนขึ้น แต่ยังง่ายต่อการ refactor ในอนาคตอีกด้วย ลองนึกภาพว่าคุณเปลี่ยนเงื่อนไข IF? คุณอาจลบบรรทัดโดยไม่ได้ตั้งใจและตัวแปรของคุณไม่ได้รับค่าที่กำหนดให้อีกต่อไป

ฉันจะพิจารณารูปแบบ C แบบโรงเรียนเก่ามากกว่านี้ มันไม่ใช่แนวปฏิบัติที่ดีจริงๆใน JavaScript ดังนั้นคุณควรหลีกเลี่ยง

คุณสามารถทำสิ่งต่างๆดังนี้:

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

ฉันมาที่นี่จากโกลังซึ่งเป็นเรื่องธรรมดาที่จะได้เห็นอะไรแบบนี้

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

ซึ่งerrกำหนดขอบเขตไว้ที่ifบล็อกนั้นเท่านั้น ด้วยเหตุนี้นี่คือสิ่งที่ฉันทำใน es6 ซึ่งดูน่าเกลียด แต่ก็ไม่ได้ทำให้กฎ eslint ที่ค่อนข้างเข้มงวดของฉันกลายเป็นเสียงหอนและประสบความสำเร็จเช่นเดียวกัน

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

เครื่องหมายวงเล็บเสริมกำหนดใหม่เอ่อ "ขอบเขตศัพท์"? ซึ่งหมายความว่าฉันสามารถใช้ได้constและerrไม่สามารถใช้ได้กับบล็อกด้านนอก