ObjectInputStream kann nicht vernünftig lesen...

[Ph3n1x]

Hallo Leute,

ich programmiere gerade schön vor mich her und stelle jetzt folgendes fest:

Ich habe folgenden Code:

// icon ist vom Typ BufferedImage
public void writeInOOS(ObjectOutputStream oos) throws IOException {
        oos.writeUTF(comment);
        oos.writeUTF(tag);
        oos.writeBoolean(icon != null);
        if (icon != null) {
            oos.writeInt(icon.getWidth());
            oos.writeInt(icon.getHeight());
            oos.writeInt(icon.getType());
            for (int x = 0; x < icon.getWidth(); x++) {
                for (int y = 0; y < icon.getHeight(); y++) {
                    oos.writeInt(icon.getRGB(x, y));
                }
            }
        }
}
public void readFromOIS(ObjectInputStream ois) throws Exception {
        comment = ois.readUTF();
        tag = ois.readUTF();
        if (ois.readBoolean()) {
            int width = ois.readInt();
            int height = ois.readInt();
            int imageType = ois.readInt();
            icon = new BufferedImage(width, height, imageType);
            for (int x = 0; x < width; x++) {
                for (int y = 0; y < height; y++) {
                    icon.setRGB(x, y, ois.readInt());
                }
            }
        }
}

Soo, wenn ich jetzt die Daten in meinen Outputstream speichere (inkl. einem 16*16 Bild) funktioniert das Speichern ohne Probleme. Beim Laden jedoch kommt an der Stelle

                    icon.setRGB(x, y, ois.readInt());

direkt bei x=0 und y=0 eine Exception:

java.io.StreamCorruptedException: unexpected EOF in middle of data block
	at java.io.ObjectInputStream$BlockDataInputStream.refill(ObjectInputStream.java:2494)
	at java.io.ObjectInputStream$BlockDataInputStream.read(ObjectInputStream.java:2570)
	at java.io.DataInputStream.readInt(DataInputStream.java:370)
	at java.io.ObjectInputStream$BlockDataInputStream.readInt(ObjectInputStream.java:2775)
	at java.io.ObjectInputStream.readInt(ObjectInputStream.java:949)

Alle anderen Daten ließt er ohne Probleme aus, aber am Bild hängt er sich auf, obwohl es nur Integer Werte sind die gespeichert und wieder gelesen werden sollen.

Wenn ich den Teil mit dem Bild speichern weglasse funktioniert alles wunderbar, aber ich verstehe nicht, wieso? Was hab ich falsch gemacht?

Hat jemand eine Idee?

Gruß

Ph3n1x

 

[Spacerat]

Ein ähnliches Problem hatte ich mal, als ich vergas, den Stream vor Programmende zu schliessen. So wurde dann eine Datei z.B. nicht vollständig geschrieben.

 

[Ph3n1x]

Wäre schön wenn es so einfach wäre, aber nein... Stream flushe und schließe ich

 

[SlaterB]

schon merkwürdig, werde ich vielleicht morgen mit eigenen Icon testen,

eine kleine mögliche Hilfe:
verwende beim Schreiben nicht icon.getWidth() usw. mehrfach, sondern lege das in einer Variablen ab,
schwankt vielleicht wenn noch nicht vollständig geladen, aber sollte ja eher nur größer werden..,
logge auch wieviel geschrieben wird, prüfe die Größe der erzeugten Datei mit verschieden großen Icons,

dass das close() usw. hier fehlt ist sehr verdächtig


---

falls nicht bekannt dann alternatives Vorgehen erwähnenswert:
speichere nicht soviel komisches einzeln, sondern speichere exakt ein Object,
writeObject(), readObject(), fertig, mehr brauch dich der Stream gar nicht interessieren

in dem einen Objekt musst du nun alle Informationen ablegen, dahin verlagert sich das Problem,
z.B. ein langes byte[], eine Liste, eine Map mit Einträgen wie "width", "isNull" usw., je nach Möglichkeit das Icon an sich,
oder naheliegend eine neue eigene Klasse, darin etwas von den vorher genannten, oder auch einfache int-Variablen, int[][] für das Raster usw.

 

[Spacerat]

Also beim ObjectOutputStream kannst dir das explitite "flush()" auch sparen, denn es wird bei "close()" automatisch ausgeführt. Möglicherweise muss das ja nur weglassen werden, ein "flush()" an falscher Stelle kann verheerend sein, dann schon lieber "drain()".

 

[bERt0r]

Du könntest auch versuchen dein Image mit ImageIO zu schreiben:

static boolean write(RenderedImage im, String formatName, OutputStream output)
Writes an image using an arbitrary ImageWriter that supports the given format to an OutputStream.

 

[Ph3n1x]

Du könntest auch versuchen dein Image mit ImageIO zu schreiben:

Das weiß ich, aber ich möchte später auf einem Android Gerät das Bild auch wieder laden können, daher speichere ich so die Daten in den Stream und hoffe dass ich das unter Android dann wieder auslesen kann...
Aber erstmal muss es generell funktionieren...

@Spacerat
flush mache ich, wie ich gerade festgestellt habe, unmittelbar vor dem close(), also nichts schlimmes...

@SlaterB
Das mit dem writeObject() hatte ich anfangs, hat genauso wenig funktioniert...

 

[SlaterB]

> hat genauso wenig funktioniert...
ohne Erklärungen auch nicht zu klären

möglich dass dein Fehler unerklärbar bleibt, ich tippe aber auf irgendwas schlechtes im Restcode wie das schon genannte falsche close() usw.
hier ein vollständiges Programm:

public class Test
{
    static BufferedImage icon;

    public static void main(String... args)
        throws Exception
    {
        icon = ImageIO.read(new File("test.png"));
        File f = new File("test.ser");
        ObjectOutputStream out = new ObjectOutputStream(new FileOutputStream(f));
        writeInOOS(out);
        out.close();
        System.out.println("length: "+f.length());
        ObjectInputStream in = new ObjectInputStream(new FileInputStream(f));
        readFromOIS(in);
    }

    public static void writeInOOS(ObjectOutputStream oos)
        throws IOException
    {
        oos.writeUTF("test");
        oos.writeUTF("Dienstag");
        oos.writeBoolean(icon != null);
        if (icon != null)
        {
            oos.writeInt(icon.getWidth());
            oos.writeInt(icon.getHeight());
            oos.writeInt(icon.getType());
            System.out.println("icon: "+icon.getType()+", "+icon.getWidth()+", "+icon.getHeight());
            for (int x = 0; x < icon.getWidth(); x++)
            {
                for (int y = 0; y < icon.getHeight(); y++)
                {
                    oos.writeInt(icon.getRGB(x, y));
                }
            }
        }
    }

    public static void readFromOIS(ObjectInputStream ois)
        throws Exception
    {
        System.out.println(ois.readUTF());
        System.out.println(ois.readUTF());
        if (ois.readBoolean())
        {
            int width = ois.readInt();
            int height = ois.readInt();
            int imageType = ois.readInt();
            System.out.println("imageType: "+imageType);
            icon = new BufferedImage(width, height, BufferedImage.TYPE_INT_RGB);
            for (int x = 0; x < width; x++)
            {
                for (int y = 0; y < height; y++)
                {
                    icon.setRGB(x, y, ois.readInt());
                }
            }
        }
        System.out.println("fertig: "+ois.available());
    }
}

welches mit dem Bild im Anhang folgenden Output liefert:

icon: 0, 20, 20
length: 1643
test
Dienstag
imageType: 0
fertig: 0

den ImageType 0 mochte der BufferedImage-Konstruktor bei mir nicht so sehr, aber das ist ja ein anderes Problem,
'fertig 0' am Ende bedeutet dass exakt alles gelesen wurde

kannst ja mein Programm bei dir ausprobieren, oder ein eigenes Icon posten,
oder mehr von deinem Code
oder auf eine sonstige nachvollziehbare Weise ein Problem, ich sehe im Moment keines


edit:
> und hoffe dass ich das unter Android dann wieder auslesen kann...

verschiedene Betriebssysteme/ Computer mit eigenen Interpretationen könnten natürlich immer irgendwie Probleme machen.., obwohl vielleicht gerade die Serialisierung dagegen gefeilt sein sollte,
schon auf nur einem PC getestet wie mit meinem Programm?

 

[Ph3n1x]

Das ist echt komisch...
Habs ausprobiert, dein Code funktioniert soweit... Bei mir im Code nochmal geguckt etc. aber ist im Grunde nicht anders...

Mal Grob skizziert:

public class C {

     List<A> listOfA = new ArrayList<A>();

     public static void main(String[] args) {
          // SAVE
          A a = new A(); // evtl auch mehrere
          B b = new B(); 

          // ....
          // Befüllen mit ein paar Daten und "b" zu irgend einem "a" in die Liste einfügen 
          // und die erstellten "a"s in die listOfA einfügen
          // ....

          ObjectOutputStream oos = new ObjectOutputStream(fos); // FileOutputStream

          oos.writeInt(listOfA.size());
          for (A a : listOfA) {
              a.writeInOOS(oos);
          }

          oos.flush();
          oos.close();  

          // LOAD 
          // alles leeren etc.
          ObjectInputStream ois = new ObjectInputStream(fis); // FileInputStream

          int size = ois.readInt();
          for (int i= 0; i< size; i++) {
                A a = new A();
                a.readFromOIS(ois);
                listOfA.add(a);
          }
          ois.close();           
     }
}

public class A {

     List<B> listOfB = new ArrayList<B>();
     String bla = "BLA";

     // ... Liste Befüllen ...

     public void writeToOOS(ObjectOutputStream oos) {
           oos.writeUTF(bla);
           oos.writeInt(listOfB.size());
           for (B b : listOfB) {
                 b.writeToOOS(oos);
           }
     }

     public void readFromOIS(ObjectInputStream ois) {
           bla = ois.readUTF();
           int size = ois.readInt();
           for (int i= 0; i<size; i++) {
                 B b = new B();
                 b.readFromOIS(ois);
                 listOfB.add(b);
           }
     }
}

public class B {

     String bla = "BLA";
     String blub = "BLUB";
     BufferedImage icon;

     // ... Image Befüllen ...

     public void writeToOOS(ObjectOutputStream oos) {
           oos.writeUTF(bla);
           oos.writeUTF(blub);
           oos.writeBoolean(icon != null);
           if (icon != null) {
              oos.writeInt(icon.getWidth());
              oos.writeInt(icon.getHeight());
              oos.writeInt(icon.getType());
              for (int x = 0; x < icon.getWidth(); x++) {
                  for (int y = 0; y < icon.getHeight(); y++) {
                     oos.writeInt(icon.getRGB(x, y));
                  }
              }
           }
     }

     public void readFromOIS(ObjectInputStream ois) {
           bla = ois.readUTF();
           blub = ois.readUTF();
           if (ois.readBoolean()) {
              int width = ois.readInt();
              int height = ois.readInt();
              int imageType = ois.readInt();
              icon = new BufferedImage(width, height, imageType);
              for (int x = 0; x < width; x++) {
                  for (int y = 0; y < height; y++) {
                      icon.setRGB(x, y, ois.readInt());
                  }
              }
           }
     }
}

Natürlich noch ein bisschen mehr als das hier, aber:
Die Bytes die in die Datei geschrieben werden sind 1:1 die, die auch wieder gelesen werden. Und es werden auch alle Dinge rein geschrieben...

Sehr kurios...

 

[Spacerat]

flush mache ich, wie ich gerade festgestellt habe, unmittelbar vor dem close(), also nichts schlimmes...

Klar, davon gehe ich aus... aber auf welchen Stream wendest du es an? Das ist wichtig, weil ein "flush()" auf einen FileOutputStream z.B. bewirkt, dass der Puffer in den Hades geleert (leere "flush()"-Implementation des OutputStreams), statt in die Datei geschrieben wird. Hier genügt ein "close()".
Fazit: Das "close()" des ObjectoutputStreams führt "flush()" automatisch aus und danach wird der übergeordnete Stream geschlossen. Versuch' mal ein simples "close()" auf den ObjectOutputStream und lass "flush()" einfach mal weg.