Dynamically fill field with product calculation

[blg002]

All I am trying to do is dynamically fill the "Your Price" field with a number that is the product of the value of "acres" "regularity " and 10. The annoying part is I had it working yesterday and can't figure out what I did to break it.


Link..
http://www.gdarb.com/jj/estimate.html

Javascript...
[HTML]function autofill() {
document.estima te.price.value = "$" + document.estima te.acres.value * document.estima te.regularity.v alue * 10 + " per week";
}[/HTML]


HTML...

[HTML]<script type="text/javascript" src="_js/estimate.js"></script>


<form id="estimate" action="" method="get">
<fieldset>
<legend>Free (Rough) Estimate</legend><br />

<label for="acres">Acr es</label>
<input type="text" name="acres" id="acres" value="" size="5" maxlength="7" onkeyup="autofi ll()" /><br />

<label for="regularity ">Regularit y of Service</label>
<select size="1" id="regularity " name="regularit y" onchange="autof ill()">
<option value="1">Once a week</option>
<option value="2">Twice a week</option>
</select><br /><br />

<label for="price">You r Price</label>
<input type="text" name="price" id="price" value="" size="5" maxlength="20" readonly="reado nly" /><br />
</fieldset>
</form>[/HTML]

[gits]

hi ...

document.estimate 

doesn't exist ... i think its a typo ... you have to use:

document.forms.estimate

or much better, give the fields an id and use:

document.getElementById('field_id').value

kind regards

[acoder]

Changed thread title.

[acoder]

You already have ids for the fields, so in your autofill(), use:

document.getElementById("price").value = "$" + (document.getElementById("acres").value * document.getElementById("regularity").value * 10) + " per week";

[gits]

You already have ids for the fields, so in your autofill(), use:

document.getElementById("price").value = "$" + (document.getElementById("acres").value * document.getElementById("regularity").value * 10) + " per week";

ummm ... i missed to have a closer look ... when someone refers the form.elements-way i always assume in my head that he's not using ids ... ;))

kind regards