Should I replace my board or there is something wrong with my code

Hi… Recently I am trying to build a secret knock and sms based door lock… but when I implement the sms the secret knock is not very responsive ? What should I do… my board is Arduino Duemilanove w/ Atmega328 and Icomsat 1.1…

Should I replace Arduino Duemilanove w/ Atmega328 to Arduino Mega2560 Rev3?
Or there is something wrong with my code,

The code is attached below…

lame_is_finsihed.ino (19.8 KB)

How do you mean "not very responsive" ?

I see your code is full of delay() calls - those alone can cause things to be "not responsive"...

What do the millis() lines do?

void loop() {
  millis();   //<--------- here
  char pos =  sms.IsSMSPresent(SMS_UNREAD);
  if (1==pos) {
    sms_detected();
  }

  millis();   //<--------- and here

I remove all of those sensor and delay.. but in some reason when I knock it detect not very well unless otherwise the gsm is off..