在if条件语句中分配变量，良好做法？[关闭]

一年前，我从Java之类的经典OO语言转移到了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);
}

为什么应该让该函数第二次运行：

alert(processorIntensiveFunction());

因为第一个版本看起来不好？我不同意这种逻辑。

我做了很多次。为了绕过JavaScript警告，我添加了两个括号：

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

您应该避免使用它，如果您真的想使用它，请在它上面写一个注释，说明您在做什么。

您也可以在Java中执行此操作。不，这不是一个好习惯。:)

（并使用===Javascript中的类型相等性。请阅读Crockford的JS的The Good Parts书。）

使用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 -statement，但是我想您的问题中也可以包含while循环。

这样做的原因是FileInputStream，每次使用时，每次调用时FileInputStream.readLine()，它都会读取文件的下一行，因此，如果您fileIn.readLine() != null不指定变量就直接从循环中调用它，而不是调用(currentLine = fileIn.readLine()) != null它，则可以，然后从中调用它。在循环内部，您也只会得到第二行。

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

您也可以在Java中的if语句中进行赋值。一个很好的例子是读一些东西然后写出来：

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的《重构改进现有代码的设计》一书！然后，在某些情况下，这将是一个很好的做法，例如 长的复杂条件，使用函数或方法调用来断言您的情况：

“动机

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

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

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

这不是一个好习惯。您很快会对此感到困惑。它看起来与一个常见错误类似：滥用“ =”和“ ==”运算符。

您应该将其分成2行代码。它不仅有助于使代码更清晰，而且将来也易于重构。假设您更改了IF条件？您可能会意外删除该行，并且变量不再获得为其分配的值。

我认为这更像是老式的C风格。在JavaScript中，这并不是真正的好习惯，因此您应该避免使用它。

您可以这样做：

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

我是从golang来到这里的，通常会看到类似

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

其中仅限err于该if块。因此，这就是我在es6中所做的事情，这看起来很丑陋，但是并没有使我相当严格的sslint规则发生变化，并且达到了同样的效果。

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

多余的花括号定义了一个新的“词汇范围”吗？这意味着我可以使用const，而err外部块不可用。