funkcja IF źle działa Arduino ESP8266

Witam,

tworze projekt, czerpie wiedze że źródła:
https://platformio.org/lib/show/1089/IRremoteESP8266

Popełniłem oto taki kod.



Monitor portu szeregowego zwraca:


czyli funkcja wykrywa kod: 800 a mimo to nie zmienia stanu output5.
I nie wiem czemu od razu wchodzi w:



Koledzy co źle robie?
Pomóżcie bo się zakopałem....

maystero wrote:

if (results.value = 800)

tym poleceniem wpisałeś do results.value wartość 800.
Porównanie robi się podwójnym "równa się":

Quote:

if (results.value == 800)

Faktycznie. Chochlik się wplątał.
Poprawiłem kod ale dalej nie działa jak trzeba



Monitor portu:

Chyba powinieneś opublikować cały kod.
Możliwe, że gdzieś użyłeś nieodpowiednich typów zmiennych.

Oto cały kod:




oraz zrzut monitora szeregowego:

maystero wrote:

Code: c

Log in, to see the code

Czegoś nie rozumiem. Ustawiasz output na "on" tylko wtedy, jeżeli już jest "on", czyli nigdy...
Nawiasem mówiąc, typ string do przechowywania stanu to nietypowy, bo niezbyt efektywny pomysł.

Hym.... poprawiłem.


Dalej nie działa.

Może źle kombinuje ale wg mnie powinno to działać tak:
jeżeli results.value == 800 i jeżeli output5State == "off"
to ustaw:
output5 na HIGH
output5State na "on"
i wyświetl napis HIGH
w przeciwny wypadku:
output5 na LOW
output5State na "off"
i wyświetl napis LOW

nie rozumiem czemu to nie działa....

maystero wrote:

eżeli results.value == 800 i jeżeli output5State == "off"

Niezupełnie.
Jeżeli (results.value == 800) oraz (output5 == HIGH && output5State == "on") to wtedy output5 ustawia na HIGH.
Jeżeli (results.value == 800) oraz (output5 != HIGH || output5State != "on") to wtedy output5 ustawia na LOW.
W każdym innym wypadku nic nie robi z output5.

Drugi wariant:


oraz monitor portu:



i wchodzi w ostatni warunek.

Sporządź prosta tabelkę i napisz, co chcesz uzyskać w zależności od stanów zmiennych: results.value, output5State oraz output5.

khoam wrote:

Sporządź prosta tabelkę i napisz, co chcesz uzyskać w zależności od stanów zmiennych: results.value, output5State oraz output5.

Popatrz na tabelkę, po spełnieniu warunku "zmieniasz" HIGH/on na HIGH/on, coś to jest bez sensu.
Rozumiem, że chcesz zmieniać stan na przeciwny czyli HIGH/on na LOW/off oraz odwrotnie.
Albo po prostu neguj znacznik i "wysyłaj" jego wartość na wyjście.

krzysiek_krm wrote:

Popatrz na tabelkę, po spełnieniu warunku "zmieniasz" HIGH/on na HIGH/on, coś to jest bez sensu.
Rozumiem, że chcesz zmieniać stan na przeciwny czyli HIGH/on na LOW/off oraz odwrotnie.
Albo po prostu neguj znacznik i "wysyłaj" jego wartość na wyjście.

Czyli jeśli dobrze cie rozumiem - powinno być tak?

Moim zdaniem - tak.
Poza tym uważam, że badanie dwóch warunków (iloczyn logiczny) jest nadmiarowe.

krzysiek_krm wrote:

Moim zdaniem - tak.
Poza tym uważam, że badanie dwóch warunków (iloczyn logiczny) jest nadmiarowe.

Poprawiłem. O to koledzie chodziło?


mam wrażenie że wogóle nie wchodzi w if



wyświetla tylko wartość: serialPrintUint64(results.value, HEX);

Tak na pierwszy rzut oka wkradła Ci się literówka Off vs. off.

krzysiek_krm wrote:

Tak na pierwszy rzut oka wkradła Ci się literówka Off vs. off.

poprawiłem, to samo

maystero wrote:

Code: c

Log in, to see the code

Jeżeli dobrze rozumiem (bo nie znam Arduino), to wypisujesz odebraną wartość szesnastkowo. Czyli odbierasz 0x800 (2048 dziesiętne), a porównanie jest na 800 dziesiętne. Nic dziwnego, że nie wchodzi do if-a.

Biblioteka którą używam zwraca poleceniem serialPrintUint64(results.value, HEX)
właśnie liczbę 800.
tu jest źródło:
https://platformio.org/lib/show/1089/IRremoteESP8266

Zmieniłem oczywiście serialPrintUint64(results.value, BIN) i sprawdziłem efekt działania. Też nie nie wchodzi w funkcje if.

maystero wrote:

Zmieniłem oczywiście serialPrintUint64(results.value, BIN) i sprawdziłem efekt działania.

Użyj po prostu serialPrintUint64(results.value)

khoam wrote:

maystero wrote:

Zmieniłem oczywiście serialPrintUint64(results.value, BIN) i sprawdziłem efekt działania.

Użyj po prostu serialPrintUint64(results.value)

Pomogło działa!

Szanowny kolego khoam może pomożesz mi jeszcze w tej części kodu.
Jak już udało się opanować funkcje if postanowiłem rozszerzyć kod aby przełączać 4x LED-y. Do tego celu użyłem funkcji switch...case
https://www.arduino.cc/reference/en/language/structure/control-structure/switchcase/

oto moja komplikacja:


a zoleji na monitorze portu szeregowego leci w nieskończoność


Wydaje mi się dobrze zastosowałem switch...case ale może znów mi się wkradł jakiś chochlik i nie chce się ujawnić.

Przede wszystkim, zamiast:powinno być:Dodatkowe sprawdzanie stanu zmiennej 'results.value' jest niepotrzebne, skoro:
już to robi.

W switch..case zwykle umieszcza się również etykietę 'default:' na wypadek, gdy żadne z wcześniejszych warunków nie wystąpiło i też trzeba podjąć akcję - masz to opisane w linku, który podałeś.

O to kolejna moje kombinacja:



Jest lepiej. Po nacisnięciu klawisza na pilocie, który powinien być =2049
układ wykrywa go jako =3268448703.
Załączony zrzut z monitora portu:

oczywiście całość sie zapętla i nie wychodzi z switch..case.

maystero wrote:

Jest lepiej. Po nacisnięciu klawisza na pilocie, który powinien być =2049 układ wykrywa go jako =3268448703.

W którym miejscu switch...case wysyłasz na Serial wartość results.value? Chodzi mi o powyższy kod.

maystero wrote:

oczywiście całość sie zapętla i nie wychodzi z switch..case.

Wychodzi, inaczej nie byłoby serii komunikatów "jakis blad".

Naprawiłem kod programu w nocy. Jak dzieci poszły spać.
Problemem okazał się irrecv.resume(); // Receive the next value:

przenieosłem to na zewnątrz funkcij switch...case



Na tą chwile program działa zgodnie z założeniami. Po testuje kilka dni i jeśli nic nie wyskoczy to chyba temat wyczerpany.