Einfache Frage zu C: so richtig?

[Hans (Gast)]

Guten morgen!

Ich habe eine einfache Frage, die mir bestimmt jeder leicht beantworten 
kann.

Ich möchte bei meinem Controller von einem Port bestimmte Ausgänge beim 
Initialisieren definiert auf low haben - die anderen Pins sind mir dabei 
egal.

Kann ich das so schreiben:

P5OUT = ( ~0x01 | ~0x04 | ~0x05 );

?

[Nico S. (nico22)]

Nein. Eher sowas hier:

P5OUT &= ~( 0x01 | 0x04 | 0x05 );

Das kann man auch leicht per Zettel und Stift nachvollziehen.

[Volker (Gast)]

nein, weil durch oder die Nullen überschriben werden.

P5OUT =  ~(0x01 | 0x04 | 0x08 );

0x05 passt hier nicht, wenn Du die Bits einselnd ansprechen wikkst.

[Hans (Gast)]

Volker schrieb:
> 0x05 passt hier nicht

LOL, ja natürlich, Tippfehler, sorry!

Danke!

[Werner B. (werner-b)]

Oder so

P5OUT = ( ~0x01 & ~0x04 & ~0x05 );

P.S.
Unter Brücksichtigung von ...
> die anderen Pins sind mir dabei egal.
... am einfachsten

P5OUT = 0;

;-)

[Tip (Gast)]

Volker schrieb:
> P5OUT =  ~(0x01 | 0x04 | 0x08 );

Besser

P5OUT =  ~((1 << 0) | (1 << 3) | (1 << 4));

P5OUT =  ~( _BV(0) | _BV(3) | _BV(4) );

Dann kann sowas wie mit der "05" auch nicht passieren.

mfg

[Hans (Gast)]

Tip schrieb:
> (1 << 0)

Also diese Schreibweise ist mir bis heute ein Rätsel, die habe ich nie 
gelernt bei den MSPs mit IAR.

[Bitschieber (Gast)]

Hallo!

(1<<x) heißt, dass Du die 1 um x Stellen nach links schiebst.

1<<2 bedeutet also, dass die 1 um 2 Stellen nach links geschoben wird:

00000001
00000010
00000100

Somit bekommst Du den Wert 4 was ja zu 2^2 passt. Somit kann man an der 
Schreibweise sofort erkennen, welches Bit manipuliert wird. Im 
Zusammenhang mit den AVR-Bibliotheken wird es dann übersichtlich:

PORTB |= (1<<PB2);    //Setzt das Portbit PB2
PORTB &= ~(1<<PB2);   //Löscht selbiges.

Viele Grüße
Bitschieber

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

Hans schrieb:
> Also diese Schreibweise ist mir bis heute ein Rätsel
Mir tuts auch weh, da lerne ich lieber die 16 möglichen 
Bit-Kombinationen in einer Hex-Zahl.
Wenn ich 5 höre, sehe ich 0101, und wenn ich A höre, sehe ich 1010...


Bitschieber schrieb:
> PORTB |= (1<<PB2);    //Setzt das Portbit PB2
> PORTB &= ~(1<<PB2);   //Löscht selbiges.
Zusammen mit den Defines hast du ja recht, aber das hier:
> P5OUT =  ~((1 << 0) | (1 << 3) | (1 << 4));
ist Unsinn, weil das niederwertigste Bit 0 ganz links kommt, und zudem 
in einem ganzen Haufen Zahlen und Sonderzeichen versteckt ist.
Ich sehe da schneller was:
> P5OUT = ~0x19;

[Tip (Gast)]

Lothar Miller schrieb:
> Zusammen mit den Defines hast du ja recht, aber das hier:
>
>> P5OUT =  ~((1 << 0) | (1 << 3) | (1 << 4));
>
> ist Unsinn, weil das niederwertigste Bit 0 ganz links kommt, und zudem
> in einem ganzen Haufen Zahlen und Sonderzeichen versteckt ist.

Der Sinn erschließt sich erst, wenn man die ganzen "magic numbers" durch 
lesbare Bitdefinitionen ersetzt. Von der Abstraktion ist die Verwendung 
des Makros "_BV" natürlich noch besser als direkt "1 <<" zu schreiben.

Und was hast du gegen das Bit 0? Normalerweise sind die Bits von 0 bis 7 
nummeriert und (1 << 0) ist halt eine Darstellung von 2^0.
Die Reihenfolge der Ausdrücke ist alleine durch die Art deiner im Code 
enthaltenen Programmdokumentation bestimmt. Dem Compiler ist das 
schnurz-piep-egal.


mfg

[Tip (Gast)]

Hans schrieb:
> Tip schrieb:
>> (1 << 0)
> Also diese Schreibweise ist mir bis heute ein Rätsel, die habe ich nie
> gelernt bei den MSPs mit IAR.

Dafür habe ich MSP und IAR nicht gelernt.   ;-)

Die Beschreibung des Links-Shift Operators "<<" findest du in jedem 
C-Buch beschrieben, z.B.
http://de.wikibooks.org/wiki/C-Programmierung:_Ausdr%C3%BCcke_und_Operatoren

[Karl H. (kbuchegg)]

Lothar Miller schrieb:
> Hans schrieb:
>> Also diese Schreibweise ist mir bis heute ein Rätsel
> Mir tuts auch weh,

Dann solltest du dir mal überlegen, ob du dir als C-Programmierer die 
Dinge nicht so zurecht legen kannst, dass sie für DICH lesbar sind.

Niemand hindert einen das zu tun.

#define ERROR_LED    (1 << PB0)

    PORTB |= ERROR_LED;      // Error LED anschalten

oder

#define ERROR_LED    PB0
#define ERROR_PORT   PORTB
#define TURN_ON( port, bit )    port |= 1 << bit


   TURN_ON( ERROR_PORT, ERROR_LED );

oder (wenn man sinnigerweise alle LED an einem Port gesammelt hat)

#define ERROR_LED   PB0
#define READY_LED   PB1

#define LED_PORT    PORTB

#define LED_ON(led)      LED_PORT &= ~( 1 << led )
#define LED_OFF(led)     LED_PORT |= ( 1 << led )

    if( error ) {
      LED_ON( ERROR_LED );
      LED_OFF( READY_LED );
    }
    else {
      LED_OFF( ERROR_LED );
      LED_ON( READY_LED );
    }

Mit ein wenig Makro Magie kann man sich in C leicht die Dinge so zurecht 
legen, dass ein Blinder mit Stock auf 5 Meter Entfernung die Bedeutung 
einer Anweisung erkennen kann UND der Code einfach wartbar ist. Die LED 
musste wegen Hardware-Änderungen den Portpin wechseln: Kein Problem: 1 
#define angepasst und den Rest erledigt der Compiler.
Nur tun muss man es.

Man beachte auch den Zugewinn an automatischer Dokumentation, der durch

      LED_OFF( ERROR_LED );

ensteht, im Vergleich zu

      PORTB &= ~( 1 << 0 );     // Fehler LED ausschalten

Der Kommentar ist direkt in den C-Code selber hineingewandert. Und das 
ist immer ein gutes Zeichen: Jede Änderung, die einen Kommentar 
überflüssig macht, ist eine gute Änderung.

Ein gutes Programm ist nicht nur dadurch gekennzeichnet, dass es 
funktioniert. Ein gutes Programm ist auch so organisiert, dass es leicht 
zu lesen und zu verstehen ist. Dazu gehört auch, dass man den Compiler 
einspannt um sich selbst das Leben leichter zu machen.