在条件陈述中分配变量,良好实践与否？

Question

一年前,我从经典的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开发人员眼中这是否是一种好的做法？

Answer 1

我不推荐它.问题是,在您尝试比较值时,它看起来像是一个常见错误,但使用单个=而不是==或===.例如,当你看到这个:

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

你不知道他们打算做什么,或者他们打算写这个:

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

如果您真的想要进行任务,我建议您进行明确的比较:

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

Answer 2

我没有看到证据证明这不是好的做法.是的,它可能看起来像是一个错误,但这很容易通过明智的评论来弥补.举个例子:

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

为什么要允许该函数第二次运行:

alert(processorIntensiveFunction());

因为LOOKS的第一个版本不好？我不能同意这种逻辑.

Answer 3

我做了很多次.为了绕过JavaScript警告,我添加了两个parens:

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

你应该避免使用它,如果你真的想使用它,在它上面写一条评论,说明你在做什么.

Answer 4

有一种情况，当你这样做时，使用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第 9 行的 -loop。在那里，读取一个新行并将其存储在变量中，然后运行循环的内容。我知道这不是一个if- 语句，但我想您的问题中也可以包含 while 循环。

这样做的原因是，当使用 a 时FileInputStream，每次调用 时FileInputStream.readLine()，它都会读取文件中的下一行，因此，如果您从循环中调用它而不fileIn.readLine() != null分配变量，而不是调用(currentLine = fileIn.readLine()) != null，然后从在循环内部，您也只会得到每隔一行的信息。

希望您能理解，祝您好运！

Answer 5

如果您要参考 Martin Fowlers 的书《重构改进现有代码的设计》！那么有几种情况是很好的做法，例如。使用函数或方法调用来断言您的情况的长复杂条件：

“动机

程序中最常见的复杂性领域之一在于复杂的条件逻辑。当您编写代码来测试条件并根据不同的条件执行各种操作时，您很快就会得到一个相当长的方法。方法的长度本身就是使其难以阅读的一个因素，但条件会增加难度。问题通常在于这样一个事实：条件检查和操作中的代码告诉您发生了什么，但很容易掩盖为什么会发生。

与任何大型代码块一样，您可以通过分解代码块并将代码块替换为以该代码块的意图命名的方法调用来使您的意图更清晰。> 有了条件，您可以通过对条件部分和每个替代方案执行此操作来获得更多好处。通过这种方式，您可以突出显示条件并清楚地表明您正在分支的内容。您还强调了分支的原因。”

是的，他的答案对于 Java 实现也有效。尽管在示例中，它并未将条件函数分配给变量。

Answer 6

你也可以用Java做到这一点.不,这不是一个好习惯.:)

(并使用===Javascript中的类型相等.阅读Crockford的关于JS的The Good Parts一书.)