first year project

blocparty

im doing a number guess game as part of my course. theres a few problems with it. the loop doesnt seem to be working properly and the statement is printed 4 times instead of once. any help?


import java.io.*;


class guessAdvanced

{

public static void main (String[] args) throws IOException

{
BufferedReader stdin =
new BufferedReader ( new InputStreamReader( System.in ) );

String InData;


int guess, count, limit;
final double answer = 56;

limit = 5;




System.out.println("Enter your guess (1-100) but remember you only have 5 guesses:"); InData = stdin.readLine();
guess = Integer.parseInt (InData);

count = 1;

while ( count < limit)
{
if ( guess < answer )
{
System.out.println("The answer is higher!"); }
else if ( guess >= answer )
{
System.out.println("The answer is lower!"); }
count = count +1; }
if ( guess == answer)
{
System.out.println("Congragulations you have guessed correctly!!!! Well done");
}
else if ( guess != answer)
System.out.println("Try again");
}

}

Tobias Greeshman

There's no input being done in the loop!

Gurgle

[b]reformatted[/b]
import java.io.*;

class guessAdvanced

{

public static void main (String[] args) throws IOException

	{
	BufferedReader stdin =
	new BufferedReader ( new InputStreamReader( System.in ) );

	String InData;

	int guess, count, limit;
	final double answer = 56;

	limit = 5;

	System.out.println("Enter your guess (1-100) but remember you only have 5 guesses:"); 
	InData = stdin.readLine();
	guess = Integer.parseInt (InData);

	count = 1;

	while ( count < limit) 
		{
		if ( guess < answer )
			{
			System.out.println("The answer is higher!"); 
			}
		else if ( guess >= answer )
			{
			System.out.println("The answer is lower!"); 
			}
		count = count +1; 
		}

	if ( guess == answer)
		{
		System.out.println("Congragulations you have guessed correctly!!!! Well done");
		}
	else if ( guess != answer) System.out.println("Try again");
	}
}

Here is your code reformatted so you can see whats going on.
Your InData and if ( guess == answer) instructions are outside your count loop.

(thx Hobbes, didn't know about code tags)

Hobbes

use the /i]code[i tags. will look even nicer reformatted.