Problema di programmazione.

buonasera a tutti.
non so se ho sbagliato sezione del forum, essendo il primo topic che apro.
il mio problema è questo,
ho applicato ad ARDUINO UNO un sensore IR Sharp2D120X, con questo sensore vorrei fare in modo che il mio servo motore si muova facendo chiudere una specie di gate al passaggio avvenuto di una persona (come un cancello automatico insomma).
lo schema di programmazione sarebbe questo:
apro il gate tramite pulsante,
il sensore rileva il passaggio della persona/macchina
chiudo il gate.

posto qui il mio programma, sperando in un consiglio su come poter risolvere il problema:

//APERTURA A PULSANTE E CHIUSURA CON IR

#include <Servo.h>
#include <DistanceGP2Y0A21YK.h>

//variabili IR
DistanceGP2Y0A21YK IR;
int distFissa;
int distVariabile;
int pinIR=A5;

//variabili servo
Servo gate;
int pinGate=3;
int pinPul=A0;
int pos=0;

void setup(){
  pinMode(pinPul, INPUT);//sceglie il pulsante come input nell'uscita A0
  IR.begin(pinIR);//attacca l'IR al pin di uscita A5
}

void detachGate(){
    gate.detach();
}

void setGatePos(){
    gate.attach(pinGate);
    gate.write(pos);
    delay(1000);
    detachGate();
}

void openGate(){
   gate.attach(pinGate);
    for(pos=0;pos<=90;pos++){
     gate.write(pos);
     delay(15);
    }
   detachGate();
}
void closeGate(){
  gate.attach(pinGate);
   for(pos=90;pos>=0;pos--){
    gate.write(pos);
    delay(15);
   }
  detachGate();
}
 
void loop(){
      setGatePos();
      distFissa=IR.getDistanceCentimeter();

      if (analogRead(pinPul)>900){
        distVariabile=IR.getDistanceCentimeter();
	openGate();
	delay(1000);
	while(distFissa>distVariabile){
	 delay(1000);
	 distVariabile=IR.getDistanceCentimeter();
	if(distVariabile>=distFissa){
	closeGate();
	}
	}
	} //valore max del pulsante = 1023 in attivazione
}

grazie infinite dell’attenzione.

Descrivi per favore il problema che hai incontrato.

perplessitá: Perché leggi il pulsante in modo analogico e non digitale? Hai messo una resistenza Pullup o pulldown?

Ciao Uwe

problema: non riesco a capire come far abbassare il gate dopo che il soggetto vi passa attraverso. il mio programma non funziona correttamente, dato che va giù a random quando vuole lui.

la resistenza è in pullup e il pulsante l'ho sempre usato così, è sbagliato?

Sono anche io neofita di Arduino ma sono un programmatore professionista.

scrivi: void loop(){ setGatePos(); distFissa=IR.getDistanceCentimeter();

Ma la funzione loop() viene eseguita di continuo da Arduino. Quei comandi mi sembrano delle funzioni da richiamare solo una prima volta (inizializzazioni), quindi da mettere nella void setup()

Poi, cosa vuol dire non riesci a chiudere il gate ? Non ti funziona il servo? Hai provato a comandare il servo senza tanti casini? Ovvero sai comandare il servo con uno sketch semplice semplice senza sensori da cui prendere input? Come alimenti il servo? Alimentando da cavo USB l'Arduino, potresti avere poca corrente

XD

Un consiglio: abituati ad usare millis() al posto di delay, ne vedo diversi... In questo modo eviti di bloccare tutto il micro, quindi letture del sensore e movimento del servo compresi

Altro consiglio, il codice dovrebbe essere indentato bene per essere leggibile al primo sguardo. Non sò se il tuo originale era indentato bene, ma quello postato è poco leggibile:

void loop()
{  setGatePos();
   distFissa=IR.getDistanceCentimeter();
   if (analogRead(pinPul)>900)
   { distVariabile=IR.getDistanceCentimeter();
     openGate();
     delay(1000);
     while(distFissa>distVariabile)
     { delay(1000);
       distVariabile=IR.getDistanceCentimeter();
       if(distVariabile>=distFissa)
       { closeGate();
       } 
     } // while
   } //valore max del pulsante = 1023 in attivazione
}

Il problema mi sembra anche di logica di programmazione.
Credo tu debba considerare una programmazione utilizzando una variabile di stato.
Ovvero una variabile dove 0=fase iniziale, 1=ha premuto il tasto, 2=altro momento.

La loop() viene eseguita di continuo. Perciò internamente alla loop secondo me devi usare uno switch() sulla variabile di stato per sapere in quale “momento” sei.

void loop()
{ ...
  switch(iState)  
  {  case 0: ...
                if(premuto pulsante) iState=1;
                break;
     case 1: //premuto il pulsante
                // allora qui verifico distanza
               break;
     case 2: 
          ...
  }
}

Guarda questo libro (purtroppo in inglese):

il capitolo 3 parla delle macchine a stato (il tipo italiano ha creato ottimi robot con i lego nxt, programmati in c)
su goglebooks sono leggibili i primi 3 capitoli. Il capitolo 3 è sulle macchine di stato.
Spero possa esserti di aiuto.

Se hai messo una resistenza pullup e il pulsante tra entrata e massa la condizione del IF nel codice sottostante é vera quando il pulsnate non é premuto e non quando qualcuno lo preme per aprire la porta. Percui la porta resta sempre aperta. Ciao Uwe

void loop(){
      setGatePos();
      distFissa=IR.getDistanceCentimeter();

      if (analogRead(pinPul)>900){
        distVariabile=IR.getDistanceCentimeter();
    openGate();
    delay(1000);
...

vi ringrazio infinitamente per i consigli, siccome stasera sono di fretta domani e lunedì leggo tutto meglio con calma, faccio tutte le prove e vi saprò dire!!.. non sono esperta per niente, sono una studentessa e sto facendo questo progetto per l'esame di maturità.

comunque in risposta a:

nid69ita: Sono anche io neofita di Arduino ma sono un programmatore professionista.

scrivi: void loop(){ setGatePos(); distFissa=IR.getDistanceCentimeter();

Ma la funzione loop() viene eseguita di continuo da Arduino. Quei comandi mi sembrano delle funzioni da richiamare solo una prima volta (inizializzazioni), quindi da mettere nella void setup()

Poi, cosa vuol dire non riesci a chiudere il gate ? Non ti funziona il servo? Hai provato a comandare il servo senza tanti casini? Ovvero sai comandare il servo con uno sketch semplice semplice senza sensori da cui prendere input? Come alimenti il servo? Alimentando da cavo USB l'Arduino, potresti avere poca corrente

XD

il gate si chiude, ma quando vuole lui. ho già programmato il servo più di una volta senza pasticci vari in mezzo di sensori e so per certo che funziona. l'arduino è alimentato esternamente, e faccio girare il programma dopo averlo caricato e staccato il cavo usb da pc.

scollega il servo e collega l' arduino al PC tramite USB. Poi metti in alcuni posti un serial.print() per controllare che valori hanno le variabili /lettura dei sensori. Puoi leggerli nel teminale del IDE.

Ciao Uwe

Come suggerito giustamente da uwefed mi sembra importante mettere un pò di Serial.println per debuggare il progamma.
Mettili dopo ogni istruzione (soprattutto nelle istruzioni condizionali) stampando anche il valore delle variabili.

nel setup()

Serial.begin(9600);
Serial.println("Fase 1");

poi nel loop()

  Serial.println("Fase 2");
  setGatePos();
  Serial.println("Fase 3");
  distFissa=IR.getDistanceCentimeter();
  Serial.println("Fase 4 distFissa="); Serial.print(distFissa);
  newvar=analogRead(pinPul);
  Serial.println("Fase 5 newvar="); Serial.print(newvar);
  if (newvar>900)
  { Serial.println("Fase 5 if vero");
    distVariabile=IR.getDistanceCentimeter();
    Serial.println("Fase 6 distVariabile="); Serial.print(distVariabile);
    openGate();
     ...etc...

Ciao, perchè stacchi il servo dopo averlo mosso?

Pelletta: Un consiglio: abituati ad usare millis() al posto di delay, ne vedo diversi... In questo modo eviti di bloccare tutto il micro, quindi letture del sensore e movimento del servo compresi

ma millis() non ritorna un valore finito di tempo da quando il programma è partito? non saprei come usarlo al posto di delay() non mi sembra la stessa cosa.

PaoloP: Ciao, perchè stacchi il servo dopo averlo mosso?

ciao, lo stacco per essere sicura che finita l'azione (se magari ci sono parti di programma che funzionano male) il servo non continui a ruotare magari accumulando tensione e rompendosi. Dici che potrebbe essere sbagliato?

Akira_Garashi:

Pelletta: Un consiglio: abituati ad usare millis() al posto di delay, ne vedo diversi... In questo modo eviti di bloccare tutto il micro, quindi letture del sensore e movimento del servo compresi

ma millis() non ritorna un valore finito di tempo da quando il programma è partito? non saprei come usarlo al posto di delay() non mi sembra la stessa cosa.

delay() ti ferma il programma. Il programa non fa nient'altro finché non è passato il tempo prestabilito. Con millis puoi fare la stessa cosa con un semplice if. Guardati lo sketch di esempio allegato all'IDE "Blink Without Delay".

ho tentato le vostre soluzioni e non mi funzionava nulla, così mi sono chiesta se il problema non fosse all'origine.. ho provato ad alimentare il sensore da solo e a misurarne l'uscita con un voltimetro, non si muove una mazza! Quindi ora mi sorge il dubbio, era rotto fin dall'inizio?.. ammetto che comunque sia il programma avrà anche i suoi problemi di funzionamento, ma se il sensore non va grazie tante! che si possa essere rotto durante la saldatura? con un'elevata temperatura? ho letto nel datasheet che può raggiungere i 60°C senza subire danni, e non penso che un saldatore possa far raggiungere tale temperatura se tenuto per pochi secondi sul pin. devo comprarne uno nuovo! http://www.sparkfun.com/datasheets/Sensors/Infrared/GP2D120XJ00F_SS.pdf

Akira_Garashi: non penso che un saldatore possa far raggiungere tale temperatura se tenuto per pochi secondi sul pin.

Ho qualche dubbio in proposito. :sweat_smile:

Il sensore funziona a riflessione e ha una distanza massima di 40cm circa. In base alla superficie riflettente posta a distanza di 30 cm dal sensore dovresti misurare tra i 250 e i 550 mV.

Per il tuo progetto non credo sia il sensore più indicato, sarebbe meglio una barriera a infrarossi, altrimenti devi vestire le persone con strisce rifrangenti. :grin:

PaoloP:

Akira_Garashi:
non penso che un saldatore possa far raggiungere tale temperatura se tenuto per pochi secondi sul pin.

Ho qualche dubbio in proposito. :sweat_smile:

Il sensore funziona a riflessione e ha una distanza massima di 40cm circa.
In base alla superficie riflettente posta a distanza di 30 cm dal sensore dovresti misurare tra i 250 e i 550 mV.

Per il tuo progetto non credo sia il sensore più indicato, sarebbe meglio una barriera a infrarossi, altrimenti devi vestire le persone con strisce rifrangenti. :grin:

^^" dafaq…

okok allora, ho trovato un altro sensore uguale nei meandri della mia casa, e ho fatto più attenzione al momento “saldatura” (devo ancora diventarne pratica… =( )
collegato a basetta e tutto e provato il programma modificato.
FUNZIONA!
e non serve nulla di rifrangente! ^^ ho provato con una mano, una persona, un modellino di macchina (mustang '74 (dettagli a parte)), un foglio di carta, un bicchiere e una macchinetta fotografica…insomma di prove ne ho fatte e funziona
XD
questo è il listato che mi è risultato infine grazie anche ai Serial.println() che ringrazio infinitamente di aver conisgliato, dato che non mi era passato per la testa :drooling_face:

//APERTURA A PULSANTE E CHIUSURA CON IR

#include <Servo.h>
//#include <DistanceGP2Y0A21YK.h>

//variabili per sensore IR
int pinIR=A1;
int dafaq;

//variabili servo
Servo gate;
int pinGate=3;
int pos=0;

//variabili pulsante
int pinPul=A0;

//settaggi dei pin e de componenti
void setup(){
  pinMode(pinPul, INPUT);//sceglie il pulsante come input nell'uscita A0
  pinMode(pinIR,INPUT);//attacca l'IR al pin A5 in input
}
//disattiva il servo
void detachGate(){
    gate.detach();
}
//setta la posizione inizaile del gate a 90°
void setGatePos(){
    gate.attach(pinGate);
    gate.write(pos);
    delay(1000);
    detachGate();
}
//apre il gate di 90° 
void openGate(){
   gate.attach(pinGate);
    for(pos=0;pos<=90;pos++){
     gate.write(pos);
     delay(15);
    }
   detachGate();
}
//chiude il gate facendolo tornare alla posizione di partenza
void closeGate(){
  gate.attach(pinGate);
   for(pos=90;pos>=0;pos--){
    gate.write(pos);
    delay(15);
   }
  detachGate();
}

void loop(){
      setGatePos();

      if (analogRead(pinPul)>900){
	openGate();
	delay(1000);
        dafaq=analogRead(pinIR);
	while(dafaq>3){
	  delay(500);
          dafaq=analogRead(pinIR);
	    if(dafaq<=3){
               delay(3000);
               closeGate();
	    }
	}
	} //valore max del pulsante = 1023 in attivazione
}

GRAZIE MILLE A TUTTI!!! ^^ AVETE CONTRIBUITO AD AVANZARE LA MIA MATURITA’… :blush: :wink:

MI piace anche come hai indentato il codice. Ed hai messo un pò di commenti. Brava. Il codice scritto così è ben leggibile.

nid69ita: MI piace anche come hai indentato il codice. Ed hai messo un pò di commenti. Brava. Il codice scritto così è ben leggibile.

ti ringrazio!! ^^

si, in effetti al primo postaggio l'ho scopiazzato male (a pezzi per cercare di metterlo apposto) e me lo hanno corretto i moderatori perchè avevo dimenticato il [code/] e l'indentazione è andata un po' a cercare sassi nei campi.. :roll_eyes: