Thumbnails worden random weergegeven

[Maverick2001]

Voor een fotopagina maak ik 'on the fly' thumbnails.

Ik heb in de output 2 rijtjes van 8 thumbs gemaakt.

Nu is het zo dat ie elke keer een paar thumbnails laat zien. Als ik de pagina refresh laat ie random weer een paar andere zien, de rest krijgt een kruisje.

Voor het maken van de thumbs heb ik het volgende:

code:

$old_error_level = error_reporting(E_ALL ^ E_NOTICE);
import_request_variables('gpc', '');
error_reporting($old_error_level);
extract($_SERVER, EXTR_REFS);

// header sturen
    header("Content-Type: image/jpeg");

// blok als iets leeg is
    if ($u=='')
        exit;

// plaatjes voor thumbnails fotonieuws
    $fotobestand= "http://(hier staat mijn url)". $u;
    $deelfactor=78;

    $image = ImagecreatefromJPEG ($fotobestand);

    $original_hoogte=imagesy($image);
    $original_breedte=imagesx($image);

    $factor = $original_breedte / $deelfactor;

    $hoogte=$original_hoogte/$factor;
    $breedte=$original_breedte/$factor;

    $image_id = ImageCreateTrueColor($breedte,$hoogte); 
    imagecopyresized($image_id,$image,0,0,0,0,$breedte,$hoogte,$original_breedte,$original_hoogte);

// vrijgeven van plaatje
    imageJPEG($image_id);

// destroy
    ImageDestroy($image_id);
    ImageDestroy($im);

[AtleX]

Waarschijnlijk heb je last van timeouts oid. Je kan beter je thumbnails cachen door ze op schijf op te slaan. Dat is ook meteen een stuk beter voor de snelheid van je fotoalbum.

[GlowMouse]

- Jij laadt de foto via http://jouwsite/foto.jpg. Dat kost je webserver dus een extra request, en onnodige resources. Beter is om via het filesystem het plaatje op te halen. Waarschijnlijk kost dit zoveel extra resources dat je timeouts krijgt.
- Leer jezelf aan superglobals te gebruiken in plaats van functies als import_request_variables en extract. Dit is vragen om problemen. In jouw script zit nu al een veiligheidslek: via thumbnail.php?old_error_level=6143 kan ik allerlei errors zien die niet voor mij bedoeld zijn. Naarmate je script groter wordt, neemt dit probleem alleen maar toe.

[ Voor 21% gewijzigd door GlowMouse op 11-02-2007 18:30 ]

[Maverick2001]

AtleX schreef op zondag 11 februari 2007 @ 18:18:
Waarschijnlijk heb je last van timeouts oid. Je kan beter je thumbnails cachen door ze op schijf op te slaan. Dat is ook meteen een stuk beter voor de snelheid van je fotoalbum.

Maar ik maak ze on the fly.. kan ik ze dan automatisch laten opslaan?

Dat van die superglobals moet ik me in verdiepen.

[AtleX]

Maverick2001 schreef op zondag 11 februari 2007 @ 18:40:
[...]


Maar ik maak ze on the fly.. kan ik ze dan automatisch laten opslaan?

Je originele afbeelding moet geüpload worden, dan kan je toch meteen een thumbnail aanmaken?

[Maverick2001]

Als je op de thumb klikt krijg je de orginele grote te zien. Vandaar dat ik ze on the fly aanmaak.

[Megamind]

Maar dan maakt het toch weinig uit in laadtijd, je kan ook gewoon dan de <img> tag aanpassen met kleinere maten op verhouding ipv een heel nieuw plaatje genereren

[Maverick2001]

Het aparte is gewoon dat het script wel werkt op dezelfde server waar ik nog een site draai.

Het gaat me er meer om dat ik niet kan achterhalen waarom het niet werkt.

[pietje63]

Misscihen de grootte van de afbeeldingen?

[Verwijderd]

Kijk eens op php.net, volgens mij komt het omdat het teveel tijd kost om de plaatjes op te halen en kapt ie er naar 30 seconden mee. Dat limiet kun je hoger instellen (zie link), maar waarom doe je niet gewoon wat Maverick zei?

[OkkE]

Ik heb zelf ook zo'n scriptje gemaakt, die doet 't volgende:

1. Kijk of er afbeeldingen in de map staan
2. Per afbeelding checken of er al een thumbnail gemaakt is
3. Als er geen thumbnail bestaat: aanmaken
4. Laat opgeslagen thumbnail zien

De stap die ik er nog in wil maken is later checken of er thumbnails zijn waarvan de originele foto niet meer bestaat en deze thumb dan ook verwijderen.

Ik denk trouwens dat het inderdaad een probleem is met de afmeting van je foto's. Mijn script laadt ook niet altijd alle foto's; als het er bijv. 100 zijn van >1024 mist 'ie er wel eens een paar. Het is sowieso slim om de thumbs op te slaan, zo voorkom je onnodige server belasting.