只是为了添加实例初始化块而创建的匿名类的意外后果

Question

这是一个关于 Java 代码的问题，例如：

List<String> list = new ArrayList<String>() {{add("hello"); add("goodbye");}}

程序员匿名扩展 ArrayList 只是为了插入实例初始化块。

问题是：如果程序员的唯一意图仅仅是实现以下目标：

List<String> list = new ArrayList<String>();
list.add("hello");
list.add("goodbye");    

那么第一种方式会产生什么意想不到的后果？

Answer 1

执行此类代码（一般情况下）的危险在于您可能会破坏 equals() 方法。 这是因为 equals() 有两个通用模板：

public boolean equals(Object ob) {
  if (!(ob instanceof MyClass)) return false;
  ...
}

第一个模板

public boolean equals(Object ob) {
  if (ob.getClass() != getClass()) return false;
  ...
}

仍然适用于您正在讨论的匿名子类，但第二个模板则不行。 事实是，第二个被认为是最佳实践，因为instanceof不一定可交换（意味着a.equals(b)可能不等于b.equals(a) ）。

特别是在这种情况下，ArrayList 使用 AbstractList.equals() 方法，该方法仅检查另一个对象是否是接口 的实例列表，所以你没问题。

然而，这是需要注意的事情。

我建议采取稍微不同的做法：

List<String> list = new ArrayList<String>(
    Arrays.asList("hello", "goodbye")
);

当然，它更冗长，但这样你不太可能遇到麻烦，因为生成的类是一个“纯”ArrayList。

Answer 2

我认为这里最大的问题是你正在创建周围类的内部类。 您创建的列表将具有对外部类的隐式 this 引用。

这通常不是问题。 但是，如果您序列化此对象（我特别考虑使用 XStream 因为我自己也遇到过这个问题) 那么序列化的形式将包含序列化的外部类。 这根本不是您通常所期望的，并且可能会导致奇怪的行为！

我经常使用上面的初始化模式。 只要您了解内部类机制，那么这不是问题。 顺便提一句。 我不认为平等问题是一个超出一般推导的平等实现的问题。

Answer 3

对代码性能和/或内存的影响可以忽略不计（即使是可测量的）。

这里真正的打击是你让人们停下来思考“这个人在做什么？”

如果我在阅读你的代码时必须这样做 1,000 次，那就存在性能问题。

代码应该易于阅读——只要满足所有性能、完整性和标准要求。

您很可能将成为维护代码的人。 善待未来的自己，让代码尽可能易于阅读。 如果您要做一些类似的奇特的事情，请有一个充分的理由并在代码中记录它。 如果唯一的原因是炫耀你作为程序员的能力，那么你就没有完成程序员的真正工作：让困难变得简单。

Answer 4

第一种方法也不利于性能。 用大量匿名类扰乱内存是不好的。 而且启动时速度也会变慢。