Please look at my code and suggest how to do it better

Some minor points:
1)
Be consistent when using const for the pin names
e.g.

int ldrnw = A0; //LDR northwest

Use pin names everywhere
e.g.

digitalWrite(10, HIGH);

Why do you sometimes use int and sometimes byte for the definition of the pin names?