AppletTalk.com

da Vinci · Guest

I completed an assignment for my Intro to Java Programming class and
then continually updated the program to make it better. This is the
third or fourth incarnation of it and I have run into a problem.

Inside of a while loop, which is controlled by a boolean (repeat), I
have an if statement. Basically, if the user enters any positive
integer, it will output the area and volume of a cube with the side
length of the entered integer. It continually loops until the user
enters 0 for the side length, which will then set repeat to false and
end the loop statement.

However, it is not ending the loop. I output the integer value to
ensure it is setting to 0 correctly and it is. However, it still wont
end the while loop. The only way I am able to end the while loop is by
adding return; into the if statements. Then it will end. But nothing
in the book shows this as being the 'correct' way to do this.

I left the return; statement in the code for anyone to peek at to see
what is going on. Any help is appreciated (Thanks in advance!).

Code follows:

import java.io.*;

public class CubeUserAdv
{
public static void main(String argv[]) throws IOException
{

int cubevalue=1;
boolean repeat = true;

// Code to get interger inputs from a string input.

BufferedReader input = new BufferedReader (new
InputStreamReader(System.in));
String inputString;

System.out.println("nnThis program will display the surface area
and volume for three cubes.");
System.out.println("Ensure that the numbers you enter for cubes are
positive integers.nn");

// While loop to get values for the cubes & display output.

while (repeat = true)
{
System.out.println("nnEnter a value for cube (0 to end): ");
inputString = input.readLine();
cubevalue = Integer.parseInt(inputString);

if (cubevalue == 0)
{
repeat = false;
return;
}
else
{
repeat = true;
}

Cube cuber = new Cube(cubevalue);

// Data Output to Console.

System.out.println("n******** Cube ********");
System.out.println("Surface Area = " + cuber.Calc_SA());
System.out.println("Volume = " + cuber.Calc_Volume());

System.out.println(cubevalue);
}

System.out.println("nn");
}
}

******* NEW FILE *********

public class Cube
{

private double Side_Length;

public Cube(double SL_Input)
{
Side_Length = SL_Input;
}

public double Calc_SA()
{
return (6 * (Side_Length * Side_Length));
}

public double Calc_Volume()
{
return ((Side_Length * Side_Length) * Side_Length);
}

}

Paul Lutus · Guest

da Vinci wrote:

Tor Iver Wilhelmsen · Guest

da Vinci <blank (AT) blank (DOT) com> writes:

da Vinci · Guest

On Sun, 22 Aug 2004 12:13:03 -0700, Paul Lutus <nospam (AT) nosite (DOT) zzz>
wrote:

da Vinci · Guest

On 22 Aug 2004 21:39:19 +0200, Tor Iver Wilhelmsen
<tor.iver.wilhelmsen (AT) broadpark (DOT) no> wrote:

Dhananjay Singh · Guest

The problem is with statement ........
while (repeat = true) { ...
this is creating the problem. change it to "equals to" operator.
while(repeat == true) [....

It will work.