Skocz do zawartości


Zdjęcie

[ajax/js] Licznik


  • Zaloguj się, aby dodać odpowiedź
1 odpowiedź w tym temacie

#1 logik_smoczek

logik_smoczek

    Początkujący

  • 75 postów

Napisano 14 05 2006 - 14:48

Jaki błąd zrobiłem w tym liczniku :?:
<script>





function Licznik () {

       var nazwa cookie = "Licznik"

       this.IleWizyt = 0;

       this.CzytajLicznik = function () {

          if (document.cookie.length>0)

          {

          poczatek=document.cookie.indexOf(nazwa cookie + "=")

          if (poczatek!=-1)

             {

                   poczatek = poczatek + naza_cookie.length + 1;

                   koniec = document.cookie.indexOf(";", poczatek)

                   if (koniec == -1)

                      koniec = document.cookie.length

                   return parseInt (unescape(document.cookie.substring(poczatek, koniec)));

             }

          }

          else

             return 0;

       }

       this.ZapiszLicznik = function () {

          this.IleWizyt +=1;

          waznosc = new Dete();

          waznosc.setTime(waznosc.getTime() + 1000 * 60 * 60 * 24 * 3650);

          document.cookie = nazwa_cookie + "=" + escape(this.IleWizyt) + ";expires="  + waznosc.toGMTString();

       }

       this.WyswietlLicznik = function () {

        document.write ("Odwiedziłe� tę strpnę " + this.IleWizyt+ "razy");

       }

       this.LicznikWStatusie= function () {

          windows.status = "Licznik odwiedzin: " +this..IleWizyt + " razy";

       }

       this.IleWizyt = this.CzytajLicznik ();

       this.ZapiszLicznik();

}

var L = new Licznik ();

L.WyswietlLicznik ();

L.LicznikWStatusie();

</script>


  • 0

#2 Kai

Kai

    Stały użytkownik

  • 237 postów

Napisano 31 03 2007 - 11:52

1. To nie Ty pisałeś, bo gdybyś to pisał Ty to umiałbyś wyszukać błąd/błędy :rolleyes:
2. Ten kto to pisał musi mieć albo dys* coś tam, albo być nieźle cofniętym :(
3. Chcesz kilka błędów? Proszę:
var nazwa cookie
...
poczatek=document.cookie.indexOf(nazwa cookie + "=")
...
poczatek = poczatek + naza_cookie.length + 1;
a) podejrzewam, że to miała być jedna zmienna (mówię o: nazwa cookie) xD Nawet przepisać nie potrafiłeś poprawnie?
waznosc = new Dete();
:( nie ma takiej funkcji wbudowanej w JS jak Dete(). Jak już to Date().
document.cookie = nazwa_cookie + "=" + escape(this.IleWizyt) + ";expires="  + waznosc.toGMTString();
c) o tu znowu inne wykorzystanie zmiennej zadeklarowanej na początku, tym razem nazwa_cookie. Graty. Radzę jednak używać ciągu znaków w zmiennych bez spacji, to taka dygresja na przyszłość :)
Reszty już mi się nie chciało sprawdzać, patrząc na ten bajeczny kod :(

  • 0

Zobacz więcej tematów z tagiem: liczniki



Użytkownicy przeglądający ten temat: 0

0 użytkowników, 0 gości, 0 anonimowych