Witam!

Otórz mam na stronie licznik odwiedzin i już drugi raz sam się wyzerował (plik z liczbą odwiedzin stał się pusty) :-/

Podaję swój skrypt, bo ja już nie wiem gdzie leży przyczyna :-/
CHMOD mam ustawiony pliku z wpisami licznika (licznik.dat) na 777.

Plik licznik.php (inkludowany: @include("index/licznik.php"); )

[PHP] pobierz, plaintext 
<?php
 
$plik = "index/licznik.dat";
$waga = @filesize($plik);
 
$fop = @fopen("$plik", "r");
@flock($fop, 1);
$czytaj_l = @fread($fop, $waga);
@flock($fop, 3);
@fclose($fop);
 
$czytaj_l++;
 
$fop = @fopen("$plik", "w");
@flock($fop, 2);
@fwrite($fop, $czytaj_l);
@flock($fop, 3);
@fclose($fop);
 
echo("$czytaj_l");
 
?>
[PHP] pobierz, plaintext 

Proszę o podpowiedzi jaka może być przyczyna samoistnego zerowania się tego licznika...
Najciekawsze jest to, że dałem blokadę flock() do zapisu i odczytu a i tak się wyzerował :-/

Pozdrawiam

a sproboj tak:

[PHP] pobierz, plaintext 
<?php
$czytaj_l = @fread($fop, $waga+1);
?>
[PHP] pobierz, plaintext 

mi kiedys to pomoglo

--------------
jak to nie pomoze to daj echo co jakis czas zeby zobaczyc co sie dzieje ze zmienna

Znów dziś mi się sam wyzerował licznik mimo, że zrobiłem jak mi tu poradzono :-(



Sprawdziłem i zmienne zachowują się w prawidłowy sposób....

Czy to nie może być skutkiem tego, że serwer robi automatycznie co jakiś czas backup? (dodam, że stronę mam na płatnym serwerze).

Dlaczego mimo zastosowania funkcji flock() ten problem występuje?
Szczerze mówiąc dziwne to wszystko, zwłaszcza że skrypt licznika jest tak banalny, że ciężko byłoby się w nim pomylić...

Pozdrawiam

Witam,

Samoistne generowanie sie licznika jest zwiazane z nastepujaca sytuacja:

Gdy zablokujesz za pomoca flock dostep do pliku najczesciej inna inwokacja skryptu nie moze odczytac jego wartosci, odczytuje wiec wartosc 0, ktora nastepnie inkrementuje. W tym samym czasie pierwsza inwokacja dokonuje zapisu i zdejkmuje blokade a druga inwokacja jest zablokowana z zapisem do chwili odblokowania zapisu.
Po odblokowaniu sie zapisu wpisuje wartosc jaka zna wiec 1 lub ciag pusty.

Ciag pusty dlatego, ze inkrementacja niczego daje w efekcie nic.

Proponuje na sam poczatek zdjecie malpek aby zobaczyc czy nie ma zadnych bledow. Po drugie proponuje linijke:

[PHP] pobierz, plaintext 
<?php
$czytaj_l++;
?>
[PHP] pobierz, plaintext 

zmienic na:

[PHP] pobierz, plaintext 
<?php
if(is_numeric($czytaj_l)){$czytaj_l++;}else{$czytaj_l=1;}
?>
[PHP] pobierz, plaintext 

Wten sposob jesli nastapi bledny odczyt wartosci licznika to zostanie wpisana 1 - wiec od zerowania bedzie dobrze zliczal. Jednak podejrzewam, ze chcesz unikanac zerowania. W tym celu mozna zrobic tak - jesli blednie odczyta to nie zapisywac. Oto kod po zmianach:

[PHP] pobierz, plaintext 
<?php
$plik = "index/licznik.dat";
$waga = @filesize($plik);
 
$fop = @fopen("$plik", "r");
@flock($fop, 1);
$czytaj_l = @fread($fop, $waga);
@flock($fop, 3);
@fclose($fop);
 
if(is_numeric($czytaj_l)){
$czytaj_l++;
 
$fop = @fopen("$plik", "w");
@flock($fop, 2);
@fwrite($fop, $czytaj_l);
@flock($fop, 3);
@fclose($fop);
 
echo("$czytaj_l");
}
?>
[PHP] pobierz, plaintext 

Blad zerowania powstaje glownie wtedy gdy jedna inwokacja chce odczytac wartosc licznika podczas gdy druga zapisuje.

pozdr.

Też tak miałem i zębami zgrzytałem. Ale doszedłem do przyczyny.
Winne jest otwieranie pliku w trybie "w". Zauważ, że zawartość pliku jest wtedy kasowana. Już wtedy, a więc jeszcze przed próbą założenia locka!

Ja doszedłem w końcu do tego co widać niżej. Zrób tak, a bedzie dobrze:

[PHP] pobierz, plaintext 
<?php
 
$df="nazwaplikulicznika.txt";
$krolicznika = 1; 
 
ignore_user_abort(true);				
 
if (file_exists($df))
{												   
 
  if ($fp=@fopen($df, "r+"))  // niedestrukcyjny tryb r+
		{											
 
				if( flock($fp, LOCK_EX))  // jesli sie uda sie zablokowac do zapisu
						{			
							$count = intval(fgets($fp, 20)) + $krolicznika;		   
							ftruncate($fp, 0);  // wyczeszczenie zawartosci pliku
							fseek($fp, 0);	   // pointer na poczatek pliku 
							fputs($fp, $count);   // zapis nowej liczby do licznika
							flock($fp, LOCK_UN);   // zdjecie blokady
							fclose($fp);									
							echo $count;   // Wyswietlenie stanu licznika
						}	
			   
			   else {
						 fclose($fp);  // zamkniecie jesli nie mozna bylo zalozyc locka 
						}
 
	  }	
 
	  else  // jesli nie udało sie otworzyc pliku do zapisu (np. z powodu braku odp. uprawnien) 
		{ echo "ERROR! Nie moge zapisywac do tego pliku!"; }  
 
}  
 
 
else   // jesli plik $df licznika NIE istnieje - trzeba go stowrzyc												
 
{				   
 
 echo "Tworze nowy plik licznika. Prosze odwiezyc strone"; 
 touch($df);  // folder w ktorym ma powstac plik licznika musi miec atr. 777
 chmod($df, 0666);  // na wszelki wypdek
 
} 
 
ignore_user_abort(false); 
 
?>
[PHP] pobierz, plaintext 

latet