Prosty kalkulator - prośba o ocenę

1

Stworzyłem prosty kalkulator w javascript. Zdaję sobie jednak sprawę, że zapewne wiele rzeczy można było rozwiązać lepiej i prościej, zwłaszcza że to jeden z moich pierwszych projektów. Dlatego proszę o ewentualne wskazanie tych rzeczy, jak i wytknięcie wszelkich błędów związanych z funkcjonowaniem kalkulatora. Mam nadzieję, że zdołam wyciągnąć odpowiednie wnioski ze swoich błędów i wiele się dzięki temu nauczyć :)

kod: https://github.com/Crono2256/Calculator
live: https://crono2256.github.io/Calculator

2
  1. Usuń wszystkie console.log.
  2. Staraj się nie używać zmiennych globalnych.
  3. Spróbuj zrobić z tego plugin np. (wtedy jedyną "zmienną" globalną będzie funkcja createCalculator)
<div id="calculator"></div>
createCalculator(document.getElementById('calculator'));
  1. Zamiast switchy możesz używać np.
const operators = new Map([[43, 'add'], [45, 'substract']]);
const operator = operators.get(e.keyCode);
1

@Markuz:

Dzięki, wprowadziłem już poprawki do kodu, na postawie tego co napisałeś w punktach 1-3. Co do switchy, to przyznam, że nie chce mi się już tego wszystkiego od nowa pisać :D Ale w przyszłych projektach już na pewno wezmę tę sugestię pod uwagę :)

1 użytkowników online, w tym zalogowanych: 0, gości: 1