|
Anzeige
|
if Verschachtelung
Gast
#3363443
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 ;-))
wie willst du aus den while-Schleifen kommen? if (Akkuspannung <= 7800)
{
...
}
else
{
if (Akkuspannung <= 7400)
{
...
}
}
}
Wie soll dabei die zweite Abfrage eine Chance haben?
Gast
#3363452
weil 7399 < 7800 ist und daher die erste If-Abfrage schon wahr ist. Die zweite wird nie erreicht.
Gast
#3363454
>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!
Gast
#3363461
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.
Gast
#3363487
hat keiner ne Idee? Beginner schrieb: > ist die Spannung <= 7800 > > > wenn die Spannung aber <= 7400 öhm, den Widerspruch bemerkst du nicht? 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. 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.
Gast
#3363519
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();
}
weil du immer noch nicht checkst, dass ein Wert, der kleiner als 7400 ist, auch kleiner als 7800 ist...
Gast
#3363532
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. 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
Gast
#3363703
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. 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 Antwort schreiben |
Anzeige
|
