[PHP]Problem z zapisem adresu IP do pliku Opcje

[Krastley]

Mam taką funkcję:

[PHP] pobierz, plaintext 
public static function uniquereader($counter_file) {
 
        $user_addr = $_SERVER['REMOTE_ADDR'];
 
        if (!is_file($counter_file)) {
            $counter_file_stream = fopen($counter_file, 'wb');
            fwrite($counter_file_stream, $user_addr);
            fclose($counter_file_stream);
        }
 
        if (!strpos(file_get_contents($counter_file), $user_addr)) {
            $counter_file_stream = fopen($counter_file, 'r+b');
            fseek($counter_file_stream, -1, SEEK_END);
            fwrite($counter_file_stream, "\n".$user_addr);
            fclose($counter_file_stream);
        }
 
        $counter = sizeof(file($counter_file));
 
        return $counter;       
    }
[PHP] pobierz, plaintext 

Jak widać, jeżeli nie ma pliku na serwerze, powinien zostać utworzony, a w nim powinien zostać zapisany adres IP. Ale kiedy plik jest tworzony po raz pierwszy, znajduje się w nim coś takiego:


127.0.0.
127.0.0.1


i funkcja zwraca wartość 2. Czy ktoś może mi powiedzieć, dlaczego?

[!*!]

[PHP] pobierz, plaintext 
function ipFile($file)
{
    $user_addr = $_SERVER['REMOTE_ADDR'];
 
    if(!file_exists($file))
    {
        file_put_contents($file, $user_addr."\n", FILE_APPEND | LOCK_EX);
        return count(file($file));
    }
 
    if(!stristr(file_get_contents($file), $user_addr))
    {
        file_put_contents($file, $user_addr."\n", FILE_APPEND | LOCK_EX);
        return count(file($file));
    }
     return count(file($file));
}
[PHP] pobierz, plaintext 

Zwraca 2, ponieważ są 2 linijki.

Ten post edytował !*! 26.07.2012, 19:35:32

[Krastley]

Dzięki za kod O wiele lepiej to wygląda i działa bez zarzutu.

Trochę źle sformułowałem pytanie. Dlaczego moja funkcja ucina pierwszy adres i zapisuje 127.0.0.? Teoretycznie wydaje mi się, że powinno być dobrze i nie bardzo rozumiem, dlaczego nie jest

OK, przypuszczam, że to kwestia LOCK_EX.

Ten post edytował Krastley 26.07.2012, 21:58:02

[!*!]

[PHP] pobierz, plaintext 
fseek($counter_file_stream, -1, SEEK_END);
[PHP] pobierz, plaintext 

A to nie jest tak że tworzysz plik zapisujesz 127.0.0.1 po czym sprawdzasz czy występuje i ustawiasz znacznik na -1, dlatego ucina i zapisuje kolejne. LOCK_EX jest od czego innego, poczytaj w manualu, bo może się okazać że będzie to dla Ciebie zbędne.

Ten post edytował !*! 27.07.2012, 09:11:11

[Krastley]

Jest dokładnie tak, jak napisałeś. W manualu znalazłem coś takiego:

"To move to a position before the end-of-file, you need to pass a negative value in offset and set whence to SEEK_END."

To mnie zmyliło, myślałem, że negatywny offset i SEEK_END przeniosą mnie na koniec pliku, ale źle to zrozumiałem.

Wielkie dzięki za pomoc.