"if(globaleVariable==0)" funktioniert nicht

Thomas D Gast 10.04.2009 00:29 #1224560

Lesenswert?

•

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.

Antwort Zitieren Bearbeiten Thread verschieben Thread sperren Anmeldepflicht aktivieren Anpinnen Thread löschen Thread mit anderem zusammenführen Beitrag melden

was-willst-du Gast 10.04.2009 00:33 #1224566

Lesenswert?

•

▲
▼

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

Was passiert in der Interruptroutine?

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Felix N. (time2ride) 10.04.2009 00:36 #1224567

Lesenswert?

•

▲
▼

volatile int inputEnable=1;

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

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Lasse S. (cowz) 10.04.2009 00:37 #1224569

Lesenswert?

•

▲
▼

Hi,

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

Gruß, Lasse

PS: Ok, Felix war schneller ;)

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Thomas D Gast 10.04.2009 00:40 #1224570

Lesenswert?

•

▲
▼

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.

Angehängte Dateien:

main.c (6,35 KB) | Codeansicht

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Thomas D Gast 10.04.2009 00:41 #1224572

Lesenswert?

•

▲
▼

Alles klar Danke!!!

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Thomas D Gast 10.04.2009 00:46 #1224576

Lesenswert?

•

▲
▼

Geht leider trotzdem noch nicht.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Lasse S. (cowz) 10.04.2009 00:47 #1224577

Lesenswert?

•

▲
▼

Dann poste bitte mal den kompletten Code!

Gruß, Lasse

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Thomas D Gast 10.04.2009 00:50 #1224579

Lesenswert?

•

▲
▼

Der ist ziemlich lang. Als Anhang...

Angehängte Dateien:

main.c (6,35 KB) | Codeansicht

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Lasse S. (cowz) 10.04.2009 00:54 #1224581

Lesenswert?

•

▲
▼

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

Teste doch mal nur den minimalnotwendigen Code.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Thomas D Gast 10.04.2009 00:59 #1224584

Lesenswert?

•

▲
▼

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.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Lasse S. (cowz) 10.04.2009 01:03 #1224587

Lesenswert?

•

▲
▼

Im Interrupt setzt du aber auch

inputEnable=0;

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Stefan E. (sternst) 10.04.2009 01:06 #1224589

Lesenswert?

•

▲
▼

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.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Thomas D Gast 10.04.2009 01:07 #1224590

Lesenswert?

•

▲
▼

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.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Lasse S. (cowz) 10.04.2009 01:10 #1224593

Lesenswert?

•

▲
▼

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

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Thomas D Gast 10.04.2009 01:15 #1224595

Lesenswert?

•

▲
▼

(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.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Lasse S. (cowz) 10.04.2009 01:18 #1224598

Lesenswert?

•

▲
▼

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

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Stefan E. (sternst) 10.04.2009 01:20 #1224599

Lesenswert?

•

▲
▼

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.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Thomas D Gast 10.04.2009 01:21 #1224600

Lesenswert?

•

▲
▼

Wie und wann löscht er sie denn?

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Stefan E. (sternst) 10.04.2009 01:23 #1224601

Lesenswert?

•

▲
▼

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;

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Thomas D Gast 10.04.2009 01:23 #1224602

Lesenswert?

•

▲
▼

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

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Lasse S. (cowz) 10.04.2009 01:24 #1224603

Lesenswert?

•

▲
▼

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

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Thomas D Gast 10.04.2009 01:24 #1224604

Lesenswert?

•

▲
▼

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

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Thomas D Gast 10.04.2009 17:22 #1225023

Lesenswert?

•

▲
▼

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?

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Lothar M. (lkmiller) Moderator (Firma: Titel) 10.04.2009 20:07 #1225164

Lesenswert?

•

▲
▼

> 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.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

"if(globaleVariable==0)" funktioniert nicht

Antwort schreiben