Problem mit Variablen auf einem Mega324

[Tim (Gast)]

Hallo,

ich habe bei mir das Problem, dass wenn ich eine Variable außerhalb der 
main deklariere, er die While-Schleife nicht verlässt:
[ ATMega324PA | avr-gcc | Atmel Studio 7 ]

Dieser Code funktioniert:

int main(void)
{
  bool Zeit_eingestellt    = false;  // Variable innerhalb der main deklariert.

  do // Belichtungszeit einstellen
  {
    _delay_ms(50);

    // Inkrementalgeber auslesen:
    Belichtungszeit += (uint16_t)Drehgeber_lesen();

    if (Belichtungszeit > 999)
    {
      Belichtungszeit = 999;
    }
    if (Belichtungszeit < 1)
    {
      Belichtungszeit = 1;
    }

    Anzeige.Anzeigen(Belichtungszeit);
    I01 = Belichtungszeit;
    
    if (!Taster_entprellt && flanke1)
    {
      Zeit_eingestellt = true;
      UART_senden((int32_t)Zeit_eingestellt, true);
    }

    flanke1 = Taster_entprellt;
  } while (!Zeit_eingestellt);
}

Dieser Code funktioniert hingegen NICHT:

bool Zeit_eingestellt    = false;  // Variable außerhalb der main deklariert.

int main(void)
{
  Zeit_eingestellt    = false;  // hier nochmal auf false gesetzt.

  do // Belichtungszeit einstellen
  {
    _delay_ms(50);

    // Inkrementalgeber auslesen:
    Belichtungszeit += (uint16_t)Drehgeber_lesen();

    if (Belichtungszeit > 999)
    {
      Belichtungszeit = 999;
    }
    if (Belichtungszeit < 1)
    {
      Belichtungszeit = 1;
    }

    Anzeige.Anzeigen(Belichtungszeit);
    I01 = Belichtungszeit;
    
    if (!Taster_entprellt && flanke1)
    {
      Zeit_eingestellt = true;
      UART_senden((int32_t)Zeit_eingestellt, true);
    }

    flanke1 = Taster_entprellt;
  } while (!Zeit_eingestellt);
}

Weiß jemand, woran das liegen kann? Was mache ich hier falsch?

LG
Tim

[holger (Gast)]

Was soll dieser Unsinn?

int main(void)
{
  do // Belichtungszeit einstellen
  {
  } while (!Zeit_eingestellt);
}

Was glaubst du was nach main() passiert wenn die Bedingung nicht
erfüllt ist? Nichts passiert dann mehr.

[Tim (Gast)]

Kleiner Nachtrag:

Da ich die Info, ob die Belichtungszeit nun gesetzt wurde, außerhalb in 
einer Funktion brauche, habe ich einfach eine zweite Variable angelegt, 
mit Unterstrich davor und beschreibe diese ebenfalls, wenn der Taster 
losgelassen wird. Auch hier wird die While-Schleife nicht verlassen.

Warum??

bool _Zeit_eingestellt = false;  zweite Variable, siehe unten

int main(void)
{
  bool Zeit_eingestellt    = false;  // Variable innerhalb der main deklariert.

  do // Belichtungszeit einstellen
  {
    _delay_ms(50);

    // Inkrementalgeber auslesen:
    Belichtungszeit += (uint16_t)Drehgeber_lesen();

    if (Belichtungszeit > 999)
    {
      Belichtungszeit = 999;
    }
    if (Belichtungszeit < 1)
    {
      Belichtungszeit = 1;
    }

    Anzeige.Anzeigen(Belichtungszeit);
    I01 = Belichtungszeit;
    
    if (!Taster_entprellt && flanke1)
    {
      Zeit_eingestellt = true;
      _Zeit_eingestellt = true;  // Wenn ich das hier einfüge, verlässt er die While-Schleife ebenfalls nicht.
      // Die "UART_Senden"-Funktion war nur zum debuggen
    }

    flanke1 = Taster_entprellt;
  } while (!Zeit_eingestellt);
}

[Tim (Gast)]

holger schrieb:
> Was soll dieser Unsinn?
>
> int main(void)
> {
>   do // Belichtungszeit einstellen
>   {
>   } while (!Zeit_eingestellt);
> }
>
> Was glaubst du was nach main() passiert wenn die Bedingung nicht
> erfüllt ist? Nichts passiert dann mehr.

Es ist eine Art Schrittkette, mit mehreren While-Schleifen 
hintereinander.
Er bleibt definitiv in der ersten Schleife hängen, wenn ich am Drehgeber 
drehe, kann ich den Wert ändern. Wenn er sie überspringen würde, würde 
er ja in die nächste Anweisung gehen, das würde ich auf dem Display 
sehen.

[Nico W. (nico_w)]

Volatile?

[Tim (Gast)]

Nico W. schrieb:
> Volatile?

Habe ich schon versucht, ohne Erfolg.

[Felix Adam (Gast)]

Probiere mal

static bool Zeit_eingestellt    = false;

(ist aber geraten, weil ich so einen ähnlichen Effekt auch mal hatte).

[Yalu X. (yalu) (Moderator)]

Wie stellst du fest, dass Zeit_eingestellt tatsächlich auf true gesetzt
wird? In deinem Eröffnungsbeitrag hast du dafür vermutlich den
UART_senden-Aufruf genutzt. Im Beispiel von 00:45 fehlt dieser Aufruf
aber.

Prinzipiell sollte es keinen Unterschied machen, ob die Variable lokal
oder global definiert ist. Wenn doch, wird die Variable vermutlich auf
Grund eines Fehlers an anderer Stelle des Programms überschrieben.

[Dieter F. (Gast)]

Praktisch wäre der komplette Code oder ein lauffähiges - 
reproduzierbares - eingeschränktes Beispiel mit gleichem Verhalten.

[Tim (Gast)]

Ich habe den Fehler beseitigt.

Als erstes habe ich die Flanke als volatile bool global deklariert.
Dann hat er sich immer aufgehängt, wenn ich den Taster losgelassen hab.

Ich hab nen Timer mit CTC alle 1ms laufen. Darin zähle ich bis 500 und 
immer wenn die 500 erreicht sind (also alle 500ms) schaue ich auf 
_Zeit_eingestellt und sende Infos bei Änderung auf den UART. Wenn ich 
das senden auskommentiere, hängt er sich nicht auf, also bleibt er in 
der UART_senden-Funktion (nur bei Änderung von "_Zeit_eingestellt" 
aufgerufen in der ISR) hängen, oder beendet diese nicht rechtzeitig.

Ich habe diese Funktion nun in die If-Abrage, welche die While-Schleife 
beendet reingepackt und es geht, ist vermutlich auch sauberer so.

Danke für die guten Anregungen und Tipps. :)

Schönes WE
Tim