Eingänge in verschiedene Ports mit for-schleife abfragen

[Frank (Gast)]

Hallo, ich habe mehrere Eingänge, die ich gerne in einer For-Schleife 
abfragen möchte. Leider war es nicht möglich, die alle auf einem Port zu 
legen.
Ist es trotzdem irgendwie möglich die Eingänge in einer For-Schleife 
abzufragen?
Die Eingänge sind bei einem Atmega328p bei
PC0-PC3 sowie PD4-PD6
Bis jetzt habe ich einfach alles untereinander geschrieben, was auch 
funktioniert. Da aber immer das selbe passiert (nur mit anderen 
PArametern, die ich mir dann aus einer Liste holen kann) wollte ich das 
kürzen.

Frank

[Ingo (Gast)]

Code?

[Peter D. (peda)]

Frank schrieb:
> Die Eingänge sind bei einem Atmega328p bei
> PC0-PC3 sowie PD4-PD6

  val = PINC & 0x0F | PIND & 0xF0;

[Rolf M. (rmagnus)]

Peter D. schrieb:
> val = PINC & 0x0F | PIND & 0xF0;

Und dann

for (uint8_t i = 0; i < 8; i++, val >>= 1)
{
    // Bit Nummer i auswerten
    bool bit = val & 1;
}

[MWS (Gast)]

Rolf M. schrieb:
>     bool bit = val & 1;

Fast ;D

[Christopher B. (chrimbo)]

MWS schrieb:
> Rolf M. schrieb:
>>     bool bit = val & 1;
>
> Fast ;D
Wieso? Passt doch!

[MWS (Gast)]

Christopher B. schrieb:
> Wieso? Passt doch!

Dann lies den Kommentar und frag' Dich nach dem Sinn in einer Schleife 
mehrere Pins abfragen zu wollen und immer mit einer eins zu ver-und-en.

     // Bit Nummer i auswerten

[Fire H. (fireheart)]

Frank schrieb:
> Ist es trotzdem irgendwie möglich die Eingänge in einer For-Schleife
> abzufragen?
> Die Eingänge sind bei einem Atmega328p bei
> PC0-PC3 sowie PD4-PD6


in   r16,PINC
andi r16,$0F
in   r17,PIND
andi r17,$70
or   r16,r17  ; now r16 contains bits 0 - 6


war eigentlich eine geniale Belegung der Ports ... nicht mal schieben 
war notwendig!

[MWS (Gast)]

Ok, stimmt, er schiebt val, dann passt's.

[Christopher B. (chrimbo)]

MWS schrieb:
> Christopher B. schrieb:
>> Wieso? Passt doch!
>
> Dann lies den Kommentar und frag' Dich nach dem Sinn in einer Schleife
> mehrere Pins abfragen zu wollen und immer mit einer eins zu ver-und-en.
> // Bit Nummer i auswerten

Brauch ich nicht, aber du solltest den Code nochmal genau lesen. 
Besonderes Augenmerk auf den Schleifenkopf.

[nicht"Gast" (Gast)]

Christopher B. schrieb:
> Brauch ich nicht, aber du solltest den Code nochmal genau lesen.
> Besonderes Augenmerk auf den Schleifenkopf.

Oo Style Diskussion incoming :)

Ich persönlich würde ja nichts in den Schleifenkopf stecken, was nicht 
unbedingt nötig ist. Das verringert die Lesbarkeit enorm( wie man 
sieht).

[Rolf M. (rmagnus)]

nicht"Gast" schrieb:
> Ich persönlich würde ja nichts in den Schleifenkopf stecken, was nicht
> unbedingt nötig ist. Das verringert die Lesbarkeit enorm( wie man
> sieht).

Unbedingt nötig ist im Schleifenkopf gar nichts. Man könnte die Schleife 
ja auch so schreiben:

uint8_t i = 0;
for (;;)
{
    if (i < 8)
    {
        // Bit Nummer i auswerten
        bool bit = val & 1;

        i++;
        val >>= 1;
    }
    else
    {
        break;
    }
}

Ich hab es im Prinzip so geschrieben, weil val für mich quasi die 
Variable ist, über die iteriert wird. Nur hat mir ein Abbruchkriterium 
gefehlt, deshalb habe ich i noch dazu geschrieben.