Code-Review (JS,HTML,CSS) - Gra w kółko i krzyżyk

pytanie zadane 25 lipca 2016 w JavaScript przez Cyborek Użytkownik (850 p.)

Witam.

Proszę was o ocenę kodu gry w kółko i krzyżyk. Bardzo zależy mi na ocenie doświadczonych programistów, aby wyciągnąć błędy,poprawić swoje umiejętności i nauczyć się czegoś nowego Jestem otwarty na krytykę,opinie i pomoc innych. Dziękuje za poświęcony czas i Pozdrawiam.

Plik: https://drive.google.com/open?id=0B7SHTFx8ps4xekdiUXBVV201VWM

1 odpowiedź

odpowiedź 25 lipca 2016 przez Comandeer Guru (601,510 p.)

Jakie to wielkie! Nie mieści się na rozdziałce 1366x768. Musiałem pomniejszyć do 67%, żeby móc grać.
```
<meta http-equiv="X-UA-Compatible" content="IE=edge,chrome=1"/>
```
Polecam poczytać: https://github.com/CodersCommunity/CodeReview-HTML-01/tree/apache#poprawka
Masakryczne formatowanie kodu.
Zamiast [onload] użyj addEventListener.
Polecam poczytać o clearfixie → http://nicolasgallagher.com/micro-clearfix-hack/
section nie pasują do gry. Już szybciej div[role=application].

var p =new Array(9);
for(var e=0;e<8;e++)
{
	p[e]=2;
}

// vs

var p = Array( 9 ).fill( 2 );

Warto się zainteresować ES6 ;)

Wszystko w globalnym scope. Polecam poczytać o modułach w JS → https://addyosmani.com/writing-modular-js/

komentarz 25 lipca 2016 przez Cyborek Użytkownik (850 p.)

Dzięki za rady Comandeer i za poświęcony czas. Zapisałem sobie rady i przy następnym projekcie wykorzystam je.

komentarz 2 sierpnia 2016 przez niezalogowany

Czemu nie poprawisz obecnego? LOL!

komentarz 3 sierpnia 2016 przez Cyborek Użytkownik (850 p.)

A po co go teraz poprawiać z tym samym doświadczeniem. Mam go na komputerze tak jak resztę prac/eksperymentów :P Zawszę wracam, do swoich starszych kodów, ale dopiero wtedy gdy zgromadzę większą ilość doświadczenia (jak w RPG hah ).