Zmienne globalne - chyba tutaj tkwi problem Opcje

[matewka]

Witam wszystkich serdecznie.
Od jakiegoś czasu głowię się z problemem, który podejrzewam, że dotyczy zmiennych globalnych. Mam taki skrypt:

[HTML] pobierz, plaintext 
  var upjoint = new array(0,0,0,0,0,0,0);
  var downjoint = new array(0,0,0,0,0,0,0);
 
  function zaznaczGodz(r,c) {
    c_int = parseInt(c);
    if(upjoint[c_int]==0) {
      $('#r-'+r+'-c-'+c).attr('bgcolor', '#f00');
      upjoint[c_int] = 1;
    }
    else {
      ...
    }
  }
[HTML] pobierz, plaintext 

Funkcja zaznaczGodz() jest wywoływana z atrybutu 'onclick' każdej z 70 komórek w pewnej tabeli. Skrypt po prostu nie działa (nie zmienia koloru). Podejrzewam, że problem tkwi gdzieś w zmiennej upjoint, bo próbowałem po kolei usuwać poszczególne elementy funkcji i, jeśli np. w warunku IF'a wpiszę samo 'c_int!=0' to funkcja działa. Co robić?

Ten post edytował matewka 15.05.2011, 21:48:34

[melkorm]

java script:

1. Zamieść cały kod - a najlepiej go nam udostępnij online,
2. FireBug i jego konsola błędów,
3. Co chcesz uzyskać,
4. Patrz punkt 1.

[matewka]

Ad.1. Cały kod to tylko prosta tabelka 7x10 + JS, który zamieściłem powyżej. Ale proszę bardzo LINK
Ad.2. Używam Chrome, który ma wbudowane takie coś, jak FireBug.
Ad.3. Tworzę tabelkę dla lekarzy przedstawiającą dni tygodnia (w kolumnach) i godziny (w wierszach). Chcę, żeby lekarze mogli sobie ustawiać godziny pracy za pomocą klikania w konkretną godzinę - od której do której chcą pracować. Potrzebuję do tego globalnych tablic, które będą miały ilość pól tyle, co "c" (cell), a wartość danego pola tyle, co "r" (row). Skrypt, który oglądasz jest bardzo uproszczony. Konkretny mam w pracy, ale tam będę dopiero jutro... Jednak nie ma to żadnego znaczenia, bo ten tutaj też nie działa.
Ad.4. Patrz Ad.1.

Ten post edytował matewka 15.05.2011, 22:06:30

[Kemsan]

"$ is not defined [Wstrzymuj na tym błędzie] $('#r-'+r+'-c-'+c).attr('bgcolor', '#f00'); " Popraw przykład, bo ściąga jQuery z localhost'a

[matewka]

już mniejsza z tym. Doszedłem, o co chodziło. debuger pokazał mi, że nie mogę zdefiniować tablicy w ten sposób:

[HTML] pobierz, plaintext 
var upjoint = new array(0,0,0,0,0,0,0);
[HTML] pobierz, plaintext 

Przetrzepałem gugla i znalazłem takie coś:

[HTML] pobierz, plaintext 
var upjoint = [0,0,0,0,0,0,0];
[HTML] pobierz, plaintext 

Działa!