带有警告'100'的CheckStyle是一个神奇的数字

Question

在我的代码中，它显示消息100的警告是一个神奇的数字。请参阅以下代码，

int randomNo = generator.nextInt(100);

我在这里读到了什么是神奇的数字，为什么它不好？但我怀疑通过创建静态变量来声明100会占用更多空间，因为我在一个地方使用它。这是解决这个问题的正确方法吗？

  public static final int HUNDRED= 100;

有什么建议吗？

Answer 1

最好不要编写最短的代码，而是编写易于理解和维护的代码。将这个100存储为常量，你可以添加一个好名字来解释为什么它真的是100。例如，如果您想要生成随机分数，并且您的最大可能分数是100，那么您可以定义

static final int MAX_SCORE = 100;

之后，你也可以在其他地方使用它。每个人都会明白为什么是100，而不是别的。如果有一天你需要将它更改为200，你将不得不只在一个地方替换它，而不必搜索代码。

此外，在程序的其他部分，可能会有100个具有不同含义（例如，MAX_PERCENT）。如果您想将MAX_SCORE更改为200，但将MAX_PERCENT保持原样，那么如果您有单独的常数，这会容易得多。

Answer 2

它实际上与存储无关，而是可读性。如果您想更改一些数字，那么在代码中很难找到，如果它位于顶部，则更好（如果它位于配置文件中，在许多情况下更好）。

是的，这是一个很好的解决方案。

如果你不需要它以外的文件，你应该使它"私有"，你可能想要更可读，并使用一个名称，表明它的真正含义，如：

MAX_RANDOM_NUMBER=100

最好包括它的用途

MAX_RANDOM_FOR_CARD_SELECTION

或者类似的。

这样，当你在5个月后查看该文件时，因为你添加了20张新卡，你甚至不用看代码就可以清楚地看到你必须更改什么。

Answer 3

嗯HUNDRED确实有点傻，但是你为什么选择100，它的含义是什么？

比如：

public static final int RANDOM_UPPER_LIMIT=100;

或者更具信息量的东西，这取决于您将值用于什么：

public static final int MAX_NUMBER_OF_COLORS=100;

会更有意义，提高可读性。

不应考虑节省空间。在这种情况下，声明变量的空间开销（如果有）完全可以忽略不计。