float selectorValueFloat = round(analogRead(selectorPin) / divider);
WTF? You divide an integer by a float, resulting in a float. You round that off, resulting in an int, which you store in a float. Why?
//cast to integer for e.g. indexing arrays
sv = selectorValueFloat;
You wouldn't have to do this if you hadn't stored the int in a float in the first place.
How would I be able to read only change of state
The change of state of what?
if(sv ==0 || stop1 || memo !=sv)
has1 = 1;
memo = sv;
WTF? You haven't learned to indent properly? Use Tools + Auto Format!
WOW Way to overreact right there. Yes the int/float rounding arrangement he has is very convoluted
but there is no reason to be so rude about it.
Indentation is something that they will get the hang of. It isn't a big deal
Every curly brace belongs on its own line. EVERY ONE!
Not true. Putting opening braces on their own line is usually nothing other than a waste of screen space.
Closing braces should be on their own line though.