[js]Kółko i krzyżyk Opcje

[Glarden]

Robię grę w kółko i krzyżyk. Próbowałem zrobić, żeby po kliknięciu przez użytkownika danego pola, jego zawartość zmieniała się na obrazek z krzyżykiem, ale coś nie działa. Kody wyglądają tak:
fragment (jedno pole) tabelki z 9 polami do gry:

[HTML] pobierz, plaintext 
<td class='gra' id='1,1' style='border: 1px solid black;' height='15%' width='10%'><a href="java script: zaznacz(1,1);" onClick="zaznacz(1,1);"><img src='nic.gif'></a></td>
[HTML] pobierz, plaintext 

Nie byłem pewien, jak wywołuje się funkcję przez odnośnik, więc użyłem obu sposobów:)
A tutaj funkcja JS:

[HTML] pobierz, plaintext 
function zaznacz (pole) {
	co = getElementById(pole);
	co.innerHTML = "<img src='krzyzyk.gif'>";
}
[HTML] pobierz, plaintext 

Po kliknięciu w pole nie ma żadnej reakcji. Gdzie są błędy?

[abc667]

, oddziela dwa argument w funkcji, nie możesz go używać w ten sposób, np po prostu go usuń, w tym wypadku to wiele nie zmienia

dodatkowo getElementById musisz wykonywać z document.
document.getElementById

[Glarden]

Racja! Wszystko działa, dzięki. Chodziło o "document.getElementById".

Ale mam kolejny problem. Kontynuowałem w pocie czoła pracę nad grą i napisałem funkcję dodającą kółko sekundę po tym, jak gracz wybierze miejsce na krzyżyk. Problem w tym, że skrypt jest jakiś kapryśny i czasem wstawia kółko w losowym miejscu, a czasem nie. Tzn., zdarza się, że po skończonej grze na planszy jest jedno kółko i osiem krzyżyków, zdarza się też, że są cztery kółka i cztery krzyżyki. Oto skrypt:

[HTML] pobierz, plaintext 
wolne = new Array();
wolne[0] = '11';
wolne[1] = '12';
wolne[2] = '13';
wolne[3] = '21';
wolne[4] = '22';
wolne[5] = '23';
wolne[6] = '31';
wolne[7] = '32';
wolne[8] = '33';
 
function find_index(t, v){
  for(var i=0,n=t.length;i<n;++i)
    if(t[i]==v)
      return i
  return null
}
 
function zaznacz (pole) {
	co = document.getElementById(pole);
	co.innerHTML = "<img src='krzyzyk.gif'>";
	z = find_index(wolne, pole); //znajduje indeks pola o wartości pole
	delete (wolne[z]); //kasuje z tablicy 'wolne'
	wolne.sort();
	s = Math.round(wolne.length*Math.random()); //losowa liczba od 0 do ilości elementów tablicy
	ID=setTimeout("zaz_komp (wolne[s]);", 1000); //zaz_komp po sekundzie
}
 
function zaz_komp (pole) {
	co = document.getElementById(pole);
	co.innerHTML = "<img src='kolko.gif'>";
	z = find_index(wolne, pole);
	delete (wolne[z]); //usuwa z tablicy 'wolne'
	wolne.sort(); //sortuje
}
[HTML] pobierz, plaintext 

Wiem, że nadłożyłem kodu pisząc tablicę, ale nie o to teraz chodzi.

[abc667]

[HTML] pobierz, plaintext 
<script type="text/javascript">
wolne = new Array();
wolne[0] = '11';
wolne[1] = '12';
wolne[2] = '13';
wolne[3] = '21';
wolne[4] = '22';
wolne[5] = '23';
wolne[6] = '31';
wolne[7] = '32';
wolne[8] = '33';
 
function find_index(t, v){
for(var i=0,n=t.length;i<n;++i)
if(t[i]==v)
return i
return null
}
 
function zaznacz (pole) {
co = document.getElementById(pole);
co.innerHTML = "krzyzyk.gif";
z = find_index(wolne, pole); //znajduje indeks pola o wartości pole
delete (wolne[z]); //kasuje z tablicy 'wolne'
wolne.sort();
wolne.length-=1;
s = Math.round((wolne.length-1)*Math.random()); //losowa liczba od 0 do ilości elementów tablicy
 
if(wolne.length > 0)
{
ID=setTimeout("zaz_komp(wolne[s]);", 1000); //zaz_komp po sekundzie
}
 
}
 
function zaz_komp (pole) {
co = document.getElementById(pole);
co.innerHTML = "kolko.gif";
z = find_index(wolne, pole);
delete (wolne[z]); //usuwa z tablicy 'wolne'
wolne.sort(); //sortuje
wolne.length-=1;
}
</script>
[HTML] pobierz, plaintext 

[Glarden]

Rzeczywiście, działa. Jaka szkoda, że sam do tego nie doszedłem... tak czy inaczej - dzięki.