如何处理Findbugs“可序列化类中的非瞬态不可序列化实例字段”？

Question

考虑下面的类。如果我对它运行Findbugs，它将在第5行而不是在第7行给我一个错误（“可序列化类中的非瞬态不可序列化实例字段”）。

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却没有实现。但是，最佳实践是针对接口而不是具体的实现进行编码。

我该如何最好地处理呢？

我可以在第3行上添加一个@Suppresswarnings（justification =“ No bug”，values =“
SE_BAD_FIELD”）。我的实际代码中有很多Sets和Lists，我担心它会乱码。

有更好的方法吗？

Answer 1

但是，最佳实践是针对接口而不是具体的实现进行编码。

我认为不可以，在这种情况下不是。Findbugs完全正确地告诉您，NotSerializableException一旦您Set在该字段中拥有不可序列化的实现，便有可能碰上a
。这是您应该处理的事情。如何，这取决于您的班级设计。

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