CRC16 von C auf Java

Gast U. (tellob) OP 03.11.2012 15:29 #2900420

Hey liebe Leute!
Mein erster Beitrag hier bei euch. Tolles Forum, tolle Seite!

Ich bin dabei ein CRC 16 Algorithmus auf Java zu portieren, um dort 
Pakete auswerden zu können. Leider will das nur nicht so wie ich und ich 
finde den Fehler einfach nicht.

Mein C Code sieht so aus:

crc16 (const uint8_t * buffer, uint32_t size)
  uint16_t crc = 0xFFFF;
  if (buffer && size)
    while (size--)
      crc = (crc >> 8) | (crc << 8);
      crc ^= *buffer++;
      crc ^= ((unsigned char) crc) >> 4;
      crc ^= crc << 12;
      crc ^= (crc & 0xFF) << 5;
  return crc;


Der Java-Port dazu:

private static short crc16(final byte[] data) {
      int crc = 0xFFFF;
      for (int i = 0; i < data.length; i++) {
          crc = (crc >> 8) | (crc << 8);
          crc ^= data[i];
          crc ^= crc >> 4;
          crc ^= crc << 12;
          crc ^= (crc & 0xFF) << 5;
      return (short) (crc & 0xffff);


Kann mir einer von euch sagen, wo hier mein Fehler liegt? Finde Ihn 
einfach nicht.

Antwort Zitieren Bearbeiten Thread verschieben Thread sperren Anmeldepflicht aktivieren Anpinnen Thread löschen Thread mit anderem zusammenführen Melden Danke

g457 Gast 03.11.2012 15:35 #2900430

> Mein C Code sieht so aus: [..]
> crc ^= ((unsigned char) crc) >> 4;
           ^^^^^^^^^^^^^
> Der Java-Port dazu: [..]
> crc ^= crc >> 4;

HTH

Antwort Zitieren Bearbeiten Löschen Melden Danke

user Gast 03.11.2012 15:39 #2900435

crc ^= ((unsigned char) crc) >> 4;

ändere das in folgendes

crc ^= (crc & 0xff) >> 4;

Antwort Zitieren Bearbeiten Löschen Melden Danke

g457 Gast 03.11.2012 15:46 #2900445

nochwas:

> crc ^= crc << 12;
> crc ^= (crc & 0xFF) << 5;

Die beiden musst Du jeweils auf 16 Bit zurechtstutzen.

Antwort Zitieren Bearbeiten Löschen Melden Danke

Gast U. (tellob) OP 03.11.2012 17:06 #2900604

Was meinst du mit Zurechtstutzen? &0xFF ?

Antwort Zitieren Bearbeiten Löschen Melden Danke

g457 Gast 03.11.2012 21:21 #2901000

> Was meinst du mit Zurechtstutzen? &0xFF ?

Ja, zum Beispiel. Aber mit 16 Bits. Danach

> crc = (crc >> 8) | (crc << 8);

ist es auch nötig.

Antwort Zitieren Bearbeiten Löschen Melden Danke

Thread beobachten

Antwort schreiben

Eine eigene Frage? Dieser Thread ist über 13 Jahre alt. Stell sie in einem eigenen Thread – dort lesen mehr Leute mit, und sie wird öfter und schneller beantwortet.

Neuen Thread starten

Eine Lösung, Ergänzung oder Rückmeldung zu diesem Thread? Dann antworte hier.

Text *

Formatierung

[c]C-Code[/c]
[code]Code in anderen Sprachen, ASCII-Zeichnungen[/code]
[math]Formel in LaTeX-Syntax[/math]
[[Titel]] – Link zu einem Artikel
Verweis auf einen anderen Beitrag: Rechtsklick auf dessen Beitrags-ID (#8078628), „Adresse kopieren“, in den Text einfügen

Mehr zur Formatierung

Anhänge

Längeren Sourcecode als Anhang, nicht in den Text. JPG nur für Fotos und Scans; Zeichnungen und Screenshots als PNG oder GIF. Siehe Bildformate.

Dein Name und deine E-Mail-Adresse

Schon registriert? Anmelden

Vorname *

E-Mail-Adresse *

Wird nicht angezeigt. Wir schicken dir einen Link; sobald du damit bestätigst, erscheint dein Beitrag.

Thread beobachten

Mit dem Abschicken legst du einen kostenlosen Account an und erkennst die Nutzungsbedingungen und die Datenschutzerklärung an.