Indeed those const int declarations probably should be global.
You're doing a digitalRead() on four pins in setup and then do nothing with it. these probably have to go in loop().
Instead of O and T for 1 and 2, why not use the numbers? It's simply clearer.