Datentypen Byte-Array aus Stream lesen

[Hutmacher]

Kann mir bitte jemand aus meiner Verwirrtheit helfen? Hab vergessen, wie ich das machen und googeln hilft nicht weiter ...

Bisher versuche ich es so:

try
{
	DataInputStream arrayDataInput = new DataInputStream(server.getInputStream());
				
	byte[] buffer = new byte[2048];
	byte[] arrayData = new byte[1234567];
	while (arrayDataInput.read(buffer) != -1)
	{
		System.arraycopy(buffer, 0, arrayData, 0, buffer.length);
	}
} 
catch (IOException e)
{
	e.printStackTrace();
}

Also ich empfange über einen Socket-Stream einen byte-Array.

(Am Ende, also nach dem Try-Catch-Block möchte ich diesen Byte-Array dann mithilfe von ImageIO ganz einfach wieder zu einem Bild machen.)

 

[SegFault]

Mal ein paar vermutungen. Du kopierst immer Buffer length in arraydata. Das scheint schon falsch zu sein. Den Buffer length sollte immer 2048 sein. Die wirklich empfangenen Bytes sind der rückgabewert von read. D.h. den Wert den Buffer.read zurückgibt in eine Var schreiben. Prüfen ob die != -1 ist und dann diese anzahl an bytes in das Zielarray kopieren. Nicht vergessen diese Bytes mit zu zählen.
mal unprobiert

int received = 0;
int written = 0;
received = arrayDataInput.read(buffer);
while ( received != -1 )
{
    System.arraycopy(buffer, 0, arrayData, written, received);
    written+=received;
}

Damit Kopierst du aber sinnlos puffer herum. Besser wäre das du einen rudimentären header an das ganze ranhängst. Die ersten 4 Byte könnten ein Long wert sein, welcher die anzahl der wirklich empfangenen Bytes beeinhaltet dann könntest du sowas machen

long bytes = dataInput.readLong();
try
{
    dataInput.read(arrayData,0,bytes);
}
catch (IndexOutOfBoundsException)

Wie schon gesagt, code ist nicht getestet und muss garantiert noch angepasst werden.

 

[Marco13]

Naja, bei dem ersten werden jetzt immer gleich viele bytes gelesen.... und eine Header sollte für sowas nicht nötig sein ...

Im Moment bin ich aber etwas verwirrt... warum wickelst du da einen DataInputStream drum? Bytes kann man von jedem InputStream lesen...

Das "Standardmuster", um einen Stream leerzulutschen ist sowas wie

try
{
    DataInputStream arrayDataInput = new DataInputStream(server.getInputStream());

    // Oder: InputStream arrayDataInput = server.getInputStream(); !?!?!
                
    byte[] buffer = new byte[2048];
    byte[] arrayData = new byte[1234567];
    while (true)
    {
        int read = arrayDataInput.read(buffer) ;
        if (read == -1) break;
        System.arraycopy(buffer, 0, arrayData, 0, read); // Anzahl GELESENER bytes kopieren!
    }
} 
catch (IOException e)
{
    e.printStackTrace();
}

(auch ungetestet, müßt aber passen...)

 

[Marco13]

Noch soals Nachtrag: Da du ja offenbar nicht weißt, wie groß der zu lesende Array ist, bietet sich da auch sowas an wie

InputStream arrayDataInput = server.getInputStream(); 
ByteArrayOutputStream baos = new ByteArrayOutputStream();
                
byte[] buffer = new byte[2048];
while (true)
{
    int read = arrayDataInput.read(buffer) ;
    if (read == -1) break;
    baos.write(buffer, 0, read);
} 
byte ergebnisArray[] = baos.toByteArray();

Der ErgebnisArray hat dann gleich die richtige Größe...