My method keeps returning 0

[ToolJob]

Hi I have the following two class files. How come my method keeps spitting out 0??? I can't for the life of me get it to calculate and display the length x height it always outputs zero?

public class UseJoe
{

  public static void main(String[] args)
  {

    Joe boxH = new Joe();
    Joe boxL = new Joe();

    boxH.setHeight(55.5);
    boxL.setLength(10.0);

    //System.out.println("The width x height is: ");


    Joe calcTotal = new Joe();
      //calcTotal.computeBox();
    double cTotal = calcTotal.computeBox();

    System.out.println("The width x height is: " + cTotal);

  }

}

And the other class

public class Joe
{

  private double length;
  private double height;

  public double getLength()
  {

    return length;

  }

  public void setLength(double l)
  {

    length = l;

  }

  public double getHeight()
  {

    return height;

  }

  public void setHeight(double h)
  {

    height = h;

  }

  double computeBox()
  {

    return height * length;

  }

}

[Fubarable]

You really don't want to create three separate Joe objects since each is completely distinct from the other. For instance you set the height of boxH, you set the length of boxL, and then do the calculations with calcTotal, and this makes no sense, since setting the fields of the two previous Joe objects has absolutely no effect on the third.

Why not just create one Joe object, set it's height and length and do the calculations with that single Joe object?

[JosAH]

  double computeBox()
  {

    return height * length;

  }

Print both the height and length values in that method and you'll see what's happening.

kind regards,

Jos

[lovelesh]

You are not setting any values in your third Joe object.

And default value for double is zero (0.0). Hence you are getting zero always.

[überfuzz]

Try to set some values before you start to calculate stuff. It seems you're tangled up in the object orientated code here. ;-)

Joe test = new Joe;
Now you have to set the values in the object you wanna tinker with. In this case it would be something like text.setHeight(10); You're making tree objects but you only give the first two height and width. Try giving the Joe object calcTotal some height and width and see what happens.

[ToolJob]

Jos

I did a

  double computeBox()
  {


    System.out.println(height + "   " + length);
    return height * length;

  }

The height and length still print out:

0.0 0.0
The width x height is: 0.0


ToolJob

[Fubarable]

Please read all the replies to your question.

[ToolJob]

Fubarable

Can you give me a simple example? Ill try it if I can see how it would work.

Thx TJ

[Fubarable]

Fubarable

Can you give me a simple example? Ill try it if I can see how it would work.

Thx TJ

You really don't need an example,... you just need to create one Joe object only, not three. Please try it, and you'll see. If it doesn't work, then please post the code.

[ToolJob]

Lovelesh

If I do:

  double computeBox()
  {

    height = 5; length = 10;
    System.out.println(height + "   " + length);
    return height * length;

  }

The values are printed when I set them to a value in the method. I guess I am way off course here - I was under the impression private variables can be used in methods of the same class?

I been trying to make this work and my brain is fried LOL

The values I set

[JosAH]

Lovelesh

If I do:

  double computeBox()
  {

    height = 5; length = 10;
    System.out.println(height + "   " + length);
    return height * length;

  }

The values are printed when I set them to a value in the method. I guess I am way off course here - I was under the impression private variables can be used in methods of the same class?

I been trying to make this work and my brain is fried LOL

The values I set

In your code you have three different Joes: named BoxH, BoxL and calcTotal; you set the length of BoxL, you set the height of BoxH and you ask calcTotal to calculate the area. It will be zero of course because you never set the height nor length of the calcTotal object. Use one Joe object instead; set its length and height and then ask it to calculate its area.

kind regards,

Jos

[Solarsonic]

As the others have said, every time you do setHeight etc. You're setting height for only that instance of Joe (whatever object you did it with). Which means that, since you never set the height for the calcTotal object, it will return 0 since the length and width values are 0 for that instance of Joe.