java - 在 while 条件下添加到集合中的良好做法？

Question

我正在尝试向集合中添加一个随机数。如果它已经在集合中，则循环应继续并重试。

哪个是更好的做法，

do {
  nextChosenInt = rand.nextInt(48) + 1;
  addFailed = !chosenInts.add(nextChosenInt);
}
while (addFailed);

或

do {
    nextChosenInt = rand.nextInt(48) + 1;
}
while (!chosenInts.add(nextChosenInt));

Answer 1

我会在这里投票支持第二个选项，但稍有改动。

您正在利用 .add 方法返回一个 boolean 值的事实，这是一个有益的事实，因为您不需要有一个冗余的标志变量，从代码的上下文来看，该变量除了终止循环之外没有其他目的。

但就我个人而言，我会选择提取:

!chosenInts.add(nextChosenInt)

进入它自己的方法，该方法更具描述性和可读性。通过这样做，您可以让任何人在不了解 Collections API 的情况下了解此情况。