[PHP] statystyki wyswietlania zdjec, a FireFox, FF pobiera dwa razy Opcje

[siutek]

Witam,
Chcialem zrobic sobie statystyki wyswietlania zdjec, do tego celu stworzylem skrypt php, ktory wyswietla na ekran odpowednie zdjecie (z uzyciem naglowka header()(IMG:style_emoticons/default/winksmiley.jpg)
W tym samym pliku kazdorazowo zapisuje do bazy id fotki i dokladna date jej wyswietlenia.

Niestety zauwazylem ze FF pobiera zdjecia dwukrotnie! w odstepie 2-4 sekund, w ten posob jedno wyswietlenie generuje mi dwa wpisy do bazy. Tego problemu nie ma np w Chromie...
Jak sobie z tym poradzic? jest mozliwosc zabezpieczenia sie? Nie chcialbym blokowac drugiego wpisu na IP :/

kod:

[PHP] pobierz, plaintext 
header('Content-type: image/jpeg');
 
error_reporting(0);
date_default_timezone_set('Europe/Warsaw');
 
include "./config.php";
 
$fotaId = base64_decode($_GET["fi"]); 
 
$fotaArr = getRecord("gallery",$fotaId);
$usrArr = getRecord("users",$fotaArr["usrId"]);
queryInsert("imgStats",array("img_id"=>$fotaId,"date"=>date("U"),"user_id"=>$fotaArr["usrId"]));
 
$link = './usersFiles/'.$usrArr["login"].'/'.$fotaArr["fileName"]; 
 
$fota = file_get_contents($link);
 
echo $fota; 
[PHP] pobierz, plaintext 

[Quadina]

A wyświetlasz te obrazki jakimś javascriptem, czy normalnie w HTML? Możesz wysłać ten fragmencik kodu? Ogólnie po za brakiem optymalizacji w kilku drobnych miejscach to Twój skrypt jest ok i na pewno będzie zliczał tylko raz.

[batman]

Stawiam na Firebuga lub inną wtyczkę. Przetestuj skrypt na "czystym" Fx.

[siutek]

nie wyswietlam obrazka ani JS, ani HTMLem. wyswietlam jak normalną fotke, tak jakbys w pasku adresu w przegladarce wpisala http://www.adresserwera.pl/zdjecie.jpg
Ten kod jakby symulowal fote, w celu zliczania statystyk.

Ps. mozesz wskazac miejsca do poprawy w kontekscie optymalizacji, o ktorej mowilas?

edit: powylaczalem wszystkie wtyczki :/ zresetowalem FF i dalej to samo...

Ten post edytował siutek 7.12.2010, 15:44:34

[Quadina]

[PHP] pobierz, plaintext 
error_reporting(0);
ob_start();
// to jest na pewno potrzebne?
//date_default_timezone_set('Europe/Warsaw');
 
include "./config.php";
 
$fotaId = base64_decode($_GET["fi"]); 
 
$fotaArr = getRecord("gallery",$fotaId);
$usrArr = getRecord("users",$fotaArr["usrId"]);
queryInsert("imgStats",array("img_id"=>$fotaId,"date"=>date("U"),"user_id"=>$fotaArr["usrId"]));
 
$link = './usersFiles/'.$usrArr["login"].'/'.$fotaArr["fileName"]; 
 
// przewidujemy, ze pliku niestety nie będzie
if(is_file($link){
    // wysylamy caly zestaw nagłówków - być moze FF nie jest pewna, czy dostała wszystko i wysyła żądanie jeszcze raz.
    header('Content-type: image/jpeg');        
    header('Content-Length: ' . filesize($link));
 
    // czyścimy context w razie jakichś niespodziewanych błędów. error_reporting nie działa na niektóre sytuacje.
    ob_clean();
    flush();
 
    // drukujemy od razu plik, nie przepisując jej do żądnej zmiennej
    readfile($link);
} else {
    echo "ERROR: brak pliku $link";
}
[PHP] pobierz, plaintext 

Tak jak napisałem w komentarzu w kodzie, FF może nie być pewne, czy pobrał całość, skoro nie wie ile tych danych ma dostać. Dodałem obsługę błędu, gdy pliku nie ma, możesz tam sobie wrzucić coś na maila, albo zapisywać błędy do jakiegoś pliku.

[PHP] pobierz, plaintext 
$fotaArr = getRecord("gallery",$fotaId);
$usrArr = getRecord("users",$fotaArr["usrId"]);
[PHP] pobierz, plaintext 

Czy nie jesteś w stanie tego wyciągnąć jednym zapytaniem z joinem?

P.S. Czepialstwo na średnim poziomie, bardziej chciałem pokazać, że można inaczej, niż że masz coś realnie źle (IMG:style_emoticons/default/smile.gif)

[batman]

Skoro nie Firebug, to dodaj buforowanie tak jak to zrobił ~Quadina i sprawdź wtyczką LiveHttpHeaders czy przypadkiem dwa razy nie wysyła Ci requesta.

[siutek]

Panowie
Poradzilem sobie (IMG:style_emoticons/default/smile.gif) doszedlem do wniosku, że przeglądarka pobiera podwójnie aby wyswietlic na ekranie i wrzucic do cache'a.

Umiescilem zatem dodatkowy naglowek: header("Cache-Control: no-cache");

Wiem ze teraz za kazdym razem zdjecie bedzie pobierane z sieci, ale inaczej nie wiem jak to rozwiazac... (IMG:style_emoticons/default/winksmiley.jpg)

[Quadina]

Rozwiązanie jest o tyle dobre, że teraz na pewno będzie Ci zliczać za każdym razem gdy zdjęcie zostanie wyświetlone. A chyba o to chodziło (IMG:style_emoticons/default/smile.gif)