01-04-2013, 17:21
Witam. Właśnie zacząłem się uczyć js, ale mam problem z stworzeniem kalkulatora. Czy mógłby ktoś mi podpowiedzieć co powinienem poprawić w moim kodzie? Proszę o wyrozumiałość, i wytłumaczenie co jest źle dlaczego, i jak powinno wyglądać. tak łopatolgicznie jeśli można. w zasadzie we wszystkich językach zaczynam się gubić przy funkcjach. Próbuje to zrozumieć, ale i tak mi nie wychodzi. Może ktoś da radę mi wytłumaczyć. A o to kod:
<html>
<head>
<script type="text/javascript">
var rozwiazanie=top.document.calc.wynik.value;
var plusik=top.document.calc.wybor.plus;
var minusik=top.document.calc.wybor.minus;
var mnoznik=top.document.calc.wybor.mnozenie;
var dzielnik=top.document.calc.wybor.dzielenie;
function dodawanie(){var pluss1=top.document.calc.liczba1.value;
var pluss2=top.document.calc.liczba2.value;
var pluss=pluss1+pluss2;
return pluss;}
function odejmowanie(){var minuss1=top.document.calc.liczba1.value;
var minuss2=top.document.calc.liczba2.value;
var minuss=minuss1-minuss2;
return minuss;}
function iloczyn(){var iloczynn1=top.document.calc.liczba1.value;
var iloczynn2=top.document.calc.liczba2.value;
var iloczynn=iloczynn1*iloczynn2;
return iloczynn;}
function iloraz(){var ilorazz1=top.document.calc.liczba1.value;
var ilorazz2=top.document.calc.liczba2.value;
var ilorazz=ilorazz1/ilorazz2;
return ilorazz}
function licz(){
switch(top.document.calc.wybor){
case plusik: rozwiazanie=dodawanie();
return rozwiazanie;
break;
case minusik: rozwiazanie=odejmowanie();
return rozwiazanie;
break;
case mnoznik: rozwiazanie=iloczyn();
return rozwiazanie;
break;
case dzielnik: rozwiazanie=iloraz();
return rozwiazanie;
break;
}
}
</script>
</head>
<body>
<form name="calc" enctype="text/plain" action="" method="post">
<input type="number" name="liczba1" value="">
<input type="number" name="liczba2" value="">
<select name="wybor">
<option value="plus" selected>+</option>
<option value="minus">-</option>
<option value="mnozenie">*</option>
<option value="dzielenie">/</option>
</select>
<input type="button" value="oblicz" onclick="licz()">
<input type="number" name="wynik" value="">
</form>
</body>
</html>
<html>
<head>
<script type="text/javascript">
var rozwiazanie=top.document.calc.wynik.value;
var plusik=top.document.calc.wybor.plus;
var minusik=top.document.calc.wybor.minus;
var mnoznik=top.document.calc.wybor.mnozenie;
var dzielnik=top.document.calc.wybor.dzielenie;
function dodawanie(){var pluss1=top.document.calc.liczba1.value;
var pluss2=top.document.calc.liczba2.value;
var pluss=pluss1+pluss2;
return pluss;}
function odejmowanie(){var minuss1=top.document.calc.liczba1.value;
var minuss2=top.document.calc.liczba2.value;
var minuss=minuss1-minuss2;
return minuss;}
function iloczyn(){var iloczynn1=top.document.calc.liczba1.value;
var iloczynn2=top.document.calc.liczba2.value;
var iloczynn=iloczynn1*iloczynn2;
return iloczynn;}
function iloraz(){var ilorazz1=top.document.calc.liczba1.value;
var ilorazz2=top.document.calc.liczba2.value;
var ilorazz=ilorazz1/ilorazz2;
return ilorazz}
function licz(){
switch(top.document.calc.wybor){
case plusik: rozwiazanie=dodawanie();
return rozwiazanie;
break;
case minusik: rozwiazanie=odejmowanie();
return rozwiazanie;
break;
case mnoznik: rozwiazanie=iloczyn();
return rozwiazanie;
break;
case dzielnik: rozwiazanie=iloraz();
return rozwiazanie;
break;
}
}
</script>
</head>
<body>
<form name="calc" enctype="text/plain" action="" method="post">
<input type="number" name="liczba1" value="">
<input type="number" name="liczba2" value="">
<select name="wybor">
<option value="plus" selected>+</option>
<option value="minus">-</option>
<option value="mnozenie">*</option>
<option value="dzielenie">/</option>
</select>
<input type="button" value="oblicz" onclick="licz()">
<input type="number" name="wynik" value="">
</form>
</body>
</html>