NullPointerException в hashCode() при десериализации графа объекта

Question

У меня есть поле final в классе Java, который обязательно будет инициализирован. Когда я сериализую и десериализую соответствующий граф объектов, я получаю NPE, потому что это поле используется в методе hashCode(), но, по-видимому, еще не прочитан. Я создал минимальный тестовый пример, который также содержит методы serialize()/deserialize() (в случае, если они являются ошибками), но я не могу понять, в чем заключается (лежащая в основе) проблема или как ее обойти.

public class TestSerializerTest { 

private static final org.slf4j.Logger logger = org.slf4j.LoggerFactory.getLogger(TestSerializerTest.class); 

private File workDir; 

@Before 
public void setUp() { 
    workDir = new File("target/tmp"); 
} 

@After 
public void tearDown() { 
    workDir.delete(); 
} 

private static class ManagingContainer implements Serializable { 

    private static final long serialVersionUID = 1L; 

    private final Set<Contained> containedElements = new HashSet<Contained>(); 

    public Contained getContained(List<String> descriptors) { 
     Contained contained = new Contained(this, descriptors); 
     containedElements.add(contained); 
     return contained; 
    } 
} 

private static class Contained implements Serializable { 

    private static final long serialVersionUID = 1L; 

    @SuppressWarnings("unused") 
    private final ManagingContainer container; 
    private final List<String> descriptors; 

    Contained(ManagingContainer container, List<String> descriptors) { 
     this.container = container; 
     if (descriptors == null) { 
      throw new NullPointerException(); 
     } 
     this.descriptors = new ArrayList<String>(descriptors); 
    } 

    @Override 
    public int hashCode() { 
     return descriptors.hashCode(); 
    } 

    @Override 
    public boolean equals(Object other) { 
     if (this == other) { 
      return true; 
     } 
     if (other == null || !(other instanceof Contained)) { 
      return false; 
     } 
     return this.descriptors.equals(((Contained) other).descriptors); 
    } 
} 

private static class OtherContainer implements Serializable { 
    private static final long serialVersionUID = 1L; 

    private final ArrayList<Contained> containedElements = new ArrayList<Contained>(); 

    public OtherContainer(Contained initialElement) { 
     this.containedElements.add(initialElement); 
    } 

    public void addContained(Contained nextElement) { 
     containedElements.add(nextElement); 
    } 
} 

void serializeObjectToFile(Serializable serializable, File file) { 
    logger.info("Saving object '{}' to file '{}'.", serializable, file.getAbsolutePath()); 
    ObjectOutputStream stream = null; 
    try { 
     if (!file.getParentFile().exists()) { 
      logger.info("Creating directory '{}'.", file.getParentFile().getAbsolutePath()); 
      file.getParentFile().mkdirs(); 
     } 
     stream = new ObjectOutputStream(new FileOutputStream(file)); 
     stream.writeObject(serializable); 
     stream.flush(); 
    } catch (Exception exception) { 
     throw new RuntimeException(exception); 
    } finally { 
     if (stream != null) { 
      try { 
       stream.close(); 
      } catch (IOException exc) { 
       logger.error("Error closing stream.", exc); 
      } 
     } 
    } 
} 

Object deserializeObjectFromFile(File file) { 
    ObjectInputStream oiStream = null; 
    try { 
     logger.info("Loading object from file '{}'.", file.getAbsolutePath()); 
     oiStream = new ObjectInputStream(new FileInputStream(file)); 
     return oiStream.readObject(); 
    } catch (Exception exc) { 
     logger.error("Exception loading object from file '{}'. Ignoring file!", file.getAbsolutePath(), exc); 
     throw new RuntimeException(exc); 
    } finally { 
     if (oiStream != null) { 
      try { 
       oiStream.close(); 
      } catch (IOException exc) { 
       logger.error("Error closing stream.", exc); 
      } 
     } 
    } 
} 

@Test 
public void testSerializeDeserialize() { 
    ManagingContainer container = new ManagingContainer(); 
    OtherContainer serializable = new OtherContainer(container.getContained(new ArrayList<String>())); 
    serializable.addContained(container.getContained(new ArrayList<String>())); 
    File file = new File(workDir, "test.ser"); 
    serializeObjectToFile(serializable, file); 
    OtherContainer result = (OtherContainer) deserializeObjectFromFile(file); 
} 
} 

Выполнение этого теста создает следующие NullPointerException:

java.lang.NullPointerException: null 
at TestSerializerTest$Contained.hashCode(TestSerializerTest.java:67) ~[test-classes/:na] 
at java.util.HashMap.put(HashMap.java:372) ~[na:1.6.0_29] 
at java.util.HashSet.readObject(HashSet.java:292) ~[na:1.6.0_29] 
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[na:1.6.0_29] 
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39) ~[na:1.6.0_29] 
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25) ~[na:1.6.0_29] 
at java.lang.reflect.Method.invoke(Method.java:597) ~[na:1.6.0_29] 
at java.io.ObjectStreamClass.invokeReadObject(ObjectStreamClass.java:974) ~[na:1.6.0_29] 
at java.io.ObjectInputStream.readSerialData(ObjectInputStream.java:1848) ~[na:1.6.0_29] 
at java.io.ObjectInputStream.readOrdinaryObject(ObjectInputStream.java:1752) ~[na:1.6.0_29] 
at java.io.ObjectInputStream.readObject0(ObjectInputStream.java:1328) ~[na:1.6.0_29] 
at java.io.ObjectInputStream.defaultReadFields(ObjectInputStream.java:1946) ~[na:1.6.0_29] 
at java.io.ObjectInputStream.readSerialData(ObjectInputStream.java:1870) ~[na:1.6.0_29] 
at java.io.ObjectInputStream.readOrdinaryObject(ObjectInputStream.java:1752) ~[na:1.6.0_29] 
at java.io.ObjectInputStream.readObject0(ObjectInputStream.java:1328) ~[na:1.6.0_29] 
at java.io.ObjectInputStream.defaultReadFields(ObjectInputStream.java:1946) ~[na:1.6.0_29] 
at java.io.ObjectInputStream.readSerialData(ObjectInputStream.java:1870) ~[na:1.6.0_29] 
at java.io.ObjectInputStream.readOrdinaryObject(ObjectInputStream.java:1752) ~[na:1.6.0_29] 
at java.io.ObjectInputStream.readObject0(ObjectInputStream.java:1328) ~[na:1.6.0_29] 
at java.io.ObjectInputStream.readObject(ObjectInputStream.java:350) ~[na:1.6.0_29] 
at java.util.LinkedList.readObject(LinkedList.java:964) ~[na:1.6.0_29] 
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[na:1.6.0_29] 
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39) ~[na:1.6.0_29] 
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25) ~[na:1.6.0_29] 
at java.lang.reflect.Method.invoke(Method.java:597) ~[na:1.6.0_29] 
at java.io.ObjectStreamClass.invokeReadObject(ObjectStreamClass.java:974) ~[na:1.6.0_29] 
at java.io.ObjectInputStream.readSerialData(ObjectInputStream.java:1848) ~[na:1.6.0_29] 
at java.io.ObjectInputStream.readOrdinaryObject(ObjectInputStream.java:1752) ~[na:1.6.0_29] 
at java.io.ObjectInputStream.readObject0(ObjectInputStream.java:1328) ~[na:1.6.0_29] 
at java.io.ObjectInputStream.defaultReadFields(ObjectInputStream.java:1946) ~[na:1.6.0_29] 
at java.io.ObjectInputStream.readSerialData(ObjectInputStream.java:1870) ~[na:1.6.0_29] 
at java.io.ObjectInputStream.readOrdinaryObject(ObjectInputStream.java:1752) ~[na:1.6.0_29] 
at java.io.ObjectInputStream.readObject0(ObjectInputStream.java:1328) ~[na:1.6.0_29] 
at java.io.ObjectInputStream.readObject(ObjectInputStream.java:350) ~[na:1.6.0_29] 
at TestSerializerTest.deserializeObjectFromFile(TestSerializerTest.java:128) [test-classes/:na] 
at TestSerializerTest.testSerializeDeserialize(TestSerializerTest.java:151) [test-classes/:na] 

Насколько я понимаю, сериализации, это не должно быть проблемой (см также Does Java Serialization work for cyclic references? и https://softwareengineering.stackexchange.com/questions/151055/what-happens-if-we-serialize-and-deserialize-two-objects-which-references-to-eac).

В настоящее время я понятия не имею, как действовать. Любые подсказки очень ценятся! Благодаря!

Answer 1

Проблема, по-видимому, вызвана круговыми отношениями между ManagingContainer и Container. Когда контейнер десериализуется, он, в свою очередь, десериализует ManagingContainer, на который ссылается свойство «container». Однако, поскольку этот десериализует, он пытается заполнить HashSet контейнером, который находится в процессе десериализации.

Если вы сериализуете/десериализуете вместо своего ManagingContainer, он должен работать нормально, поскольку контейнеры будут полностью загружены до вызова hashCode(). В качестве альтернативы, переосмыслите свой граф объектов, чтобы удалить циклическую зависимость или написать методы чтения/записи пользовательских объектов.