Невозможно прочитать сериализуемый класс из файла .dat

Question

Я пытаюсь написать keyholder, и я хочу написать пароли в .dat-файл с помощью ObjectOutputStream, а затем прочитать их с помощью ObjectInputStream. Это мой код для записи объектов:

public void toFile() 
{  
    try 
    { 
     ObjectOutputStream oos = new ObjectOutputStream(new FileOutputStream("passwords.dat"));  
     for(int i = 0; i<this.nrOfPW; i++) 
     { 
      if(this.PWlist[i] instanceof longPW) 
      { 
       oos.writeObject((longPW)this.PWlist[i]); 
      } 
      else 
      { 
       oos.writeObject((PinPW)this.PWlist[i]); 
      } 
     } 
     oos.close(); 
    } 
    catch(IOException e) 
    { 
     e.getStackTrace(); 
    } 
} 

Это похоже на работу, но когда я пытаюсь прочитать файл снова и поместить объекты в моем PWlist массиве он говорит, что PinPW не сериализации, хотя PinPW реализует Serializable и импортируется. Базовый класс PinPW (Info) также реализует Serializable и импортирует его. Это код, где я прочитал файл:

public void fromFile() 
{ 
    try 
    { 
     ObjectInputStream objIn = new ObjectInputStream(new FileInputStream("passwords.dat")); 
     while(objIn.readObject() != null) 
     { 
      if(this.nrOfPW == this.PWlist.length) 
      { 
       expand(10); 
      } 
      if(objIn.readObject() instanceof PinPW) 
      { 
       this.PWlist[this.nrOfPW] = (PinPW)objIn.readObject(); 
       this.nrOfPW++; 
      } 
      else 
      { 
       this.PWlist[this.nrOfPW] = (longPW)objIn.readObject(); 
       this.nrOfPW++; 
      } 
     } 
     objIn.close(); 
    } 
    catch(EOFException e) 
    { 
     e.getStackTrace(); 
    } 
    catch(IOException ex) 
    { 
     ex.printStackTrace(); 
    } 
    catch(ClassNotFoundException ex) 
    { 
     ex.printStackTrace(); 
    } 
} 

Массив PWlist является массив информации, и PinPW и longPW распространяется информация.

Что мне делать, чтобы исправить эту проблему?

Answer 1

Давайте исправим "первую ошибку, первым" ...

В этом коде:

while(objIn.readObject() != null)       // reads object, tests then *discards* it 
{ 
    ... 

    if(objIn.readObject() instanceof PinPW)     // reads object, tests then *discards* it 
    { 
    this.PWlist[this.nrOfPW] = (PinPW)objIn.readObject(); // conditionally read an object 
    this.nrOfPW++; 
    } 
    else 
    { 
    this.PWlist[this.nrOfPW] = (longPW)objIn.readObject(); // conditionally read an object 
    this.nrOfPW++; 
    } 
} 

Каждый раз вокруг итерации цикла, вы на самом деле читать объекты. В первый раз, когда вы читаете объект, чтобы проверить, был ли он в потоке, в следующий раз, когда вы его прочитаете и определите его тип, отбросьте его. Затем вы прочитали третий объект и бросили его в зависимости от того, какой тип был отброшен. объект был.

Кроме того, как правильно указывает EJP, правильный способ определить Конец потока для ObjectInputStream должен поймать конец исключения файла.

Вы хотите сделать это вместо того, чтобы:

try 
{ 
    while (true) 
    { 
    final Object o = objIn.readObject();   // read the object from the stream 

    ... 

    if (o instanceof PinPW) 
    { 
     this.PWlist[this.nrOfPW] = (PinPW) o;   // cast to correct type 
     this.nrOfPW++; 
    } 
    else 
    { 
     this.PWlist[this.nrOfPW] = (longPW) o;  // cast to correct type 
     this.nrOfPW++; 
    } 
    } 
} 
catch (EOFException e) 
{ 
    // end of stream reached ... 
    // ... close the file descriptor etc ... 
} 

Answer 2

У вас есть проблема здесь.

while(objIn.readObject() != null) 
    { 
     if(this.nrOfPW == this.PWlist.length) 
     { 
      expand(10); 
     } 
     if(objIn.readObject() instanceof PinPW) 
     { 
      this.PWlist[this.nrOfPW] = (PinPW)objIn.readObject(); 
      this.nrOfPW++; 
     } 
     else 
     { 
      this.PWlist[this.nrOfPW] = (longPW)objIn.readObject(); 
      this.nrOfPW++; 
     } 
    } 

Вы читаете объект много раз. Постарайтесь сохранить его, а затем работать с ним. if (objIn.readObject() instanceof PinPW) читает один, читает дважды, this.PWlist [this.nrOfPW] = (PinPW) objIn.readObject(); читается три раза, когда это должно быть только один раз. PS: используйте синтаксис Грега Копфа внутри и без ключевого слова final, потому что вы хотите сохранить в нем больше объектов.

Answer 3

Я просто хотел указать, что блок if-else в функции toFile() совершенно бессмыслен. writeObject() принимает аргумент Object. Ему все равно, какой тип объекта он есть, если он сериализуем.

В то время как ваш метод fromFile() серьезно испорчен, он не вызвал бы NotSerializableException. Я считаю, что исключение, о котором вы говорите, действительно произошло в toFile().

Причина очень проста: вы не полностью прочитали и поняли документацию ObjectOutputStream. Чтобы быть конкретным, Object и все его непереходные поля и все непереходные поля в его классах предков должны реализовать Serializable. Он также должен иметь открытый конструктор no-arg.