关于使用神奇数字，还有一个问题没有被提及……

如果你有很多这样的数字，很有可能你有两个不同的目的，你使用魔法数字，其中的值碰巧是相同的。

然后，果然，你需要改变值…只有一个目的。

Magic Number是一个硬编码的值，它可能在以后的阶段更改，但因此很难更新。

例如，假设您有一个页面，在“您的订单”概览页面中显示最近50个订单。50在这里是一个神奇的数字，因为它不是通过标准或惯例设置的，它是您根据规范中概述的原因虚构的数字。

现在，你要做的是你在不同的地方有50个-你的SQL脚本(SELECT TOP 50 * FROM orders)，你的网站(你的最后50个订单)，你的订单登录(for (i = 0;I < 50;i++))和可能的许多其他地方。

那么，如果有人决定把50岁改成25岁，会发生什么呢?还是75年?还是153年?现在你必须在所有的地方替换50，你很可能会错过它。Find/Replace可能不起作用，因为50可能用于其他事情，盲目地将50替换为25可能会产生一些其他不良副作用(即你的Session)。Timeout = 50呼叫，也设置为25，用户开始报告太频繁的超时)。

此外，代码可能很难理解，即。"if a < 50那么bla"——如果你在一个复杂的函数中遇到这种情况，其他不熟悉代码的开发人员可能会问自己"WTF是50?? "

这就是为什么最好在1个地方有这样模糊和任意的数字-“const int NumOrdersToDisplay = 50”，因为这使代码更具可读性(“如果< NumOrdersToDisplay”，这也意味着你只需要在1个定义良好的地方更改它。

适用Magic Numbers的地方是通过标准定义的所有内容，即SmtpClient。DefaultPort = 25或TCPPacketSize =任何(不确定是否标准化)。此外，只在一个函数中定义的所有内容都可能是可接受的，但这取决于上下文。

在类的顶部用默认值初始化一个变量怎么样?例如:

public class SomeClass {
    private int maxRows = 15000;
    ...
    // Inside another method
    for (int i = 0; i < maxRows; i++) {
        // Do something
    }

    public void setMaxRows(int maxRows) {
        this.maxRows = maxRows;
    }

    public int getMaxRows() {
        return this.maxRows;
    }

在这种情况下，15000是一个神奇的数字(根据CheckStyles)。对我来说，设置一个默认值是可以的。我不想做的事情是:

private static final int DEFAULT_MAX_ROWS = 15000;
private int maxRows = DEFAULT_MAX_ROWS;

这会让它更难读吗?在安装CheckStyles之前，我从未考虑过这一点。

你看过维基百科上关于魔法数字的词条了吗?

它详细介绍了魔术数字引用的所有方式。下面是关于魔术数字是一种糟糕的编程实践的引用

“魔数”一词也指在源代码中直接使用数字而不作解释的糟糕编程实践。在大多数情况下，这会使程序更难阅读、理解和维护。尽管大多数指南对数字0和1做了例外处理，但在代码中将所有其他数字定义为命名常量是个好主意。

关于使用神奇数字，还有一个问题没有被提及……

如果你有很多这样的数字，很有可能你有两个不同的目的，你使用魔法数字，其中的值碰巧是相同的。

然后，果然，你需要改变值…只有一个目的。

提取一个神奇数字作为常数的另一个优点是可以清楚地记录业务信息。

public class Foo {
    /** 
     * Max age in year to get child rate for airline tickets
     * 
     * The value of the constant is {@value}
     */
    public static final int MAX_AGE_FOR_CHILD_RATE = 2;

    public void computeRate() {
         if (person.getAge() < MAX_AGE_FOR_CHILD_RATE) {
               applyChildRate();
         }
    }
}