Curious, why so many of these?
while (!Serial.available()) {};
And the semi-colon at the end of each one won't help.
And as a side note, all your variables are declared inside the setup(). Thus, they are "local" and can only be accessed by setup(). If, in the future, you need to have them available to other functions, such as loop(), you will have to move them outside of setup(). Thus making them "global".