Bit in Byte mit Schleife setzen/abfragen

[Markus P. (sebastianwurst)]

Hallo,
ich habe eine For-Schleife mit der ich je nach Status eines Arrays 
(ausgang _status[i]) ein Bit in der Variable  (StatusPortx) setzen will. 
Leider klappt das nur für das erste Byte (StatusPort1) Kann mir einer 
sagen was ich da falsch mache?

 uint8_t StatusPort1,StatusPort2,StatusPort3,StatusPort4,StatusPort5,StatusPort6,StatusPort7,StatusPort8,StatusPort9,StatusPort10,
    StatusPort11,StatusPort12,StatusPort13,StatusPort14,StatusPort15,StatusPort16 = 0;

    for (uint16_t i = 0; i < 128; i++)
    {
      ausgaenge_status[i] = (ausgaenge_status[i] ^ ausgaenge_schalten[i]);
      ausgaenge_schalten[i] = 0;    
      
      if (ausgaenge_status[i]) 
      {
        if      (i< 8)  StatusPort1 |= ( 1 << i );
        else if      (i< 16) StatusPort2 |= ( 1 << i );
        else if      (i< 24) StatusPort3 |= ( 1 << i );
        else if      (i< 32) StatusPort4 |= ( 1 << i );      
        else if      (i< 40) StatusPort5 |= ( 1 << i );  
        else if      (i< 48) StatusPort6 |= ( 1 << i );  
        else if      (i< 56) StatusPort7 |= ( 1 << i );  
        else if      (i< 64) StatusPort8 |= ( 1 << i );  
        else if      (i< 72) StatusPort9 |= ( 1 << i );  
        else if      (i< 80) StatusPort10 |= ( 1 << i );  
        else if      (i< 88) StatusPort11 |= ( 1 << i );  
        else if      (i< 96) StatusPort12 |= ( 1 << i );  
        else if      (i< 104) StatusPort13 |= ( 1 << i );  
        else if      (i< 112) StatusPort14 |= ( 1 << i );  
        else if      (i< 120) StatusPort15 |= ( 1 << i );  
        else if      (i< 128) StatusPort16 |= ( 1 << i );
      }        
      else
      {
        if      (i< 8) StatusPort1 &= ~(1 << i);
        else if (i< 16) StatusPort2 &= ~(1 << i);
        else if (i< 24) StatusPort3 &= ~(1 << i);
        else if (i< 32) StatusPort4 &= ~(1 << i);
        else if (i< 40) StatusPort5 &= ~(1 << i);
        else if (i< 48) StatusPort6 &= ~(1 << i);
        else if (i< 56) StatusPort7 &= ~(1 << i);
        else if (i< 64) StatusPort8 &= ~(1 << i);
        else if (i< 72) StatusPort9 &= ~(1 << i);
        else if (i< 80) StatusPort10 &= ~(1 << i);
        else if (i< 88) StatusPort11 &= ~(1 << i);
        else if (i< 96) StatusPort12 &= ~(1 << i);
        else if (i< 104) StatusPort13 &= ~(1 << i);
        else if (i< 112) StatusPort14 &= ~(1 << i);
        else if (i< 120) StatusPort15 &= ~(1 << i);
        else if (i< 128) StatusPort16 &= ~(1 << i);
                
      }
    } // For Schleife Ende

Das Status Bit frage ich dann so ab:

if (StatusPort6&0b00000001) {......
if (StatusPort6&0b00010000) {......

[Noname (Gast)]

Offen gesagt habe ich den Eindruck, das Du mit dem Schlachtschiff ne 
Pfütze überwinden willst. Wenn Du mal genauer erklärst worum es geht, 
kann man das sicher noch wesentlich einfacher erledigen.

Aber meine Glaskugel sagt mir sowas wie:

        else if      (i< <y>) StatusPorty<x> |= ( 1 << (i%8) );

[Wolfgang (Gast)]

Erste Maßnahme wäre, deine StatusPortX in ein Array StatusPort[] zu 
packen und dann statt der if/else if-Kette direkt über das High-Byte von 
i zu adressieren.

[PittyJ (Gast)]

for (uint16_t i = 0; i < 128; i++)
    ...
          StatusPort1 |= ( 1 << i );


es wird i um 127 Stellen nach rechts geschoben. Das kann kein Prozessor 
mehr darstellen, weil die Register oft nur 32 Bit breit sind.

und das Ziel hat nur 8 Bit. Da ist spätestens beim 9ten Bit Schluss mit 
einer Veränderung.


Man sollte evtl ein Array der statusports nehmen

uint8_t Statusports[16].

und dann mit
Statusports[ Port / 8] |= 1<<( Port % 8)
das Bit setzen. (oder so ähnlich)

[Markus P. (sebastianwurst)]

PittyJ schrieb:
> es wird i um 127 Stellen nach rechts geschoben. Das kann kein Prozessor
> mehr darstellen, weil die Register oft nur 32 Bit breit sind.

Da war mein Denkfehler... Danke schonmal... Jetzt überlege ich wie ich 
es am besten hin bekomme, also ein Array StatusPort[x] will ich nicht 
machen, es soll schone jeweils ein Byte sein.

[Tom R. (Gast)]

Markus P. schrieb:
> Da war mein Denkfehler... Danke schonmal... Jetzt überlege ich wie ich
> es am besten hin bekomme, also ein Array StatusPort[x] will ich nicht
> machen, es soll schone jeweils ein Byte sein.

Das mußt du jetzt mal erklären. Was bringt das für einen Vorteil?

[Noname (Gast)]

Ich denke auch das es Dir Vorteile bringt wenn Du das Problem mal 
beschreibst, so das wir Dir was dazu sagen können.

Solche ellenlangen Listen von Parametern oder bedingten Anweisungen mit 
nur numerisch unterschiedlichen Bedingungen und in den abhängigen 
Anweisungen sind fast immer ein Hinweis darauf, das das Konzept einen 
bedeutenden Denkfehler enthält.

[Markus P. (sebastianwurst)]

Weil ich eine Funktion habe um auf einer Webseite die im Flash des yc 
läuft (AVR Net IO) eine Checkbox zu setzen (checked)

Die kann man mit Sicherheit auch auf ein Array erweitern, würde dann 
aber wieder ein Rattenschwanz hinter sich ziehen... und so fit bin ich 
da drin auch nicht, was die porgrammierung angeht...

  //---------------------------------------------------------------
      // Einsetzen des Port Status %PORTxy durch "checked" wenn Portx.Piny = 1
      // x: A..G  y: 0..7 
      //---------------------------------------------------------------

      if (strncasecmp_P("PORT",http_entry[index].new_page_pointer,4)==0)
      {
        unsigned char pin  = (pgm_read_byte(http_entry[index].new_page_pointer+5)-48);  
        b = 0;
        switch(pgm_read_byte(http_entry[index].new_page_pointer+4))
        {
        
          case 'A':            
            b = (StatusPort1  & (1<<pin));
            //usart_write_str0("-Port1-");
            break;
            
          case 'B':
             b = (StatusPort2 & (1<<pin));
            //usart_write_str0("-Port2-");
            break;

[Flohzirkus (Gast)]

Probier' einfach

        if      (i< 8)  StatusPort1 |= ( 1 << i );
        else if      (i< 16) StatusPort2 |= ( 1 << (i-8) );
        else if      (i< 24) StatusPort3 |= ( 1 << (i-16) );
        else if      (i< 32) StatusPort4 |= ( 1 << (i-24) );      
        else if      (i< 40) StatusPort5 |= ( 1 << (i-32) );

Aber das Umschreiben in ein Array wäre sicherlich zu empfehlen.

Gruß

[Markus P. (sebastianwurst)]

Habs jetzt so gelöst:
else if      (i< <y>) StatusPorty<x> |= ( 1 << (i%8) );

Also:

 if (ausgaenge_status[i]) 
      {
        if      (i< 8)  StatusPort1 |= ( 1 << (i%8) );
        else if      (i< 16) StatusPort2 |= ( 1 << (i%8) );
        else if      (i< 24) StatusPort3 |= ( 1 << (i%8) );
        else if      (i< 32) StatusPort4 |= ( 1 << (i%8) );      
        else if      (i< 40) StatusPort5 |= ( 1 << (i%8) );  
        else if      (i< 48) StatusPort6 |= ( 1 << (i%8) );  ......