Witam. Mam skrypt prostego licznika, ktory zlicza ilosc odwiedzajacych osob do pliku ilosc.dat.

[PHP] pobierz, plaintext 
<?php
 
//licznik tekstowy
 
$licznik_plik = "./ilosc.dat";
	if(!($fp = fopen($licznik_plik, "r"))) // Próba otwarcia pliku do odczytu
	die ("Nie mogę otworzyć pliku $licznik_plik.");
$licznik = (int) fread($fp, 20);
	fclose($fp);
 
$licznik++;
 
	echo "Jesteś gościem nr: $licznik."; //Wyświetla stan licznika
 
$fp = fopen($licznik_plik, "w"); //Otwiera plik do zapisu
	fwrite($fp, $licznik); //Zapisuje
	fclose($fp); //Zamyka plik
?>
[PHP] pobierz, plaintext 

Gdy utworze plik ilosc.dat wszystko jest OK jednak gdy nie ma tego pliku to pokazuje mi błędy:


Dodam że testowałem ten licznik na lokalnym serwerze jakim jest KRASNAL Serv.

Prosze o jakies wskazowki, rady jak pozbyć się tych błedów. Jestem w php początkujący i dopiero się ucze narazie na prościutkich skryptach jakim są liczniki i jeśli źle zabieram się do tworzenia kodu to też proszę zwrócić mi uwagę :/ Może wyświetlanie stanu licznika dać na koniec?

w twoim kodzie nie ma błędu... tzn nie do końca, jest tu błąd przewidziany, który błędem nie jest

[PHP] pobierz, plaintext 
<?php
 
    if(!($fp = @fopen($licznik_plik, "r"))) // Próba otwarcia pliku do odczytu (jeśli nie uda się otworzyć, wywołujesz die i komunikat. Wiesz że błąd może 
wystąpić.
    die ("Nie mogę otworzyć pliku $licznik_plik.");
 
?>
[PHP] pobierz, plaintext 

aby pozbyć się wyświetlania błędu wstawiasz @ przed fopen

Można tak, a można i ładniej

[PHP] pobierz, plaintext 
<?php
 
if(file_exists($licznik_plik)){ // jezeli plik istnieje, otwiera go
	fopen($licznik_plik, "r");
} else { //jezeli nie, 'umiera'
	die ("Nie mogę otworzyć pliku $licznik_plik.");
}
 
?>
[PHP] pobierz, plaintext 

Licznik to z założenia prosty skrypt, ale nie taki prosty jak się na początku wydaje. Są pewne pułapki, np. sytuacja gdy 2 userów w tym samym momencie wywoła licznik - jesli nie zabezpieczysz się przed tym, to ulegnie on wyzerowaniu (cecha trybu "w").

Ja metoda prób i błędów, potu i łez doszedłem w końcu do takiego, mysle, ze bezpiecznego i niezawodnego licznika:

[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 

Pozdrawiam,

latet

Bardzo dziękuję za pomoc, proszę tylko powiedziec jak moge zabezpieczyć mój kod przed wyzerowaniem pliku ilosc.dat (Bede wdzieczny jak powiecie jakich komend/funkcji uzyć...) ?

Np. tak jak w przykładzie, który podałem wyżej w pełnym brzmieniu.

latet