delay(10);
delay(10);
delay(10);
delay(10);
delay(10);
delay(10);
That's no different than "delay(60);"
The purpose to bring a large delay() into smaller chunks is so that in between those chunks, you can do something else - aka unblocking a delay().
On the bright side, you have something working.
the pin changes should have been made from the very beginning
It depends on how you code. I typically code to a logic pin and once the code is taking up shape, I start to firm up the pin assignment. The benefit of that approach is that it enhances its portability: if you were to change the pin assignment, you just need to change the defines and recompile.