Can't answer your question, but why code this way:
if(myhour == 21 && myminute == 18) state = 1;
else if(myhour == 21 && myminute == 00) state = 2;
else if(myhour == 21 && myminute == 10) state = 3;
else if(myhour == 23 && myminute == 00) state = 4;
The else may not be doing what you expect and is superfluous.
Just make all 4 "IF" with no "else".
Paul