ElectroYou

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; } } }

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

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?

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.

O_/

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(); } } ...................

MarcoD ha scritto: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

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

IlGuru ha scritto: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',

MarcoD ha scritto: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; } } }

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;
}
}
}

danielealfa ha scritto: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.

danielealfa ha scritto: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; } } //========================================================

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

ElectroYou

tempo tra un lampeggio e il successivo

tempo tra un lampeggio e il successivo

Re: tempo tra un lampeggio e il successivo

Re: tempo tra un lampeggio e il successivo

Re: tempo tra un lampeggio e il successivo

Re: tempo tra un lampeggio e il successivo

Re: tempo tra un lampeggio e il successivo

Re: tempo tra un lampeggio e il successivo

Re: tempo tra un lampeggio e il successivo

Re: tempo tra un lampeggio e il successivo

Re: tempo tra un lampeggio e il successivo