seltsames Verhalten if-Bedingung

[nebie (Gast)]

Hey!

weiß einer warum die zweite if-Bedingung nicht aufgerufen wird obwohl 
die Bedingung gegeben ist? Port toggelt immer noch...

 if (s_IO.ucChargerPresent == 1){
    __disable_interrupt();
  PORTC &= ~ (1 << PC0 | 1 <<PC4 | 1 << PC1 | 1 << PC5 | 1 << PC3 | 1 << PC2);
  
  PORTB |=  ( 1 << PB2 );
  _delay_ms(6000);
  PORTB &= ~ ( 1 << PB2 );
  _delay_ms(6000);
    
  
  ChargeCurrent_Write();
  ChargeVoltage_Write();
  InputCurrent_Write();
  //__enable_interrupt();
  
  if (s_IO.ucChargerPresent == 0)
  {  __disable_interrupt();
    PORTB &= ~ ( 1 << PB2); 
    PORTC |=  (1 << PC0 | 1 << PC4 | 1 << PC1 | 1 << PC5 | 1 << PC3 | 1 << PC2);
  }
  }

[Kurt (Gast)]

Die zweite if-Bedingung steht innerhalb der ersten...

[Karl H. (kbuchegg)]

Wer oder was ist s_IO?
Wie wird es verändert? Wo wird es verändert?

[mike (Gast)]

Wenn s_IO.ucChargerPresent == 1 kann es nicht gleichzeitig 0 sein.
(Interrups ausgeschlossen)

[nebie (Gast)]

Kurt schrieb:
> Die zweite if-Bedingung steht innerhalb der ersten...

ah wie blind von mir... du hast vollkommen recht Kurt

Karl Heinz Buchegger schrieb:
> Wer oder was ist s_IO?
> Wie wird es verändert? Wo wird es verändert?

// Ladespannung abfragen
  if ( (PIND&0x20) > 0 ){
    s_IO.ucChargerPresent = 1;
  } else {
    s_IO.ucChargerPresent = 0;
  }

[Karl H. (kbuchegg)]

nebie schrieb:
> Kurt schrieb:
>> Die zweite if-Bedingung steht innerhalb der ersten...
>
> ah wie blind von mir... du hast vollkommen recht Kurt

Tja.
Wenn dich so ein Konstrukt

  }
  }

im Code nicht stutzig macht.
2 geschlossene Klammen können bei korrekter Einrückung nie untereinander 
stehen.
Rückt man aber korrekt ein, dann würde man so manchen Fehler in den 
Code-Schachtelungen sehen.

>
> Karl Heinz Buchegger schrieb:
>> Wer oder was ist s_IO?
>> Wie wird es verändert? Wo wird es verändert?
>
> // Ladespannung abfragen
>   if ( (PIND&0x20) > 0 ){
>     s_IO.ucChargerPresent = 1;
>   } else {
>     s_IO.ucChargerPresent = 0;
>   }

OK. Und wo findet sich der Aufruf in deinem Codestück, mit dem dieses 
Auswertung laufen würde?
Oder war die Sache mit der fehlerhaften Einrückung / Schachtelung schon 
des Rätsels Lösung?

[Andreas H. (andario)]

Kurt schrieb:
> Die zweite if-Bedingung steht innerhalb der ersten...

Deshalb der Übersichtlichkeit halber so schreiben, dass öffnende und 
schließende Klammern eines Blocks immer untereinander stehen:

  if (Bedingung)
  {
    Code;
    Code;
  }
  else
  {
    Code;
    Code;
  }

  if (Andere Bedingung)
  {
    ...
  }

  ...

Grüße,
Andario

[Sebastian W. (sebastian_w29)]

Das ist auch im kompakten Format möglich (wenn man sich dran hält):

  if (Bedingung) {
    Code;
    Code;
  } else {
    Code;
    Code;
  }

  if (Andere Bedingung) {
    ...
  }

  ...

Vorteil ist der m.M.n. bessere Lesefluss.

LG, Sebastian

[Kurt (Gast)]

Da hat sich das Python lernen doch gelohnt :D
Dadurch hatte ich nie wieder solche Probleme.
Eigentlich erstaunlich wie viele "Fehler" man nur durch einen sauberen 
Code-Stil verhindern kann...