Thread: Switch in a For loop

		for(int r=25;r<=125;r+=25)
		{
			int color = 1;
			switch(color)
			{
 
				case 1 : g.setColor(Color.black);  Expo.fillCircle(g,625,175,r); color++; break;
				case 2 : g.setColor(Color.white);  Expo.fillCircle(g,625,175,r); color++; break;
				case 3 : g.setColor(Color.blue);  Expo.fillCircle(g,625,175,r); color++; break;
				case 4 : g.setColor(Color.red);  Expo.fillCircle(g,625,175,r); color++; break;
				case 5 : g.setColor(Color.yellow);  Expo.fillCircle(g,625,175,r); color++; break;
			}
 
 
		}
		g.setColor(Color.black); 	//reset colors.

I am attempting to draw 5 concentric circles with increasing radius and chose the colors that each circle is filled with via a switch of an int. The result is a circle of maximus radius (which would indicate that the for loop is working, right?), but it is all the same color, the color of black which is case 1. What am I doing wrong, why won't the other circles change colors?

You've declared and initialized your color variable inside of the for loop. So even though you increment it, it reverts back to 1 with every iteration of the loop.

Thanks!! After changing the int declaration to before the for loop there was still a problem where the largest radius circle of the last color showed up, but I easily fixed that by changing r within the for loop. Thanks again!

	Random rand = new Random();
		int color = 0;	//1 = black, 2 = white, 3 = blue, 4 = red, 5 = yellow
		for(int r=125;r<=125;r-=25)		//execute loop as long as the radius is 125(the max value used) or less, and decrease radius of circles as executed.
		{
			color = rand.nextInt(5)+1;
			switch(color)	//for every circle check the color integer to determine the color the circle is to be filled with.
			{
 
				case 1 : g.setColor(Color.black);  Expo.fillCircle(g,625,175,r); break;
				case 2 : g.setColor(Color.white);  Expo.fillCircle(g,625,175,r); break;
				case 3 : g.setColor(Color.blue);  Expo.fillCircle(g,625,175,r); break;
				case 4 : g.setColor(Color.red);  Expo.fillCircle(g,625,175,r);  break;
				case 5 : g.setColor(Color.yellow);  Expo.fillCircle(g,625,175,r);  break;
			}
 
 
		}
		g.setColor(Color.black)

I am attempting to generate random colors for each circle out of the 5 color options, and it works except if you wait long enough after the circle has been drawn, the window will begin flashing random colors incessantly. Why is this?