Loudspeaker code not working when LCD section added

Hi

Just wondering whether anyone could help with a piece of code which basically triggers LEDs and note on a loudspeaker when LRDS(switches) hi or low. I wanted to add an LCD screen in to display a message however when i add the code for this is messes with the exsisting code and the sound makes a horrible sound.

I have added into the loop() but not sure whether it should go here or within the setup()

/*
Code for Ange's flashy light thing
*/
#include <SoftwareSerial.h>

SoftwareSerial mySerial(-1, 2); // RX9 DISABLED, TX ON D2

// Pins where the shift registers are conected:
int latchPin = 5;
int clockPin = 6;
int dataPin = 4;

// Pins where the LDRs are connected:
const int LDR1 = 13;
const int LDR2 = 12;
const int LDR3 = 11;
const int LDR4 = 10;
const int LDR5 = 9;
const int LDR6 = 8;
const int LDR7 = 7;
const int SPEAKER = 3;

int LDR1State = 0;
int LDR2State = 0;
int LDR3State = 0;
int LDR4State = 0;
int LDR5State = 0;
int LDR6State = 0;
int LDR7State = 0;

int x=0;

// Initial output to shift registers:
unsigned char reg1 = 0;
unsigned char reg2 = 0;
unsigned char reg3 = 0;
unsigned char reg4 = 0;

#define NOTE_B0  31
#define NOTE_C1  33
#define NOTE_CS1 35
#define NOTE_D1  37
#define NOTE_DS1 39
#define NOTE_E1  41
#define NOTE_F1  44
#define NOTE_FS1 46
#define NOTE_G1  49
#define NOTE_GS1 52
#define NOTE_A1  55
#define NOTE_AS1 58
#define NOTE_B1  62
#define NOTE_C2  65
#define NOTE_CS2 69
#define NOTE_D2  73
#define NOTE_DS2 78
#define NOTE_E2  82
#define NOTE_F2  87
#define NOTE_FS2 93
#define NOTE_G2  98
#define NOTE_GS2 104
#define NOTE_A2  110
#define NOTE_AS2 117
#define NOTE_B2  123
#define NOTE_C3  131
#define NOTE_CS3 139
#define NOTE_D3  147
#define NOTE_DS3 156
#define NOTE_E3  165
#define NOTE_F3  175
#define NOTE_FS3 185
#define NOTE_G3  196
#define NOTE_GS3 208
#define NOTE_A3  220
#define NOTE_AS3 233
#define NOTE_B3  247
#define NOTE_C4  262
#define NOTE_CS4 277
#define NOTE_D4  294
#define NOTE_DS4 311
#define NOTE_E4  330
#define NOTE_F4  349
#define NOTE_FS4 370
#define NOTE_G4  392
#define NOTE_GS4 415
#define NOTE_A4  440
#define NOTE_AS4 466
#define NOTE_B4  494
#define NOTE_C5  523
#define NOTE_CS5 554
#define NOTE_D5  587
#define NOTE_DS5 622
#define NOTE_E5  659
#define NOTE_F5  698
#define NOTE_FS5 740
#define NOTE_G5  784
#define NOTE_GS5 831
#define NOTE_A5  880
#define NOTE_AS5 932
#define NOTE_B5  988
#define NOTE_C6  1047
#define NOTE_CS6 1109
#define NOTE_D6  1175
#define NOTE_DS6 1245
#define NOTE_E6  1319
#define NOTE_F6  1397
#define NOTE_FS6 1480
#define NOTE_G6  1568
#define NOTE_GS6 1661
#define NOTE_A6  1760
#define NOTE_AS6 1865
#define NOTE_B6  1976
#define NOTE_C7  2093
#define NOTE_CS7 2217
#define NOTE_D7  2349
#define NOTE_DS7 2489
#define NOTE_E7  2637
#define NOTE_F7  2794
#define NOTE_FS7 2960
#define NOTE_G7  3136
#define NOTE_GS7 3322
#define NOTE_A7  3520
#define NOTE_AS7 3729
#define NOTE_B7  3951
#define NOTE_C8  4186
#define NOTE_CS8 4435
#define NOTE_D8  4699
#define NOTE_DS8 4978

// note durations: 4 = quarter note, 8 = eighth note, etc.:
int noteDurations[] = {
  4 };



void setup() 
{  
  pinMode(latchPin, OUTPUT);
  pinMode(dataPin, OUTPUT);  
  pinMode(clockPin, OUTPUT);
  
  pinMode(LDR1, INPUT);
  pinMode(LDR2, INPUT);
  pinMode(LDR3, INPUT);
  pinMode(LDR4, INPUT);
  pinMode(LDR5, INPUT);
  pinMode(LDR6, INPUT);
  pinMode(LDR7, INPUT);
  
  reg1= B00111100;
  reg2= B00111100;
  reg3= B00111100;
  reg4= B00111100;
  
  updateShiftRegister();
  Serial.begin(9600);
 
}
 
void loop() 
{

mySerial.begin(9600);
  clearScreen();
  mySerial.print(" Move you hand");
  mySerial.print(" &make a tune!");
  
updateShiftRegister();


LDR1State = digitalRead(LDR1);
LDR2State = digitalRead(LDR2);
LDR3State = digitalRead(LDR3);
LDR4State = digitalRead(LDR4);
LDR5State = digitalRead(LDR5);
LDR6State = digitalRead(LDR6);
LDR7State = digitalRead(LDR7);

  // check if the pushbutton is pressed.
  // if it is, the buttonState is HIGH:
  x=0;
  
  if (LDR1State == HIGH) {
    x = 1;
  }
  
  if (LDR2State == HIGH) {
    x = 2;
  }
  
  if (LDR3State == HIGH) {
    x = 3;
  }
  
  if (LDR4State == HIGH) {
    x = 4;
  }
  
  if (LDR5State == HIGH) {
    x = 5;
  }
  
  if (LDR6State == HIGH) {
    x = 6;
  }
  
  if (LDR7State == HIGH) {
    x = 7;
  }

  
  switch (x) {
  case 0:    // your hand is on the sensor
    reg1= B00111100;
    reg2= B00111100;
    reg3= B00111100;
    reg4= B00111100;
    updateShiftRegister();
    break;
  case 1:    
    col1Green();
    tone(SPEAKER, NOTE_G3, 100);
    break;
  case 2:   
    col2Green();
    tone(SPEAKER, NOTE_A3, 100);
    break;
  case 3: 
    col3Green();
    tone(SPEAKER, NOTE_B3, 100);
    break;
  case 4:    
    col4Green();
    tone(SPEAKER, NOTE_C4, 100);
    break;
  case 5:    
    col5Green();
    tone(SPEAKER, NOTE_D4, 100);
    break;
  case 6:    
    col6Green();
    tone(SPEAKER, NOTE_E4, 100);
    break;
  case 7:    
    col7Green();
    tone(SPEAKER, NOTE_F4, 100);
    break;
  } 
  
}
 
void updateShiftRegister()
{
   digitalWrite(latchPin, LOW);
   shiftOut(dataPin, clockPin, MSBFIRST, reg4);
   shiftOut(dataPin, clockPin, MSBFIRST, reg3);
   shiftOut(dataPin, clockPin, MSBFIRST, reg2);
   shiftOut(dataPin, clockPin, MSBFIRST, reg1);
   digitalWrite(latchPin, HIGH);
}

void col1Green()
{
      reg1= B00000001;
      reg2= B00111100;
      reg3= B00111100;
      reg4= B00111100;
updateShiftRegister();
      reg1= B00111101;
      reg2= B00111100;
      reg3= B00111100;
      reg4= B00111100;
updateShiftRegister();
}

void col2Green()
{
      reg1= B00000010;
      reg2= B00111100;
      reg3= B00111100;
      reg4= B00111100;
updateShiftRegister();
      reg1= B00111110;
      reg2= B00111100;
      reg3= B00111100;
      reg4= B00111100;
updateShiftRegister();
}

void col3Green()
{
     reg1= B00000000;
     reg2= B00000011;
     reg3= B00000000;
     reg4= B00000000;
updateShiftRegister();
     reg1= B00111100;
     reg2= B00111101;
     reg3= B11111100;
     reg4= B00111100;
updateShiftRegister();
}

void col4Green()
{
     reg1= B00000000;
     reg2= B00000010;
     reg3= B00000001;
     reg4= B00000000;
updateShiftRegister();
      reg1= B00111100;
      reg2= B00111110;
      reg3= B00111101;
      reg4= B00111100;
updateShiftRegister();
}

void col5Green()
{
     reg1= B00000000;
     reg2= B00000000;
     reg3= B00000010;
     reg4= B00000000;
updateShiftRegister();
     reg1= B00111100;
     reg2= B00111100;
     reg3= B00111110;
     reg4= B00111100;
updateShiftRegister();
}

void col6Green()
{
     reg1= B00000000;
     reg2= B00000000;
     reg3= B00000000;
     reg4= B00000001;
updateShiftRegister();
     reg1= B00111100;
     reg2= B00111100;
     reg3= B00111100;
     reg4= B00111101;
updateShiftRegister();
}

void col7Green()
{
     reg1= B00000000;
     reg2= B00000000;
     reg3= B00000000;
     reg4= B00000010;
updateShiftRegister();
     reg1= B00111100;
     reg2= B00111100;
     reg3= B00111100;
     reg4= B00111110;
updateShiftRegister();
}

 void clearScreen(void){
 mySerial.write(0xFE);
 mySerial.write(0x01); 
}
int LDR1State = 0;
int LDR2State = 0;
int LDR3State = 0;
int LDR4State = 0;
int LDR5State = 0;
int LDR6State = 0;
int LDR7State = 0;

Aren't arrays great?
Aren't arrays great?
Aren't arrays great?
Aren't arrays great?
Aren't arrays great?
Aren't arrays great?
Aren't arrays great?

I wanted to add an LCD screen

Is it a serial LCD?

yes it is a serial LCD