[F.....k]

#arduino #elektronika

Fragment kodu wygląda następująco:

if (status == A) {
 Timeout = millis() +1000;
 if (millis() > Timeout){
 status = B
 }
}

Po sekundzie dalej jest w stanie A. Czego nie chwytam?

Zasadniczo gdzieś w środku ma być jeszcze wywołana funkcja od wpisania kodu czterocyfrowego.

[Chlebowy_makaron]

Średnik xD

[F.....k]

@Chlebowy_makaron: w programie jest ;p

[Norskee]

@Fostrzyk: nie znam sie, ale czy aby timeout = millis() + 1000 nie oznacza ze wlasnie stworzyles zmienna millis() + 1000 jako jakis string czy ki #!$%@??

[Zrzyck]

@Fostrzyk: no dobrze jest. Timeout zawsze jest o 1k większy od millis, czyli nigdy nie wejdzie w ifa ustawiającego status na B. Nie wiem co robi millis ale zakładam, że zwraca to samo przy obu wywołaniach.

[F.....k]

Ogólnie kradzione stąd (obsługa hasła i patent z timerem): https://stackoverflow.com/questions/40966687/arduino-keypad-4x4-to-lcd-activate-deactivate-home-security-system

[F.....k]

@Zrzyck: dzięki, nie pomyślałem!

idę na kiepa i przemyślę

[Santacrruz]

@Fostrzyk: Jeśli chcesz czekać sekundę w przypadku gdy _status będzie równy A to walnij tam jakiegoś sleepa bo aktualnie zapamietujesz czas i od razu sprawdzasz czy minął. Z tego co widzę to możesz użyć delay(ms)_ tutaj info

[F.....k]

@Santacrruz: sleep nie, bo ta "sekunda" ma być na wpisanie kodu

[F.....k]

if (statusalarmu == 2) {
....timeoutA = millis() + 5000;
....rozbroj();
....if (timeoutA < millis()) {
........statusalarmu = 3;
....}
}

tak to dokładnie wygląda

[Santacrruz]

@Fostrzyk: Zatem w funkcji disarm() wczytuj stan przycisków dopóki nie minie zadany czas. Coś w stylu: while(millis() < targetTime) { readButtons(); ... } i zwrócisz odpowiedniego boola.
Wybacz brak formatowania ale niewygodnie pisze się na komórce.

[F.....k]

Dobra, ogarnąłem, zmienna pośrednia, to mi wpadło do głowy, doszukałem na forbocie:
https://forbot.pl/blog/kurs-arduino-ii-wielozadaniowosc-opoznienia-z-millis-id18418

if (statusalarmu == 2) {
rozbroj();
aktualnyczas = millis();
roznicaczasu = aktualnyczas - zapamietanyczas;
if (roznicaczasu >= 5000UL) {
zapamietanyczas = aktualnyczas;
statusalarmu = 3;
}
}

[A.....k]

@Fostrzyk: twój pierwszy kod jest ok a @Zrzyck: jak nie wie co zwraca millis to dupa cicho.
Jedynym mankamentem jest to, że if sprawdza warunek raz, zanim millis się zmieni. Potrzebujesz pętli
while(timeout > millis()) {};
W końcu millis będzie wieksze od timeout, warunek w pętli zmieni się na false i pętla się zakończy.

[Zrzyck]

@Analityk: niemiły arduinowiec :< jak odpisywałem to nie było do końca jasne co ma robić kod. elo