Witam
Z PHP mam kontakt od dzisiaj, a więc proszę o wyrozumiałość.
Udało mi się ostatnimi czasy uruchomić Serwer za pomocą IIS'a i zacząłem się bawić z prywatną stronką na no-ip. Wpadłem na pomysł, że fajnie będzie móc zapisywać sobie informacje o użytkownikach, którzy mnie odwiedzili, np. ich IP.
Napisałem taki kod, ale niestety nie działa, siedzę nad nim już ponad godzinę i już się troszkę tym zmęczyłem a nic nie widzę, wydaje mi się, że problem jest z którymś while, ale niestety to jest "kopiuj", "wklej" bo dopiero dzisiaj pierwszy raz miałem kontakt z PHP i nie znam komend na odczyt i zapis plików.

Kod:

[PHP] pobierz, plaintext 
//$ip pobieram wcześniej za też pomocą wklejki, ale dobrze działa.
//plik state.txt zawiera jedną cyfrę z informacją ile IP jest zapisanych w folderze "dane" pod kolejnymi cyframi od "0". 
//(teraz zauważyłem problem, że przy 10tym użytkowniku, przekręci mi state.txt ponieważ odczyta 1 znak
//no trudno, później się tym zajmę, ale chciał bym, żeby kod zaczął działać).
$read = fopen("state.txt","r");
while (!feof($read)){
	$lp = fgetc($read);    //odczytanie liczby IP zapisanych
	fclose($read);
	for ($i = 0; $i < $lp ; $i++){
		$read2 = fopen("dane/".$i.".txt", "r"); //otwarcie pliku
		while (!feof($read2)){
			$ipFromFile[$i] = fgets($read2);    //odczytanie IP z pliku
			fclose($read2);
			if($i++ == $lp){			//nie umiem posługiwać się globalnymi zmiennymi, a więc w taki sposób wszystko wcisnąłem lokalnie
				$check = 0;
				for ($j = 0; $j < $lp ; $j++){
					if($ip == $ipFromFile[$i]){
						$check++;				//zwiększam wartość $check tylko wtedy, gdy IP już mam zapisane
					}							//czyli $ip użytkownika jest równe $ip z pliku 
				}
				if($check == 0){				//jeżeli check nie został zwiększony, to znaczy, że jest to nowe IP
					$save2 = fopen("dane/".($lp+1).".txt", "w"); //zapisanie nowego IP w kolejnym pliku
					fwrite($save2, $ip);
					fclose($save2);
					$save = fopen("state.txt","w");
					fwrite($save, $lp++);						 //zapisanie w pliku state.txt liczby większej o 1
					fclose($save);								 //ponieważ jest jedno nowe IP
				}
			}
		}
	}
}
[PHP] pobierz, plaintext 

Może ktoś to przepuści przez jakiś kompilator i zobaczy czy nie ma błędów w ten sposób, ja nie mam żadnego, piszę kod w Notepad++ i patrzę na efekty na stronie.


z góry dziękuję bardzo za pomoc!
Prosiłbym o nakierowanie mnie co gdzie jest źle, nie interesuje mnie gotowiec, wykonany 100 razy lepiej przez doświadczonego informatyka, ja czerpie przyjemność z pisania kodu samemu i debugowania go, ale chciałbym, żeby strona dalej działa i chciałbym już zbierać informację i gościach.

Prosciej bedzie uzyc file_get_contents rozbic na tablice, potem sprawdzic in_array i zapisujesz file_put_contents, prosciej

Można to napisać dużo lepiej bez wnikania co nie działa w tym miejscu.

[PHP] pobierz, plaintext 
$ip = $_SERVER['REMOTE_ADDR'];
$found = false;
$filePath = 'sciezka/do/pliku/z/lista_ip.txt'; // jeden plik, w którym masz listę IP
 
if(is_readable($filePath)) { // jeśli istnieje to ładujemy go
	$file = fopen($filePath,'r');
 
	while(!feof($file)) { // sprawdzamy czy dana linia jest równa adresowi IP
		$line = fgets($file);
		if($line == $ip) { // jeśli tak to kończymy
			$found = true; 
			break;
		}
	}
 
	fclose($file);
}
 
if(!$found) { // jeśli nie znaleziono to zapisujemy plik (dopisujemy nowe IP)
	$file = fopen($filePath,'a+');
	if(flock($file, LOCK_EX)) { // ale blokujemy plik by nie było równoległych zapisów
		fwrite($file, "{$ip}\n");
		fflush($file);
		flock($file, LOCK_UN)
	}
	fclose($file);
}
[PHP] pobierz, plaintext 

@E30Tomas włącz wyświetlanie błędów, a wszystko stanie się jasne.

[PHP] pobierz, plaintext 
error_reporting(E_ALL); // umiesc to na początku pliku
[PHP] pobierz, plaintext 

@Sephirus - nie utrudniaj innym życia ;)

[PHP] pobierz, plaintext 
$file = 'ip.txt'; // plik z adresami
$ip = $_SERVER['REMOTE_ADDR']; // pobranie IP
 
$ipAll = file($file, FILE_IGNORE_NEW_LINES | FILE_SKIP_EMPTY_LINES); // wczytanie wszystkich ip z pliku do tablicy
 
if (!in_array($ip, $ipAll)) // jesli nie jestnieje
{
    file_put_contents($file, $ip."\n", FILE_APPEND | LOCK_EX); //zapis ip do pliku
}
[PHP] pobierz, plaintext 

@!*! Touche - ale ale przyzwyczajony jestem do działania na wydajności i przerabiania sporych plików - toteż wolałem napisać coś co nie zajmie dużo ramu

Skrajność:

plik z Ip ma już 1mb - wchodzi 100 gości - u Ciebie ram skacze o ~200mb
u mnie o jakieś ~60mb

plik 2mb - u Ciebie 300mb
a u mnie ~60mb

Przelicz

Stąd taki a nie inny kod

Oczywiście w przypadku autora nie ma to większego sensu

Dzięki bardzo za pomoc!

kod od "Sephirus" nie działa, strona nie ładuje, ale kod od "!*!" działa świetnie!

Dzięki bardzo!! Oboje dostajecie po pomocy


Tutaj link do strony, możecie zobaczyć jak to działa, a ja przy okazji zobaczę czy IP mi zapisze i licznik odwiedzin wzrośnie
5k.no-ip.org