Bitweiser Schiebeoperator über 8 stellen

pszy Gast #2504130
Lesenswert?
• ▲
▼ 
Hallo,
Ich habe im EEPROM 4 Bytes, die ich in eine Variable vom Typ unsigned 
long speichern muss.
Dazu habe ich folgenden Code geschrieben:

1
unsigned long epoch = 0;

2


3
epoch = (EEPROM.read(1) << 24);

4
epoch += (EEPROM.read(2) << 16);

5
epoch += (EEPROM.read(3) << 8);

6
epoch += (EEPROM.read(4));


In der variable steht jetzt aber immer statt z.b. 4F11A2D5 der wert 
FFFFA2D5. Warum? Die Daten sind im eeprom auf jeden Fall 
richtigbabgelegt und werden auch korrekt ausgelesen.

Grüße
Antwort Zitieren Bearbeiten Thread verschieben Thread sperren Anmeldepflicht aktivieren Anpinnen Thread löschen Thread mit anderem zusammenführen Beitrag melden
pszy Gast #2504225
Lesenswert?
• ▲
▼ 
Die Antwort war ja schnell :)

Wie genau gehts das denn in meinem Fall? Ich habe so probiert:
1
epoch = static_cast<unsigned long> (EEPROM.read(1) << 24);

Compiler hat nicht gemeckert. Funktionieren tuts dennoch nicht.
Antwort Zitieren Bearbeiten Löschen Beitrag melden
marcus6100 Gast #2504893
Lesenswert?
• ▲
▼ 
Nimm lieber uint32_t anstatt unsigned long. Das ist deutlicher.

Auch wenn der Compiler den Aufwand der Addition vermutlich wegoptimiert 
solltest du | benutzen.

Die Verwendung einer Addition um bitweise einen Wert zusammenzusetzen 
ungewöhnlich. Die Operator Precedence von bitwise or passt mit addition 
auch nicht zusammen.

Die Operator precedence ist:
erst: typecast
dann: addition
dann: bitwise shift
dann: bitwise or

Du kannst das also ohne die ganzen Klammern auch so schreiben:
1
uint32_t epoch;

2
epoch = (uint32_t)EEPROM.read(1) << 24

3
        | (uint32_t)EEPROM.read(2) << 16

4
        | (uint16_t)EEPROM.read(3) << 8

5
        | EEPROM.read(4);

6


7
oder:

8
epoch = (uint32_t)EEPROM.read(1) << 24 | (uint32_t)EEPROM.read(2) << 16 | (uint16_t)EEPROM.read(3) << 8 | EEPROM.read(4);
Antwort Zitieren Bearbeiten Löschen Beitrag melden
Oo0 Gast #2504924
Lesenswert?
• ▲
▼ 
Lothar Miller schrieb:
> marcus6100 schrieb:
>> Nimm lieber uint32_t anstatt unsigned long. Das ist deutlicher.
> Ähm...
> Kannst du das mal begründen?

uint32_t ist ein 32 Bit Integer ohne Vorzeichen. unsigned long ist von 
der Plattform abhaengig in der Laenge, ohne Vorzeichen.

Da im EEPROM 4 Byte abgespeichert sind die einen Integer-Wert abbilden, 
ist es durchaus sinnvoll, einen dazu passenden Datentyp zu verwenden und 
erst anschliessend in den im Programm verwendeten Typ zu konvertieren. 
Was im Idealfall sowieso wegfaellt.
Antwort Zitieren Bearbeiten Löschen Beitrag melden
Thread beobachten

Antwort schreiben

Bitte melde dich an, um einen Beitrag zu schreiben.

Bestehender Account

oder

Mit Google-Account einloggen

Noch kein Account?

Die Registrierung ist kostenlos und dauert nur eine Minute.

Jetzt registrieren