Ocena kalkulatora JS

Question

Proszę o kodu. Dopiero zaczynam z JS i muszę popracować nad optymalizacją w dużym stopniu.

Kalkulator ma również być obsługiwany przez klawiaturę numeryczną.

Link: https://jsfiddle.net/163tj40z/

Comments

Popraw walidacje. Chyba, że tak ma być. http://puu.sh/umqkK/b9cae7f30a.png

---

Do tego po użyciu nawiasów nie da się wykonać żadnej operacji.

---

Ok problem leży na evencie keydown a raczej jego walidacji dzięki wielkie.

Answer 1

• mieszana konwencja w nazewnictwie, proponuję camelCase, czyli typową jeśli chodzi o JS
• tablice możesz deklarować za pomocą literałów, raczej nie widzę praktycznego zastosowania deklarowania z góry konkretnej ilości elementów w pamięci, przynajmniej w tym wypadku
• fajnie by było, gdybyś wykorzystał event delegation zamiast iterować poszczególne elementy i przypisywać im po kolei obsługę zdarzeń
• brak możliwości wpisywania znaku z klawiatury to nieistniejąca funkcjonalność czy też bug?
• w wypadku zbyt dużej liczby znaków następuje overflow elementu, ogólnie liczbę tą możesz odgórnie ograniczyć
• purystycznym okiem, właściwość innerHTML możesz zastąpić textContent, oszczędzisz obliczeń przeglądarce i zapobiegniesz wstawieniu jakiegoś HTML'a w element

Comments

Dziękuję, za konkretne wskazanie wad. Postaram się je poprawić. 

Answer 2

Z tego co widzę to można wykonać tylko jedną operację , trochę słabo. Poczytaj o odwróconej notacji polskiej.

Answer 3

• Nie zakładaj, że metoda forEach istnieje na wyniku zwracanym przez document.querySelectorAll, bo jej tam nie powinno być (nie ma tego w standardzie).
• Marna ta obsługa klawiaturą, bo np. mój mak nie ma klawiatury numerycznej… W sumie coraz mniej komputerów ją ma.