Relay Problem

I am trying to get the button to turn on and off the relay channel once. But somehow, it is performing this action 2 times. Can you please help me make the action do it once? Thanks
CODE:
if(readString.indexOf(‘23’)>0)
{
digitalWrite(RELAY1,LOW);
delay(2000);
digitalWrite(RELAY1,HIGH);
}

sketch_mar10a.ino (11 KB)

  keypad.getKey();

Determine which key, if any, is being pressed. But, throw away the results. Why the hell bother?

    if (digitalRead(pirPin1) == HIGH)
    {
      zone = 0;
      alarmTriggered();
    }
    //if (digitalRead(pirPin1) == LOW)
    //{
      //zone = 1;
      //alarmTriggered();
    //}

Just in case someday digitalRead() returns something other than HIGH or LOW? Not a chance in hell of that happening.

    if(digitalRead(Grove_Water_Sensor) == LOW){
                digitalWrite(LED,HIGH);
  digitalWrite(Grove_Piezo_Buzzer, HIGH);
                delay(2);
                digitalWrite(Grove_Piezo_Buzzer, LOW);
                delay(40);
        }else{
                digitalWrite(Grove_Piezo_Buzzer, LOW);
                digitalWrite(LED,LOW);
        }
[/code
Even drunk,I can indent more consistently than THAT.

[code]    keypadEvent;

Perhaps you meant

47;
            if(readString.indexOf('23')>0)

Please post a picture of your keyboard, with the ONE key that you pressed to get the ONE character in the single quotes to show up.