Aus 2 char ein unsigned int erzeugen

Sebastian Gast 20.06.2007 17:14 #586368

Lesenswert?

•

Hallo @ all

Ich möchte Daten von den Ports einlesen. Der Datenbus ist 8 Bit breit. 
Zuerst übertrage ich das Low-Byte und anschließend das High-Byte. Danach 
möchte ich das Low-Byte und High-Byte zusammenfassen zu einem unsigned 
int. Hier mein Quellcode:

bit status;
char low_byte, high_byte;


if(status == LOW_BYTE)
{
   low_byte = PINC;
   status = HIGH_BYTE;
}
else
{
   high_byte = PINC;

   ptr_datenpuffer[i] = high_byte;
   ptr_datenpuffer[i] = ptr_datenpuffer[i] << 8;
   ptr_datenpuffer[i] = low_byte;

   status = LOW_BYTE;
}

Ich dachte das durch die Wertzuweisung die unteren 8 Bits des unsigned 
int beschrieben werden. Anschließend wollte ich das ganze um 8 Stellen 
nach links schieben und dann das Low-Byte zuweisen. Dies scheint so aber 
nicht zu funktionieren

Grüsse

Sebastian

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

Obelix Gast 20.06.2007 17:16 #586373

Lesenswert?

•

▲
▼

Du überschreibst den wert damit.
x=(hi << 8) + lo;

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Sebastian Gast 20.06.2007 17:17 #586374

Lesenswert?

•

▲
▼

Ist x=(hi << 8) + lo; denn die Lösung?

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Rahul D. (rahul) 20.06.2007 17:18 #586376

Lesenswert?

•

▲
▼

>   ptr_datenpuffer[i] = high_byte;
>   ptr_datenpuffer[i] = ptr_datenpuffer[i] << 8;
>   ptr_datenpuffer[i] = low_byte;

Oder einfacher und kürzer:

ptr_datenpuffer[i] = ((int) high_byte )<<8) | low_byte;

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Obelix X. (obelix) 20.06.2007 17:21 #586379

Lesenswert?

•

▲
▼

@Sebastian : ja, aber schau dir noch mal das Beispiel von Rahul an. 
(Typenumwandlung)


x=((unsigned int)hi << 8) + lo;

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Tobias R. (tobiasrohde) 20.06.2007 19:06 #586450

Lesenswert?

•

▲
▼

oder so:

union{ unsigned short u_s;
       char u_c[2];
     }convert_var;

convert_var.u_c[0]=byte_1;
convert_var.u_c[1]=byte_2;

Ergebniss:

convert_var.u_s;

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Andreas S. (andreas) Admin 20.06.2007 19:15 #586455

Lesenswert?

•

▲
▼

Die Lösung mit der Union ist kein sauberes C, weil man je nach 
Compiler/Prozessor verschiedene Ergebnisse bekommen kann (Stichwort 
Endianness), deshalb sollte man lieber Shifts verwenden.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Sebastian Gast 21.06.2007 07:41 #586679

Lesenswert?

•

▲
▼

ptr_datenpuffer[i] = (((int) high_byte )<<8) | low_byte;

Wieso muss der Datentyp vom low_byte nicht verändert werden? Schließlich 
führe ich doch eine Veroderung von zwei unterschiedlichen Datentypen 
durch.

Grüsse

Sebastian

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Lötnagel Gast 21.06.2007 08:06 #586687

Lesenswert?

•

▲
▼

Vorschlag: ausschießlich unsigned verwenden, zb so:

bit status;
unsigned char low_byte, high_byte;
// unsigned short ptr_datenpuffer[300];
if(status == LOW_BYTE)
   low_byte = PINC;
   status = HIGH_BYTE;
   high_byte = PINC;
   ptr_datenpuffer[i] = (high_byte<<8)+low_byte;
   status = LOW_BYTE;

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Sebastian Gast 21.06.2007 08:14 #586691

Lesenswert?

•

▲
▼

Wieso braucht man denn mit einmal keine Typenumwandlung mehr?

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Lötnagel Gast 21.06.2007 08:36 #586699

Lesenswert?

•

▲
▼

>Wieso braucht man denn mit einmal keine Typenumwandlung mehr?

Das erledigt der Compiler. Unsigned char und unsigned short haben
ja schon den gleichen Typ, nur unterschiedliche Größe.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Random .. (thorstendb) 21.06.2007 08:40 #586703

Lesenswert?

•

▲
▼

Hi,

wie wärs mit


int intvalue=0;
char *low_byte = (char*) &intvalue;
char *high_byte = (char*) (&intvalue)+1;

Schneller gehts nicht, weil hier nicht gerechnet werden muss. Die ARMs 
haben idR. nen Barrelshifter drin, so dass man da in einem Takt bis zu 
32Bit shiften kann. Die AVRs glaub ich nicht...


Greetz,
/th.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Sebastian Gast 21.06.2007 09:05 #586721

Lesenswert?

•

▲
▼

Ich habe mich jetzt für diese Lösung entschieden

(((int) high_byte )<<8) | low_byte;

Vielen Dank für die Antworten. Es war sehr Aufschlußreich

Grüsse

Sebastian

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Johannes M. (johnny-m) 21.06.2007 09:11 #586725

Lesenswert?

•

▲
▼

Sebastian wrote:
> Ich habe mich jetzt für diese Lösung entschieden
>
> (((int) high_byte )<<8) | low_byte;
Das kann aber auch nach hinten losgehen. Schiebeoperationen sollte man 
möglichst nur mit vorzeichenlosen Datentypen machen.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Willi W. (williwacker) 21.06.2007 09:16 #586730

Lesenswert?

•

▲
▼

wieso castest Du nach int, Du wolltest doch eigentlich unsigned?

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Sebastian Gast 21.06.2007 09:41 #586749

Lesenswert?

•

▲
▼

Mein Fehler im Quellcode habe ich natürlich

ptr_datenpuffer[i] = (((unsigned int) high_byte )<<8) | low_byte;

benutzt

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Andreas S. (andreas) Admin 21.06.2007 11:20 #586825

Lesenswert?

•

▲
▼

Thorsten De buhr wrote:
> int intvalue=0;
> char *low_byte = (char*) &intvalue;
> char *high_byte = (char*) (&intvalue)+1;

Auch das ist nicht portabel. Je nach Prozessor kann es sein dass das 
low_byte hier gar nicht das low byte ist, sondern das high byte.

> Schneller gehts nicht, weil hier nicht gerechnet werden muss. Die ARMs
> haben idR. nen Barrelshifter drin, so dass man da in einem Takt bis zu
> 32Bit shiften kann. Die AVRs glaub ich nicht...

Der Compiler setzt das beim AVR in zwei Byte-Zugriffe um, geschoben wird 
da gar nicht.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

STK500-Besitzer Gast 21.06.2007 12:12 #586864

Lesenswert?

•

▲
▼

>> int intvalue=0;
>> char *low_byte = (char*) &intvalue;
>> char *high_byte = (char*) (&intvalue)+1;

>Auch das ist nicht portabel. Je nach Prozessor kann es sein dass das
>low_byte hier gar nicht das low byte ist, sondern das high byte.

Das was Thorsten vorgeschlagen hat, macht doch eigentlich der Compiler, 
wobei dieser dann auch die richtige Zuordnung (Position der Bytes) 
trifft, oder?

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Random .. (thorstendb) 21.06.2007 13:29 #586941

Lesenswert?

•

▲
▼

shiftet denn der avr 8x in einem takt?

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Andreas S. (andreas) Admin 21.06.2007 13:35 #586947

Lesenswert?

•

▲
▼

Der AVR shiftet überhaupt nicht, weil der Compiler schlau genug ist 
einen 8-Bit-Shift durch einen entsprechenden Bytezugriff zu ersetzen.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Aus 2 char ein unsigned int erzeugen

Antwort schreiben