need help plz, about Wav Shield

Hey guys, I am new here and have started learning Arduino for about one moths, now I am facing a problem when build a sound system with the sound kit.
I want to make the sound to generate one by one when push the button. For ex, when push the button 1 first time, it will play “sound1.wav” and if push it again it will play “sound2.wav”, then again “sound3.wav”, but there is something wrong with my code, it keep playing “sound2.wav” again and again, here is the code:
#include <FatReader.h>
#include <SdReader.h>
#include <avr/pgmspace.h>
#include “WaveUtil.h”
#include “WaveHC.h”
SdReader card; // This object holds the information for the card
FatVolume vol; // This holds the information for the partition on the card
FatReader root; // This holds the information for the filesystem on the card
FatReader f; // This holds the information for the file we’re play
WaveHC wave; // This is the only wave (audio) object, since we will only play one at a time
#define DEBOUNCE 100 // button debouncer
// this handy function will return the number of bytes currently free in RAM, great for debugging!
int freeRam(void)
{
extern int __bss_end;
extern int *__brkval;
int free_memory;
if((int)__brkval == 0) {
free_memory = ((int)&free_memory) - ((int)&__bss_end);
}
else {
free_memory = ((int)&free_memory) - ((int)__brkval);
}
return free_memory;
}
void sdErrorCheck(void)
{
if (!card.errorCode()) return;
putstring("\n\rSD I/O error: “);
Serial.print(card.errorCode(), HEX);
putstring(”, ");
Serial.println(card.errorData(), HEX);
while(1);
}
void setup() {
// set up serial port
Serial.begin(9600);
putstring_nl(“WaveHC with 6 buttons”);

putstring(“Free RAM: “); // This can help with debugging, running out of RAM is bad
Serial.println(freeRam()); // if this is under 150 bytes it may spell trouble!

// Set the output pins for the DAC control. This pins are defined in the library
pinMode(2, OUTPUT);
pinMode(3, OUTPUT);
pinMode(4, OUTPUT);
pinMode(5, OUTPUT);

// pin13 LED
pinMode(13, OUTPUT);

// enable pull-up resistors on switch pins (analog inputs)
digitalWrite(14, HIGH);
digitalWrite(15, HIGH);
digitalWrite(16, HIGH);
digitalWrite(17, HIGH);
digitalWrite(18, HIGH);
digitalWrite(19, HIGH);

// if (!card.init(true)) { //play with 4 MHz spi if 8MHz isn’t working for you
if (!card.init()) { //play with 8 MHz spi (default faster!)
putstring_nl(“Card init. failed!”); // Something went wrong, lets print out why
sdErrorCheck();
while(1); // then ‘halt’ - do nothing!
}

// enable optimize read - some cards may timeout. Disable if you’re having problems
card.partialBlockRead(true);

// Now we will look for a FAT partition!
uint8_t part;
for (part = 0; part < 5; part++) { // we have up to 5 slots to look in
if (vol.init(card, part))
break; // we found one, lets bail
}
if (part == 5) { // if we ended up not finding one :frowning:
putstring_nl(“No valid FAT partition!”);
sdErrorCheck(); // Something went wrong, lets print out why
while(1); // then ‘halt’ - do nothing!
}

// Lets tell the user about what we found
putstring(“Using partition “);
Serial.print(part, DEC);
putstring(”, type is FAT”);
Serial.println(vol.fatType(),DEC); // FAT16 or FAT32?

// Try to open the root directory
if (!root.openRoot(vol)) {
putstring_nl(“Can’t open root dir!”); // Something went wrong,
while(1); // then ‘halt’ - do nothing!
}

// Whew! We got past the tough parts.
putstring_nl(“Ready!”);
}
void loop() {
//putstring(”.”); // uncomment this to see if the loop isnt running
int i;
switch (check_switches()) {
case 1:
i=0;
if (i=0){
i++;
playfile(“SOUND1.WAV”);
break;
}
if(i=1){
i++;
playfile(“SOUND2.WAV”);
break;
}
if(i=2){
i++;
playfile(“SOUND3.WAV”);
break;
}
if(i=3){
i++;
playfile(“SOUND4.WAV”);
break;
}
if (i=4){
i=1;
playfile(“SOUND1.WAV”);
break;
}
case 2:
playfile(“SOUND2.WAV”);
break;
case 3:
playfile(“SOUND3.WAV”);
break;
case 4:
playfile(“SOUND4.WAV”);
break;
case 5:
playfile(“SOUND5.WAV”);
break;
case 6:
playfile(“SOUND6.WAV”);
}
}
byte check_switches()
{
static byte previous[6];
static long time[6];
byte reading;
byte pressed;
byte index;
pressed = 0;
for (byte index = 0; index < 6; ++index) {
reading = digitalRead(14 + index);
if (reading == LOW && previous[index] == HIGH && millis() - time[index] > DEBOUNCE)
{
// switch pressed
time[index] = millis();
pressed = index + 1;
break;
}
previous[index] = reading;
}
// return switch number (1 - 6)
return (pressed);
}
// Plays a full file from beginning to end with no pause.
void playcomplete(char *name) {
// call our helper to find and play this name
playfile(name);
while (wave.isplaying) {
// do nothing while its playing
}
// now its done playing
}
void playfile(char *name) {
// see if the wave object is currently doing something
if (wave.isplaying) {// already playing something, so stop it!
wave.stop(); // stop it
}
// look in the root directory and open the file
if (!f.open(root, name)) {
putstring("Couldn’t open file "); Serial.print(name); return;
}
// OK read the file and turn it into a wave object
if (!wave.create(f)) {
putstring_nl(“Not a valid WAV”); return;
}

// ok time to play! start playback
wave.play();
}
need some help
thx :slight_smile:

You have the switches plugged into all the analog pins? Why? Why not plug the digital switches into digital pins?

    i=0;
    if (i=0){
     i++;
      playfile("SOUND1.WAV");
     break;
     }
    if(i=1){

In general, this will work better if you use the equality operator (==) instead of the assignment operator (=).

I have not completely finished study the WaveHC libarary, so the only way for me now is to follow the examples they showed. But by following your suggest I changed the "=" into "==" and put the defination of "int i=0;" outside the "void loop()", it worked!!
Thank you dude, your advice greatly helped me!
thx!!