Dezimal zu Hex

[Philipp M. (lord-maricek)]

Moin,

ich hab schon gesucht aber nichts passendes gesucht, sorry wenn ich was 
übersehen habe.

Ich decodiere einen NMEA GPS Satz und möchte die Checksumme überprüfen.
Ich erstelle die Checksumme mit einer for Schleife, und speicher sie in 
einem unsigned char.

Die Checksumme, die am Satz hinten dranhängt sind aber 2 Ascii 
Buchstaben bzw. kann man die auch in einen Integer konvertieren.

Aber wie kann ich die Checksummen vergleichen?

MfG
Philipp

unsigned char checksum = 0x00;
        for(unsigned char i=0;i<=this->indexer;i++)
        {
            if(this->buffer[i]=='*')
            {
                break;
            }
            else if(this->buffer[i]!='$')
            {
                checksum ^= this->buffer[i];
            }
        }

this ist ein Pointer auf ein Struct

[Karl H. (kbuchegg)]

Philipp Maricek schrieb:

> Die Checksumme, die am Satz hinten dranhängt sind aber 2 Ascii
> Buchstaben bzw. kann man die auch in einen Integer konvertieren.

natürlich.

wenn du 2 Ziffern hast   3 und 4
dann ist die entsprechende Dezimalzahl
   x = erste_Ziffer * 10 + zweite_Ziffer

Aus 3 und 4  wird so     3 * 10 + 4 -> 34

jetzt hast du aber keine Dezimalziffern, sondern Hex-Ziffern. Und die 
auch noch nicht in Zahlenform sondern als 'Buchstaben'. Also wird es 
wohl schlau sein, sich diese 'BUchstaben' erst mal in richtige Zahlen zu 
verwandeln und dann .... bei Dezimalziffern wird die erste Zahl mit 10 
Multipliziert, bei Hexadezimalziffern dann eben mit 16

> Aber wie kann ich die Checksummen vergleichen?

Du kannst aber auch deine errechnete Checksumme in einen String 
verwandeln und dann einen Stringvergleich machen. Ist halt in der 
umgekehrten Richtung. Aber aus dem NMEA die CHecksumme numerisch zu 
machen dürfte etwas einfacher und auch schneller sein.

[Jürgen S. (jsachs)]

Dein Wert in "checksum" ist ja schon Hex.
Du musst also nur deine Checksumme von ASCII in "hex" umrechnen, dazu 
gibt es Funktionen oder eben manuell in der Art:

unsigned char chk = 0x00;
// von ASCII nach hex (binär)
if ((this->buffer[i] >= 0) && (this->buffer[i] <= 9 ))
     chk+=(this->buffer[i] - '0');
else if ((this->buffer[i] >= 'a') && (this->buffer[i] <= 'f'))
     chk+=(this->buffer[i] - 'a' + 10);
// auch für "i+1" machen

if (chk == checksumme)
     -> alles gut
else
     -> ups schlecht.

Der Code setzt voraus das dein ASCII in kleinbuchstaben ist, sonst eben 
anapssen.
Habs nicht gecheckt, aber als Richtung sollte es reichen :-)

Gruss
Juergen

[Philipp M. (lord-maricek)]

Ok, danke,

aber müsste es hier nicht heißen:

if ((this->buffer[i] >= '0') && (this->buffer[i] <= '9' ))

statt

if ((this->buffer[i] >= 0) && (this->buffer[i] <= 9 ))

MfG
Philipp

[Jürgen S. (jsachs)]

Correct.

Wie gesagt das war nur pro forma.

Gruss
Juergen

[Philipp M. (lord-maricek)]

Hi,
leider funtzt das bei mir nicht so richtig.

Als test benutzte ich diesen NMEA Satz.

$GPRMC,191410,A,4735.5634,N,00739.3538,E,0.0,0.0,181102,0.4,E,A*19

Ich habe die Funktion noch erweitert, dass groß und Kleinbuchstaben 
verwendet werden können.

Hier der Code: Die letzten beiden Chars lasse ich mit auch ausgeben und 
bekomme auch 19 angezeigt, also ist das nicht das Problem und er gehts 
in
der Abfrage in 0-9, also das auch richtig.

unsigned char checksum = 0x00;

        printf("Last chars: %c, %c\n",this->buffer[this->indexer-1],this->buffer[this->indexer]);
        unsigned char chk = 0x00;
        if ((this->buffer[this->indexer-1] >= '0') && (this->buffer[this->indexer-1] <= '9' ))
        {
            chk+=(this->buffer[this->indexer-1] - '0');
            printf("\nChar 1: 0-9\n");
        }
        else if ((this->buffer[this->indexer-1] >= 'A') && (this->buffer[this->indexer-1] <= 'F'))
        {
            chk+=(this->buffer[this->indexer-1] - 'A' + 10);
            printf("\nChar 1: A-F\n");
        }
        else if ((this->buffer[this->indexer-1] >= 'a') && (this->buffer[this->indexer-1] <= 'f'))
        {
            chk+=(this->buffer[this->indexer-1] - 'a' + 10);
            printf("\nChar 1: a-f\n");
        }


        if ((this->buffer[this->indexer] >= '0') && (this->buffer[this->indexer] <= '9' ))
        {
            chk+=(this->buffer[this->indexer] - '0');
            printf("\nChar 2: 0-9\n");
        }
        else if ((this->buffer[this->indexer] >= 'A') && (this->buffer[this->indexer] <= 'F'))
        {
            chk+=(this->buffer[this->indexer] - 'A' + 10);
            printf("\nChar 2: A-F\n");
        }
        else if ((this->buffer[this->indexer] >= 'a') && (this->buffer[this->indexer] <= 'f'))
        {
            chk+=(this->buffer[this->indexer] - 'a' + 10);
            printf("\nChar 2: a-f\n");
        }
        printf("Check: %x\n",chk);

Wenn ich mir das am ende ausgeben lasse, bekomme ich als Hex nur 'a' 
ausgeben.

Was ist falsch?

MfG
Philipp

[visitor (Gast)]

Moin,

Du kannst auch strtol verwenden um die ASCII Zeichen der Checksumme in 
einen numerischen Wert zu wandeln. Bei strtol kann man die Basis mit 
angeben.

[visitor (Gast)]

Du rechnest 1 + 9 = 10.  10 in hex angezeigt ist a.
Die Ausgabe ist korrekt, aber die erste Stelle gibt ja
das Vielfache von 16 an. Also 1*16 + 9 = 25.
25 in hex ist dann 19h.

[Karl H. (kbuchegg)]

Un dtu dir selbst einen Gefallen und schreib dir ein paar Funktionen zu 
diesem thema

unsigned char hexDigit( char c )
{
  if( c >= '0' && c <= '9' )
    return c - '0';

  if( c >= 'a' && c <= 'f' )
    return c - 'a' + 10;

  if( c >= 'A' && c <= 'F' )
    return c - 'A' + 10;

  return 0;
}

unsigned char toHex( char c1, char c2 )
{
  return hexDigit(c1) * 16 + hexDigit(c2);
}


....



  unsigned char checksumNMEA = toHex( this->buffer[this->indexer-1],
                                      this->buffer[this->indexer] );

[Philipp M. (lord-maricek)]

Ok, danke jetzt läufts wunderbar.

MfG
Philipp