fading leds code question

Hi.

Why will it not run when I uncomment lines 18 & 19?

#define leds
#define green_led    11 
#define yellow_led   10
#define red_led      9

int i = 0; //green     
int j = 0; //yellow
int k = 0; //red

//line 10
void setup(leds) {pinMode(green_led, OUTPUT);pinMode(yellow_led, OUTPUT);pinMode(red_led, OUTPUT);} 

void loop(leds){ 
////green led in & out
  for (i = 0; i < 255; i++) {analogWrite(green_led, i);delay(10);}
  for (i = 255; i > 0; i--) {analogWrite(green_led, i);delay(10);}
////yellow led in & out
//  for (j = 0; j < 255; j++) (analogWrite(yellow_led, j);delay(10);)
//  for (j = 255; j > 0; j--) (analogWrite(yellow_led, j);delay(10);)
//line 20
  for (j = 0; j < 255; j++) {analogWrite(yellow_led, j);delay(10);}
  for (j = 255; j > 0; j--) {analogWrite(yellow_led, j);delay(10);}
////red led in & out
  for (k = 0; k < 255; k++) {analogWrite(red_led, k);delay(10);}
  for (k = 255; k > 0; k--) {analogWrite(red_led, k);delay(10);}
;}

It works great, but when I uncomment lines 18 & 19. This is what I get:

leds_fade.cpp: In function 'void loop()':
leds_fade:17: error: expected )' before ';' token leds_fade:17: error: expected primary-expression before ')' token leds_fade:17: error: expected ;' before ')' token
leds_fade:18: error: expected ;' before ')' token leds_fade:18: error: expected primary-expression before ')' token leds_fade:18: error: expected ;' before ')' token
leds_fade:20: error: expected `;' before ')' token

I'm not sure if it wants a ); or a :wink:

I have version 1.0.1 of the software, and I'm using an Uno.

what about { } instead of ( )

in some cases :wink:

Thank you for the reply, so I tried replacing all of lines 18 & 19. This is what I got:

leds_fade.cpp: In function 'void loop()':
leds_fade:17: error: expected (' before '{' token leds_fade:17: error: expected primary-expression before '{' token leds_fade:17: error: expected ;' before '{' token
leds_fade:17: error: expected primary-expression before '{' token
leds_fade:17: error: expected ;' before '{' token leds_fade:17: error: expected primary-expression before '{' token leds_fade:17: error: expected )' before '{' token
leds_fade:17: error: expected ;' before '{' token leds_fade:17: error: expected ;' before '{' token
leds_fade:18: error: expected (' before '{' token leds_fade:18: error: expected primary-expression before '{' token leds_fade:18: error: expected ;' before '{' token
leds_fade:18: error: expected primary-expression before '{' token
leds_fade:18: error: expected ;' before '{' token leds_fade:18: error: expected primary-expression before '{' token leds_fade:18: error: expected )' before '{' token
leds_fade:18: error: expected ;' before '{' token leds_fade:18: error: expected ;' before '{' token

Code blocks should use { and }. Function parameters should use ( and ). Your lines 18 and 19 should read:

  for (j = 0; j < 255; j++) {analogWrite(yellow_led, j);delay(10);}
  for (j = 255; j > 0; j--) {analogWrite(yellow_led, j);delay(10);}

Thank you. It works great now.