using if/else statments

**new2java** · September 24th, 2009, 03:09 PM

my program is supposed to show "correct" when color box goes into "colors" box, dark box goes into "darks" box and white box goes into "whites" box. it is also supposed to create a new color if the user is correct. What am I doing wrong?

if(colorsum <600 && colorsum>230 && laundrydark.contains(point) )

correct++;
swatch.setFillColor(randomcolor);

if ((colorsum < 230) && laundrycolor.contains(point) )

correct++;
swatch.setFillColor(randomcolor);

if((colorsum > 600) && laundrywhite.contains(point))

correct++;
swatch.setFillColor(randomcolor);

if( colorsum <=600 && colorsum >=230 && laundrywhite.contains(point) ||
colorsum <=230 && laundrydark.contains(point) ||
colorsum >=600 && laundrycolor.contains(point))

incorrect ++;
}

**helloworld922** · September 25th, 2009, 12:21 AM

I have a feeling it's because you're forgetting to bracket each if statement. I'd also recommend doing if/else statements, since only 1 of these can ever be executed each time through. By re-arranging your tests, you can also remove a lot of checks.

if ((colorsum < 230) && laundrycolor.contains(point) )
{
correct++;
swatch.setFillColor(randomcolor);
}
else if(colorsum <600 && laundrydark.contains(point) )
{
correct++;
swatch.setFillColor(randomcolor);
}
else if(laundrywhite.contains(point))
{
correct++;
swatch.setFillColor(randomcolor);
}
else
{
incorrect ++;
}

Thread: using if/else statments

LinkBack

Thread Tools

Display

using if/else statments

Related threads:

Re: using if/else statments