tempo tra un lampeggio e il successivo

[danielealfa]

buona sera a tutti , vorrei chidervi qualche consiglio su un programma che ho scritto prendendo esempio
da cio che ho trovato in giro .
si tratta di misurare il tempo tra un lampeggio ed il successivo .
nel caso che il led lampeggi ad una distanza minima di 2 secondi non fa nulla
se lampeggia veloce, effettua un reset, attende 39 secondi e si riavvia.
secondo voi ce qualche miglioria da fare? e sicuramente delle correzioni
grazie e buona serata

Codice: Seleziona tutto

boolean flag = 0;
boolean total_reset = 1;
byte premuto = 0; //segnale_memoria
long tt = 0; //tempo_trascorso
long tm = 0; //tempo_memoria
unsigned long T_pertenza  = 0; // tempo avvio macchina
unsigned long o = 0;
float T_media = 0.0;
byte conta = 0;
int sensorValue = 0;
void setup() {
  pinMode(13, OUTPUT);
  pinMode(A5, INPUT);
  Serial.begin(9600);
  Serial.println (F("  xxxxx "));
}
void loop() {
  sensorValue = analogRead(A5);
  if (sensorValue >= 500  ) {
    o = millis();
    if (premuto == 0) {
      premuto = 1;
      tt = millis() - tm;
      Serial.print (F("  premuto  "));
      Serial.println (conta);
      tm = millis();
    }
  }
  if (sensorValue < 500) {
    if (premuto == 1) {
      premuto = 0;
      conta = conta + 1;
      T_media += tt;
    }
  }
  //||se total_reset == calcolo il tempo delle pressioni
  if (total_reset == 0) {
    //||se non aggiorno sensorValue resetta il sistema
    if (millis() - o > 10000) {
      Serial.println (F("reset tempo"));
      total_reset = 1;// avvio procedura reset
    }
    //conto fino a 5 e faccio la media
    if (conta >= 5 && premuto == 0) {
      conta = 0;
      Serial.print (F("tt ")); Serial.println (tt );
      Serial.print (F("T_media "));
      Serial.println (T_media / 5 );
      Serial.println(F(" controllo"));
      //dentro a conta >= 5   valuto  il tempo T_media / 2
      if (T_media / 5 < 2000 ) { //se vera
        total_reset = 1;// avvio procedura reset
      }
      else {
        Serial.println(F("  no azioni"));
      }
      T_media = 0.0;
    }
  }
  if (total_reset == 1 ) {
    if (flag == 0 ) {
      T_media = 0.0;
      conta = 0;
      T_pertenza = millis();
      Serial.println(F(" reset HIGH"));
      digitalWrite(13, HIGH);
      delay(1000);
      Serial.println(F(" reset LOW"));
      digitalWrite(13, LOW);
      flag = 1;
    }
    // tempo di avvio da reset
    if (millis() - T_pertenza > 30000) {
      Serial.println(F(" avvio"));
      flag = 0;
      total_reset = 0;
    }
  }
}


[WALTERmwp]

Ciao @danielealfa.
Chiedo, potresti essere un po' più esplicito nella descrizione ?
E magari scrivere cosa non accade rispetto a quel che invece ti aspetti ?

Saluti

[IlGuru]

Mi perplime un po' veder usare il tipo float su un AVR ad 8 bit.
Ma quando assegni tt di tipo long a T_media di tipo float senza castarlo il compilatore non si incazza?

[MarcoD]

Premetto che è molto faticoso e sgradevole esaminare un listato altrui.

Ti propongo una semplificazione marginale consistente nel modificare il sensorValue da intero a boolean e a scriverlo una sola volta.
Inoltre suggerisco di allineare le parentesi { } in modo da facilitare la comprensione.

Codice: Seleziona tutto

  sensorValue = analogRead(A5);
  if (sensorValue >= 500  ) {
    o = millis();
    if (premuto == 0) {
      premuto = 1;
      tt = millis() - tm;
      Serial.print (F("  premuto  "));
      Serial.println (conta);
      tm = millis();
    }
  }


..........modifica proposta:...............
boolean  sensorValue;
.......................
  if (analogRead(A5)>500) {sensorValue = high } else {sensorValue = low }
  if (sensorValue )
   {
     o = millis();
     if (premuto == 0)
       {
        premuto = 1;
        tt = millis() - tm;
        Serial.print (F("  premuto  "));
        Serial.println (conta);
        tm = millis();
       }
    }
...................



[WALTERmwp]

Premetto che è molto faticoso (...)

certo, dall'OP mi pare anche mal posta la questione.
Si chiede perché non si sa come fare, comprensibile, ma è auspicabile un pochino d'impegno nell'esposizione: eviterebbe di dover interpretare quel che dovrebbe essere chiaro.

Saluti

[danielealfa]

grazie delle vostre attenzioni.
no il listato funziona, era una domanda per sapere se poteva essere migliorato.

Mi perplime un po' veder usare il tipo float su un AVR ad 8 bit.
Ma quando assegni tt di tipo long a T_media di tipo float senza castarlo il compilatore non si incazza?

no non mi da errori, niente di niente, questa e una cosa che mi farebbe piacere come la faresti tu, anche per imparare qualcosa di piu',

Ti propongo una semplificazione marginale consistente nel modificare il sensorValue da intero a boolean e a scriverlo una sola volta.
Inoltre suggerisco di allineare le parentesi { } in modo da facilitare la comprensione.

come suggerito metto il tutto corretto, per il momento vi ringrazio delle vostre critiche e consigli

Codice: Seleziona tutto

boolean flag = 0;
boolean total_reset = 1;
byte premuto = 0; //segnale_memoria
long tt = 0; //tempo_trascorso
long tm = 0; //tempo_memoria
unsigned long T_pertenza  = 0; // tempo avvio macchina
unsigned long o = 0;
float T_media = 0.0;
byte conta = 0;
boolean sensorValue = 0;
void setup() {
  pinMode(13, OUTPUT);
  pinMode(A5, INPUT);
  Serial.begin(9600);
  Serial.println (F("  xxxxx "));
}
void loop() {
  if (analogRead(A5)>500)
  {
    sensorValue = HIGH;
  }
  else
  {
    sensorValue = LOW;
    }
  if (sensorValue== HIGH )
   {
     o = millis();
     if (premuto == 0)
       {
        premuto = 1;
        tt = millis() - tm;
        Serial.print (F("  premuto  "));
        Serial.println (conta);
        tm = millis();
  }
  }
 
  if (sensorValue == LOW)
  {
    if (premuto == 1)
    {
      premuto = 0;
      conta = conta + 1;
      T_media += tt;
    }
  }
  //||se total_reset == calcolo il tempo delle pressioni
  if (total_reset == 0)
  {
    //||se non aggiorno sensorValue resetta il sistema
    if (millis() - o > 10000)
    {
      Serial.println (F("reset tempo"));
      total_reset = 1;// avvio procedura reset
    }
    //conto fino a 5 e faccio la media
    if (conta >= 5 && premuto == 0)
    {
      conta = 0;
      Serial.print (F("tt ")); Serial.println (tt );
      Serial.print (F("T_media "));
      Serial.println (T_media / 5 );
      Serial.println(F(" controllo"));
      //dentro a conta >= 5   valuto  il tempo T_media / 2
      if (T_media / 5 < 2000 ) { //se vera
        total_reset = 1;// avvio procedura reset
      }
      else {
        Serial.println(F("  no azioni"));
      }
      T_media = 0.0;
    }
  }
  if (total_reset == 1 )
  {
    if (flag == 0 ) {
      T_media = 0.0;
      conta = 0;
      T_pertenza = millis();
      Serial.println(F(" reset HIGH"));
      digitalWrite(13, HIGH);
      delay(1000);
      Serial.println(F(" reset LOW"));
      digitalWrite(13, LOW);
      flag = 1;
    }
    // tempo di avvio da reset
    if (millis() - T_pertenza > 30000)
    {
      Serial.println(F(" avvio"));
      flag = 0;
      total_reset = 0;
    }
  }
}


[MarcoD]

ho fatto solo qualche abbellimento estetico funzionale al programma, sperando di non aver introdotto errori.

Devi scrivere qualche riga di commento e descrizinoe di cosa fa all'inizio del programma, e una data di inizio sviluppo, così ti rendi conto di quanto tempo ci passi sopra.

Un programma è un poco come una poesia o uno scritto impaginato, deve facilitare la lettura e la comprensione, se no, dopo qualche mese, non comprendi che cosa hai fatto.

Poi ognuno ha il suo stile di programmazione....e non si cura molto di quelli altrui...

//15/2/2021
//programma di misura del tempo.....
//

boolean flag = 0;
boolean total_reset = 1;
byte premuto = 0; //segnale_memoria
long tt = 0; //tempo_trascorso
long tm = 0; //tempo_memoria
unsigned long T_pertenza = 0; // tempo avvio macchina
unsigned long o = 0;
float T_media = 0.0;
byte conta = 0;
boolean sensorValue = 0;

void setup() {
pinMode(13, OUTPUT);
pinMode(A5, INPUT);
Serial.begin(9600);
Serial.println (F(" xxxxx "));
}

void loop() {
if (analogRead(A5)>500) { sensorValue = HIGH } else { sensorValue = LOW;}

if (sensorValue== HIGH )
{
o = millis();
if (premuto == 0)
{
premuto = 1;
tt = millis() - tm;
Serial.print (F(" premuto "));
Serial.println (conta);
tm = millis();
}
}

else // di sicuro allora è LOW if (sensorValue == LOW)
{
if (premuto == 1)
{
premuto = 0;
conta = conta + 1;
T_media += tt;
}
}

//||se total_reset == calcolo il tempo delle pressioni
if (total_reset == 0)
{
//||se non aggiorno sensorValue resetta il sistema
if (millis() - o > 10000)
{
Serial.println (F("reset tempo"));
total_reset = 1;// avvio procedura reset
}
//conto fino a 5 e faccio la media
if (conta >= 5 && premuto == 0)
{
conta = 0;
Serial.print (F("tt ")); Serial.println (tt );
Serial.print (F("T_media "));
Serial.println (T_media / 5 );
Serial.println(F(" controllo"));
//dentro a conta >= 5 valuto il tempo T_media / 2
if (T_media / 5 < 2000 )
{ //se vera
total_reset = 1;// avvio procedura reset
}
else {
Serial.println(F(" no azioni"));
}
T_media = 0.0;
}
}
if (total_reset == 1 )
{
if (flag == 0 ) {
T_media = 0.0;
conta = 0;
T_pertenza = millis();
Serial.println(F(" reset HIGH"));
digitalWrite(13, HIGH);
delay(1000);
Serial.println(F(" reset LOW"));
digitalWrite(13, LOW);
flag = 1;
}
// tempo di avvio da reset
if (millis() - T_pertenza > 30000)
{
Serial.println(F(" avvio"));
flag = 0;
total_reset = 0;
}
}
}

[lemure64]

grazie delle vostre attenzioni.
no il listato funziona, era una domanda per sapere se poteva essere migliorato.

Come detto, interpretare un sorgente non scritto da sé stessi è veramente impegnativo e vedo che stanno partecipando al thread persone che la sanno molto più lunga di me quindi taccio :) Però ho visto ancora dei float in giro; per il poco che ne so, vanno evitati a meno che non esista un'altra soluzione formulata in termini di interi. Se si ha davvero ma davvero bisogno di un float su un micro sospetto che possa non essere un problema da risolvere con un micro.

Se può esserti d'aiuto a iniziare a pensare diversamente (laddove occorra) a me è comodo inventare unità di misura e scalare le grandezze in modo che possano entrare nello spazio dei tipi interi. Per cose afferenti al seguire alba e tramonto per esempio mi sono tornati comodi i "decasecondi" (1 unità == 10 s), e così via. In altri casi mi è stato utile 1 unità == 10 ms. Forse qualcosa del genere potrà tornarti utile per i prossimi progetti.

[djnz]

per sapere se poteva essere migliorato

Déjà vu... si può usare una logica molto più semplice:

Codice: Seleziona tutto

//========================================================
byte          conta = 0;
byte          primoGiro = 1;
byte          premuto = 0;
unsigned long tm;  //tempo_memoria
//========================================================
void setup(){
    pinMode(13, OUTPUT);
    pinMode(A5, INPUT);
}
//========================================================
void loop(){
    unsigned long now = millis();
    byte sensorValue = analogRead(A5) > 500;
    byte fronte = sensorValue && !premuto;
    premuto = sensorValue;

    if (fronte){
        if (!primoGiro) conta = (now-tm < 2000) ? conta+1 : 0;
        primoGiro = 0;
        tm = now;
    }

    if (5 == conta  ||  (now-tm > 10000)){
        digitalWrite(13, HIGH);
        delay(1000);
        digitalWrite(13, LOW);
        delay(30000);
        conta = 0;
        primoGiro = 1;
        tm = now;
    }
}
//========================================================

[danielealfa]

grazie delle info

Codice: Seleziona tutto

if (!primoGiro) conta = (now-tm < 2000) ? conta+1 : 0;

non riesco a capirla, nel uso del ? e del 0 finale.

unsigned long now = millis();
unsigned long now = millis();
byte sensorValue = analogRead(A5) > 500;
byte fronte = sensorValue && !premuto;
premuto = sensorValue;

cosa cambia se vengono inizializzate prima del setup o nel loop a questo modo?

now = millis();
sensorValue = analogRead(A5) > 500;
fronte = sensorValue && !premuto;
premuto = sensorValue;

grazie ancora delle spiegazioni