如果有条件，则返回set.add（）的布尔值？

set类的add运算符返回一个布尔值，如果元素（要添加的元素）不存在则为true，否则为false。在写字

if (set.add(entry)) {
    //do some more stuff
}

在编写简洁的代码方面被认为是好的风格？我想知道既然您一次执行两项操作。1）添加元素，2）检查元素是否存在。

是的。

使操作返回布尔值的通常点是，您可以使用它来进行决策，即在if构造内。您可以意识到这种潜力的唯一另一种方法是将返回值存储在变量中，然后立即在中使用它if，这很愚蠢，而且肯定不比编写任何方式可取if(operation()) { ... }。因此，只要继续执行此操作，我们就不会（因此）判断您。

我会说它不是尽可能干净，因为它迫使维护者已经知道或查找返回值的含义。这是否意味着已经成功插入了不存在的值？如果您不经常使用它，您将不会知道，即使这样做，也要花很多心思。

我希望以下内容：

boolean added = set.add(entry);

if (added) {
    //do some more stuff
}

是的，有点冗长，但是编译器应该生成几乎完全相同的字节码，即使多年未使用Java集的人也可以遵循逻辑而无需查找任何内容。

如果为true表示成功，那么这是很好的清晰代码。

有一个普遍的约定，即函数或方法在成功时返回true（或某些结果为true）。只要您的代码遵循该规则，我认为将方法置于条件中就可以了。

在我看来，这样的代码不必要地混乱了：

boolean frobulate_succeeded = thing.frobulate();

if (frobulate_succeeded) {
    ...
}

感觉就像您在重复自己。

但是，问题在于返回值的含义。您说“一个布尔值，它指示添加的元素是否已经存在”，这可能意味着true表示该元素存在（并且未发生添加）。如果真是这样，我理想上将方法的返回行为更改为更常规。如果不可能，我将添加一个额外的中间变量，该变量使您可以在代码中清楚地标记返回结果（如其他人所建议）。

我会说这非常像C。在大多数情况下，我希望使用一个描述性名称的变量来表示突变结果，并且在某种if情况下不会发生突变。

如果立即重用此变量，则编译器将消除该变量。人类将更容易阅读源代码。对我来说，这更重要。

如果有人必须通过在if条件中添加and/ or子句来扩展条件，则.add()由于短路评估，在某些情况下，他们可能最终不会调用。除非特别预计会发生短路，否则可能会导致错误。

您的代码似乎破坏了命令查询分离。清洁代码书和功能结构视频中对此进行了讨论。因此，从“简洁代码”的角度来看，我认为它不是好样式。

“干净的代码既简单又直接。干净的代码读起来像写得很好的散文。简洁的代码永远不会掩盖设计者的意图，而是充满清晰的抽象和直接的控制线。

-Grady Booch的《面向对象的分析和应用程序设计》一书的作者”

对我来说，您代码的意图尚不清楚。如果在添加条目成功时成功执行了if，或者在已经存在时成功执行了if操作？什么add()回报？该项目？错误代码？