Zamulałem się na wykładach z flasha więc napisałem sobie klasę liczącą wejścia na stronę. Co o niej sądzicie?

[PHP] pobierz, plaintext 
<?php
//wersja pod php 4
class licznik {
	var $file;
	var $datatab;
	var $dzis;
 
	function licznik() { //pobierz info
		$this->file = 'licz.slm';
		$handle = @fopen($this->file, 'a+');
		if (!file_exists($this->file)) {
			 print 'Brak pliku '.$this->file.'!';
		}
		else {
			$data = @fread($handle, filesize($this->file));
			$this->datatab = explode(' ', $data);
		}
		@fclose($handle);
	}
 
	function ustaw() {
		if ($this->datatab[2] != date("w")){
			$this->datatab[1] = $this->datatab[0];
			  $this->datatab[0] = 0;
		}
		if ($this->datatab[5] != date("n")) {
			  $this->datatab[4] = 0;
		}
		if ($this->datatab[2] == 0 && $this->datatab[7] == 0) {
			$this->datatab[6] = 0;
			$this->datatab[7] = 1;
		}
		else {
			if ($this->datatab[2] == 0) {
				$this->datatab[7] = 1;
			}
			elseif ($this->datatab[2] != 0) {
				$this->datatab[7] = 0;
			}
 
		}
 
		$this->datatab[0]++;
		$this->datatab[3]++;
		$this->datatab[4]++;
		$this->datatab[6]++;
 
		$handle = @fopen($this->file, 'w');
		$array = array($this->datatab[0], $this->datatab[1], date("w"), $this->datatab[3], $this->datatab[4], date("n"), $this->datatab[6], $this->datatab[7]);
		$impl = implode(' ', $array);
		@fwrite($handle, $impl);
		@fclose($handle);
	}
 
	function wyswietl() {
		  $br = nl2br("n");
		print 'Dzis: '.$this->datatab[0].$br; //dzis
		print 'Wczoraj: '.$this->datatab[1].$br; //wczoraj
		print 'Ten tydzien: '.$this->datatab[6].$br; //tydzien
		print 'Ten miesiac: '.$this->datatab[4].$br; //miesiac
		print 'Wszystkich: '.$this->datatab[3].$br; //calosc
	}
 
	function cookieProtect() {
		setcookie('prtct', md5('1'), time()+3600*24*31);
		if ($_COOKIE['prtct'] == md5('1')) {
			  return false;
		}
		else {
			  return true;
		}
	  }
}
 
$objLicznik = new licznik();
if ($objLicznik->cookieProtect()) {
  $objLicznik->ustaw();
}
$objLicznik->wyswietl();
 
 
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
$handle = @fopen($this->file, 'a+');
		if (!file_exists($this->file)) {
			 print 'Brak pliku '.$this->file.'!';
		}
		else {
			$data = @fread($handle, filesize($this->file));
			$this->datatab = explode(' ', $data);
		}
		@fclose($handle);
?>
[PHP] pobierz, plaintext 

Czemu najpierw otwierasz plik, potem sprawdzasz czy istnieje?

Gdzie kasujesz odwiedziny poprzedniego dnia?

Ups faktycznie
nie kasuje odwiedzin poprzedniego dnia tylko zastępuje wynikami z tego dnia:
$this->datatab[1] = $this->datatab[0];