[Code Review] Prosty kalkulator w js

pytanie zadane 6 sierpnia 2017 w Nasze projekty przez Cyborek Użytkownik (850 p.)
zmienione kategorie 7 sierpnia 2017 przez Cyborek

Witam

Stworzyłem prosty kalkulator w js i chciałbym poznać waszą opinie na jego temat. Głównie zależy mi na ocenie kodu co warto poprawić i jakie błędy popełniłem przy jego tworzeniu. Dziękuje za pomoc

Link do strony: https://cyborek.github.io/Kalkulator/

Link do repozytorium: https://github.com/Cyborek/Kalkulator

2 odpowiedzi

odpowiedź 7 sierpnia 2017 przez Comandeer Guru (600,330 p.)

<button class="calc-btn--function" type="button" value="*">*</button>

W BEM klasę z modyfikatorem zawsze nadaje się razem z klasą podstawową. Ogólnie to nie bardzo BEM, bo porozrywane są relacje pomiędzy blokami i elementami.

<p id="screen" class="calc-tile__score"></p>

A może output?

Po co stosujesz let skoro większość tych zmiennych nigdy się nie zmieni (jak choćby mainModule)? No i czemu nie stosujesz składni modułów ES?

Użyj event delegation, żeby nie przypinać tylu listenerów.

A samo liczenie też nie jest jakoś super sprawne. Wykonanie w ciągu działań typu 7*2-5 podaje jakiś wynik z kosmosu.

komentarz 7 sierpnia 2017 przez Cyborek Użytkownik (850 p.)

Dzięki Comandeer

Przy okazji chciałbym się Ciebie zapytać o materiały do nauki js. Nie chodzi mi o materiały dla początkujących, lecz materiały zaawansowane. Większość mojej pracy w javascript ograniczała się do przypisywania prostych zdarzeń na stronie, lecz ta wiedza powoli mi nie wystarcza i chciałbym podnieść swoje umiejętności na wyższy poziom.

Czy dobry rozwiązaniem byłoby dalej korzystać z poradników polskich i zagranicznych, czytać dokumentacje albo zakupić książkę (tutaj najbardziej wpadła mi w oko książka Twojego autorstwa) ?

1	komentarz 7 sierpnia 2017 przez Comandeer Guru (600,330 p.) Z polskich raczej nie ma jakichś mega zaawansowanych poradników, więc skupiłbym się na zagranicznej Sieci. A z książek to raczej też zagraniczne (chociaż, z oczywistych względów, swoją książkę polecam ;)).