Why is this method return 0

I have a class with instance variable price.

I have a method getThePrice() which gets this just fine.

However I have decided to use another method getThePrice(int weight) and simply pass zero as an argument for those products that need not be weighed. The problem is that the method I have written does not work, I am a loss as to what I have done wrong. Any help would be appreciated:

This is the method that does not work:

Code:

---


public int getThePrice(int weighedObject){


            if(weighedObject == 0){

                return price;

            }
            else if(weighedObject > 0){

                return price * weighedObject;

            }

            return price;

            }

---

But when I use this one everything works fine. I am sure it is something wrong with the method I have written:

Code:

---


public int getThePrice(){

        return price;

        }

---

Quote:

---

Originally Posted by africanhacker

I have a class with instance variable price.

I have a method getThePrice() which gets this just fine.

However I have decided to use another method getThePrice(int weight) and simply pass zero as an argument for those products that need not be weighed. The problem is that the method I have written does not work, I am a loss as to what I have done wrong. Any help would be appreciated:

This is the method that does not work:

Code:

---


public int getThePrice(int weighedObject){


            if(weighedObject == 0){

                return price;

            }
            else if(weighedObject > 0){

                return price * weighedObject;

            }

            return price;

            }

---

But when I use this one everything works fine. I am sure it is something wrong with the method I have written:

Code:

---


public int getThePrice(){

        return price;

        }

---

---

That snippet of code seems fine; can you show us how you're calling that method and how you set a (non-zero) value to the price member variable?

kind regards,

Jos

System.out.println("weighedObject Value [" + weighedObject + "]");
System.out.println("Boolean Comparison [" + (weighedObject == 0) + "]");

Quote:

---

Originally Posted by ioexcept

System.out.println("weighedObject Value [" + weighedObject + "]");
System.out.println("Boolean Comparison [" + (weighedObject == 0) + "]");

---

ioexcept, I did not get that.

Quote:

---

Originally Posted by JosAH

That snippet of code seems fine; can you show us how you're calling that method and how you set a (non-zero) value to the price member variable?

kind regards,

Jos

---

This works:

Code:

---

" £" + productArray.get(i).getThePrice();

---

This does not work:

Code:

---

" £" + productArray.get(i).getThePrice(25);

---

Thanks for taking the time to help guys (or girls)


.

What is returned by the second method? Not 25*price I presume because you wouldn't have asked this question here ...

kind regards,

Jos

It returns two possible values. If the argument passed was 0 it should return the base price of the product. If the argument passed was 25 then it would indeed return 25 * price. I don't get why I would not have asked the question here. I am desperately confused as to why something this simple is not working.

Quote:

---

Originally Posted by africanhacker

It returns two possible values. If the argument passed was 0 it should return the base price of the product. If the argument passed was 25 then it would indeed return 25 * price. I don't get why I would not have asked the question here. I am desperately confused as to why something this simple is not working.

---

As already suggested: print the price and the weight factor in the body of that method and see if it clarifies things ...

kind regards,

Jos