KNOCKLOCK: my lock automatically unlock (without knocking)

#include<Servo.h>
Servo myServo ;

const int piezo = A0 ;
const int switchPin = 2 ;
const int yellowLed = 3 ;
const int greenLed = 4 ;
const int redLed = 5 ;

int knockValue ;
int switchValue ;

const int quietknock = 10 ;
const int loudknock = 1000 ;

boolean locked = false ;
int numberOfknock = 0 ;

void setup()
{
myServo.attach (10) ;
pinMode (yellowLed, OUTPUT) ;
pinMode (greenLed, OUTPUT) ;
pinMode (redLed, OUTPUT) ;
pinMode (switchPin, INPUT) ;
Serial.begin (9600) ;

digitalWrite(greenLed, HIGH) ;
myServo.write (0) ;
Serial.println (“the box is unlock !”) ;

}

void loop()
{
if ( locked == false )
{
switchValue = digitalRead (switchPin) ;
if (switchValue == HIGH )
{
locked = true ;
digitalWrite(greenLed, LOW) ;
digitalWrite(redLed, HIGH) ;
myServo.write (90) ;
Serial.println (“the box is lock !”) ;
delay (1000) ;

}
}

if ( locked == true )
{
knockValue = analogRead (piezo) ;

if ( numberOfknock < 3 && knockValue > 0 )
{
if ( checkForknock (knockValue) == true )
{
numberOfknock++ ;
}

Serial.print (3 - numberOfknock) ;
Serial.println (“more knock to go”) ;

}

if ( numberOfknock >= 3)
{
locked = false ;
myServo.write (0);
delay (20);
digitalWrite (greenLed, HIGH) ;
digitalWrite (redLed, LOW) ;
Serial.println (“the box is unlocked !”) ;
numberOfknock = 0 ;
}
}
}

boolean checkForknock (int value )
{
if ( value > quietknock && value < loudknock )
{
digitalWrite (yellowLed, HIGH) ;
delay (50) ;
digitalWrite (yellowLed, LOW) ;
Serial.print ("valid knock of value = ") ;
Serial.println (value);

return true ;
}

else
{
Serial.print ("bad knock value ") ;
Serial.println (value);

return false ;

}
}

What do your Serial.print() statements tell you is happening?