As mentioned earlier . . .
-
What does this do?
while (!digitalWrite, sw) -
When will this be true?
if (currentMillis == 1000 && ledState1 == HIGH) { -
Put each line of code on a line by itself.
-
Place each { and } on a line by them self, makes it easier to find them.
-
A week from now, comments would remind you what’s happening.
When asking for help, your code should be commented so we can figure out what you are trying to do.