servo case structure issue

Here it is:

    Timer1 = millis();
    if (millis() - Timer1 > 5000)
    {
      state = two;
    }

I'm not clear what you're trying to do here but you store millis in Timer1 and immediately check to see whether five seconds have passed since you did it. That will never be the case so you never change to state two.