Zu doof für While Schleife? Arduino

[Frank B. (susefreax)]

Hallo zusammen,

ich bin wohl einfach zu doof.

Ich Versuche folgendes:
Von einem 433MHz Empfänger wird abgefragt ob eine bestimmte Taste 
gedrückt wird. Ist dies der Fall soll ein Ausgang so lange einen 
bestimmten Wert haben wie die Taste gedrückt wird.
Mein Problem: Sobald ich in der While-Schleife bin, komme ich da nicht 
mehr raus? Ich habe wohl Tomaten auf den Augen.

#include <RCSwitch.h>
RCSwitch mySwitch = RCSwitch();


int LEDr = 11;
int RCWert = 0;
int menu = 0;
int m1 = 0;
int m2 = 0;


int RCauslesen()
{
    if (mySwitch.available()) {
    
    RCWert = mySwitch.getReceivedValue();


    mySwitch.resetAvailable();
  }
}



void setup()

{ 
pinMode(LEDr, OUTPUT);
mySwitch.enableReceive(0);
   Serial.begin(9600);


}

void loop()
  { 
    RCauslesen();
        while(RCWert == 5393)
          {            
          analogWrite(LEDr, 110);
          RCauslesen();
          }
          digitalWrite(LEDr, HIGH);
          delay(250);
          Serial.println(RCWert);
  }

Solange ich nicht in der Schleife bin zeigt mir das serial.print 
wunderbar alle Tastencodes an. Sobald ich aber in der Schleife bin ist 
RCWert immer gleich? Wie kann ich denn dieses (wahrscheinlich mega 
einfache) Problem lösen?

Er soll die While-Schleife verlassen sobald der RC-Wert sich ändert. Es 
kommt mir aber so vor als würde er gar nicht mehr in die Routine 
"RCauslesen();" rein kommen?

Bin für jeden Tipp dankbar.

[J. (Gast)]

volatile

[Bernd (Gast)]

Funktioniert mySwitch.available()?

[Max H. (hartl192)]

J. schrieb:
> volatile
Oder die globale Variable ganz weglassen und RCWert als Rückgabewert der 
Funktion RCauslesen().

[Karl H. (kbuchegg)]

Sicher, dass der ResetAvailable hier

int RCauslesen()
{
    if (mySwitch.available()) {
    
    RCWert = mySwitch.getReceivedValue();


    mySwitch.resetAvailable();
  }
}

notwendig ist?

Ich geh mal davon aus, dass du von deinem Gerät nicht ständig über den 
Tastenstatus informiert wirst, sondern nur dann, wenn sich da was 
ändert.
Und tja, wenn man mutwillig aus dem Kommunikationskanal mit Bausch und 
Bogen einfach rauslöscht, dann sollte man sich nicht wundern, wenn dabei 
auch derartige Benachrichtigungen verloren gehen.
In a Nutshell: Es ist fast NIE eine gute Idee, derartige Eingangspuffer 
einfach komplett zu löschen.

Wenn da nicht das Problem besteht, dann spick halte mal die Funktion mit 
zusätzlichen Serial.println. Das ist ja schliesslich nicht verboten, 
dass du dir hier ein paar Debug-Ausgaben einbaust

[Frank B. (susefreax)]

Ohje so viel auf einmal. Also um es vlt klarer zu machen folgendes.

Das Beispiel in der Bibliothek sieht so aus:

#include <RCSwitch.h>

RCSwitch mySwitch = RCSwitch();

void setup() {
  Serial.begin(9600);
  mySwitch.enableReceive(0);  // Receiver on inerrupt 0 => that is pin #2
}

void loop() {
  if (mySwitch.available()) {
    
    int value = mySwitch.getReceivedValue();
    
    if (value == 0) {
      Serial.print("Unknown encoding");
    } else {
      Serial.print("Received ");
      Serial.print( mySwitch.getReceivedValue() );
      Serial.print(" / ");
      Serial.print( mySwitch.getReceivedBitlength() );
      Serial.print("bit ");
      Serial.print("Protocol: ");
      Serial.println( mySwitch.getReceivedProtocol() );
    }

    mySwitch.resetAvailable();
  }
}

Heißt vor der letzten } hat man immer eine "0" außer es wird eine Taste 
gedrückt. Ich habe für meinen kleinen Test einfach die prints raus 
genommen und eine Routine draus gemacht.
Und genau diese 0 wollte ich mir für die While-Schleife zu nutze machen. 
Aber das klappt nicht. Solange ich die Taste (nennen wir sie A+) nicht 
drücke, aber jede x-Beliebige zeigt er mir die taste an, dann wieder 0. 
Außer ich habe die A+ einmal gedrückt. Ab diesem Zeitpunkt ändert sich 
der Wert RCWert nicht mehr. Der bleibt dann bei dem Code der Taste A+

[Frank B. (susefreax)]

Max H. schrieb:
> J. schrieb:
>> volatile
> Oder die globale Variable ganz weglassen und RCWert als Rückgabewert der
> Funktion RCauslesen().

Sorry das verstehe ich nicht. Wie meinst du das?

[Karl H. (kbuchegg)]

Frank B. schrieb:
> Max H. schrieb:
>> J. schrieb:
>>> volatile
>> Oder die globale Variable ganz weglassen und RCWert als Rückgabewert der
>> Funktion RCauslesen().
>
> Sorry das verstehe ich nicht. Wie meinst du das?

int foo()
{
  return 5;
}

void bar()
{
  int j;

  j = foo();
}

so wie man halt mit dem Returnwert von Funktionen arbeitet.

Das wäre zwar eleganter, aber dein Problem hat damit erst mal nichts zu 
tun.

[Holger L. (max5v)]

Schmeiß in der while - Schleife einfach mal das "analogWrite(LEDr, 
110);" heraus, die PWM-Funktion stört eventuell das auslesen deines 
Empfängers.

Die While - Schleife wird nicht verlassen weil RCWert sich nicht ändert, 
das sollte bedeuten das RCauslesen(); nicht mehr ausgeführt wird. Da in 
der while nur eine weitere Anweisung steht sollte es nahe liegen das es 
an ihr liegt.
Eventuell wird ein Timer für die PWM sowie das Empfangsmodul verwendet, 
Abhilfe könnte eventuell auch einfach ein anderer Pin für die Led 
schaffen, bei einem Arduino mit ATmega168/328 zum Beispiel Pin 5/6 oder 
Pin 9/10.