Wie uint16_t bitweise verarbeiten

Nö, das ist soweit korrekt.

Allerdings kann man das auch ein bisschen anders schreiben:

1

uint16_t Vergleich = 1 << 15;

2

uint8_t BitCount = 0;

3

4

for (BitCount = 0; BitCount < 16; BitCount++)

5

{

6

  if (Vergleich & sbyte)

7

    puls(); //sendet langen Impuls

8

  else

9

    pause(); //sendet kurzen Impuls

10

  Vergleich >>= 1;

11

}

Inhaltlich ist das aber exakt das gleiche, es wird nacheinander Bit 15 
bis Bit 0 von "sbyte" untersucht und entsprechend "puls" oder "pause" 
aufgerufen.

Nur am Rande:
Der Variablenname "sbyte" ist ungeschickt gewählt.

Denkfehler meinerseits:

Der Wert wird korrekt ausgegeben.
Nur ich Depp habe ihn falsch ins Binärformat umgerechnet und somit etwas 
anderes erwartet.

Danke für die schnelle Antwort und die Tipps zur Schreibweise.

Gruß

Günter

Ich würds so machen:

1

for(int8_t i=15; i>=0; i-=1)

2

  (sbyte & (1<<i)) ? puls() : pause();

Ich weis, dass viele Leute das ? nicht so toll finden, aber ich finde es 
bei sehr kurzen Codesequenzen übersichtlicher als auf vier Zeilen 
gestreckt.

Und wer kein >=c99 verwendet, muss das int8_t vor die for-schleife 
setzen.


W.S. schrieb:
> sowas leiden wie die Pest:
> Rufus Τ. Firefly schrieb:
>> uint16_t Vergleich = 1 << 15;
>
> Schreib lieber
> uint16_t Vergleich;
>
> ...
>  Vergleich = 1<<15;
>  while (....
Genau das versuche ich zu vermeiden, da man sonst möglicherweise 
ausversehen die Variable benutzt bevor man sie initialisiert hat. Und je 
nach dem was vorher im RAM stand, bekommt man schwer reproduzierbare 
Ergebnisse.

> Es ist ne Sache der Vermeidung von Schusselfehlern,
> direkt vor der Verwendung seine Schleifenvariable zu setzen, weil man
> sonst gar zu häufig bei etwas längeren Funktionen selbiges vergißt (Ich
> hab's ja ganz oben schon gemacht..)
Ich deklariere meine Variablen erst dort, wo ich sie benötige, 
idealerweise im Schleifenkopf selbst. Nervt, wenn man wieder zum 
Funktionsanfang scrollen muss, nur um nachzusehen, ob die Variable jetzt 
ein int8_t oder ein uint8_t war. Und schön aussehen tut so ein 
Variablenblob am Anfang auch nicht.

Simon S. schrieb:
> Genau das versuche ich zu vermeiden, da man sonst möglicherweise
> ausversehen die Variable benutzt bevor man sie initialisiert hat. Und je
> nach dem was vorher im RAM stand, bekommt man schwer reproduzierbare
> Ergebnisse.

Das sollte ein vernueftiger Compiler schon anmeckern.

Simon S. schrieb:
> Ich würds so machen:for(int8_t i=15; i>=0; i-=1)
>   (sbyte & (1<<i)) ? puls() : pause();
> Ich weis, dass viele Leute das ? nicht so toll finden, aber ich finde es
> bei sehr kurzen Codesequenzen übersichtlicher als auf vier Zeilen
> gestreckt.

Ist ganz nett, aber man muss sich bewusst sein, dass der erzeugte Code 
je nach Architektur nicht unbedingt ideal wird. Auf einem AVR wird das 
(1 << i) mangels Barrel-Shifters nicht unbedingt performant.

Gruß
Marius

Ich würds so schreiben:

1

void sendebyte( uint16_t val )

2

{

3

  DDRB=0x01; //schaltet den PIN auf Ausgang

4

  for( uint8_t i = 16; i--; val <= 1 )

5

    val & 1<<15 ? puls() : pause();

6

  DDRB=0x00; //schaltet den PIN wieder auf Eingang

7

}

Peter Dannegger schrieb:
>   for( uint8_t i = 16; i--; val <= 1 )
______________________________^^^^^^^^
val <<= 1