比较Java枚举成员：==或equals（）？

我知道Java枚举是用私有构造函数和一堆公共静态成员编译成类的。当比较给定枚举的两个成员时，我总是使用.equals（），例如。

public useEnums(SomeEnum a)
{
    if(a.equals(SomeEnum.SOME_ENUM_VALUE))
    {
        ...
    }
    ...
}

然而，我刚刚遇到一些使用equals运算符==而不是.equals（）的代码：

public useEnums2(SomeEnum a)
{
    if(a == SomeEnum.SOME_ENUM_VALUE)
    {
        ...
    }
    ...
}

我应该使用哪个操作员？

当前回答

enums容易与==一起工作的原因是每个定义的实例也是一个单实例。因此，使用==进行身份比较总是有效的。

但使用==，因为它与枚举一起使用，意味着所有代码都与该枚举的使用紧密耦合。

例如：枚举可以实现接口。假设您当前正在使用实现Interface1的枚举。若稍后，有人将其更改或引入一个新的类Impl1作为同一接口的实现。然后，如果您开始使用Impl1的实例，由于前面使用了==，您将有很多代码需要更改和测试。

因此，除非有任何正当利益，否则最好遵循被视为良好的做法。

其他回答

在枚举的情况下，两者都是正确的！！

两者在技术上都是正确的。如果查看.equals（）的源代码，它只需遵循==。

然而，我使用==，因为这将是空安全的。

enums容易与==一起工作的原因是每个定义的实例也是一个单实例。因此，使用==进行身份比较总是有效的。

但使用==，因为它与枚举一起使用，意味着所有代码都与该枚举的使用紧密耦合。

因此，除非有任何正当利益，否则最好遵循被视为良好的做法。

使用==比较两个枚举值是有效的，因为每个枚举常量只有一个对象。

另一方面，如果您像这样编写equals（），实际上不需要使用==来编写空安全代码：

public useEnums(final SomeEnum a) {
    if (SomeEnum.SOME_ENUM_VALUE.equals(a)) {
        …
    }
    …
}

这是一个被称为“从左侧比较常量”的最佳实践，您绝对应该遵循。

我更喜欢使用==而不是equals：

除了这里已经讨论过的其他原因之外，还有一个原因，那就是你可能会在没有意识到的情况下引入一个bug。假设你有一个完全相同的enums，但在不同的包中（这并不常见，但可能会发生）：

第一个枚举：

package first.pckg

public enum Category {
    JAZZ,
    ROCK,
    POP,
    POP_ROCK
}

第二个枚举：

package second.pckg

public enum Category {
    JAZZ,
    ROCK,
    POP,
    POP_ROCK
}

然后假设您在item.category中使用了类似next的equals，即first.pckg.category，但您导入了第二个enum（second.pckkg.category），而不是第一个enum，而没有意识到它：

import second.pckg.Category;
...

Category.JAZZ.equals(item.getCategory())

因此，尽管item.getCategory（）是JAZZ，但由于是一个不同的枚举，所以您将始终得到false。这可能有点难以理解。

因此，如果改用运算符==，将出现编译错误：

运算符==不能应用于“second.pkg.Category”、“first.pkg.Ccategory”

import second.pckg.Category; 
...

Category.JAZZ == item.getCategory()

推荐文章