Wert aus verschiedenen Porteingängen erzeugen

[Andreas (Gast)]

Guten Morgen,

kann ich mit diesem Code eine Variable beschreiben, deren Werte von 
verschiedenen PORTs und PINs kommen?

Die Pins sind mit einem 8 Fach DIP-Schalter versehen. Das 7. Bit soll 
immer null sein. Auf Grund des Layouts ist es nicht möglich alle 
Schalter an einem Port zu legen.

uint8_t zahl = 0;

zahl |= ((PINB & (1<<PB7))<<6)|
        ((PIND & (1<<PD5))<<5)|
        ((PIND & (1<<PD6))<<4)|
        ((PIND & (1<<PD7))<<3)|
        ((PINB & (1<<PB0))<<2)|
        ((PINB & (1<<PB1))<<1)|
        ((PINB & (1<<PB2))<<0);

Danke

[Programmierer (Gast)]

Ich vermute das funzt so nicht (oder?), weil du damit die Bits weit über 
die Bytegrenzen rausschiebst. Versuch mal die kompakte if/then Syntax:

wert = (port & pin) ? (1<<6) : 0 ¦ ...

[Klaus W. (mfgkw)]

oder ähnlicher zum Original:

 zahl |= ( ((PINB & (1<<PB7))!=0 ) << 6 )|
         ( ((PIND & (1<<PD5))!=0 ) << 5 )|
         ( ((PIND & (1<<PD6))!=0 ) << 4 )|
         ( ((PIND & (1<<PD7))!=0 ) << 3 )|
         ( ((PINB & (1<<PB0))!=0 ) << 2 )|
         ( ((PINB & (1<<PB1))!=0 ) << 1 )|
         ( ((PINB & (1<<PB2))!=0 ) << 0 );

[Klaus W. (mfgkw)]

PS: Das Oder in der Zuweisung ist natürlich überflüssig.

[Andreas (Gast)]

Danke für die Antworten.

Kann es momentan leider nicht praktisch testen,
aber das sieht sehr gut aus.

Gruß
Andreas

[Klaus W. (mfgkw)]

P.S.: Solange die Bits schon an der richtigen Stelle stehen, kann man 
das Schieben natürlich auch weglassen:

 zahl |= ( ((PINB & (1<<PB7))!=0 ) << 6 )|
         ( ((PIND & (1<<PD5))!=0 ) << 5 )|
         ( ((PIND & (1<<PD6))!=0 ) << 4 )|
         ( ((PIND & (1<<PD7))!=0 ) << 3 )|
         ( ((PINB & (1<<PB0))!=0 ) << 2 )|
         ( ((PINB & (1<<PB1))!=0 ) << 1 )|
         ( ((PINB & (1<<PB2))!=0 ) << 0 );

wird zu:

 zahl |= ( ((PINB & (1<<PB7))!=0 ) << 6 )|
         ( ((PIND & (1<<PD5))    )      )|
         ( ((PIND & (1<<PD6))!=0 ) << 4 )|
         ( ((PIND & (1<<PD7))!=0 ) << 3 )|
         ( ((PINB & (1<<PB0))!=0 ) << 2 )|
         ( ((PINB & (1<<PB1))    )      )|
         ( ((PINB & (1<<PB2))!=0 ) << 0 );

[Programmierer (Gast)]

Klaus Wachtler schrieb:
> wird zu:


Überlass dem Compiler das optimieren und schreib wartbaren Code.

SCNR.

[Klaus W. (mfgkw)]

Wenn der Compiler das hinbekommt...

Außerdem finde ich diese Version jetzt auch nicht sooo unübersichtlich.
Wenn ich es noch klarer formulieren will, sind auch diese 
Bitschiebereien nicht elegant.

[Peter D. (peda)]

  uint8_t zahl;

  zahl = (PINB & 1<<PB7 ? 64 : 0)
       | (PIND & 1<<PD5 ? 32 : 0)
       | (PIND & 1<<PD6 ? 16 : 0)
       | (PIND & 1<<PD7 ?  8 : 0)
       | (PINB & 1<<PB0 ?  4 : 0)
       | (PINB & 1<<PB1 ?  2 : 0)
       | (PINB & 1<<PB2 ?  1 : 0);

Peter

[Andreas (Gast)]

Danke für alle Vorschläge...

Gruß
Andreas