おそらく始めたばかりで、何を含め、何を削除すればよいか分からないことは承知しています。ただし、ここでの良いアドバイスは、要素を理解できるように明確にラベルを付け、それを機能させるために必要な最小のコードに切り詰めることです (それを構築できるように、さらに少なくします)。
これがあなたのコードを作り直したものです:
HTML
<div>
<input type="text" id="multiplicand" value="4">
<input type="text" id="multiplier" value="10">
<button type="button" id="multiply">Multiply</button>
</div>
<p id="result">
The product is: <span id="product"> </span>
</p>
Javascript
window.onload = function(){
var button = el('multiply'),
multiplicand = el('multiplicand'),
multiplier = el('multiplier'),
product = el('product');
function el(id) {
return document.getElementById(id);
};
function multiply() {
var x = parseFloat(multiplicand.value) || 0,
y = parseFloat(multiplier.value) || 0;
product.innerHTML = x * y;
}
button.onclick = multiply;
};
http://jsfiddle.net/userdude/EptAN/6/
加算/減算/乗算/除算を使用した、もう少し洗練されたアプローチ:
http://jsfiddle.net/userdude/EptAN/9/