Arduino Polska Forum

Witam. O arduino dowiedziałem się 3 tygodnie temu i od tamtego momentu uczę się programować. Do tamtego momentu nie miałem styczności z jakimkolwiek językiem programowania. Małymi kroczkami postanowiłem zbudować system alarmowy. Wzorując się na tym poradniku https://forbot.pl/blog/kurs-arduino-ii-k...wy-id18341. Pojawił się pierwszy problem.

kod:

#include <Keypad.h>

int diodaZielona = 13;
int diodaZulta = 12;
int diodaCzerwona = 11;

 
const byte ROWS = 4; 
const byte COLS = 4;

byte rowPins[ROWS] = {9, 8, 7 ,6 };
byte colPins[COLS] = {5, 4, 3, 2};

char keys[ROWS][COLS] = {
  {'1','2','3','A'},
  {'4','5','6','B'},
  {'7','8','9','C'},
  {'*','0','#','D'}
};

Keypad klawiatura = Keypad( makeKeymap(keys), rowPins, colPins, ROWS, COLS ); //inicjalizacja klawiatury

int stanAlarmu = 1;

void setup() {
  pinMode(diodaZielona, OUTPUT);
  pinMode(diodaZulta, OUTPUT);
  pinMode(diodaCzerwona, OUTPUT);

}
void loop() {
 
  char klawisz = 0;
 
  switch(stanAlarmu) { //Wykonywanie akcji odpowiedniej dla danego stanu
    case 1: 
    digitalWrite(diodaZielona, HIGH);
    digitalWrite(diodaZulta, LOW);
    digitalWrite(diodaCzerwona, LOW);
   
    klawisz = klawiatura.getKey();
   
    if (klawisz == 'A') { // aktywacja alarmu
        stanAlarmu = 2; // przechodzi do case 2
    }

    break;
   
    case 2: 
    digitalWrite(diodaZielona, LOW);
    digitalWrite(diodaZulta, HIGH);
    digitalWrite(diodaCzerwona, LOW);
   
    if (klawisz == 'B') { // dezaktywacja alarmu
        stanAlarmu = 1; // przechodzi do case 1
    }
 
    break;
  }
 
}


Naciskając przycisk A alarm przechodzi do case2. Dodając funkcję if (klawisz == 'B') {  stanAlarmu = 1; }  w case2 chciałem aby po naciśnięciu przycisku B alarm się dezaktywował i przeszedł do case1, niestety tak się nie dzieje.
O ile po naciśnięciu przycisku A alarm rzeczywiście przechodzi do case2 ( świeci żółta dioda), tak po naciśnięciu B nic się nie dzieje. Żółta dioda nadal się świeci, a powinna świecić się zielona.
Proszę o pomoc w rozwiązaniu tego problemu.

W case 2 brakuje odczytu stanu klawiatury


Ale i tak miło widzieć, że kursy z Forbota cieszą się popularnością

Dzięki. Nie będę kłamał, że nie zauważyłem tego błędu... po prostu nie wiedziałem, że tak powinno być Będę już wiedział, że potrzeba odczytu stanu klawiatury. Projekt alarmu będę chciał zmodyfikować. Planuję dołożyć do niego moduł GSM i  ekran LCD. Może się uda, a jak nie to trudno. Jeśli spadać, to z wysokiego konia 

Napotkałem nowy problem którego nie mogę rozgryźć. Kod:
#include <LiquidCrystal.h>
#include <Keypad.h>

int diodaZielona = 13;
int diodaZulta = 12;
int diodaCzerwona = 11;
int buzzer = 10;
int czujnikRuchu = 1;

const byte ROWS = 4;
const byte COLS = 4;

byte rowPins[ROWS] = {9, 8, 7 ,6 };
byte colPins[COLS] = {5, 4, 3, 2};

char keys[ROWS][COLS] = {
{'1','2','3','A'},
{'4','5','6','B'},
{'7','8','9','C'},
{'*','0','#','D'}
};

Keypad klawiatura = Keypad( makeKeymap(keys), rowPins, colPins, ROWS, COLS ); //inicjalizacja klawiatury

volatile int stanAlarmu = 1;
int aktywacjaAlarmuPozycja = 1;
char aktywacjaCyfra1 = '1';
char aktywacjaCyfra2 = '2';
char aktywacjaCyfra3 = '3';
char aktywacjaCyfra4 = '4';
char aktywacjaCyfra5 = 'A';

int dezaktywacjaAlarmuPozycja = 1;
char dezaktywacjaCyfra1 = '1';
char dezaktywacjaCyfra2 = '2';
char dezaktywacjaCyfra3 = '3';
char dezaktywacjaCyfra4 = '4';
char dezaktywacjaCyfra5 = 'D';

void setup() {
pinMode(diodaZielona, OUTPUT);
pinMode(diodaZulta, OUTPUT);
pinMode(diodaCzerwona, OUTPUT);
pinMode(buzzer, OUTPUT);
pinMode(czujnikRuchu, INPUT_PULLUP);

}
void loop() {

char klawisz = 0;

switch(stanAlarmu) {
case 1:
//Czuwanie
digitalWrite(diodaZielona, HIGH);
digitalWrite(diodaZulta, LOW);
digitalWrite(diodaCzerwona, LOW);

klawisz = klawiatura.getKey();
if (klawisz) {
if (aktywacjaAlarmuPozycja == 1 && klawisz == aktywacjaCyfra1) {
aktywacjaAlarmuPozycja++;
} else if (aktywacjaAlarmuPozycja == 2 && klawisz == aktywacjaCyfra2) {
aktywacjaAlarmuPozycja++;
} else if (aktywacjaAlarmuPozycja == 3 && klawisz == aktywacjaCyfra3) {
aktywacjaAlarmuPozycja++;
} else if (aktywacjaAlarmuPozycja == 4 && klawisz == aktywacjaCyfra4) {
aktywacjaAlarmuPozycja++;
} else if (aktywacjaAlarmuPozycja == 5 && klawisz == aktywacjaCyfra5) {
stanAlarmu = 2;
}
}

break;

case 2://Monitorowanie
digitalWrite(diodaZielona, LOW);
digitalWrite(diodaZulta, HIGH);
digitalWrite(diodaCzerwona, LOW);

klawisz = klawiatura.getKey();
if (klawisz) {
if (dezaktywacjaAlarmuPozycja == 1 && klawisz == dezaktywacjaCyfra1) {
dezaktywacjaAlarmuPozycja++;
} else if (dezaktywacjaAlarmuPozycja == 2 && klawisz == dezaktywacjaCyfra2) {
dezaktywacjaAlarmuPozycja++;
} else if (dezaktywacjaAlarmuPozycja == 3 && klawisz == dezaktywacjaCyfra3) {
dezaktywacjaAlarmuPozycja++;
} else if (dezaktywacjaAlarmuPozycja == 4 && klawisz == dezaktywacjaCyfra4) {
dezaktywacjaAlarmuPozycja++;
} else if (dezaktywacjaAlarmuPozycja == 5 && klawisz == dezaktywacjaCyfra5) {
stanAlarmu = 1;
}
}
break;

}
}

W celu aktywacji wpisuje pin "1 2 3 4 A" , następnie dezaktywuje alarm wpisując pin "1 2 3 4 D" i rzeczywiście alarm jest dezaktywowany. Przy ponownej próbie aktywacji wystarczy nacisnąć na klawiaturze ostatni znak pinu "A" , a przy dezaktywacji tylko "D". Oczywiście takiego efektu nie oczekiwałem. Chcę aby wymagany był cały pin, a nie tylko ostatnie znaki. Analizując kod nie mogę zrozumieć dlaczego tak się dzieje.

Trzeba ustawić na 1 zmienne aktywacjaAlarmuPozycja i dezaktywacjaAlarmuPozycja po lub przed zmienną stanAlarmu.
Dodam, że tylko trzeba coś zrobić aby się ustawiały na 1 po każdym niewłaściwym naciśnięciu przycisku oraz po jakimś czasie od ostatniego naciśnięcia jakiegokolwiek przycisku.

Właśnie to zauważyłem. Błędnie podany znak w środku pinu nie przeszkadza w aktywacji i wyłączeniu alarmu. W obecnym momencie ważna jest tylko kolejność znaków. Jutro będę nad tym myślał.

Miałem 2 problemy z lcd, jeden rozwiązałem, na drugim utknąłem, mianowicie za żadne skarby nie potrafię skasować poprzedniego napisu na LCD.



W case1 wyświetla się napis "CZUWANIE WYL". Dlaczego po wpisaniu prawidłowego znaku pinu, nie kasuje się napis na lcd? Zamiast niego powinien pojawić się "x".  Mam na myśli ten fragment kodu. Użyłem lcd.clear();  ale nic to nie daje.


Dlaczego tak się dzieje?

Ponieważ w case1 masz:


Po wciśnięciu prawidłowego przycisku kasowany jest lcd i wpisywany jest 'x', ale zaraz masz znowu


A dzieje się to tak szybko, że nie zauważasz.

Druga sprawa, to nie musisz po każdym wciśnięciu klawisza pisać:

ponieważ po wysłaniu znaku na lcd, kursor jest przesuwany o jeden znak w prawo, więc wystarczy:


Trzecia sprawa, to w case2 gdy wcisnę jakikolwiek inny klawisz niż któryś z dezaktywacjaCyfra1-5, to od razu stanAlarmu = 5;
a powinno to wyglądać inaczej. Niezależnie czy wpisuję prawidłowe znaki czy złe, to na ekranie powinny pojawiać się iksy. Po zakończeniu wpisywania należy porównać wprowadzony ciąg z kodem alarmu i na tej podstawie zmieniać stanAlarmu.
Te Twoje wypociny można uprościć, zredukować ilość linii i przy okazji stworzyć bardziej niezawodną aplikację.

Doszedłem gdzie był błąd dziś w nocy. Kod Będę chciał w wersji ostatecznej znacznie uprościć. Wiedza obecna na to mi nie pozwala, więc wygląda to jak wygląda, ale mam nadzieję, że się to zmieni.

Ostatnio miałem troszkę czasu i zmieniłem kod.  Wydaje mi się że wygląda to znacznie lepiej. 
Mam problem z wyświetlaniem wpisywanych znaków , mianowicie podczas wpisywania znaki nie wpisują się jeden obok drugiego , tylko  nakładają się na siebie. Morduję  się z tym cały dzień i nie daję rady ogarnąć tego. Wiem, że naciśnięcie klawisza wywołuje za każdym razem funkcję processNumberKey,  która wyświetla  znak ciągle w tym samym miejscu. Jak to ogarnąć? Proszę o pomoc.