Ocena kodu JavaScript

Question

Cześć, czy mógłbym prosić o zerknięcie na kod JavaScript i porady co poprawić, czy struktura plików jest dobra itd.

Kod umieściłem na GitHub, link:
https://github.com/mat89c/galaxy_shooter

demo:
http://galaxy-shooter.czajka-mateusz.pl/

Z góry dziękuję za uwagi.

Comments

#start:focus {

outline:none;

}

bo widać granice buttona podczas startu 

---

> widać granice buttona podczas startu

I bardzo dobrze.

---

średnio to wygląda

---

Dzięki za uwagę. To w sumie detal, bardziej chciałem poradzić się o kod JavaScript, ale dzięki za komentarz.

---

Niemniej przycisk powinien reagować na focus klawiaturą i kliknięcie. Obecnie tego nie robi.

Ogólnie jeśli chodzi o gry, to WCAG jest średnio przydatny: http://gameaccessibilityguidelines.com/

---

Witam. Byłbym wdzięczny jeśli mógłby Pan zerknąć i ocenić kod lub stronę. Nie ukrywam że wiele informacji odnośnie kodu podpatrywałem z Pana strony https://blog.comandeer.pl/ oraz tutoriala https://tutorials.comandeer.pl/html5-blog.html.

---

Wieczorem rzucę okiem. I żaden Pan ;)

Answer 1

• https://github.com/mat89c/galaxy_shooter/blob/3f0b2436fb62c1550f92eba1c8322136ba616ebd/package.json#L2 – o ile mnie pamięć nie myli, nazwy z podkreślnikami są nieprawidłowe w npm.

• https://github.com/mat89c/galaxy_shooter/blob/3f0b2436fb62c1550f92eba1c8322136ba616ebd/package.json#L6-L8 – fajnie byłoby mieć jakieś bardziej dopasowane do projektu skrypty (build, test).

• No i raczej powinno być jeszcze pole private: true.

• https://github.com/mat89c/galaxy_shooter/blob/3f0b2436fb62c1550f92eba1c8322136ba616ebd/package.json#L4 – nie widzę Babylon.js w dependencies.

• https://github.com/mat89c/galaxy_shooter/blob/3f0b2436fb62c1550f92eba1c8322136ba616ebd/src/index.js#L8 – jeśli używasz modułów ES, to nie ma potrzeby tworzenia IIFE. W modułach żadna zmienna nie wycieka do globalnej przestrzeni nazw. Dodatkowo każdy moduł jest domyślnie w trybie ścisłym, więc 'use strict' też nie jest potrzebne. A wgl to czemu var a nie const?

• https://github.com/mat89c/galaxy_shooter/blob/3f0b2436fb62c1550f92eba1c8322136ba616ebd/src/index.js#L1-L3 – komentarz powinien być inaczej wcięty:

   /**
    *@author Czajka Mateusz <mateusz.czajka89@gmail.com>
    */

• https://github.com/mat89c/galaxy_shooter/blob/3f0b2436fb62c1550f92eba1c8322136ba616ebd/src/game/Asteroids.js#L10 – czemu to nie jest klasa?

• https://github.com/mat89c/galaxy_shooter/blob/3f0b2436fb62c1550f92eba1c8322136ba616ebd/src/game/Asteroids.js#L27 – przy każdym wywołaniu Asteroids nadpisujesz prototyp. Nie ma to sensu.

• https://github.com/mat89c/galaxy_shooter/blob/3f0b2436fb62c1550f92eba1c8322136ba616ebd/src/game/Asteroids.js#L28 – jeśli nie zmieniasz w trakcie życia programu przypisania do danej zmiennej, lepiej użyć const.

• Ogólnie widziałbym raczej Asteroids jako kolekcję obiektów Asteroid.

• Dodatkowo nie widzę za bardzo potrzeby robienia ze wszystkich klas singletonów.

• https://github.com/mat89c/galaxy_shooter/blob/3f0b2436fb62c1550f92eba1c8322136ba616ebd/src/game/Enemy.js#L7-L11 – tutaj ładnie sprawdziłby się destructuring.

• https://github.com/mat89c/galaxy_shooter/blob/3f0b2436fb62c1550f92eba1c8322136ba616ebd/src/game/Helpers.js – to spokojnie można rozegrać jako moduł z named exports. Nie ma potrzeby, by pakować to do klasy.

• Ogólnie mam wrażenie, że cała logika gry jest ściśle powiązana z renderowaniem na ekranie. Osobiście bym te dwa aspekty całkowicie rozdzielił. Renderowanie powinno jedynie odzwierciedlać to, co wylicza logika gry.

Comments

Dziękuję za tak dokładną analizę. To dla mnie bardzo cenne wskazówki, które pomogą mi pisać lepszy kod. Siadam do poprawiania :)

Answer 2

Można przelecieć przez kamień nie :) dodatkowo pojawia się napis zawróć ale nie wiemy kiedy konczy sie plansza można by wprowadzić jakiś radar na dole strony pokazujacy pozycje i koniec strony to jeśli chodzi o funkcjonalność

Comments

W wersji gry na stronie głównej https://czajka-mateusz.pl/, jest zaimplementowana kolizja naszego statku z asteroidami. Ale boty mogą przelecieć, ponieważ przy sprawdzaniu kolizji również dla botów, ilość fpsów znacznie spada. Co do granicy świata i radaru to kiedyś może dodam. Dzięki za uwagę.