int buttonState = 0;
Don't know if this is your problem, but get rid ofCode: [Select]int buttonState = 0; in your else clause.
It's odd that you're using 4 outputs for two 3-color LEDs... I think we need to see a schematic. And, you your comments don't define the colors.How about if you simply write all-low, do the LEDs go off? Do they go on when you write all-high? (That's without the switch or any if-statements, etc.)Can you control the colors individually without using the switch or if-statement?