Search:

Leaving off the {}s can be a problem if later you add more code and forget to add them. Best to always use them.


if(some == all)
doThis();
doThat(); //<<<<<<< This...

I'd move the colors array outside the method next to where the option variable is defined and make it final to show that isn't to be updated.