[PHP] Thumbs vanuit directory creeren

[Verwijderd]

Ik wil graag van een map met allemaal foto's thumbnails creeren. Ik heb de volgende code:

code:

foreach($bestanden as $file)
{
  echo $file. "<br />";
  $name= explode(".",$file); 
  createImage("img/", $name[0], $name[1], 60, "_small");
}

$bestanden is de array met de bestanden uit directory. Hij maakt nu echter maar een thumbnail in plaats van het aantal in de map.

Iemand enig idee hoe ik dit voorelkaar krijg?

[Pete]

Print hij wel alle filename's? (dus komt hij wel x aantal keer in de loop?)

[Verwijderd]

phsmit schreef op dinsdag 15 mei 2007 @ 14:30:
Print hij wel alle filename's? (dus komt hij wel x aantal keer in de loop?)

Die print hij allemaal netjes. de loop werkt verder dus goed, alleen de functie voert hij blijkbaar maar een keer uit.

[Noork]

Misschien zit er wel een fout of een fend/exit statement in de createImage functie

[Verwijderd]

dit is de functie:

PHP:

function createImage($imgDir, $imageName, $extension, $width, $suffix)
{  
    //De hoogte en breedte ophalen van het plaatje
    $dimensions = getimagesize($imgDir . $imageName . "." . $extension); 
    
    //Hoogte en breedte toekennnen aan nieuwe variabelen
    $bronbreedte = $dimensions[0]; 
    $bronhoogte  = $dimensions[1];
    
    //Dit is de breedte die alle plaatjes krijgen
    if($width > 640)
    {
      $doelbreedte = 640;
    }
    else
    {
      $doelbreedte = $width;
    }
    
    //De nieuwe hoogte berekenen aan de gegevens van het oude plaatje en de doel breedte
    $doelhoogte = ($bronhoogte * $doelbreedte) / $bronbreedte;
    
    //De hoogte, als het nodig is, afronden
    $doelhoogte = round($doelhoogte, 0);
    
    //Zeggen dat dit bestand een plaatje is
    header('content-type: image/jpeg'); 
    
    //Het plaatje inlezen in de variabele $image
    $image = imagecreatefromjpeg($imgDir . $imageName . "." . $extension);
    
    //een nieuw klein plaatje maken met de gewenste grootte
    $destination = imagecreatetruecolor($doelbreedte, $doelhoogte);
    
    //Het nieuwe plaatje vullen met verkleinde plaatje
    imagecopyresampled($destination, $image, 0, 0, 0, 0, $doelbreedte, $doelhoogte, $bronbreedte, $bronhoogte);
    
    $new_filename = "$imgDir" . "$imageName" . "$suffix" . ".$extension";
    
    //Het plaatje opslaan en weergeven
    imagejpeg($destination, $new_filename);
    
    //Het bronplaatje verwijderen
    imagedestroy($image);
    
    //Het doelplaatje verwijderen
    imagedestroy($destination); 
}

[Pete]

Noork schreef op dinsdag 15 mei 2007 @ 14:32:
Misschien zit er wel een fout of een fend/exit statement in de createImage functie

Nee, dan zou het hele script er mee stoppen. Maar kan de TS misschien de createImagefunctie laten zien. Daar zit waarschijnlijk wel een fout.
(ik denk dan zelf dat er misschien resources opnieuw gebruikt worden die niet opnieuw gebruikt kunnen worden)

[mithras]

Je stuurt een header in je functie: waarom?

[Janoz]

Wat doet die header aanroep in die functie? Geeft je script verder nog error meldingen?

[Noork]

Dat zal het probleem best eens kunnen zijn. Het script poept geen jpegje uit op het scherm, dus de header is niet nodig.

[Verwijderd]

Ik heb de header eruit gehaald, maar maakt geen verschil. Verder krijg ik geen foutmeldingen.

[Voutloos]

Verwijderd schreef op dinsdag 15 mei 2007 @ 14:43:
Ik heb de header eruit gehaald, maar maakt geen verschil. Verder krijg ik geen foutmeldingen.

Je dient zelf bijvoorbeeld de returnwaarde van imagejpeg() te controleren.

Daarnaast controleer je nu nergens of de thumb al bestaat en maak je die thumbs steeds opnieuw.

[mithras]

Verwijderd schreef op dinsdag 15 mei 2007 @ 14:43:
Ik heb de header eruit gehaald, maar maakt geen verschil. Verder krijg ik geen foutmeldingen.

En je hebt error reporting wel op e_all gezet? Als je opzettelijk een ; mist moet je dat zien met een error reporting, dus test eens of dat aan staat

Doe anders ook gewoon wat extra debug werk in de trant van:

PHP:

//De nieuwe hoogte berekenen aan de gegevens van het oude plaatje en de doel breedte 
    echo "Start berekening...";
    $doelhoogte = ($bronhoogte * $doelbreedte) / $bronbreedte; 
     
    //De hoogte, als het nodig is, afronden 
    $doelhoogte = round($doelhoogte, 0); 
    echo "Ok!<br>\n";
     
    //Zeggen dat dit bestand een plaatje is
    //Weggegooid dus
    //header('content-type: image/jpeg');  
     
    echo "Afbeelding inlezen...";
    //Het plaatje inlezen in de variabele $image 
    $image = imagecreatefromjpeg($imgDir . $imageName . "." . $extension); 
    echo "Ok!<br>\n";

    //etc

En check trouwens nog eens of je functies wel wat juist retourneren:

PHP:

if($image = imagecreatefromjpeg($imgDir . $imageName . "." . $extension)){
  echo "Gelukt";
}
if(imagejpeg($destination, $new_filename)){
  echo "Gelukt";
}

Dus

[ Voor 10% gewijzigd door mithras op 15-05-2007 14:49 ]

[Noork]

Werk het scriptje wel naar behoren wanneer je deze voor 1 afbeelding gebruikt (desnoods even de url van de afbeelding vast in de code zetten)

[Verwijderd]

Hej verdraaid hij doet het na nader in zien toch wel

Ik had eerst 1 bestand in de map (bestand1.jpg) na het uitvoeren van het script heb ik twee in de map:

bestand1.jpg
bestand1_small.jpg

Als ik nog een keer het script uitvoer verwachtte ik eigenlijk vier bestanden in de map maar dit is natuurlijk niet het geval omdat bestand1_small.jpg overschrijft wordt.

Was even een denkfoutje vanmezelf...

In ieder geval bedankt voor de feedback.

[Verwijderd]

En weet je zeker dat alle bestanden (plaatjes) maar 1 punt in de bestandsnaam hebben? Anders gaat die explode in je loop al fout. Ik zou daar dan ook dit van maken:

PHP:

$pos = strpos($file,'.'); 
createImage("img/",substr($file,0,$pos),substr($file,$pos+1,"_small");

Als je dit script twee keer aanroept op dezelfde directory gaat hij van die thumbnails ook weer thumbnails maken.