"if(globaleVariable==0)" funktioniert nicht

[Thomas D (Gast)]

Hallo

ich habe hier eine selbst gebaute 3x3 LED-Matrix und eine 3x3 
Tastermatrix.

Die LEDs und Taster werden mit ca 100 Hz durch Interrupt angesteuert.

Wenn ein Taster gedrückt wird, wird die globale Variable inputEnable im 
Interrupt auf 0 gesetzt und die zum Taster entsprechende LED leuchtet.

Komischerweise erkennt er if(inputEnable==0) nicht in der while 
Schleife. Hier der Code:

int inputEnable=1;

int main(void)
{
  int temp;
  init();
  while(1)
  {
    if(inputEnable==0)    // Spieler hat einen Stein gesetzt
    {
      feld[0][0]=1;    
    }
  }
  return 0;
}

Zum Testen soll also einfach die LED in der Ecke [0][0] angehen. Tut sie 
aber nicht.

[was-willst-du (Gast)]

Wenn das Programm die Gleichheit nicht feststellt, so wird sie auch 
nicht vorhanden sein. Der Wunsch hilft da nicht.

Was passiert in der Interruptroutine?

[Felix N. (time2ride)]

volatile int inputEnable=1;

Ist des Rätsels Lösung, da der Compiler die Variable sonst wegoptimiert.

[Lasse S. (cowz)]

Hi,

Globale Variablen, die in Interrupts verwendet werden müssen als 
volatile deklariert werden.

Gruß, Lasse

PS: Ok, Felix war schneller ;)

[Thomas D (Gast)]

Oh, tut mir Leid hab ich vergessen:

Ich weiß, dass interruptEnable==0 ist da das anschalten der anderen LEDs 
gesperrt ist. Ich häng mal das File an. Gleich oben ist der Interrupt. 
Egal welchen Taster man drückt, die entsprechende LED geht an. Danach 
sind die anderen gesperrt und trotzdem leuchtet die LED in der Ecke 
nicht.

[Thomas D (Gast)]

Alles klar Danke!!!

[Thomas D (Gast)]

Geht leider trotzdem noch nicht.

[Lasse S. (cowz)]

Dann poste bitte mal den kompletten Code!

Gruß, Lasse

[Thomas D (Gast)]

Der ist ziemlich lang. Als Anhang...

[Lasse S. (cowz)]

Kann es sein, dass die Timerroutine das Flag wieder löscht?

Teste doch mal nur den minimalnotwendigen Code.

[Thomas D (Gast)]

Ja. Der Timer wird alle 0,01/3 s neu gestartet und dabei der Interrupt 
ausgelöst. Das Programm ist ja schon sehr reduziert. Die ganzen 
Funktionen unten werden garnicht benutzt. Durch den Interrupt weiß ich 
ja, dass darin inputEnable!=1. In der while Schleife kapiert er das 
nicht.

[Lasse S. (cowz)]

Im Interrupt setzt du aber auch

inputEnable=0;

[Stefan E. (sternst)]

feld[0][0] muss hier auch volatile sein, um ein Zurückschreiben der 
geänderten Variable in der Schleife zu erzwingen. Ein lokaler Cast wäre 
hier das Beste:

    if(inputEnable==0)    // Spieler hat einen Stein gesetzt
    {
      (volatile int)feld[0][0]=1;    
    }

PS: Und mache dir mal Gedanken über die Größe der Variablen. Die sind 
alle größer als nötig, was eine schlechte Idee ist.

[Thomas D (Gast)]

Richtig. Und in der while(1) Schleife in der Main steht, dass dann das 
feld[0][0]=1;

Demach müsste jetzt die LED in der Ecke leuchten. Tut sie aber nicht.

[Lasse S. (cowz)]

Der Interrupt setzt inputEnable=0;. Kein Wunder, dass du das in der Main 
nicht mehr mitbekommst...

Alternativ: Kann es sein, dass die LED einfach nur nicht leuchtet, weil 
beim Anzeigemechanismus was kaputt ist? Mal ohne die If-Abfrage einfach 
direkt gesetzt?

Gruß, Lasse

[Thomas D (Gast)]

(volatile int)feld[0][0]=1;

main.c:188: error: lvalue required as left operand of assignment



Ich hatte neulich Probleme mit chars, da hab ich jetzt erstmal int 
genommen. Der Platz reicht schon aus.



Alternative stimmt auch nicht. Ich kann überall wo er sonst durchgeht 
feld[0][0]=1 setzten und die LED leuchtet. Das geht mit allen LEDs.

[Lasse S. (cowz)]

Und die nicht Alternative, dass er das im Interrupt löscht ignorierst du 
einfach mal so? ;)

[Stefan E. (sternst)]

Lasse S. wrote:
> Und die nicht Alternative, dass er das im Interrupt löscht ignorierst du
> einfach mal so? ;)

Das ist doch genau das, was er will. Er fragt doch in main auf Null ab.

[Thomas D (Gast)]

Wie und wann löscht er sie denn?

[Stefan E. (sternst)]

Thomas D wrote:
> (volatile int)feld[0][0]=1;
>
> main.c:188: error: lvalue required as left operand of assignment

Dann eben mit etwas Verrenkung:

*(volatile int*)&feld[0][0] = 1;

[Thomas D (Gast)]

Löschen im Sinne von =0 setzen. Ja, das soll er. Und die main erkennt es 
nicht.

[Lasse S. (cowz)]

hust ;) Stimmt ja, sorry... Zu spät, ich geh dann jetzt mal lieber ins 
Bett ;)

[Thomas D (Gast)]

Und es klappt tatsächlich. Was auch immer in dieser Zeile geschehen 
mag... sie funktioniert. Danke!!!

[Thomas D (Gast)]

So, da dies nur ein kleiner Test war und ich das Feld ziemlich oft 
brauche kommt es immer wieder zu Fehlern.


Wie schafft man es also ein Feld sowohl in einem Interrupt als auch in 
der main Funktion schreiben/lesen zu können.


Reicht es wenn ich das Feld ebenfalls

volatile

definiere?

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

> also ein Feld sowohl in einem Interrupt als auch in
> der main Funktion schreiben/lesen zu können.
Das ist eher ein Kapselungsproblem (Semaphore). Du darfst nicht 
gleichzeitig vom Interrupt und vom Hauptprogramm aus auf das selbe 
Element im Feld schreibend zugreifen. Das geht garantiert schief.