What is this for(;;) supposed to so. Isn't it an endless loop?
What is this if (0 == (millis()%1000)) supposed to do?
Can you describe in English the sequence of things that should happen within loop() including how often they should happen.
At the moment the code seems confused (and confusing).
...R