[PHP]Licznik, Co jakiś czas się zeruje Opcje

[ziom]

Napisałem sobie dość prosty licznik na plikach, ale jest problem, bo co jakiś czas przeskakuje mi w dół. Nie jestem w stanie powiedzieć, czy do zera, ale raczej nie.

[PHP] pobierz, plaintext 
<?
class LICZNIK 
{ 
   static private $odw; 
 
   static private $file; 
 
   public function __construct($var) 
   { 
	  self::$file = $var; 
	  self::$odw = self::get(); 
	   
	  setcookie('licznik',true,0); 
	   
	  self::add(); 
   } 
 
   public function odw() 
   { 
	  return self::$odw; 
   } 
 
   private function add() 
   { 
	  if(!isset($_COOKIE['licznik'])) 
	  { 
		 if(!file_exists(self::$file)) 
		 { 
			file_put_contents(self::$file, '1'); 
		 } 
		 else 
		 { 
			++self::$odw; 
			file_put_contents(self::$file,self::$odw); 
		 } 
	  } 
   } 
 
   private function get() 
   { 
	  if(!file_exists(self::$file)) 
	  { 
		 return '1'; 
		 file_put_contents(self::$file, '1'); 
	  } 
	  else 
	  {		  
		 return file_get_contents(self::$file); 
	  } 
   } 
 
}
?>
[PHP] pobierz, plaintext 

Gdzie popełniłem błąd?

Ten post edytował ziom 26.05.2007, 11:11:53

[slaug]

Ja bym proponował zmienić Twoją klasę na coś takiego i ew. poniższe dopracować:

[PHP] pobierz, plaintext 
<?php
class Licznik
{ 
   private $file; 
 
   public function __construct($var) 
   { 
	  $this->file = $var;
	  if(!file_exists($this->file)) 
	  { 
			file_put_contents($this->file, '1');
	  } 
	  setcookie(md5($_SERVER['HTTP_HOST'] . 'licznik'),true,0); 
	  $this->add(); 
   } 
 
   public function get() 
   { 
	  return file_get_contents($this->file); 
   }
   
   private function add() 
   { 
	  if(!isset($_COOKIE[md5($_SERVER['HTTP_HOST'] . 'licznik')])) 
	  { 
		 file_put_contents($this->file, $this->get() + 1); 
	  } 
   } 
}
 
$licznik = new Licznik('test');
echo $licznik->get();
?>
[PHP] pobierz, plaintext 

Kilka uwag dot. Twojej klasy:

• Nazywanie ciacha 'licznik' moze spowodować że nie będą naliczane wejścia użytkownika, który był na stronie gdzie webmaster pomyślał tak jak ty, albo gdzie używana jest twoja klasa

• [PHP] pobierz, plaintext 
  <?php
  return '1'; 
  		 file_put_contents(self::$file, '1');
  ?>
  [PHP] pobierz, plaintext 

  
  to file_put_contents ci się nie wywoła bo jest po return
• nie ma potrzeby używania zmiennych statycznych w twojej klasie
• w metodzie add nie musisz ponownie sprawdzać czy plik istnieje skoro zrobiłeś to przed chwilą w metodzie get, którą wywołałeś wcześniej. Ewentualnie możesz zakończyć działanie jeśli zapisanie do pliku się nie udało.
• klas raczej nie nazywa się dużymi literami

Niestety nie udało mi się powtórzyć błędu, o którym piszesz, a na pierwszy rzut oka nie widzę co mogłoby spowodować wyczyszczenie licznika. Spróbuj z wersją, którą podałem wyżej. Raczej powinna działać dobrze. W sumie jedyna możliwość kiedy wyzeruje się licznik w tej wersji to brak pliku.

[nexis]

Nazywanie ciacha 'licznik' moze spowodować że nie będą naliczane wejścia użytkownika, który był na stronie gdzie webmaster pomyślał tak jak ty, albo gdzie używana jest twoja klasa

Co za bzdura! Ciasteczko jest ustawiane dla konkretnej strony i tylko ona ma do niego dostęp (prawo do odczytu, zapisu oraz usuwania), więc nie ma takiej możliwości, aby inna strona je nadpisała używając tej samej nazwy.

[kwiateusz]

nie strony a domeny, o ile w parametrach nie poda się inaczej więc jak nexis napisał ciacha nie da sie nadpisać z poziomu innej witryny

[nexis]

nie strony a domeny

Chciałem to określić jasno, żeby nikt nie pomyślał, że dla domeny wszystkie ciasteczka są wspólne, bo subdomeny są traktowane jak oddzielne domeny.