Gra 2048 JavaScript

Question

Cześć,

Uczę się programowania w JavaScript i stworzyłem sobie grę 2048.

Kod niby działa jednak domyślam się iż nie jest napisany w sposób czysty, poprawny

Czy mogę prosić o wskazówki co jest nie halo?

Dodatkowo jak to poprawić aby gra odpalana na poniższej stronie działała jak oryginalna, czyli przesuwając palcem po ekranie przesuwają się klocki?

https://michalqq.github.io/

Answer 1

Okno sie scrolluje po nacisnieciu strzalki w dol. Tak to chyba dzialac nie powinno. Nie jest to błąd, ale te elseif z keyCode zamieniłbym na switch case. Wszystkie vary na const/let, mamy 2018 rok. Funkcja moveBoxesLeft jest straszna. Nie można tego wydzielić? W sumie wszystkie są straszne. Bardzo, bardzo dużo powtórzeń. Jak dla mnie kod nie do sprawdzenia, przynajmniej mi się nie chce zagłębiać w ten ogromny, jeden plik.

Co to ma oznaczać - var text =0.0  ?

ps. Wiadomo, że się uczysz. Powodzenia !

Comments

Dziękuję za uwagi. Na pewno postaram się ulepszyć moją wersję.

Answer 2

DOMy są stałe i można je pobrać tylko raz. var wyszło z mody. Teraz jest let oraz const. Ładniej jest for(let i = 0; ...). Wygląd gry jest ble. Podpowiedź jak grać. Wystarczy grafika strzałek. Od nowa zawsze ustawia liczby tak samo co strasznie nudzi. I to document.getElementById(k + "x" + (i+1)) jeżuuu... Wrzuć te pola w tablicę. Czasami masz pojedyncze, a innym razem podwójne ciapki. text=text + 0.1 można zapisać łatwiej text += 0.1, ale i to nie jest bezpieczne bo operujesz na liczbach zmiennoprzecinkowych. Czasami masz nie równe formatowanie bo używasz spacji zamiast tabulacji. Zauważ, że przy tabulacji jak przeskoczysz za daleko to widać to dokładnie.

Comments

Dziękuję za uwagi. Na pewno postaram się ulepszyć moją wersję.