[PHP]zapisywanie obrazka i zatrzymanie pętli while Opcje

[maksik]

W pętli while mam skrypt który z linku obrazka.jpg sklauje go i zapisuje na serwer

[PHP] pobierz, plaintext 
$link= 'http://upload.wikimedia.org/wikipedia/commons/e/ee/Mango_and_cross_section_edit.jpg';
$imagetam ='nazwa.jpg';
 
$ch = curl_init($link);
curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
file_put_contents($imagetam ,curl_exec($ch));
 
 
function zmiana_rozmiaru($orginalny_obrazek,$docelowa_szerokosc,$docelowa_wysokosc){
 
    // Pobranie parametrów
    list($szerokosc, $wysokosc) = getimagesize($orginalny_obrazek);
    $xskala=$szerokosc/$docelowa_szerokosc;
    $yskala=$wysokosc/$docelowa_wysokosc;
 
    // Kalkulacja 
    if ($yskala>$xskala){
        $nowa_szerokosc = round($szerokosc * (1/$yskala));
        $nowa_wysokosc = round($wysokosc * (1/$yskala));
    }
    else {
        $nowa_szerokosc = round($szerokosc * (1/$xskala));
        $nowa_wysokosc = round($wysokosc * (1/$xskala));
    }
 
    // Zmiana rozmiaru 
    $obraz_zmiana_wielkosci = imagecreatetruecolor($nowa_szerokosc, $nowa_wysokosc);
    $obrazek_tymczasowy     = imagecreatefromjpeg ($orginalny_obrazek);
    imagecopyresampled($obraz_zmiana_wielkosci, $obrazek_tymczasowy, 0, 0, 0, 0, $nowa_szerokosc, $nowa_wysokosc, $szerokosc, $wysokosc);
 
    return $obraz_zmiana_wielkosci;
}
 
$obraz_zmiana_wielkosci = zmiana_rozmiaru($imagetam, '228', '228');
 
header('Content-Type: image/jpeg');
 
//zapisywanie zdjecia
imagejpeg($obraz_zmiana_wielkosci,'nowy_folder/'.$imagetam.'');
 
unlink($imagetam);
[PHP] pobierz, plaintext 

Problem w tym że podczas uruchomienia pojawia się komunikat "obrazek nie może zostać otworzony poniewaz zawiera błędy" co zatrzymuje pętle, a musi ona obsłużyć jeszcze inne obrazki, obrazek mimo komunikatu oczywiscie się zapisuje w folderze co można zrobić by pętla się nie zatrzymywała?

Ten post edytował maksik 2.08.2014, 11:43:29

[SmokAnalog]

Uwagi, które mam do Twojego kodu:

1. Po co tworzysz plik nazwa.jpg tylko po to, żeby za chwilę go usunąć? Możesz operować bezpośrednio na źródle obrazka
2. Pierwsze 6 linii można zastąpić jedną instrukcją copy.
3. Linie 13-24 można znacznie skrócić:
   

   [PHP] pobierz, plaintext 
   $skala = min($docelowa_szerokosc / $szerokosc, $docelowa_wysokosc / $wysokosc);
   $nowa_szerokosc = round($szerokosc * $skala);
   $nowa_wysokosc = round($wysokosc * skala);
   [PHP] pobierz, plaintext 

4. W PHP jest coś takiego jak liczba, po co apostrofy przy przekazywaniu wymiarów w linii 34.?
5. Po co ustawiasz Content-type, skoro tylko zapisujesz do pliku?
6. Po co dodajesz .'' na końcu stringa? Nie pierwszy raz zdaje się.

Na razie tyle, póki co za dużo śmieci tu żeby zacząć się zastanawiać czemu nie działa.

Ten post edytował SmokAnalog 2.08.2014, 11:56:00