检查 Typescript/Javascript 中与“true”、“false”和“undefined”相关的布尔值

Question

我可以确定布尔检查的唯一结果是“真”还是“假”的明确方式是什么？换句话说，我想尽可能地排除“未定义”的可能性。两种选择是：

功能一：

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

简短的回答是否，它们不等价：

• 无论如何，第一个都不会返回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; 将确保您在何时返回false this.client.services 是 undefined。
• 以这种方式在for 循环中使用return 语句是一个非常非常糟糕的主意，实际上您只会进行一次迭代。