[PHP] Upload obrazków - problem Opcje

[RubiX]

Witam
napisałem taki kod do uploadu obrazków:

[PHP] pobierz, plaintext 
$obrazek = $_FILES['obrazek'];
$nazwaobrazka = $_FILES['obrazek']['name'];
$num_start = rand(20 , 9999); //losowa liczba między 20 a 9999
$nowanazwaobrazka = $num_start.$nazwaobrazka; // tworzy nową nazwe pliku
if($obrazek['type']=='image/jpeg') { 
$source = imagecreatefromjpeg($obrazek[tmp_name]);
} elseif($obrazek['type']=='image/png') { 
$source = imagecreatefrompng($obrazek[tmp_name]);
} elseif($obrazek['type']=='image/gif') {
$source = imagecreatefromgif($obrazek[tmp_name]);
} elseif($obrazek['type']=='image/pjpeg') {
$source = imagecreatefromjpeg($obrazek[tmp_name]);
} elseif($obrazek['type']=='image/x-png') {
$source = imagecreatefrompng($obrazek[tmp_name]);
} 
 
//obraz
if (imagesy($source)<600 AND imagesx($source)<600) {
 
if(imagesy($source)>imagesx($source)) { 
$new_height = imagesx($source);
$new_width = floor(imagesx($source)*($new_height/imagesy($source)));
} else {
$new_width = imagesy($source);
$new_height = floor(imagesy($source)*($new_width/imagesx($source)));
}
$dest = imagecreatetruecolor($new_width,$new_height);
imagecopyresampled($dest,$source,0,0,0,0,$new_width,$new_height,imagesx($source),imagesy($source));               imagejpeg($dest,'../images/duze/'.$nowanazwaobrazka.'',99);
imagedestroy($dest);
} else {
if(imagesy($source)>imagesx($source)) { 
$new_height = 600;
$new_width = floor(imagesx($source)*($new_height/imagesy($source)));
} else {
$new_width = 600;
$new_height = floor(imagesy($source)*($new_width/imagesx($source)));
}
$dest = imagecreatetruecolor($new_width,$new_height);
imagecopyresampled($dest,$source,0,0,0,0,$new_width,$new_height,imagesx($source),imagesy($source));               imagejpeg($dest,'../images/duze/'.$nowanazwaobrazka.'',99);
imagedestroy($dest);
}
 
//miniaturka
if(imagesy($source)>imagesx($source)) { 
$new_height = 60;
$new_width = floor(imagesx($source)*($new_height/imagesy($source)));
} else {
$new_width = 60;
$new_height = floor(imagesy($source)*($new_width/imagesx($source)));
}
$dest = imagecreatetruecolor($new_width,$new_height);
imagecopyresampled($dest,$source,0,0,0,0,$new_width,$new_height,imagesx($source),imagesy($source));               imagejpeg($dest,'../images/mini/'.$nowanazwaobrazka.'',99);
imagedestroy($dest);
imagedestroy($source);
[PHP] pobierz, plaintext 

i mam problem otóż jak obraz jest większy niż 600px x 600px to zmniejsza go i dobrze wgrywa, natomiast jak obraz jest mniejszy od 600px x 600px to zamiast wgrac go dobrze tzn stworzyc od nowa jak 1 bok 1 za duzy to zmniejszyc troche i wrzucic to zmniejsza go całkowicie nieraz nawet do 100x100px pomimo iz obraz ma 400x400px ? gdzie zrobiłem błąd ?

Powód edycji: Dodałem tag. /webdice

[thek]

Powiem tylko tyle, że kod masz mocno nieoptymalny.
Powinno to wyglądać tak:
1. Sprawdzasz format (ta część poprawnie),
2. Sprawdzasz rozmiar (tu bym się kłócił czy poprawnie),
3. Jeśli większy niż wymagane to oblicz stopień skalowania dla każdego z boków. Ten który ma mniejszy - dla niego ustaw 600, temu z większym oblicz nowy rozmiar obliczając wcześniej ratio pomiędzy bokami oryginalnego by zachować proporcje boków.
4. Jeśli jest mniejszy... Po prostu skopiuj obraz przysłany do katalogu docelowego.
5. A teraz zrób sobie miniaturkę (IMG:style_emoticons/default/winksmiley.jpg)

I jeszcze jedno... Czy masz mieć maksymalny rozmiar jednego z nich większy od 600 czy wystarczy że jeden z nich przekroczy 600? bo od tego zależy czy masz w sprawdzaniu warunków AND czy OR. A to jest poważna różnica, bo obraz 5000x400 nie będzie skalowany gdy masz AND, zaś w przypadku OR jak najbardziej zostanie potraktowany zmniejszaniem.

[RubiX]

dobra już działa a co do

5. zrób sobie miniaturke to ten kod juz robi miniaturkę.
Błąd był w tym że miałem w 1 imagesx i imagesy zamienione x z y (IMG:style_emoticons/default/smile.gif)

[thek]

[PHP] pobierz, plaintext 
$obrazek = $_FILES['obrazek'];
$nazwaobrazka = $_FILES['obrazek']['name'];
$num_start = rand(20 , 9999); //losowa liczba między 20 a 9999
$nowanazwaobrazka = $num_start.$nazwaobrazka; // tworzy nową nazwę pliku
switch( $obrazek['type'] ) { 
//w tym switchu sprawdzamy typ typ pliku. Jeśli to nie jest według MIME obrazek o żadnym z podanych typów, wrzuć do $source FALSE
case 'image/jpeg':
case 'image/pjpeg':
    $source = imagecreatefromjpeg($obrazek[tmp_name]);
    break;
case 'image/png':
case 'image/x-png':
    $source = imagecreatefrompng($obrazek[tmp_name]);
    break;
case 'image/gif':
    $source = imagecreatefromgif($obrazek[tmp_name])
    break;
default:
    $source = FALSE;
}
 
//jesli plik nie został rozpoznany lub wczytanie się nie powiodło to wywal informację o błędzie
if( $source ) {
    $src_w = imagesx($source); //pobierz szerokość
    $src_h = imagesy($source); //pobierz wysokość
    $dest_w = $src_w; //ustal rozmiary docelowego identyczne ze źródłowym
    $dest_w = $src_h; 
    $destm_w = 60; //ustal wymiary miniaturki na 60 x 60
    $destm_h = 60;
    $ratio = $src_w / $src_h; //ustal współczynnik proporcji
    if( $src_w > 600 OR $src_h > 600 ) {  //sprawdź, czy któryś z wymiarów przekracza 600
        if( $src_w > $src_h ) {  //jeśli szerokość większa to...
            $dest_w = 600;         //..szerokość ustaw na 600...
            $dest_h = ceil( 600 / $ratio );  //...wysokość policz...
            $destm_h = ceil( $destm_h / $ratio);   // ...i policz wysokość miniaturki
        } else {    //jesli wysokość większa...
            $dest_h = 600;  // ...ustaw wysokość na 600...
            $dest_w = ceil( 600 * $ratio );  //...policz nową szerokość...
            $destm_w *= ceil( $destm_w * $ratio);   //... i oblicz szerokość miniaturki
        }
    }
    if( $src_w > $src_h ) {  //jeśli szerokość większa to...
        $destm_h = ceil( $destm_h / $ratio);   // policz wysokość miniaturki
    } else {    //jesli wysokość większa..
         $destm_w *= ceil( $destm_w * $ratio);   //oblicz szerokość miniaturki
    }
    $dest = imagecreatetruecolor( $dest_w,$dest_h );
    imagecopyresampled( $dest, $source, 0, 0, 0, 0, $dest_w, $dest_h, $src_w, $src_h );  //stwórz obraz wynikowy
    imagejpeg($dest,'../images/duze/'.$nowanazwaobrazka.'',75);  //zapisz go  (większe niż 75 dla tej rozdzielczości są bezsensem )
    $dest = imagecreatetruecolor( $destm_w,$destm_h ); 
    imagecopyresampled( $dest, $source, 0, 0, 0, 0, $destm_w, $destm_h, $src_w, $src_h ); //stwórz miniaturkę
    imagejpeg($dest,'../images/mini/'.$nowanazwaobrazka.'',60);  //zapisz miniaturkę (większe niż 60 dla miniatur są bezsensem )
    imagedestroy($dest); //niszczenie plików biblioteki GD
    imagedestroy($source);
} else
    echo "Błąd przetwarzania";
[PHP] pobierz, plaintext 

Dlaczego tak? Bo jest mniej bajzlu w kodzie, widać co się dzieje. Mogłem wrzucić konwersje głównego obrazka do osobnego i w razie gdyby oba wymiary były mniejsze niż 600 oraz byłby to jpg po prostu skopiować źródłowy bezpośrednio gdzie indziej, bo nawet konwersji bym robić nie musiał. A wpisywanie 99 w imagejpg jest głupotą, bo tworzy ogromne pliki. Na potrzeby www z reguły wystarcza 70-75 a dla miniatur maksimum to 60-65. Kod skracałes sobie wywołując co chwilę imagesx i imagesy, a przypisanie do zmiennych skraca czas dostępu niż notoryczne ich używanie (IMG:style_emoticons/default/winksmiley.jpg)
Kod jest zupełnie niezabezpieczony przeciwko takim "atakom" jak obrazki 5000x1 czy 1x5000 bo warunek skalowania wtedy nie ruszy (IMG:style_emoticons/default/smile.gif) A layout strony Ci się rozjedzie całkiem. Nie jest tez zabezpieczony przeciwko innym typom plików bo nawet nie sprawdzasz, czy to co dostałeś w ogóle jest plikiem obrazu. Kompletnie więc zaniedbałeś bezpieczeństwo skryptu i wystawiłeś się na atak. Ja i tak zanim swój skrypt bym puścił to sprawdziłbym co mi właściwie ludzie do zmiennej $_FILES wrzucili. Bo jeśli jakiś skrypt... to mógłbyś mieć nieprzyjemną sytuację.

Ten post edytował thek 1.09.2009, 04:18:06

[webdice]

Przenoszę do działu Przedszkole.