Allarme con lampeggio led e suono sirena

International Italiano Software
6

Per la formattazione del codice, usare CTRL+T è utile per trovare le parentesi graffe chiuse male.
Ad esempio nel tuo loop() DI SICURO hai messo la parte di codice da eseguire quando val è low NON nel else perchè hai chiuso male le graffe.
Io ti consiglio di mettere le graffe sempre su riga da sola NON su righe con altre cose, esempio:

void loop() 
{
  val=digitalRead(BUTTON);
  if (val==HIGH)
  { 
    //Serial.println("Allarme ON INTERR Aperto");
    digitalWrite(LED,HIGH);
    makeCall();
    Serial.println("Allarme ON INTERR Aperto");
    delay(2000);
  }
  else
  {
    digitalWrite(LED,LOW);
    Serial.println("Allarme OFF INTERR Chiuso");
    delay(2000);
  }
}

Occupi più spazio, ma incolonnando le graffe è più facile vedere corrispondenza aperte/chiuse. Inoltre IDE quando ti posizioni su graffa, ti evidenzia la corrispondente.
Nel pezzo di codice ti ho corretto l'errore delle graffe chiuse male. Confrontalo col tuo.

Ma ci sono altri errori. Esempio non ha senso quel BUTTON == LOW; ne il pezzo ("da vivi");
E poi c'e' errore di logica sull'uso del millis(). (e millis() ritorna unsigned long non un int)

1 Like