Bit Position für Array verwenden

Du kannst das Ganze in eine Schleife packen. Dadurch wird es im 
Quellcode schöner aber evtl. auch etwas langsamer.

Simon schrieb:

> errechnen kann und zwar anhand des bits in changed.

Na ja.
Das ist ja nicht weiter schwer.
Die Verundung mit einem Maskenwert bleibt weiterhin.
Einziger UNterschied.
Bisher waren deine Maskenwerte 0x01, 0x02, 0x04, 0x08, ....
WEnn man sich die aber mal auf Bitebene ansieht, dann sieht man, dass 
das im Grunde ja immer nur ein einzige 1 Bit ist, welches in jeder 
weiteren Maskenstufe um 1 Position nach links rutscht.

1

Bei x gleich 0     0x01       00000001

2

Bei x gleich 1     0x02       00000010

3

Bei x gleich 2     0x04       00000100

4

Bei x gleich 3     0x08       00001000

5

...

Diese 'weiterrutschen' kann man natürlich auch programmierern.

1

  uint8_t mask = 0x01;

2

3

  for (x=0; x<8; x++) {

4

    if (changed & mask)

5

      digital[x].value += 1;

6

    mask = mask << 1;

7

  }

PS: Bitte vergiss mein voriges Posting wieder - ich habs bereits 
gelöscht. Das war Blödsinn und ist so in deinem konkreten Fall nicht 
anwendbar.

Simon schrieb:
> Vielen Dank für deine Antwort.
>
> ok... das wird funktionieren,
> aber habe ich damit wirklich irgendwas gewonnen?
>
> Ich meine der code ist nicht einfacher geworden und vermutlich auch
> nicht
> schneller.

Das ist schwer zu sagen.
Im Zweifelsfall musst du es kompilieren und dir das Ergebnis auf 
Assembler-Ebene ansehen.
Kürzer ist er auf Assembler-ebene schon - würde ich mal sagen.
Obs auch schneller ist, ist eine andere Frage.

Noch sparender geht es wohl so:

1

digital[0].value += changed & 1; changed>>=1;

2

digital[1].value += changed & 1; changed>>=1;

3

digital[2].value += changed & 1; changed>>=1;

4

digital[3].value += changed & 1; changed>>=1;

5

digital[4].value += changed & 1; changed>>=1;

6

digital[5].value += changed & 1; changed>>=1;

7

digital[6].value += changed & 1; changed>>=1;

8

digital[7].value += changed & 1;

oder als schleife

1

uint8_t i;

2

for (i=0; i<8; ++i){

3

  digital[i].value += changed & 1;

4

  changed>>=1;

5

}

Gruß

Simon schrieb:
> Gibt es hier eine elegantere Lösung als das einzelnd aufzählen zu
> müssen?

Mit GCC:

digital[0].value += __builtin_popcount (changed);

http://gcc.gnu.org/onlinedocs/gcc/Other-Builtins.html

Je nach Target ist das in Assembler imlementiert, z.B. für die aktuellen 
Versionen von avr-gcc (ab 4.7).