AVR-gcc Wie IF-Abfrage richtig nachvollziehen

[Christian S. (roehrenvorheizer)]

Hallo allerseits,


momentan befasse ich mich mit den Anpassungen dieses vorgefertigten auf 
IRMP basiernden Moodlight-Programms

Beitrag "noch ein AVR Moodlight"
Re: noch ein AVR Moodlight
AVR-Moodlight-20120308.tar.gz (86,6 KB, 103 Downloads)

an meine bestehende Schaltung und an diejenigen Fernbedienungen, die mir 
zur Verfügung stehen.

Nachdem nun die Anpassung an meine NEC-Fernbedienung gelungen war, 
wollte ich dies auch für die RC5-Fernbedienung vornehmen. Klappte auch. 
Ebenso für das Kaseikyo-Protokoll.

Nun sollten beide Fernbedienungen wahlweise erkannt werden und genau da 
beginnt mein Problem.
Die Protokolle werden einzeln erkannt, bei jedem Tastendruck blitzt die 
Toggle-LED kurz auf, bleibt aber niemals an und mir ist vor allem die 
Wirkungsweise der IF-Abfrage unklar, da ich mich beim Durchdenken immer 
in Widersprücken verstricke.

Ich verstehe die Wirkungsweise dieser Zeilen folgendermaßen:

In My_PROTO und MY_REMOTE werden Protokoll und Geräte-Code derjenigen 
Fernbedienung festgelegt, auf die das Programm reagieren soll.
Diese werden dann beide verglichen mit "irmp_data.protocol" und 
"irmp_data.address".
Falls die "richtigen" Signale empfangen wurden, soll eine Aktion 
abhängig von der gedrückten Taste geschehen, sofern es eine der 
definierten Tasten ist, hier also nur die vier Tasten ON/OFF und 1, 2, 
3.

Die IF-Abfrage benutzt !=, das ich mit "ungleich" übersetze.
Falls ("Vergleich1 auf Ungleichheit" ODER "Vergleich2 auf Ungleichheit") 
toggle die LED und return.
Das würde die Aktionen auslösen, sofern nur einer der Partner ungleich 
wäre. Das ist aber falsch.

Was genau macht eigentlich das Return?
Return kenne ich, um einen Rückgabewert an das übergeordnete Programm zu 
geben. Z.B. return (0) oder return variablenname.
Falls nicht die richtige Fernbedienung (Protokoll und Gerätecode) 
erkannt wurden, soll die nachfolgende switch-case-Abfrage garnicht 
durchlaufen werden.

Ich wäre um Tipps dankbar.
mit freundlichem Gruß
____________________________________________
in moodlight.c (Darstellung etwas gekürzt):

/* remote control address and command codes */

#define MY_PROTO    IRMP_NEC_PROTOCOL   /* IR protocol and */
#define MY_REMOTE   0x916e              /* system code */


....
....


/* last received IR signal */

IRMP_DATA irmp_data;


/* evaluate an IR remote signal */
void moodlight_remote(void)
{

/*  es geht um genau diese IF-Abfrage, die ich nicht genau verstehe <<<<<<<<<<<<<--<<<<<<<<<---<<<<<<<<---*/
    if (   irmp_data.protocol != MY_PROTO
        || irmp_data.address  != MY_REMOTE) {
    PORTC ^= 1 << PC5;    // LED toggeln
        return;
    }

    switch (irmp_data.command) {

    case KEY_OPERATE: /* on/off */
        {
            if ((irmp_data.flags & IRMP_FLAG_REPETITION) == 0) {
                ml.flags ^= ML_FLAG_OFF;
                if (ml.flags & ML_FLAG_OFF) {
                    level_r = 0;
                    level_g = 0;
                    level_b = 0;
                } else {
                    moodlight_wakeup();
                }
            }
            break;
        }

    case KEY_1:
        {
            ml.mode = 1;
            ml.delay = 6;
            ml.flags = 0;
            ml.cycle = 1;
            tick = 1;
            break;
        }

    case KEY_2:
        {
            ml.mode = 2;
            ml.delay = 4;
            ml.flags = 0;
            ml.cycle = 1;
            tick = 1;
            break;
        }

    case KEY_3:
        {
            ml.mode = 3;
            ml.delay = 4;
            ml.flags = 0;
            ml.cycle = 0;
            tick = 1;
            break;
        }
    }
}

[Rufus Τ. F. (rufus)]

Christian S. schrieb:
> Die IF-Abfrage benutzt !=, das ich mit "ungleich" übersetze.
> Falls ("Vergleich1 auf Ungleichheit" ODER "Vergleich2 auf Ungleichheit")
> toggle die LED und return.


> Das würde die Aktionen auslösen, sofern nur einer der Partner ungleich
> wäre.

Was meinst Du mit "die Aktionen auslösen"? Das Ansteuern der LED?

> Das ist aber falsch.

So aber stehts da.


> Was genau macht eigentlich das Return?

Die Funktion verlassen. Da die Funktion als void deklariert ist, gibt 
es keinen Rückgabewert, also steht auch nichts hinter return.

Der Code, der auf das return -Statement folgt, wird nur durchgeführt, 
wenn BEIDE, also Protokoll und Adresse korrekt sind.

[Christian S. (roehrenvorheizer)]

Hallo nochmal,

mit "Aktionen auslösen" meine ich das Durchlaufen der 
Switch-case-Anweisungen. Wobei sie auch dann durchlaufen werden, wenn 
falsche Tasten gedrückt werden. Mit falschen meine ich solche, die 
keinerlei Aktion auslösen sollen.

Nachdem nun klar ist, was das Return genau macht, ist die ganze Sache 
einfacher zu verstehen:
Die IF-Abfrage macht einen Vergleich auf Ungleichheit. Falls etwas 
ungleiches vorkommt, führe toggeln und return aus. Springe aus 
"moodlight_step" heraus. Die Switch-case-Abfrage wird nicht mehr 
durchlaufen, so daß keinerlei Tastendrücke eine "Aktion" auslösen 
können.
Falls Gleichheit vorliegt, die switch-case Abfragen durchlaufen und bei 
"richtiger" Taste eine Aktion ausführen.

Danke für's Helfen, das hat mich schon weiter gebracht.

mit freundlichem Gruß

[Rufus Τ. F. (rufus)]

Christian S. schrieb:
> Danke für's Helfen, das hat mich schon weiter gebracht.

Bei so elementaren Dingen einer Programmiersprache bringt einen 
übrigens auch ein Buch weiter -- "Programmieren in C", Brian Kernighan 
& Dennis Ritchie, 2. Ausgabe, Hanser-Verlag.