Jak naprawić kod - instrukcja switch-case i LED?

[jas123]

Dzisiaj próbowałem nauczyć się sterować kilkoma funkcjami jednym przyciskiem za pomocą "switch case" i coś mi nie wyszło. Z założenia program po wciśnięciu przycisku miał zapalać  diodę, gasić ją po 2 sekundach .Po kolejnym przyciśnięciu przycisku zapalać następną diodę , gasić po 2 sekundach i tak dalej... Zamiast tego diody same zapalają się i gasną po kolei . Z góry dziękuje za odpowiedzi.

szkic programu:

#define przycisk 6
#define led1 7
#define led2 8
#define led3 9
#define led4 10
byte x = 0; 
void setup (){
   digitalWrite(led1,LOW);
    digitalWrite(led2,LOW);
    digitalWrite(led3,LOW);
    digitalWrite(led4,LOW);
  pinMode(led1,OUTPUT);
   pinMode(led2,OUTPUT);
   pinMode(led3,OUTPUT);
   pinMode(led4,OUTPUT);
  pinMode(przycisk,INPUT_PULLUP);
}
void loop(){
  if(digitalRead(przycisk == LOW)){
  switch(x){
    case 0:
    digitalWrite(led1,HIGH);  
  delay(2000); 
  digitalWrite(led1,LOW);
    x=1;
    break;
     case 1:
    digitalWrite(led2,HIGH);  
  delay(2000);
  digitalWrite(led2,LOW);
    x=2;
    break;
     case 2:
    digitalWrite(led3,HIGH); 
  delay(2000); 
  digitalWrite(led3,LOW);
    x=3;
    break;
     case 3:
    digitalWrite(led4,HIGH);  
  delay(2000);
  digitalWrite(led4,LOW);
    x=0;
    break;
  }
  }else{
    digitalWrite(led1,LOW);
    digitalWrite(led2,LOW);
    digitalWrite(led3,LOW);
    digitalWrite(led4,LOW);
  }
}

 

[adam30010]

A jesteś pewny co do stanu aktywnego przycisku? Spróbuj podmienić 

if(digitalRead(przycisk == LOW)){

na 

if(digitalRead(przycisk == HIGH)){

I sprawdź jak będzie działać.

[ethanak]

W obu przypadkach będziesz odczytywać stan pinu 0 🙂

@adam30010 - nie pisz bzdur.

[Belferek]

if(digitalRead(przycisk == LOW))

A co to za dziwna konstrukcja?

Powinno być:

if(digitalRead(przycisk) == LOW)

Takie literówki ciężko wyłapać.

[Polecacz 101]

Produkcja i montaż PCB - wybierz sprawdzone PCBWay!
   • Darmowe płytki dla studentów i projektów non-profit
   • Tylko 5$ za 10 prototypów PCB w 24 godziny
   • Usługa projektowania PCB na zlecenie
   • Montaż PCB od 30$ + bezpłatna dostawa i szablony
   • Darmowe narzędzie do podglądu plików Gerber
Zobacz również » Film z fabryki PCBWay

[jas123]

Trochę mi się myli te ustawianie nawiasów , szkoda że program nie uznaje takich pomyłek jako błąd.Teraz program działa poprawnie. Jeszcze raz dziękuje😀 nie wpadłbym na to .

[jas123]

Działanie programu po dodaniu paru linijek kodu😀:

 

 

[ethanak]

7 godzin temu, jas123 napisał:

szkoda że program nie uznaje takich pomyłek jako błąd.

A dlaczego miałby uznawać za błąd całkowicie prawidłową konstrukcję?

Jeśli chcesz pomnożyć dwie liczby i zamiast * napiszesz +, to też kompilator ma to za Ciebie wykryć?

Inaczej: czy w słowie "morze" jest błąd ortograficzny?

 

[Treker (Damian Szymański)]

7 godzin temu, jas123 napisał:

szkoda że program nie uznaje takich pomyłek jako błąd

Z punktu widzenia kompilatora to nie są błędy. Podczas kompilacji sprawdzane jest tylko to czy program jest poprawnie zapisany, komputer nie wie jak ma działać dany układ, więc nie jest w stanie wykryć błędów wynikających z takich "dwuznaczności". To, że jakiś kod jest zupełnie bezsensowny z punktu widzenia (nie)działającego urządzenia to nie znaczy, że jest błędnie napisany 😉

Aby wykrywać problemy tego typu należałoby pisać testy, ale to zupełnie inny, dużo bardziej rozbudowany temat i ma mało wspólnego z Arduino. Z ciekawości możesz zerknąć na ten artykuł: Czym jest Test Driven Development? Wprowadzenie, przykłady

[ethanak]

1 godzinę temu, Treker napisał:

Z punktu widzenia kompilatora to nie są błędy.

Dodam jeszcze, że współczesne kompilatory traktują jako podejrzane niektóre konstrukcje, które co prawda są prawidłowe ale bardzo często błędnie zastosowane. Przykładowo: całkiem prawidłowe

while (a=b()) { //...

zostanie uznane za podejrzane (kompilator zakłada, że chciałeś użyć operatora porównania a nie podstawienia), czy też brak break przed case w instrukcji switch. Takich ostrzeżeń oczywiście można się pozbyć informując kompilator, że właśnie o to nam chodziło  - np. ujmując podstawienie w dodatkowe nawiasy:

while ((a=b())) { // ...

czy też umieszczając odpowiedni komentarz przed case:
 

// pass through
case cośtam:

Niestety - w opisywanym przypadku kompilator nawet nie ma szans podejrzewania że to błąd, ponieważ:

• LOW i HIGH już na etapie preprocesora są zastąpione odpowiednio 0 i 1
• przycisk jest w tym przypadku stałą, czyli zastąpi go po prostu 6
• operator porównania daje w wyniku wartość int, czyli "przycisk ==HIGH" zostanie zastąpiony przez 0 (podobnie jak "przycisk == LOW", gdyż niezmiernie rzadko się zdarza aby 6 było równe zero lub jeden, nawet dla bardzo, bardzo małych szóstek).
• Wyrażenie które dostaje w efekcie kompilator, czyli "digitalRead(0)" jest całkowicie prawidłowe i nie bardzo można tu kogokolwiek podejrzewać o jakiekolwiek niecne zamiary...

Podobnie zresztą, jak wszelkie "pokazywacze błędów ortograficznych" nie podkreślą na czerwono błędu w zdaniach "może było bardzo wzburzone" i "morze pójdziemy na spacer" (do tego potrzebny by był analizator gramatyki, który i tak polegnie na zdaniach wieloznacznych, gdzie "morze" i "może" mogą wystąpić w tym samym miejscu, zmieniając tylko sens zdania).

Wniosek: trzeba uważać na to, co się pisze. Komputery mają tę brzydką cechę, że dokładnie i bezbłędnie wykonują polecenia wydane przez człowieka. Również te błędne...

 

 

 

Edytowano Kwiecień 21, 2020 przez ethanak

[adam30010]

Ooo, faktycznie nie zauważyłem że tam nawiasu brakuje, a w zasadzie to jest w innym miejscu 😄