Arduino Polska Forum

(21-10-2023, 10:58)kopcor07 napisał(a): [ -> ]Witam,

chcę sterować przekaźnikami za pomocą pilota 433mhz. Projekt oparty jest na Arduino nano, rxb6 i przekaźniku. Chcę stworzyć takie sterowanie, że po wciśnięciu jednego przycisku załączam i wyłączam jeden przekaźnik. Oto kod:

Kod:

      int value = mySwitch.getReceivedValue();
      if ( mySwitch.getReceivedValue() == 1653416)

 
Po wgraniu powyższego kodu zrobiłem z przekaźnika ,, dzwonek ". Nie wiem jak wyjść z warunku if po wciśnięciu przycisku. Proszę o jakąś pomoc lub sugestię co robię źle.

(21-10-2023, 14:56)oscarX napisał(a): [ -> ]

(21-10-2023, 10:58)kopcor07 napisał(a): [ -> ]Witam,

chcę sterować przekaźnikami za pomocą pilota 433mhz. Projekt oparty jest na Arduino nano, rxb6 i przekaźniku. Chcę stworzyć takie sterowanie, że po wciśnięciu jednego przycisku załączam i wyłączam jeden przekaźnik. Oto kod:

Kod:

      int value = mySwitch.getReceivedValue();
      if ( mySwitch.getReceivedValue() == 1653416)

 
Po wgraniu powyższego kodu zrobiłem z przekaźnika ,, dzwonek ". Nie wiem jak wyjść z warunku if po wciśnięciu przycisku. Proszę o jakąś pomoc lub sugestię co robię źle.

W if-ie sprawdzaj zmienną value, nie wołaj drugi raz getReceivedValue(). Nie dostajesz ostrzeżenia od kompilatora, że wartość podstawiona do value nie jest nigdzie użyta?

---

Spawdzam value i zgłasza błąd:

Kod:

#include <RCSwitch.h>

RCSwitch mySwitch = RCSwitch();
             
boolean stan = LOW;

void setup()
{
Serial.begin(9600);
  pinMode(3, OUTPUT);
  digitalWrite(3,LOW);

  mySwitch.enableReceive(0);
}

void loop()
{

  if (mySwitch.available())
  {
     
     // int value = mySwitch.getReceivedValue();
      if ( int value == 1653416)
      {
      stan = !stan;
      Serial.println(stan);
      digitalWrite(3, stan);
      delay(100);
      } 

       
  }

}

 

(21-10-2023, 14:03)kaczakat napisał(a): [ -> ]Pokazujesz połowę programu, a problem może być w tej drugiej, gdzie odczytujesz stan przycisku. W obu częściach kodu trzeba nauczyć się mierzyć czas, na początek migać ledem z użyciem millis, eliminować drgania styków.
W nadajniku eliminujesz drgania styków i decydujesz czy jest sens wysłać kolejny komunikat, w odbiorniku tak samo eliminujesz komunikaty, które z dużą pewnością są skutkiem tego samego naciśnięcia.
W pokazanej części od biedy jest OK, jeśli  nadajnik może nadawać dwa komunikaty, zmieniać je w kolejnych wciśnięciach.
Jak masz gotowy pilot komercyjny to wystarczy ignorować komunikaty w taki sam sposób jak eliminuje się drgania styków.  Czyli zmieniłeś stan, zapisujesz czas, przez kolejne 200ms odbierasz komunikaty i je zerujesz - nic z nimi nie robisz.
Może nawet wystarczy tak jak w przykładzie blinkwithoutdelay wywołać kod raz na 200ms, bez tego delay.

---

To jest cały kod. Pilot pod przyciskiem A geneuje tylko jeden kod : 1653416

(21-10-2023, 15:56)kopcor07 napisał(a): [ -> ]

(21-10-2023, 14:56)oscarX napisał(a): [ -> ]W if-ie sprawdzaj zmienną value, nie wołaj drugi raz getReceivedValue(). Nie dostajesz ostrzeżenia od kompilatora, że wartość podstawiona do value nie jest nigdzie użyta?

---

Spawdzam value i zgłasza błąd:

Kod:

void loop()
{

  if (mySwitch.available())
  {
     
     // int value = mySwitch.getReceivedValue();
      if ( int value == 1653416)
      {
      stan = !stan;
      Serial.println(stan);
      digitalWrite(3, stan);
      delay(100);
      } 

      
  }

}

 

(21-10-2023, 20:28)oscarX napisał(a): [ -> ]

(21-10-2023, 15:56)kopcor07 napisał(a): [ -> ]

(21-10-2023, 14:56)oscarX napisał(a): [ -> ]W if-ie sprawdzaj zmienną value, nie wołaj drugi raz getReceivedValue(). Nie dostajesz ostrzeżenia od kompilatora, że wartość podstawiona do value nie jest nigdzie użyta?

---

Spawdzam value i zgłasza błąd:

Kod:

void loop()
{

  if (mySwitch.available())
  {
     
     // int value = mySwitch.getReceivedValue();
      if ( int value == 1653416)
      {
      stan = !stan;
      Serial.println(stan);
      digitalWrite(3, stan);
      delay(100);
      } 

      
  }

}

 

Sorki, ale napisalem nieprawdę, zakładając, że jest logika jak w read - raz odczytana wartość przepada. Ale nie, sprawdziłem w źródłach i kod jest następujący:

Kod:

bool RCSwitch::available() {
  return RCSwitch::nReceivedValue != 0;
}

void RCSwitch::resetAvailable() {
  RCSwitch::nReceivedValue = 0;
}

unsigned long RCSwitch::getReceivedValue() {
  return RCSwitch::nReceivedValue;
}

Wnioski: - wołanie available() jest bez sensu - można od razu sprawdzać wynik getReceivedValue(), jak wynik jest != 0 to odczytano wartość. Ale potem należy wyzerować obiekt, by jedna wartość nie była odczytywana wielokrotnie - bo jej odczyt "nie jest niszczący". Ponieważ sprawdzasz wynik na jedną konkretną wartość to nie ma sensu wcześniej sprawdzać na zero.  W sumie to pominąłeś wywołanie resetAvailable().

Sprawdź następujący fragment kodu:

Kod:

void loop()
{
    if ( mySwitch.getReceivedValue() == 1653416)
    {
        mySwitch.resetAvailable();
        stan = !stan;
        Serial.println(stan);
       digitalWrite(3, stan);
       delay(100);
    }
} 

(22-10-2023, 01:37)oscarX napisał(a): [ -> ]To nie jest taki przycisk bezpośrednio na płytce. To bardziej podobne jest do klawiatury komputera - ona wysyła do komputera kod naciśniecia klawisza i inny kod puszczenia klawisza. Dodatkowo trzymany klawisz wysyła co jakiś czas kod. Tutaj jest łączność radiowa, gdzie trzeba się dodatkowo liczyć z zakłóceniami. Nie znam dokładnie tematu, nie bawiłem się takim sterowaniem, przypuszczam, że działa to podobnie do pilota do TV - gdzie też mogą być zakłócenia, a wręcz można pilotem nie trafić w odbiornik.
Napisz prosty kod, który po prostu wypisze ci wszystkie kolejno odbierane z pilota kody i trzeba zobaczyć, czy są jakieś kody powtarzane przy trzymaniu, czy coś idzie przy puszczaniu itp. Ale ciągle istnieje możliwość, że nie da się rozróżnić przytrzymania przycisku od kolejnego naciśniecia.
No i moim zdaniem ta biblioteka jest dość słaba - trochę dziwny sposób odbioru poszczególnych kodów z pilota - brak tzw handshakingu - może gubić kody jeśli się jakiś czas jej nie odczytuje.