Datentypen byte[] to int

[hosch]

Hallo,
ich habe ein byte[] das versendet wird. Die Daten sind wie folgt aufgebaut:1byte Länge der Daten, Rest Daten.
Wenn ich das byte[] erhalte, lese ich die Länge aus, und erstelle mir anschließend ein byte[] mit genau dieser Länge, in dem die Daten gespeichert werden. Nun habe ich jedoch ein Problem. Die Daten die gesendet werden sehen so aus:

00 00 00 9d ac ed 00 05 73 72 00 1f 6a 61 76 61 78 ...

Insgesamt sind es 157 byte an Daten, die gesendet werden.
Wenn diese Daten erhalten werden, lese ich wie oben beschrieben zuerst die ersten 4 byte in ein neues byte[]. Aus diesem wird die Länge der Daten berechnet.

public int getLength() {
	return (length[0] << 24) + (length[1] << 16) + (length[2] << 8) + length[3];
}

jedoch ist die berechnete Länge der Daten stehts -99 (00 00 00 9d).
D.h. das byte[] das anschließend die Daten speichern soll, kann nicht initialisiert werden.

 

[Gas]

& 0xff

 

[hosch]

return ((length[0] << 24) + (length[1] << 16) + (length[2] << 8) + length[3]) & 0xff;

geht nicht!

 

[Landei]

Bytes in Java sind vorzeichenbehaftet, also denke ich Gas wollte eher

return ((length[0] & 0xff) << 24) 
+ ((length[1] & 0xff) << 16)
+ ((length[2] & 0xff) << 8)
+ (length[3] & 0xff);

vorschlagen.

 

[SlaterB]

@hosch
doch geht!








neben noch mehr Geschrei hilft Beweis durch Programm statt leerer Hülsen,
das hier läuft z.B.

public class Test
{
    byte[] length = new byte[4];

    public Test()
    {
        length[3] = (byte)0x9d;

        System.out.println(getLength());
    }

    public int getLength()
    {
        return ((length[0] << 24) + (length[1] << 16) + (length[2] << 8) + length[3]) & 0xff;
    }

    public static void main(String[] args)
    {
        new Test();
    }
}

du lieferst eher wenig Beispiele

---

allgemein ist es aber besser, das & 0xff auf jeden der 4 Werte einzeln auszuführen, damit es auch geht wenn die ersten drei Werte != 0 sind

 

[Gas]

Bytes in Java sind vorzeichenbehaftet, also denke ich Gas wollte eher

return ((length[0] & 0xff) << 24) 
+ ((length[1] & 0xff) << 16)
+ ((length[2] & 0xff) << 8)
+ (length[3] & 0xff);

vorschlagen.

Richtig. Sorry für diese wirklich fast schon zu kurz gefasste Antwort, aber Kollegen drängten mich zum Mittagessen.

 

[hosh_]

public int getLength()
    {
        return ((length[0] << 24) + (length[1] << 16) + (length[2] << 8) + length[3]) & 0xffff;
    }

geht!

 

[Gas]

Ist aber nicht richtig.

 

[SlaterB]

bevor man einen Block wie

return ((length[0] & 0xff) << 24)
+ ((length[1] & 0xff) << 16)
+ ((length[2] & 0xff) << 8)
+ (length[3] & 0xff);

auch nur einmal schreibt, bietet sich übrigens nach Möglichkeit eine weitere Hilfsmethode an, so dass diese Zeile dann nur noch
return (geFF(0) << 24) + (getFF(1) << 16) + (getFF(2) << 8) + getFF(3);
lautet oder auch
return geFF(0,24) + getFF(1,16) + getFF(2,8) + getFF(3,0);

 

[gosu]

Oh cool, genau das habe ich auch gebraucht.
Nur habe ich da ein problem.
Ich habe mir 2 Funktionen gebaut die wie folgt aussehen:

public static int byteArrayToInt(byte[] byteArray){
		if (byteArray.length == 4)
			return ((byteArray[0] & 0xff) << 24) 
			+ ((byteArray[1] & 0xff) << 16)
			+ ((byteArray[2] & 0xff) << 8)
			+ (byteArray[3] & 0xff);
		else{
			try {
				throw new Exception("byteArray exception");
			} catch (Exception e) {
				e.printStackTrace();
			}
			
			return 0;
		}
	}
	
	public static byte[] intToByteArray(int integer){
		byte[] array = new byte[4];
		for (int i = 0; i < 4; ++i) {
			int shift = i << 3; // i * 8
			array[3-i] = (byte)((integer & (0xff << shift)) >>> shift);
		}
		return array;
	}

Nun habe ich eine kasse mit gettern und settern in den jeweiligen getter/setter-Metoden rufe ich dann die static funktionen auf.
Nun bekomme ich eine Exception

Exception in thread "Thread-518" java.lang.OutOfMemoryError: Java heap space

Wenn jedoch die getter bzw setter alle implementiert sind und nicht die static methoden nutzen. Läuft alles fein.
hat wer ne Idee?