[stew_downunder]

I have constructed one of my first (novice) calculator programs, and I am not real schooled up on getting this to repeat over (rather than have the prgram end and then have to restart it every time to perform another calculation.

Please help as I don't know how to do this yet!

At the moment, my code will compile but it goes straight to the "Press any key to continue" prompt.

Check out my code and pay attention to the for loop.

import java.io.*;

public class CalculateBox
{
	public static void main(String args []) throws IOException
	{
		String strHeight = "0", strWidth = "0";
		float height = 0, width = 0, total = 0;
		boolean heightDone = false, widthDone = false;
		BufferedReader dataIn = new BufferedReader(new InputStreamReader(System.in));

		for(int j = 0; j > 0; System.exit(0))
		{

		String yesOrNo;

		while(!heightDone)
		{
			try
			{
				System.out.println();
				System.out.println("Enter the height of the box: ");
				System.out.println();
				strHeight = dataIn.readLine();
				height = Float.parseFloat(strHeight);
				if(height <= 0) throw new NumberFormatException ();
				else heightDone = true;
			}

			catch(NumberFormatException e)
			{
				System.out.println();
				System.out.println("Invalid height entry! - Try again");
			}
		} // end of while

		while(!widthDone)
		{
			try
			{
				System.out.println();
				System.out.println("Enter the width of the box: ");
				System.out.println();
				strWidth = dataIn.readLine();
				width = Float.parseFloat(strWidth);
				if(width <= 0) throw new NumberFormatException ();
				else widthDone = true;
			}

			catch(NumberFormatException e)
			{
				System.out.println();
				System.out.println("Invalid width entry! - Try again:");
			}
		} // end of while

		total = height * width;

		System.out.println();
		System.out.println("The area = " + total);
		System.out.println();

		// this ain't a workin'
		//=====================
		System.out.print("Another calculation? y/n");
		yesOrNo = dataIn.readLine();
		if(yesOrNo == "n") j++;
		//=====================

		} // end of for loop

	} // end of main

} // end of class CalculateBox

[Programmist]

I'm thinking an easier way to do this would be to replace that for loop with another while loop. create a boolean called "done" or "quit" or something like that and set it to false. Then: "while(!done) { ...", and set done to true when the user doesn't want to continue.

[stew_downunder]

ok done that, now the program will run but now whether i enter a yes or a no it now keeps returning to the "The area is ..." Another calculation y/n" over and over...

import java.io.*;

public class CalculateBox
{
	public static void main(String args []) throws IOException
	{
		String strHeight = "0", strWidth = "0";
		float height = 0, width = 0, total = 0;
		boolean heightDone = false, widthDone = false;
		BufferedReader dataIn = new BufferedReader(new InputStreamReader(System.in));

	   	String yesOrNo;
	   	boolean quit = false;
	   	while(!quit)
	   	{


			while(!heightDone)
			{
				try
				{
					System.out.println();
					System.out.println("Enter the height of the box: ");
					System.out.println();
					strHeight = dataIn.readLine();
					height = Float.parseFloat(strHeight);
					if(height <= 0) throw new NumberFormatException ();
					else heightDone = true;
				}

				catch(NumberFormatException e)
				{
					System.out.println();
					System.out.println("Invalid height entry! - Try again");
				}
			} // end of height while

			while(!widthDone)
			{
				try
				{
					System.out.println();
					System.out.println("Enter the width of the box: ");
					System.out.println();
					strWidth = dataIn.readLine();
					width = Float.parseFloat(strWidth);
					if(width <= 0) throw new NumberFormatException ();
					else widthDone = true;
				}

				catch(NumberFormatException e)
				{
					System.out.println();
					System.out.println("Invalid width entry! - Try again:");
				}
			} // end of width while

			total = height * width;

			System.out.println();
			System.out.println("The area = " + total);
			System.out.println();

			// keeps looping back to here for some reason!?
			//=====================
			System.out.print("Another calculation? y/n");
			yesOrNo = dataIn.readLine();
			if(yesOrNo == "n") quit = true;
			//=====================

		} // end of quit while loop

	} // end of main

} // end of class CalculateBox

[1lacca]

Yup, because your heightDone and widthDone variables are still set to true, so it just skips the part where it should input the parameters.

[Programmist]

Instead of using yesOrNo == "n", try yesOrNo.equalsIgnoreCase("n");

Edit: Here is a page explaining the difference between the == operator and the equals method: http://www-128.ibm.c...j-ebb0917a.html
The equalsIgnoreCase String method is basically the same thing as the equals method with one obvious difference.

This post has been edited by alcdotcom: 11 April 2007 - 06:40 AM

[1lacca]

lol, and that too.

[stew_downunder]

Thanks alcdotcom and 1lacca works like a gem now!

This post has been edited by stew_downunder: 12 April 2007 - 12:42 AM