Code-Review - Kółko i krzyżyk

Question

Cześć

Jeśli masz czas i chęć to proszę o ocenienie mojego kodu gry w kółko i krzyżyk. Jestem ciekaw Twojej opinii. 
Jest to mój jeden z pierwszych projektów więc jestem świadomy że popełniłem wiele błędów, dlatego zwracam się do ciebie doświadczony użytkowniku forum o pomoc. Dziękuje za poświęcony czas. 
Pozdrawiam 

Pliki: https://www.dropbox.com/sh/wu4k9avduaxo0ty/AADk4SPOiQb9PIwPseBx7d7ta?dl=0

GitHub: https://github.com/MarianTyle/kolko_i_krzyzyk

Comments

Jeśli masz czas i chęć, ogarnij gita. Naprawdę to 5 poleceń. W zamian za to wszystkie projekty w jednym miejscu, historia edycji kodu, i łatwy dostęp do wszystkiego.

---

Okej, poczytałem trochę o git i zacznę teraz tego używać dziękuje. Dodałem już link do GitHub w pytaniu, mam nadzieje że jest w porządku.

Answer 1

• Wywal bzdurne [onclick] z kodu (zwłaszcza generowanego przez JS!) i przypnij wszystkie handlery przez JS.
• W sumie generowanie kodu mógłbyś przerzucić do HTML-a i zastosować np. trick z .no-js.

• document.getElementById(pole).onclick = nic();

  Tutaj do obsługi kliku przypisujesz undefined a nie pustą funkcję (najpierw następuje wywołanie tej funkcji a zwrócona wartość jest przypisywana).

• Problem globalnych zmiennych rozwiąże IIFE.

W sumie to Ci przygotuję PR ;) W najbliższym czasie napiszę przykładową implementacją kółka i krzyżyka w ES6.

Comments

Ostatnio, robiłem AI do tic-tac-toe w ES6, możesz rzucić okiem :)

https://github.com/pietrzakacper/tic-tac-toe-ai

Answer 2

if(ruch%2==0)
{
}
else if(ruch%2!=0)
{
}

Trochę zabawne, taka "oczywista oczywistość".

funkcja sprawdz ma za dużo zagnieżdżeń.

Answer 3

x = "";
y = "";

To są domniemane zmienne globalne. Zapomniałeś o var, a to zła praktyka.

setTimeout("sprawdz_remis()",1000);

Zamiast tego lepiej:

setTimeout( function(){ sprawdz_remis(); }, 1000);

Za dużo zmiennych globalnych.  

Answer 4

Nie wiem czy dobrze wszystko zrozumiałem i czy dobrze zrobiłem, niemniej uporałem się chyba z podanymi błędami :)  Jak uważasz?

https://github.com/MarianTyle/kolko_i_krzyzyk/tree/poprawka_2

Comments

a to: onload="poczatek()", onClick="start()", onClick="stworz()", czemu nie użyłeś addEventListener?

A takie cudo: float="right" valign="middle" w obrazkach, to skąd?