检查Typescript / Javascript中的布尔值，分别与“ true”，“ false”和“ undefined”有关

Question

我可以确保布尔检查的唯一结果是“ true”或“ false”的显式方法是什么？ 换句话说，我想尽可能排除“ undefined”。 两种选择是：

功能一：

private canMove = (currentOptionSelected): boolean => {
    if (this.client.services) {
        for (const service of this.client.services) {
            if (service === currentOptionSelected) {
                if (service.currentStage === 'enrolling') {
                    return true;
                }
            }
        }
    } 
}

功能二：

private canMove = (currentOptionSelected): boolean => {
    if (this.client.services) {
        for (const service of this.client.services) {
            if (service === currentOptionSelected) {
                return service.currentStage === 'enrolling';
            }
        }
    }
}

编辑：在评论者的回应下，一个更可靠的选择是显式返回'false'，如下所示：

private canMove = (currentOptionSelected): boolean => {
    if (this.client.services) {
        for (const service of this.client.services) {
            if (service === currentOptionSelected) {
                //You should also rethink this return statement
                return service.currentStage === 'enrolling';
            }
        }
    }
    return false;
}

我将通过询问是否存在“ client.services”的情况添加一个额外的“ return'false'”来跟进，但是“ currentStage！=='enrolling”会更好吗？ 还是在这种情况下第二个“ else”子句是多余的？

其次，他写道，我应该根据函数中的返回位置重新考虑return语句。 替代方案是什么？ 简而言之，我正在尝试找到最健壮但简洁的方法来编写此函数。

Answer 1

这些不相等。 在第一个版本返回undefined某些情况下，第二个版本返回false 。 调用者检查该值是否明确=== false将观察到不同的行为。

两者都可以返回undefined ，这可能不是很大。 最好总是返回true或false的实际值

Answer 2

简短的回答是No ，它们不是等效的：

• 在任何情况下，第一个都不会返回false 。
• 如果service.currentStage !== 'enrolling'则第二个可以返回false 。

但是正如Ryan所述，两者都可以返回undefined ，您应该避免这种情况，只要不满足条件，就需要显式返回false 。

这应该是您的代码：

private canMove = (currentOptionSelected): boolean => {
    if (this.client.services) {
        for (const service of this.client.services) {
            if (service === currentOptionSelected) {
                //You should also rethink this return statement
                return service.currentStage === 'enrolling';
            }
        }
    }
    return false;
}

注意：

• return false; 当this.client.services undefined时，这里将确保您返回false 。
• 以这种方式在for循环中使用return语句是一个非常糟糕的主意，实际上您只会进行一次迭代。