[php] upload i zmniejszanie obrazków w locie Opcje

[wzd]

Witam, proszę ekspertów o rzucenie okiem na ten kod. Już nie mam pojęcia gdzie popełniam błąd.
Chodzi mi o to, aby pobrać plik obrazka, zmniejszyć go do określonej wielkości i przekopiować do katalogu upload.

Oto mój kod

Formularz

[HTML] pobierz, plaintext 
<form action="test.php" method="post" enctype="multipart/form-data">
<input type="hidden" name="MAX_FILE_SIZE" value="50000">
<p>Upload Image: <input type="file" name="logo"><br/>
<font size="1">Podaj plik</font><br/>
<br/>
<input type="submit" name="submit" value="Upload">
</form>
[HTML] pobierz, plaintext 

Kod PHP

[PHP] pobierz, plaintext 
<?php
 
if(isset($_POST['submit']))
{
	$logo_tmp = $_FILES['logo']['tmp_name'];
	  $logo_name = $_FILES['logo']['name'];
 
	$path_thumbs = "/tmp";
 
	$losowa_nazwa = md5(time());
	$losowa_nazwa = rand(0,999999999);
 
	$pobierz_rozszerzenie = explode ('.', $logo_name);
	$rozszerzenie = $pobierz_rozszerzenie[count($pobierz_rozszerzenie)-1];
 
	$dozwolone = array(".gif",".jpg",".png",".jpeg");
 
	$thumb_path = substr($logo_tmp,0,strrpos($logo_tmp, '.')).'_min.jpg';
	$max_width = $max_height=100;
 
	if(!is_uploaded_file($logo_tmp))
	{
		echo "Brak pliku";
		exit();
	}
 
	$img_attr = getimagesize($logo_tmp);
 
	if($img_attr[0]>$img_attr[1])
		{
			 $scale = $img_attr[0] / $max_width;
		 }
	 else
		 {
			 $scale = $img_attr[1] / $max_height;
		 } 
 
	 $w = floor($img_attr[0]/$scale);
	 $h = floor($img_attr[1]/$scale);
	 
	 $ext = strrchr($logo_name,'.');
	 $ext = strtolower($ext);
	 
	 if (in_array($ext,$dozwolone)) 
	 {
		 $nowy = imagecreatefromjpeg($logo_tmp);
		 $zmniejszony_obrazek = imagecreatetruecolor($w,$h);
		imagecopyresized($zmniejszony_obrazek, $nowy, 0, 0, 0, 0, $w, $h, $img_attr[0], $img_attr[1]);
		imagejpeg($zmniejszony_obrazek,"$path_thumbs/$losowa_nazwa.$rozszerzenie",100);
		imagedestroy($zmniejszony_obrazek);
		imagedestroy ($nowy);
		move_uploaded_file ($file_tmp, "/upload/$losowa_nazwa.$rozszerzenie");
	 }
 
 
}
?>
[PHP] pobierz, plaintext 

Efekt działania jak narazie :

[HTML] pobierz, plaintext 
<?php Warning: imagejpeg() [function.imagejpeg]: Unable to open '/tmp/739776611.jpg' for writing in C:Program FilesWebServhttpd\test.php on line 49
?>
[HTML] pobierz, plaintext 

Ten post edytował wzd 8.01.2008, 00:02:02

[Cysiaczek]

Może opiszesz, co nie działa? Większość osób nie ma ochoty uruchamiać kodu, po to, aby zobaczyć jakiś parse error ;p

Pozdrawiam.

[aiv]

Odpalasz to pod windowsowym serwerem php a podajesz ścieżkę do /tmp

[dadexix]

Hm.. wygląda to tak jak by plik który tworzy php odrazu przybierał atrybut "tylko do odczytu"... sprawdz czy katalog o nazwie /tmp nie ma atrybutu "tylko do odczytu"
PS.

Kod

    $losowa_nazwa = md5(time());
    $losowa_nazwa = rand(0,999999999);

Najpierw robisz losową nazwę za pomocą md5(time()) a potem nadpisujesz to za pomocą rand()... to nie ma przecież sensu ponieważ robisz coś po to by to usunąć... mądrszejsze będzie

Kod

    $losowa_nazwa = md5(time());
    $losowa_nazwa .= rand(0,999999999);

wtedy do md5(time()) doda na końcu wynik z rand... tylko po co? przecież time zwraca liczbę sekund... zakładasz że w jednej sekundzie będzie uploadować kilka osób? to chyba lepiej

Kod

$losowa_nazwa = md5(microtime());

tutaj praktycznie nie ma możliwości że ktoś będzie w tym samym momęcie... no a teoretycznie się da - ale trudniej trafić niż w dużego lotka:)
@aiv

Odpalasz to pod windowsowym serwerem php a podajesz ścieżkę do /tmp

ale kto powiedział że on nie zrobil sobie katalogu tmp? prawdą jest że poprawnie powinno być ./tmp ale /tmp też powinno działać:)

[wzd]

wprowadziłem wszelkie wasze sugestie i zaczęło działać przyzwoicie ale nie mam pojęcia dlaczego tworzone miniaturki są słabej jakości ?

[mike]

imagecopyresized() -> imagecopyresampled()

[wzd]

imagecopyresized() -> imagecopyresampled()

dzięki nie zauważyłem tego działa wyśmienicie