Was macht dieser Code?

[jollyroger]

Hi,

und sorry für das Thema, aber was Besseres ist mir echt nicht eingefallen.

Ich bin gerade auf eine Funktion eines Kollegen von mir gestoßen, der leider gerade im Urlaub ist.

Die Funktion schreibt einen String in eine AMQ / JMS-Bytemessage und sieht so aus:

	private void writeString(BytesMessage message, String s) throws JMSException  {
      
		int strlen = s.length();
		
	    if (strlen<128) {
	         message.writeByte((byte)strlen);
	    }
	    else {
	    	message.writeInt(strlen|0x80000000);
	    }
	    for (int i = 0; i < strlen; i++) {
	         message.writeByte((byte)(s.charAt(i)));
	    }
   }

Ok, offensichtlich enthält der Anfang der Bytemessage die Länge der gesamten Payload.

Aber was macht denn

message.writeInt(strlen|0x80000000);

"strlen" wird "Oder" verknüft mit dem Hex-Wert 0x80000000, das seh ich richtig, oder?

Nur wieso?

 

[Marco13]

Wieso? Die Intention wird man wohl kaum nachvollziehen können. Vielleicht hilft die Information, dass (wenn ich mich nicht gerade schwer irre) dadurch strlen schlicht und einfach negativ gemacht wird.... (das erste Bit steht für's Vorzeichen)

 

[SlaterB]

weil dann ein anderer Wert rauskommt und dieser anscheinend erwartet wird
aufgrund anderer Bit-Zahldarstellung oder ähnliches


edit:
also schlicht negativ ist es nicht ganz:

public class Test
{
    public static void main(String[] args)
        throws Exception
    {
        int s = 4;
        System.out.println(Integer.toBinaryString(s));
        s = 4|0x80000000;
        System.out.println(Integer.toBinaryString(s));
        s = -4;
        System.out.println(Integer.toBinaryString(s));
    }
}


-------

100
10000000000000000000000000000100
11111111111111111111111111111100

 

[0x7F800000]

es wird schon negativ, aber ändert sich gewaltig betragsmäßig, n dürfte zu -(2^31-n) werden, so auf +-1 genau... :bahnhof:

 

[jollyroger]

Hey ho,

vielen Dank für die Antworten.

Ich seh schon, mein Kollege hat entweder Mist gebaut oder der tiefere Sinn bleibt mir verborgen....

 

[maki]

Weisst du wie die Gegenstelle das ausliest?

 

[SlaterB]

wahrscheinlich meckert die Gegenseite auch zum Himmel, warum der jollyroger das bloß so kompliziert schickt

 

[Zetagi]

Hi,

"Mist" hat dein Kollege sicherlich nicht gebaut. Er hätte es nur kommentieren sollen.

Für mich sieht es wie ein Vorrendner erwähnte nach einer anderen Zahlendarstellung aus. In einem Byte befinden sich 8Bit. Damit lassen sich 256 Zahlen darstellen.

Je nach Zahlendarstellung (nur positive Zahlen) wären das 0-255.
Wenn auch negative zahlen dargestellt werden können, wäre das der Bereich -127 <-> 128

Daher die Abfrage: wenn < 128 OK SONST ändere Darstellung....

Aber wenn dein Kollege dir erkärt hat wozu der Code dient, erleuchte bitte auch das Forum mit der Erklärung ;-)

 

[Guest]

Weisst du wie die Gegenstelle das ausliest?

Wenn ich mich die nächsten 3 - 4 Stunden durch nicht dokumentierten C-Code wühle, ja.... :wink:

"Mist" hat dein Kollege sicherlich nicht gebaut. Er hätte es nur kommentieren sollen.

Amen ..... :wink:

Aber wenn dein Kollege dir erkärt hat wozu der Code dient, erleuchte bitte auch das Forum mit der Erklärung

Mach ich auf jeden Fall.

 

[jollyroger]

Sorry,

vergessen einzuloggen.....

 

[maki]

Wenn ich mich die nächsten 3 - 4 Stunden durch nicht dokumentierten C-Code wühle, ja.... icon_wink.gif

Das ist ja schonmal sehr interessant.

Könnte mir vorstellen, dass die Gegenstelle prüft, ob die Zahl negativ ist und damit über 128, oder positiv und damit kleiner als 128.

 

[Marco13]

Oh ja von wegen negativ die kleine Besonderheit bei der 2-Komplement-Darstellung :roll:

 

[Gast12]

offensichtlich hat die gegenstelle probleme, wenn der "header" einer message (die länge) mit 0 beginnt. denn genau das kann passieren, wenn writeInt() aufgerufen wird. das schiebt 4 bytes in den strom, high to low bytes. durch die veroderung beginnt der header immer mit einer 1, an der bit darstellung ändert sich bis auf das höchstwertigste bit allerdings nichts, da die zahlen garantiert positiv sind.

 

[Gast12]

äh, nicht "immer mit einer 1", sondern immer mit "nicht 0", so wars gemeint