Salve a tutti
ho realizzato un contapezzi dove ad ogni pressione di un pulsante mi va ad aumentare di una unità un contatore e il valore viene visualizzato su un display 7 segment.
Il tutto funziona senonchè ho aggiunto un buzzer che emette un tono ad ogni pressione del pulsante contatore, ed oltre ad emettere il bip c'è sempre un fischio continuo che non riesco a togliere.
Qualcuno può aiutarmi a capire perchè? Grazie.
[code]
//State change detection (edge detection)
//We always have to include the library
#include "LedControl.h"
/*
Now we need a LedControl to work with.
pin 12 is connected to the DataIn
pin 11 is connected to the CLK
pin 10 is connected to LOAD
We have only a single MAX72XX.
*/
LedControl lc=LedControl(12,11,10,1);
/* The circuit:
- buttonpin attached to pin 2 from +5V
- buttonreset attached to pin 4 from +5V
- 10 kilohm resistor attached to pin 2&4 from ground
*/
// this constant won't change:
const int buttonPin = 2;// the pin that the pushbutton is attached to
const int buttonReset = 4;// the pin that the buttonreset is attached to
const int buzzer = 8;
// Variables will change:
int contatore = 0; // counter for the number of button presses
int buttonState = 0; // current state of the button
int lastButtonState = 0; // previous state of the button
int unita = 0;
int decine = 0;
int centinaia = 0;
int tmp=0;
void setup(){
/*
The MAX72XX is in power-saving mode on startup,
we have to do a wakeup call
*/
lc.shutdown(0,false);
/* Set the brightness to a medium values */
lc.setIntensity(0,8);
/* and clear the display */
lc.clearDisplay(0);
// initialize the button pin as a input:
pinMode(buttonPin, INPUT);
pinMode(buttonReset, INPUT);
}
void loop() {
// read the pushbutton input pin:
buttonState = digitalRead(buttonPin);
// compare the buttonState to its previous state
if (buttonState != lastButtonState) {
// if the state has changed, increment the counter
if (buttonState == HIGH) {
// if the current state is HIGH then the button went from off to on:
contatore++;
tone (8,800,20);
tmp=contatore;
unita=tmp%10;
tmp=tmp/10;
decine=tmp%10;
centinaia=tmp/10;
lc.setDigit(0,2,(byte)centinaia,false);
lc.setDigit(0,1,(byte)decine,false);
lc.setDigit(0,0,(byte)unita,false);
}
if (buttonState != lastButtonState) {
if (buttonState == LOW) {
noTone (8);
}
}
// Delay a little bit to avoid bouncing
delay(50);
}
// save the current state as the last state, for next time through the loop
lastButtonState = buttonState;
// if the buttonreset is pressed reset the buttonpushcounter
if (digitalRead(buttonReset) == HIGH){
(contatore = 0);
lc.clearDisplay(0);
}
}
Non vedo cose particolari, se non alcune piccole correzioni.
Intanto il noTone(), ossia perché usi noTone() se il comando tone() lo imposti per 20 millisecondi di durata? Mi pare superfluo. Anche la if() dove è contenuto è comunque inutile visto che sei già dentro ad una "if(buttonState != lastButtonState)".
E lastButtonState, sempre per ottimizzare il codice, meglio metterlo dentro la if(), inutile settarla se non è cambiata.
Il pin del buzzer non lo hai definito in setup().
Poi il pin del buzzer lo definisci all'inizio come "const int buzzer = 8;" ma poi non lo usi mai.
Per definire i pin, invece delle variabili (seppur "const") meglio usare i #define.
Insomma, intanto il codice lo cambierei così, vedi tu di provarlo e facci sapere:
//State change detection (edge detection)
//We always have to include the library
#include "LedControl.h"
/*
Now we need a LedControl to work with.
pin 12 is connected to the DataIn
pin 11 is connected to the CLK
pin 10 is connected to LOAD
We have only a single MAX72XX.
*/
LedControl lc=LedControl(12,11,10,1);
/* The circuit:
- PIN_BUTTON attached to pin 2 from +5V
- PIN_RESET attached to pin 4 from +5V
- 10 kilohm resistor attached to pin 2&4 from ground
*/
// this constant won't change:
#define PIN_BUTTON 2 // the pin that the pushbutton is attached to
#define PIN_RESET 4 // the pin that the PIN_RESET is attached to
#define PIN_BUZZER 8 // buzzer pin
// Variables will change:
int contatore = 0; // counter for the number of button presses
int buttonState = 0; // current state of the button
int lastButtonState = 0; // previous state of the button
int unita = 0;
int decine = 0;
int centinaia = 0;
int tmp=0;
void setup(){
/*
The MAX72XX is in power-saving mode on startup,
we have to do a wakeup call
*/
lc.shutdown(0,false);
/* Set the brightness to a medium values */
lc.setIntensity(0,8);
/* and clear the display */
lc.clearDisplay(0);
// initialize the button pin as a input:
pinMode(PIN_BUTTON, INPUT);
pinMode(PIN_RESET, INPUT);
pinMode(PIN_BUZZER, OUTPUT);
}
void loop() {
// read the pushbutton input pin:
buttonState = digitalRead(PIN_BUTTON);
// compare the buttonState to its previous state
if (buttonState != lastButtonState) {
// if the state has changed, increment the counter
if (buttonState == HIGH) {
// if the current state is HIGH then the button went from off to on:
contatore++;
tone (PIN_BUZZER,800,20);
tmp=contatore;
unita=tmp%10;
tmp=tmp/10;
decine=tmp%10;
centinaia=tmp/10;
lc.setDigit(0,2,(byte)centinaia,false);
lc.setDigit(0,1,(byte)decine,false);
lc.setDigit(0,0,(byte)unita,false);
}
// Delay a little bit to avoid bouncing
delay(50);
// save the current state as the last state, for next time through the loop
lastButtonState = buttonState;
}
// if the RESET is pressed reset the buttonpushcounter
if (digitalRead(PIN_RESET) == HIGH){
(contatore = 0);
lc.clearDisplay(0);
}
}
Ciao doc, il noTone () l' avevo usato per fare anche quella prova (non sapendo più che pesci pigliare!)...
comunque ho provato il tuo codice ma il fischio permane, il beep lo fa, ma il fischio non se ne va!
Ho anche provato a inteporre una resistenza da 10 ohm tra il contatto del buzzer e il pin 8 ma non cambia nulla.
ok allora fai una prova, collega il negativo al gnd di arduino e il positivo al pin 3,3V di arduino.... meglio con la resistenza da 10-100 Ohm in mezzo, e vedi se con l'alimentazione diretta in CC suona o se ne sta muto... poi ne parliamo
Patrick_M:
ok allora fai una prova, collega il negativo al gnd di arduino e il positivo al pin 3,3V di arduino.... meglio con la resistenza da 10-100 Ohm in mezzo, e vedi se con l'alimentazione diretta in CC suona o se ne sta muto... poi ne parliamo
#include "LedControl.h"
LedControl lc=LedControl(12,11,10,1);
#define PIN_BUTTON 2 // the pin that the pushbutton is attached to
#define PIN_RESET 4 // the pin that the PIN_RESET is attached to
#define PIN_BUZZER 8 // buzzer pin
int contatore = 0; // counter for the number of button presses
int buttonState = 0; // current state of the button
int lastButtonState = 0; // previous state of the button
int unita = 0;
int decine = 0;
int centinaia = 0;
int tmp=0;
void setup()
{
lc.shutdown(0,false);
lc.setIntensity(0,8);
lc.clearDisplay(0);
pinMode(PIN_BUTTON, INPUT);
pinMode(PIN_RESET, INPUT);
pinMode(PIN_BUZZER, OUTPUT);
tone(PIN_BUZZER,1000,1000);
}
Comunque pinMode(PIN_BUZZER, OUTPUT); non è necessario per usare tone().
Flavi71:
Ok farò anche questa prova...certo che usare una libreria per far fare un beep a un piezo!!
Una libreria???...
La resistenza in serie serve solo per non sovraccaricare le uscite usando altoparlanti dinamici da 4...32 ohm. Non serve per i cicalini piezoelettrici. Tanto meno può fare qualcosa per risolvere il problema in oggetto.
Flavi71:
No Patrick_M...è il piezo dello starter Kit.
Così per qualche altro test, prova intanto a cambiare il pin del buzzer da 8 a 9 (ovviamente cambiando anche il valore nella #define).
Però un dubbio: nel reference della tone() leggo: Use of the tone() function will interfere with PWM output on pins 3 and 11
Ed io vedo che la LedControl usa il Pin 11 (CLK), non avendo mai usato quella libreria non so se all'interno usi qualche interrupt, quindi magari il pin del buzzer "risente" del clock, non so dirti.
Prova a fare uno sketch dove gestisci con un pulsante SOLO il buzzer, senza led e vedi se fa la stessa cosa.
Se lo fa ugualmente, non so darti risposte. Se non lo fa, è un qualche conflitto tra tone() e LedControl (in tal caso prova a cambiare il pin da 11 ad un altro).
non è che hai usato un buzzer attivo al posto di uno passivo.... 
no, io intendevo solo di usare una resistenza da 1K