Thread: Setter Methods Query

Hi,

I want to use a setter method to set the value of insurance that is set depending on the value of cost how would you do this?

A little confused how to get this one to work

Thanks.

Show what you've tried, and describe how it could be better. "depending on" suggests the setter would take a parameter (most do) and then execute logic to set the value of insurance per the parameter.

 /**
    * Set insurance value of the football player
    */
   public void setInsuranceValue() 
   {    
      if (this.cost <= 10) {
         this.value = 5;
      } else if (this.cost >10 && this.cost <30) {
         this.value = 10;
      } else {
         this.value = 15;
      }
   }

The problem is once I set the cost value the value of value does not change.

Originally Posted by jamesh

 /**
    * Set insurance value of the football player
    */
   public void setInsuranceValue() 
   {    
      if (this.cost <= 10) {
         this.value = 5;
      } else if (this.cost >10 && this.cost <30) {
         this.value = 10;
      } else {
         this.value = 15;
      }
   }

The problem is once I set the cost value the value of value does not change.

Do you expect value to be recalculated each time you change cost? If so it's wrong. You need to add setInsuranceValue() call to method where you set cost.

Thanks worked straight away!