whats wrong here

can not get any leds to come on any ideas

int redPin=10; //set red LED pin to 11 int greenPin=9; //set green LED pin to 10 int bluePin=8; //set blue LED pin to 6 int brightness=255; //Set brightness to 75 String colorChoice; //Will hold users input of color choice

void setup() { // put your setup code here, to run once:

Serial.begin(9600); //Turn on Serial port pinMode(redPin, OUTPUT); //Set redPin to be an output pinMode(greenPin, OUTPUT); //Set greenPin to be an output pinMode(bluePin, OUTPUT); //set bluePin to be an output

}

void loop() {

Serial.println("What color would you like the LED? (red, green, or blue)"); //Prompt user for color while (Serial.available()==0) { } //Wait for input colorChoice = Serial.readString();

if (colorChoice== "red" ) {

digitalWrite(redPin, HIGH); //turn on red pin analogWrite(greenPin, 0); //turn off green pin analogWrite(bluePin, 0); //write off blue pin }

if (colorChoice== "blue") {

analogWrite(redPin, 0); //turn off red pin analogWrite(greenPin, 0); //turn off green pin analogWrite(bluePin, brightness); //write on blue pin }

if (colorChoice=="green") {

analogWrite(redPin, 0); //turn off red pin analogWrite(greenPin, brightness); //turn on green pin analogWrite(bluePin, 0); //write off blue pin }

if (colorChoice!="red" && colorChoice!="green" && colorChoice != "blue") { Serial.println("That is not a valid color choice, please try again"); Serial.println(""); }

}

Are you sure that your colorChoice is being read correctly? Print out the colorChoice when you think you read it to make sure.

Are all your pins PWM enabled? Try just using a digitalWrite to see if that works.

Also, why do you need a word and not just R, G and B? Just asking, as the typing would be quicker.

Also, to address programming form:

int brightness=255; //Set brightness to 75

The comment is pointless as it does not tell me what 'brightness' it is referring to. it also becomes outdated/confusing (as you can see here) if you then decide to change the value. A better comment would be

int brightness=255; // LED PWM output level to control brightness

As this explains what the variable does and you can infer from the value what the level is anyway.