kalkulator w js

[Kartofelek]

Zle.
Pamiętaj o 2 rzeczach:
1) staraj się rozbijać funkcje tak by robiły konkretne zadania. Ale ten punkt nie jest bardzo wymagany
2) Podpinasz zdarzenie pod guzik. Po jego kliknięciu wywołujesz funkcję, która pobiera AKTUALNE dane, po czym wykonujesz przeliczenie i pokazanie wyniku.

Kod:

<form name="calc">
  <div>
    <input type="text" name="liczba1" id="liczba1"></input>
    <input type="text" name="liczba2" id="liczba2"></input>
    <input type="button" id="dodawanie" value="+" />
    <input type="button" id="odejmowanie" value="-" />
    <input type="text" name="wynik" id="wynik" />
  </div>
</form>

<script type="text/javascript">
  //dla nie powtarzania kodu, robię funkcję-selektor
  function $(id) {
     return document.querySelector(id);
  }

  function dodawanie(liczba1, liczba2) {
     return Number(liczba1) + Number(liczba2)
  }

  function odejmowanie(liczba1, liczba2) {
     return Number(liczba1) - Number(liczba2)
  }

  function pokazDodawanie() {
     var liczba1 = $('#liczba1').value;
     var liczba2 = $('#liczba2').value;
     console.log('s')
     $('#wynik').value = dodawanie(liczba1, liczba2);
  }

  function pokazOdejmowanie() {
     var liczba1 = $('#liczba1').value;
     var liczba2 = $('#liczba2').value;
     console.log('ss')
     $('#wynik').value = odejmowanie(liczba1, liczba2);
  }

  //podpinam zdarzenia
  $('#dodawanie').onclick = pokazDodawanie;
  $('#odejmowanie').onclick = pokazOdejmowanie;
</script>