Thread: 1-100 counter

Welcome to the forum.
Always post the full text of the error message with your code and question.
Use code tags when posting code, instructions can be found on the Announcements - What's Wrong With My Code?.

Start with the syntax error first, once you get it to compile and run again, work on the rest. See this tutorial on if-else in Java
It seems the if statement was terminated early with a semicolon in a bad place.

Please post your code in code tags. You can learn how to do that and many other useful things about the forum in the Announcements topic at the top of the page.

It's a best practice to include the bodies of loops inside braces, '{}' even if they're only one line long. In this case, there should be several lines in the body of your for loop, yet you have only one. Your for loop should look like:

Pseudo-code:

for ( counter = 1 to 100 )
{
	// print the counter WITHOUT a linefeed
 
	if ( the counter is even )
	{
		// print "even" 
	}
	else
	{
		// print "odd"
	}
}

I'm not sure why you have the println( 100 ) statement in the middle. That shouldn't be there at all.

I have fixed the syntax error and I also took out the useless println for the 100. I still can't figure out what I need to do to print odd and even beside the appropriate numbers. UGH!

public class counter
{
 
 
 
 
 
	public static void main(String[] args) {
		int count;
		for(count = 1; count <= 100;count++)
			System.out.print( count+",");
 
 
			if((count++%2)!=2)
				System.out.print("odd");
				else
				System.out.print("even"); 
 
 
}
}

There's no point in asking for advice if you're not going to use it.

Put the entire for loop clause, including the if statement, inside braces, '{}'.

The 'if' statement needs work. Since you want to determine if the count is even, you can use the modulo (or remainder) operator:

if ( count % 2 == 0 ) // the expression is true if count is even

You'll also want to add a linefeed (using println()) after "odd" or "even". To make it look nice, you might add a space either after the ',' in the previous print() statement or before " odd" or " even".

I was taught the method of using brackets I originally posted. I have attempted to use what you all have said but it is hard to take words and transfer it to code. Here is where I am at:
[highlight = java]public class counter

{




public static void main(String[] args) {
int count;
{ for(count = 1; count <= 100;count++)
System.out.print( count+",");{
}
{
if(count % 2 ==2)
System.out.print ("odd");}

{ if (count % 1 ==1);

}}
{ System.out.print("even"); }
}
}
[/highlight]

Still no luck as far as printing odd or even beside the numbers.

It'd be much easier to understand if you used the code tags here... but I can help.

GregBrannon gave a simple way of testing if an integer is even.

If an integer is not even, then it must be odd, so you only need to test for an even integer.

If ( count % 2 == 0 ), integer is even, so print count + ", even"
Else, integer is odd, so print count + ", odd"

You can streamline your code a bit by defining count inside of the loop statement itself.

Your syntax looks messy, and you have an unnecessary print statement at the beginning of your code. Use println instead, and handle all the printing inside the if/else statement.

if((count++%2)!=2)

That statement is needlessly convoluted. If you're going to use an increment or decrement operator like count++ or count--, you should do it all by itself on its own line. Never mix it with other statements. Inside the head of a for loop, such as for(count=0; count<10; count++), is fine because the count++ is its own independent statement. When you mix something like that in with the rest of a statement, you make things unpredictable because your variable will be incremented somewhere midstream and you'll lose track of the variables.

Also, that if statement will always be true regardless of what the value of count is. Modular division only considers the remainder. If you divide any number by 2, the highest remainder you can get is 1. So any number % 2 will != 2.

You're on the right track, except for your misplaced curly brackets, as others have mentioned. Whenever you do a loop, if, or switch, the only thing that is affected by it is the very next statement. In this case, your for-loop has only one statement affected--the System.out.print(count +", "). To get around this and include more statements, you need to wrap your code up in curly brackets. Everything in curly brackets following a conditional will be affected by that conditional. So....

for(int x=1; x<10; x++)
...only this statement is affected by the for loop...

for(int y=10; y>10; y--)
{
...this statement is affected by the loop....
....so is this one....
..... this one too...
}
...this one is not...

Some folks say that as a best practice you should do a curly bracket after every conditional, even if you only have one statement. There's no harm in doing that.

Your if statement needs adjustment too. Ask yourself this: what is 4 % 2 (the remainder when you do 4 / 2). What is 5 % 2? What is 6 % 2? Get rid of that count++ in the condition.. it's incrementing a variable that doesn't need to be incremented and as I said that kind of thing is ridiculously confusing.

--- Update ---

Also, use println instead of print. Println automatically adds a newline at the end of each line and makes output much easier to read.