Problem using tone function to play a song with a peizo speaker

this my first project with arduino could somebody tell me why this code makes a random noise instead of playing the intentional song. I mad test code to see if peizo can give single tones and it can so I think that the issue is with my code. Perhaps the tone function durations don't work correctly? Thank you so much!

songs.ino (4.21 KB)

Put some delays in there.
example:

tone(11, NOTE_Re2, Quarter);
delay(Quarter);
tone(11, NOTE_La3, Quarter);
delay(Quarter);
tone(11, NOTE_La3, Quarter);
delay(Quarter);
etc.

Also make your variable initializations globals.

How do you have this wired?

As a first project, this is a bit ambitious, because you obviously don't yet know how to declare pins as inputs or outputs. This is absolutely essential, so start with the Arduino Blink example.

FYI, Happy Birthday

// Reference:  This list was adapted from the table located here:
// 		http://www.phy.mtu.edu/~suits/notefreqs.html
const float  note_C0 = 16.35;  //C0
const float  note_Db0 = 17.32;  //C#0/Db0
const float  note_D0 = 18.35;  //D0
const float  note_Eb0 = 19.45;  //D#0/Eb0
const float  note_E0 = 20.6;  //E0
const float  note_F0 = 21.83;  //F0
const float  note_Gb0 = 23.12;  //F#0/Gb0
const float  note_G0 = 24.5;  //G0
const float  note_Ab0 = 25.96;  //G#0/Ab0
const float  note_A0 = 27.5;  //A0
const float  note_Bb0 = 29.14;  //A#0/Bb0
const float  note_B0 = 30.87;  //B0
const float  note_C1 = 32.7;  //C1
const float  note_Db1 = 34.65;  //C#1/Db1
const float  note_D1 = 36.71;  //D1
const float  note_Eb1 = 38.89;  //D#1/Eb1
const float  note_E1 = 41.2;  //E1
const float  note_F1 = 43.65;  //F1
const float  note_Gb1 = 46.25;  //F#1/Gb1
const float  note_G1 = 49;  //G1
const float  note_Ab1 = 51.91;  //G#1/Ab1
const float  note_A1 = 55;  //A1
const float  note_Bb1 = 58.27;  //A#1/Bb1
const float  note_B1 = 61.74;  //B1
const float  note_C2 = 65.41;  //C2 (Middle C)
const float  note_Db2 = 69.3;  //C#2/Db2
const float  note_D2 = 73.42;  //D2
const float  note_Eb2 = 77.78;  //D#2/Eb2
const float  note_E2 = 82.41;  //E2
const float  note_F2 = 87.31;  //F2
const float  note_Gb2 = 92.5;  //F#2/Gb2
const float  note_G2 = 98;  //G2
const float  note_Ab2 = 103.83;  //G#2/Ab2
const float  note_A2 = 110;  //A2
const float  note_Bb2 = 116.54;  //A#2/Bb2
const float  note_B2 = 123.47;  //B2
const float  note_C3 = 130.81;  //C3
const float  note_Db3 = 138.59;  //C#3/Db3
const float  note_D3 = 146.83;  //D3
const float  note_Eb3 = 155.56;  //D#3/Eb3
const float  note_E3 = 164.81;  //E3
const float  note_F3 = 174.61;  //F3
const float  note_Gb3 = 185;  //F#3/Gb3
const float  note_G3 = 196;  //G3
const float  note_Ab3 = 207.65;  //G#3/Ab3
const float  note_A3 = 220;  //A3
const float  note_Bb3 = 233.08;  //A#3/Bb3
const float  note_B3 = 246.94;  //B3
const float  note_C4 = 261.63;  //C4
const float  note_Db4 = 277.18;  //C#4/Db4
const float  note_D4 = 293.66;  //D4
const float  note_Eb4 = 311.13;  //D#4/Eb4
const float  note_E4 = 329.63;  //E4
const float  note_F4 = 349.23;  //F4
const float  note_Gb4 = 369.99;  //F#4/Gb4
const float  note_G4 = 392;  //G4
const float  note_Ab4 = 415.3;  //G#4/Ab4
const float  note_A4 = 440;  //A4
const float  note_Bb4 = 466.16;  //A#4/Bb4
const float  note_B4 = 493.88;  //B4
const float  note_C5 = 523.25;  //C5
const float  note_Db5 = 554.37;  //C#5/Db5
const float  note_D5 = 587.33;  //D5
const float  note_Eb5 = 622.25;  //D#5/Eb5
const float  note_E5 = 659.26;  //E5
const float  note_F5 = 698.46;  //F5
const float  note_Gb5 = 739.99;  //F#5/Gb5
const float  note_G5 = 783.99;  //G5
const float  note_Ab5 = 830.61;  //G#5/Ab5
const float  note_A5 = 880;  //A5
const float  note_Bb5 = 932.33;  //A#5/Bb5
const float  note_B5 = 987.77;  //B5
const float  note_C6 = 1046.5;  //C6
const float  note_Db6 = 1108.73;  //C#6/Db6
const float  note_D6 = 1174.66;  //D6
const float  note_Eb6 = 1244.51;  //D#6/Eb6
const float  note_E6 = 1318.51;  //E6
const float  note_F6 = 1396.91;  //F6
const float  note_Gb6 = 1479.98;  //F#6/Gb6
const float  note_G6 = 1567.98;  //G6
const float  note_Ab6 = 1661.22;  //G#6/Ab6
const float  note_A6 = 1760;  //A6
const float  note_Bb6 = 1864.66;  //A#6/Bb6
const float  note_B6 = 1975.53;  //B6
const float  note_C7 = 2093;  //C7
const float  note_Db7 = 2217.46;  //C#7/Db7
const float  note_D7 = 2349.32;  //D7
const float  note_Eb7 = 2489.02;  //D#7/Eb7
const float  note_E7 = 2637.02;  //E7
const float  note_F7 = 2793.83;  //F7
const float  note_Gb7 = 2959.96;  //F#7/Gb7
const float  note_G7 = 3135.96;  //G7
const float  note_Ab7 = 3322.44;  //G#7/Ab7
const float  note_A7 = 3520;  //A7
const float  note_Bb7 = 3729.31;  //A#7/Bb7
const float  note_B7 = 3951.07;  //B7
const float  note_C8 = 4186.01;  //C8
const float  note_Db8 = 4434.92;  //C#8/Db8
const float  note_D8 = 4698.64;  //D8
const float  note_Eb8 = 4978.03;  //D#8/Eb8

//#include "musical_notes.h"

int speakerPin = 11; // speaker connected to digital pin 9 

unsigned long currentMillis2;
unsigned long delayTime;
byte note = 0;

void setup()    
{   
  pinMode(13,OUTPUT); //debug stuff
  
  pinMode(speakerPin, OUTPUT); // sets the speakerPin to be an output 
  currentMillis2 = millis();
  delayTime = 0;
}    

void loop()
{    
  digitalWrite(13,!digitalRead(13)); //debug stuff
  
  if( millis() - currentMillis2 >= delayTime)
  {
    happyBirthday();
  }
}     

void beep (int speakerPin, float noteFrequency, long noteDuration)
{    
  int x;
  // Convert the frequency to microseconds
  float microsecondsPerWave = 1000000/noteFrequency;
  // Calculate how many HIGH/LOW cycles there are per millisecond
  float millisecondsPerCycle = 1000/(microsecondsPerWave * 2);
  // Multiply noteDuration * number or cycles per millisecond
  float loopTime = noteDuration * millisecondsPerCycle;
  // Play the note for the calculated loopTime.
  for (x=0;x<loopTime;x++)   
  {   
    digitalWrite(speakerPin,HIGH); 
    delayMicroseconds(microsecondsPerWave); 
    digitalWrite(speakerPin,LOW); 
    delayMicroseconds(microsecondsPerWave); 
  } 
}     


void happyBirthday(){
  switch(note)
  {
  case 0:
    beep(speakerPin, note_Bb4,200); //B b  
    currentMillis2 = millis();
    delayTime = 25;
    note++;
    break;

  case 1:
    beep(speakerPin, note_Bb4,100); //B b  
    currentMillis2 = millis();
    delayTime = 25;
    note++;
    break;

  case 2:
    beep(speakerPin, note_C5,300); //C
    currentMillis2 = millis();
    delayTime = 25;
    note++;
    break;

  case 3:
    beep(speakerPin, note_Bb4,300); //B b  
    currentMillis2 = millis();
    delayTime = 100;
    note++;
    break;

  case 4:
    beep(speakerPin, note_Eb5,300); //E b      
    currentMillis2 = millis();
    delayTime = 100;
    note++;
    break;

  case 5:
    beep(speakerPin, note_D5,450); //D
    currentMillis2 = millis();
    delayTime = 400;
    note++;
    break;

  case 6:
    beep(speakerPin, note_Bb4,200); //B b  
    currentMillis2 = millis();
    delayTime = 25;
    note++;
    break;

  case 7:
    beep(speakerPin, note_Bb4,100); //B b  
    currentMillis2 = millis();
    delayTime = 25;
    note++;
    break;

  case 8:
    beep(speakerPin, note_C5,300); //C
    currentMillis2 = millis();
    delayTime = 25;
    note++;
    break;

  case 9:
    beep(speakerPin, note_Bb4,300); //B b  
    currentMillis2 = millis();
    delayTime = 100;
    note++;
    break;

  case 10:
    beep(speakerPin, note_F5,300); //F
    currentMillis2 = millis();
    delayTime = 100;
    note++;
    break;

  case 11:
    beep(speakerPin, note_Eb5,450); //E b      
    currentMillis2 = millis();
    delayTime = 400;
    note++;
    break;

  case 12:
    beep(speakerPin, note_Bb4,200); //B b  
    currentMillis2 = millis();
    delayTime = 25;
    note++;
    break;

  case 13:
    beep(speakerPin, note_Bb4,100); //B b  
    currentMillis2 = millis();
    delayTime = 25;
    note++;
    break;

  case 14:
    beep(speakerPin, note_Bb5,300); //B b up an octave
    currentMillis2 = millis();
    delayTime = 25;
    note++;
    break;

  case 15:
    beep(speakerPin, note_G5,300); //G
    currentMillis2 = millis();
    delayTime = 100;
    note++;
    break;

  case 16:
    beep(speakerPin, note_Eb5,300); //Eb   
    currentMillis2 = millis();
    delayTime = 100;
    note++;
    break;

  case 17:
    beep(speakerPin, note_D5,400); //D
    currentMillis2 = millis();
    delayTime = 100;
    note++;
    break;

  case 18:
    beep(speakerPin, note_C5,500); //C
    currentMillis2 = millis();
    delayTime = 400;
    note++;
    break;

  case 19:
    beep(speakerPin, note_Ab5,200); //A b  
    currentMillis2 = millis();
    delayTime = 25;
    note++;
    break;

  case 20:
    beep(speakerPin, note_Ab5,100); //A b  
    currentMillis2 = millis();
    delayTime = 25;
    note++;
    break;

  case 21:
    beep(speakerPin, note_G5,300); //G
    currentMillis2 = millis();
    delayTime = 100;
    note++;
    break;

  case 22:
    beep(speakerPin, note_Eb5,300); //E b  
    currentMillis2 = millis();
    delayTime = 100;
    note++;
    break;

  case 23:
    beep(speakerPin, note_F5,300); //F
    currentMillis2 = millis();
    delayTime = 100;
    note++;
    break;

  case 24:
    beep(speakerPin, note_Eb5,500); //E b      
    currentMillis2 = millis();
    delayTime = 2500;
    note++;
    break;

  case 25:
    note = 0;
    break;

  }
}

Your note duration (beep length) should always be shorter than the actual delay. Otherwise, the notes run into each other, and it sounds odd. Also, you couldn't play the same note twice, as it would just sound like one long note.

If you have a beep length of 3/4 the delay length, it sounds ok.