在布尔/条件表达式中包含副作用代码是不好的做法吗？

Question

而不是做传统的

if

表达：

if(user.isActive) dispatch(RedirectUser(...)); 

你像这样内联它：

user.isActive && dispatch(RedirectUser(...));

截取的代码工作正常，只是想知道这是否是糟糕的编程风格。

Answer 1

好吧，我认为无论哪种方式，它们都是条件表达式，所以这部分是不错的做法。

只是表达式的返回值不同，使用

&&

你最终会得到一些

false

或返回类型

dispatch

的可能性，而使用显式 if 语句，调度的返回类型是唯一的可能性。

此外，它的可读性主要受到影响，但并不那么重要，取决于您正在处理的内容。