Go Down

Topic: Project 07 Keyboard Instrument (Read 42773 times) previous topic - next topic

nass

Now to work out why the circuit doesn't work properly... tomorrow night

crazzzik

I tend to agree. Most of the projects I've done had some sort of mistakes in them. It added to the challenge, don't get me wrong. However this project wasn't as fun simply due to vaguely working breadboard. It took me 4 attempts to get all four buttons to work. I made a mistake with one, but the other two that weren't working simply needed a different spot on the board. I had to move last button over to the other side of the board in order to get it to work. Not ideal. Discouraging at times.

harmhero

There is no mistake in this code. The first 2 lines:

int buttons[6];
int buttons[0] = 2;

are EXAMPLE lines, to show how it works. You can identify them because they have no number. The code always starts at
1.
2.

etc.
anything before that is 'code' to explain things.

B10N1C

Hello, I am a newcomer to arduino and I too have been having a problem with this project. The first key (notes[0]) is the only one not working and this is after I made the change to the code suggested by mikeucfl.

Code: [Select]
void setup() {
  buttons[0] = 2;
  Serial.begin(9600);
}


Furthermore when I tried changing the ranges for the first two keys to see if I could get a wider range for the first key to play (1000-1023 for first key and 990-1000 for second), the second key instead played the note the first key was supposed to play and the first key was still silent (I got this idea from the fact that since the last key was near the highest value and had a small range of values to read the actual note would have a slight stutter when played, so I tried tweaking the range values). My breadboard is the same as the one in the manual. Does anyone have any suggestions for me? Thanks.

giro

Had same problems w/ '0' being returned; turned out I had (at least) 2 bad buttons in my kit. IMHO, they should have put a little less emphasis on making the book look pretty (got to love the 10 point font for readability too!), and more(?) on better components at this price...

Either way, still enjoying it immensely :)

chrismacp

Just wanted to report that I had similar issues on this project. The first issue I've had really since starting with the book.

It was kind of weird though and after getting it to work I can't really explain what was wrong.

Looking at the diagram in the book it was the top push button that was not activating the piezo and reading zero in the serial monitor. Removing the push button and replacing with a normal wire instead made the circuit work as intended, giving me 1023 in the serial monitor, so the breadboard seemed fine.

I then got my multimeter and set it to resistance and measured across the push button noticing it change to zero resistance when I depressed the button. This seemed to suggest the button was working.

Then, not sure what was going on, I read this forum and decided to move the button one row down on the breadboard. It worked. I then swapped the button over for one of the other ones and they both worked.

So not sure what the problem is to be honest. Anyway, thanks for the tip, hope it doesn't catch me out again.

brianearwood

My problem with the buttons not working is that the pins don't reach all the way down into the breadboard.

I was able to fix by bending a set of pins outward and inserting the other deeper into the breadboard. After that it worked.

Andrwsk23

I think that may be a typo in the book (I see it in mine as well). AWOL's suggestion would fix it, or you could move 'int buttons[0] = 2' into the setup method and drop the int declaration.

e.g.:
Code: [Select]

void setup() {
  buttons[0] = 2;
  Serial.begin(9600);
}

THANKS!!

jrobarts

Those two lines aren't actually for the code. They are only an explanation of how arrays are written.
The actual code starts at "int notes []= ..."

You can see that on the right side line 1 only starts there

Gorduino283

The buttons part is NOT part of the code.  It was just an example.

jedi5200

my last 2 buttons don't do anything. help?

Pinnock

SOLVED IT

SORRY FOR MY ENGLISH, I AM FROM COSTA RICA, ONLY SPEAK SPANISH
CHANGE THE TWO FIRST LINES FOR THIS.. WORK IT


int buttons[6] = {2,0,0,0,0,0};
/*int buttons[0] = 2;
*/

int notes[] = {262,294,330,349};

void setup() {
  Serial.begin(9600);
  }

  void loop() {
    int keyVal = analogRead(A0);
    Serial.println(keyVal);

    if(keyVal == 1023) {
      tone(8, notes[0]);
    }
    else if(keyVal >= 990 && keyVal <= 1010){
      tone(8, notes[1]);
    }
    else if(keyVal >= 505 && keyVal <= 515){
      tone(8, notes[2]);
    }
    else if(keyVal >= 5 && keyVal <=10){
      tone(8, notes[3]);
    }
  else{
    noTone(8);
  }
  }
 

Gluce

#27
May 05, 2017, 12:46 am Last Edit: May 05, 2017, 12:51 am by Gluce
Hello Everybody,
I am also having the same problem on this project. The last switch, the one connected to the 10 Kilohm resistor, only reads 0.

The problem is not the switch. If I switch it with one of those that work, it makes the sound. Instead, the one that was working, when put as 4th switch, doesn't work anymore.

Also, I tried to change the position on the breadbord, and the result is still the same. Any idea? =(

This is what the wiring looks like:


CrossRoads

Is anything connected to the leg in row 27? It looks empty to me.
Designing & building electrical circuits for over 25 years.  Screw Shield for Mega/Due/Uno,  Bobuino with ATMega1284P, & other '328P & '1284P creations & offerings at  my website.

Gluce

#29
May 06, 2017, 04:50 pm Last Edit: May 06, 2017, 05:04 pm by Gluce
Sorry, I had moved a couple of things around and forgot to put it back in the right place.










This should be like it is explained in the book.

Plus, I don't understand one thing: the last resistance. In the book, it says "connect tall the switches' output in one junction. Connect this junction to ground with a 10 kilohm resistor". I don't understand the function of this last resistance, considering that every single switch has got his own resistance....

Thank you in advance...

Go Up