java - 如何处理 Findbugs "Non-transient non-serializable instance field in serializable class"？

Question

考虑下面的类(class)。如果我对它运行 Findbugs，它会在第 5 行但不在第 7 行给我一个错误(“可序列化类中的非 transient 非可序列化实例字段”)。

1 public class TestClass implements Serializable {
2
3  private static final long serialVersionUID = 1905162041950251407L;
4
5  private Set<Integer> mySet;      // Findbugs error
6
7  private HashSet<Integer> myOtherSet;
8
9 }

这是正确的，因为 java.util.Set 从未在其层次结构中实现 Serializable 而 java.util.HashSet 实现了。但是，最佳实践是针对接口(interface)而不是具体实现进行编码。

我怎样才能最好地处理这个问题？

我可以在第 3 行添加 @Suppresswarnings(justification="No bug", values="SE_BAD_FIELD")。我的实际代码中有很多集合和列表，我担心它会乱扔我的代码太多了。

有更好的方法吗？

Answer 1

However it is best practice to code against interfaces instead of concrete implementations.

我认为不，在这种情况下不是。 Findbugs 非常正确地告诉您，一旦在该字段中具有不可序列化的 Set 实现，您就有可能遇到 NotSerializableException。这是你应该处理的事情。如何，这取决于你的类的设计。

• 如果这些集合在类中初始化并且从未从外部设置，那么我认为声明该字段的具体类型绝对没有错，因为无论如何字段都是实现细节。请务必在公共(public)接口(interface)中使用接口(interface)类型。
• 如果通过公共(public)接口(interface)将集合传递到类中，则必须确保它们实际上是 Serializable。为此，请创建一个接口(interface) SerializableSet extends Set, Serializable 并将其用于您的字段。然后，要么:
  • 在公共(public)接口(interface)中使用SerializableSet，并提供实现它的实现类。
  • 检查通过 instanceof Serializable 传递给类的集合，如果不是，则将它们复制到存在的对象中。