if Verschachtelung

[Beginner (Gast)]

Hallo,

mein Programm bleibt in der ersten if Abfrage hängen die LED blinkt aber 
wenn die Spannung <= 7400 geht es nicht mehr in die andere if Abfrage.

Wo is der Logikfehler? (und ja vor dem PC schon klar ;-))

int VoltageMonitoring (void)
{
 __disable_interrupt();

  uint16_t WandleWertIn16Bit (uint8_t Address)
  {
    uint16_t Wert;
    uint8_t  lowByte, highByte;
    smb_start_wait(AdresseSlave_1+SMB_WRITE);
    smb_write (Address);
    smb_rep_start(AdresseSlave_1+SMB_READ);

    lowByte = smb_readAck();
    highByte = smb_readNak();
    smb_stop();

    Wert = ( highByte << 8 ) | lowByte;

    return Wert;
  }

uint16_t Akkuspannung;
Akkuspannung = WandleWertIn16Bit(0x09);

    if (Akkuspannung <=  7800)
    {
      PORTC |=  (1 << PC0 | 1 << PC4 | 1 << PC1 | 1 << PC5 | 1 << PC3 | 1 << PC2);
      while (1)
      {
        Akkuspannung = WandleWertIn16Bit(0x09);
        PORTB |= (1 << PB2);  // LED_ROT blinken
        _delay_ms(1000);
        PORTB &= ~ (1 << PB2);
        _delay_ms(50);
        PORTB |= (1 << PB1);  
        _delay_ms(1000);
        PORTB &= ~ (1 << PB1);
        _delay_ms(50);
      }
    }
    
    else
    { 
      if (Akkuspannung <= 7400)
      {  
        while (1)
        {  PORTC &=~  (1 << PC0 | 1 << PC4 | 1 << PC1 | 1 << PC5 | 1 << PC3 | 1 << PC2);  
          PORTB |= (1 << PB1);  // LED_ROT blinken
          _delay_ms(500);
          PORTB &= ~ (1 << PB1);
          _delay_ms(500);
        }
      }
    }
__enable_interrupt(); 
}

[Justus S. (jussa)]

wie willst du aus den while-Schleifen kommen?

[(prx) A. K. (prx)]

if (Akkuspannung <=  7800)
{
  ...
}
else
{
  if (Akkuspannung <= 7400)
  {
    ...
    }
  }
}

Wie soll dabei die zweite Abfrage eine Chance haben?

[Chris (Gast)]

weil 7399 < 7800
ist und daher die erste If-Abfrage schon wahr ist.
Die zweite wird nie erreicht.

[Bastler (Gast)]

>mein Programm bleibt in der ersten if Abfrage hängen die LED blinkt aber
>wenn die Spannung <= 7400 geht es nicht mehr in die andere if Abfrage.


Ja, klar! Wie auch?

Du hast eine while(1) Schleife programmiert!

[Beginner (Gast)]

wie müsste ich denn die Abfrage gestalten?

es sollte wie folgt logisch ablaufen:

ist die Spannung <= 7800

dann soll die LED erst grün dann rot blinken und noch alle anderen LEDs 
sollen an sein

wenn die Spannung aber <= 7400

ist dann sollen alle LEDs aus sein nur die Statusled soll rot schnell 
blinken.

Wie schreiben ich das in C

switch / case geht ja auch nicht da dort kein Bereichsüberprüfung 
gemacht werden kann.

[Beginner (Gast)]

hat keiner ne Idee?

[Justus S. (jussa)]

Beginner schrieb:
> ist die Spannung <= 7800
>
>
> wenn die Spannung aber <= 7400

öhm, den Widerspruch bemerkst du nicht?

[Wegstaben V. (wegstabenverbuchsler)]

die allgemeine Syntax für if Abfragen lautet:

if BEDINGUNG then ANWEISUNG

BEDINGUNG kann durchaus auch ein zusammengesetzter, mehrteiliger 
Ausdruck sein. Damit alles zusammen passt was zusammen gehört solltest 
du großzügig Klammern spendieren.

[Udo S. (urschmitt)]

Die while (1) nach aussen.
Dann die Werte so abfragen, daß die erste Abfrage nicht gleich die 2te 
mit einschliesst.
Versuch das mal selbst, du kriegst das hin. Wenn wir dirs hier 
vorprogrammieren lernst du fast nix dabei.

[Beginner (Gast)]

hab es jetzt so umgeschrieben funktioniert leider immer noch nicht. Ich 
sehe aber nicht warum..


int VoltageMonitoring (void)
{
 __disable_interrupt();

  uint16_t WandleWertIn16Bit (uint8_t Address)
  {
    uint16_t Wert;
    uint8_t  lowByte, highByte;
    smb_start_wait(AdresseSlave_1+SMB_WRITE);
    smb_write (Address);
    smb_rep_start(AdresseSlave_1+SMB_READ);

    lowByte = smb_readAck();
    highByte = smb_readNak();
    smb_stop();

    Wert = ( highByte << 8 ) | lowByte;

    return Wert;
  }

while (1){
uint16_t Akkuspannung;
Akkuspannung = WandleWertIn16Bit(0x09);

      if (Akkuspannung <=  7800)
      {
        PORTC |=  (1 << PC0 | 1 << PC4 | 1 << PC1 | 1 << PC5 | 1 << PC3 
| 1 << PC2);
        Akkuspannung = WandleWertIn16Bit(0x09);
        PORTB |= (1 << PB2);  // LED_ROT blinken
        _delay_ms(1000);
        PORTB &= ~ (1 << PB2);
        _delay_ms(50);
        PORTB |= (1 << PB1);
        _delay_ms(1000);
        PORTB &= ~ (1 << PB1);
        _delay_ms(50);
      }

      else
        {
          if (Akkuspannung <= 7400)
          {

            PORTC &=~  (1 << PC0 | 1 << PC4 | 1 << PC1 | 1 << PC5 | 1 << 
PC3 | 1 << PC2);
            PORTB |= (1 << PB1);  // LED_ROT blinken
            _delay_ms(500);
            PORTB &= ~ (1 << PB1);
            _delay_ms(500);

          }
        }


     }

__enable_interrupt();
}

[Justus S. (jussa)]

weil du immer noch nicht checkst, dass ein Wert, der kleiner als 7400 
ist, auch kleiner als 7800 ist...

[Gerhard (Gast)]

Vertausch mal die 7400 und die 7800, dann wird die 7400 zuerst behandelt 
und wenn Wert größer dann wird alles was kleiner 7800 ist behandelt.

[Claude M. (stoner)]

Beginner schrieb:
> wie müsste ich denn die Abfrage gestalten?

Entweder du schaust, dass du nicht schon in den ersten Block fällst wenn 
die Akkuspannung kleiner gleich 7400 ist. Also:

if ((Akkuspannung <= 7800) && (Akkuspannung > 7400))
{
  ...
}
else if (Akkuspannung <= 7400)
{
  ...
}


oder du tauschst die Reihenfolge der if Statements (inkl. dem was 
zwischen den Klammern steht):

if (Akkuspannung <= 7400)
{
  ...
}
else if (Akkuspannung <= 7800) // d.h. 7400 < Akkuspannung <= 7800
{
  ...
}

D.h. ist Akkuspannung <= 7800 endest du in einer der beiden 
Endlosschleifen (von da geht's dann aber nie mehr raus). Ist 
Akkuspannung > 7800 kehrst du aus der Funktion zurück? Ist das das was 
du willst? Wenn ja, dann funktioniert das jetzt zwar, du solltest es 
aber trotzdem nicht tun. Eine Funktion, die manchmal endet und manchmal 
nicht, ist seeeehr hässlich und unbedingt zu vermeinden.


Gruss
Claude

[Bastler (Gast)]

Normalerweise setzt man nur einen Status für die LED und handelt diesen 
in einem Timer ab.

_delay_ms(500); ist nur eine Notlösung, warum wirst du erkennen wenn der 
Controller noch etwas Anderes machen soll...

und die uint16_t Akkuspannung; muss nicht in der while(1) sein.

[Wegstaben V. (wegstabenverbuchsler)]

Bastler schrieb:
> und die uint16_t Akkuspannung; muss nicht in der while(1) sein.

bitte präziser:

Die DEKLARATION der Variable sollte nicht in der While schlefe sein, 
sondern kann / soll außerhalb liegen