Code Review - TicTacToe

Powiecie mi o wszystkich złych nawykach jakie tu zastosowałem? Chciałbym żebyście nie pozostawili po mnie suchej nitki. Wiem na pewno że pierwszy błąd to zmienne globalne tylko nie bardzo wiem jak ich w modułach bo inne funkcje ich nie widzą. 



var turaa = 1; 
var znak = "";
var znak1= "X";
var znak2 = "O";
var licznik = 0;

//wypisywanie pol X lub O, turowosc-----------------------------------------------------------------------
function podmien(nr)
{
	if(turaa == 1) {
		znak = znak1;
		turaa--;
		document.getElementById("tura").innerHTML = "Tura gracza O"
	}
 else  if(turaa == 0)
	{
		znak = znak2;
		turaa++;
		document.getElementById("tura").innerHTML = "Tura gracza X"
	}
	var myid = "pole" + nr;
	document.getElementById(myid).innerHTML = znak;
	document.getElementById(myid).setAttribute("onclick",";");
	document.getElementById(myid).style.cursor = "default";	
	document.getElementById(myid).onmouseover = function() {
    this.style.backgroundColor = "coral";}
	licznik++;
	sprawdz();
}

//sprawdzanie wygranej----------------------------------------------------------------------------------
function sprawdz()
{
	var uchwyt = document.getElementsByClassName("box");
	
var p1 = uchwyt[0].innerHTML;   var p2 = uchwyt[1].innerHTML;   var p3 = uchwyt[2].innerHTML;
var p4 = uchwyt[3].innerHTML;   var p5 = uchwyt[4].innerHTML;   var p6 = uchwyt[5].innerHTML;
var p7 = uchwyt[6].innerHTML;   var p8 = uchwyt[7].innerHTML;   var p9 = uchwyt[8].innerHTML;

//sprawzdanie wygranej dla O
	if((p1==znak2&&p2==znak2&&p3==znak2)||(p4==znak2&&p5==znak2&&p6==znak2)||(p7==znak2&&p8==znak2&&p9==znak2)||(p1==znak2&&p4==znak2&&p7==znak2)||(p2==znak2&&p5==znak2&&p8==znak2)||(p3==znak2&&p6==znak2&&p9==znak2)||(p1==znak2&&p5==znak2&&p9==znak2)||(p3==znak2&&p5==znak2&&p7==znak2))
	{
		document.getElementById("wygrana").innerHTML = "<a href=\"index.html\">Wygrał O! Jeszcze raz?</a>";
		koniec();
	}
//sprawzdanie wygranej dla X
	if((p1==znak1&&p2==znak1&&p3==znak1)||(p4==znak1&&p5==znak1&&p6==znak1)||(p7==znak1&&p8==znak1&&p9==znak1)||(p1==znak1&&p4==znak1&&p7==znak1)||(p2==znak1&&p5==znak1&&p8==znak1)||(p3==znak1&&p6==znak1&&p9==znak1)||(p1==znak1&&p5==znak1&&p9==znak1)||(p3==znak1&&p5==znak1&&p7==znak1))
	{
		document.getElementById("wygrana").innerHTML = "<a href=\"index.html\">Wygrał X! Jeszcze raz?</a>";
		koniec();
	}
//sprawdzanie remisu---------------------------------------------------------------------------------------
	if(licznik == 9) 
		{
		document.getElementById("wygrana").innerHTML = "<a href=\"index.html\">Remis! Jeszcze raz?</a>";
		koniec();
	}
}
//co ma sie stac po wygranej------------------------------------------------------------------------
function koniec()
{
		document.getElementById("pole1").setAttribute("onclick",";");
		document.getElementById("pole2").setAttribute("onclick",";");
		document.getElementById("pole3").setAttribute("onclick",";");
		document.getElementById("pole4").setAttribute("onclick",";");
		document.getElementById("pole5").setAttribute("onclick",";");
		document.getElementById("pole6").setAttribute("onclick",";");
		document.getElementById("pole7").setAttribute("onclick",";");
		document.getElementById("pole8").setAttribute("onclick",";");
		document.getElementById("pole9").setAttribute("onclick",";");
		
		document.getElementById("pole1").onmouseover = function() {
		this.style.backgroundColor = "coral";}
		document.getElementById("pole2").onmouseover = function() {
		this.style.backgroundColor = "coral";}
		document.getElementById("pole3").onmouseover = function() {
		this.style.backgroundColor = "coral";}
		document.getElementById("pole4").onmouseover = function() {
		this.style.backgroundColor = "coral";}
		document.getElementById("pole5").onmouseover = function() {
		this.style.backgroundColor = "coral";}
		document.getElementById("pole6").onmouseover = function() {
		this.style.backgroundColor = "coral";}
		document.getElementById("pole7").onmouseover = function() {
		this.style.backgroundColor = "coral";}
		document.getElementById("pole8").onmouseover = function() {
		this.style.backgroundColor = "coral";}
		document.getElementById("pole9").onmouseover = function() {
		this.style.backgroundColor = "coral";}
		
		document.getElementById("pole1").style.cursor = "default";
		document.getElementById("pole2").style.cursor = "default";
		document.getElementById("pole3").style.cursor = "default";
		document.getElementById("pole4").style.cursor = "default";
		document.getElementById("pole5").style.cursor = "default";
		document.getElementById("pole6").style.cursor = "default";
		document.getElementById("pole7").style.cursor = "default";
		document.getElementById("pole8").style.cursor = "default";
		document.getElementById("pole9").style.cursor = "default";
}