Radio buttons change variable in JavaScript

Question

I have two fields which calculate a total, but I have a radio button pair that determines a variable within the equation. Here's the radio button code:

<input type="radio" name="estimate_pool_shape" value="1" /> Angle
<input type="radio" name="estimate_pool_shape" value=".86392" /> Freeform

Immediately after that, this JavaScript is then calculating the surface area of the pool based of these fields and script:

    (Length <input type="text" size="5" name="estimate_pool_length" 
onBlur="conv2inches(document.forms.mainform);" />) x 
    (Width <input type="text" name="estimate_pool_width" size="5" 
onBlur="conv2inches(document.forms.mainform);" />)

The following JavaScript already works to update the fields, I just need the variable it uses to set the field values to change based on the radio buttons.

        function conv2inches(mainform) {            

        var oin;    
        var oinches;

        if(isNaN(mainform.estimate_pool_width.value)||isNaN(mainform.estimate_pool_length.value)){  
            alert("Please enter numbers only"); 
            return false;   
        }

        oin = ((mainform.estimate_pool_width.value) + (mainform.estimate_pool_length.value) * %%%RADIO BUTTON VARIABLE HERE%%%);    
        oinches = (oin);    

        if(oinches==0){ 
            alert("Please enter valid values into the boxes");  
        }   
            mainform.estimate_pool_sqft.value = oinches;    
            return false;
        }

((mainform.estimate_pool_width.value) + (mainform.estimate_pool_length.value) * (mainform.estimate_pool_shape.value)) bugs out — dmanexe
– dmanexe, Commented Apr 2, 2010 at 17:10

Tim · Accepted Answer · 2010-04-02 18:57:36Z

1

var estimate_pool_shape = 0; // Or some default value.  Maybe 1?
for(var i = 0; i < mainform.estimate_pool_shape.length;i++){
 if(mainform.estimate_pool_shape[i].checked){
   estimate_pool_shape = mainform.estimate_pool_shape[i].value;
   break;
 }
}

Should give you what you need.

edited Apr 2, 2010 at 18:57

answered Apr 2, 2010 at 17:21

Tim

2,43919 silver badges16 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

Tim Over a year ago

Above the line with %%%RADIO BUTTON VARIABLE HERE%%% Then replace %%%RADIO BUTTON VARIABLE HERE%%% with estimate_pool_shape

dmanexe Over a year ago

Returns NaN to the target field

Tim Over a year ago

Sorry I made some mistakes in the code above, I'll edit it so its right.

Collectives™ on Stack Overflow

Radio buttons change variable in JavaScript

1 Answer 1

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related