Imageshack - zrobiłem ale... Opcje

[Nightwalker]

Witam
Napisalem skrypt a'la imageshack.us

[PHP] pobierz, plaintext 
<?php
 
#########################################################
#														#
#	Skrypt a'la imageshack.us lub fotosik.pl			#
#	Jego zadaniem jest wgranie pliku na serwer,			#
#	utworzenie miniaturki oraz utworzenie				#
#	gotowych linkow ze znacznikami na forum				#
#	oraz strone www.									#
#														#
#	Skrypt napisany przez Nightwalkera					#
#	Mozesz go uzywac do celow nie komercyjnych			#
#	oraz nie mozesz usuwac tego naglowka!				#
#														#
#	email autora: admin@smallville.pl					#
#	www autora: http://smallville.pl/					#
#	GG autora: 5608039									#
#														#
#	Wersja skryptu: 1.1 beta							#
#	Data napisania: 11 Marca 2006						#
#	Ostatnia aktualizacja: 12 Marzec 2006				#
#														#
#########################################################
 
if(isset($_POST['Submit']))
 
{
	// Wczytywanie pliku z configiem
 
		include("config.php");
 
	$uzyj_nazwy = $_FILES['image']['name'];
	$uzyj_nazwy_min = $_FILES['image']['tmp_name'];
	$uzyj_nazwy_roz = $_FILES['image']['size'];
	$uzyj_nazwy_typ = $_FILES['image']['type'];
 
	if (isset($_FILES['image']['name'])) 
	{
		$obrazek = $katalog.$numer.$uzyj_nazwy;
		$image = $adres.$katalog.$numer.$uzyj_nazwy;
 
		$obrazek_min = $katmin.$prefix.$numer.$uzyj_nazwy;
		$image_thumb = $adres.$obrazek_min;
 
		move_uploaded_file($uzyj_nazwy_min, $obrazek);
		chmod ($obrazek, octdec($upraw));
 
		$rozmiary = getimagesize($obrazek);
 
		$proporcje = $rozmiary[1]/$rozmiary[0]; 
 
		if ($rozmiary[1] <= $rozmiar)
		{
			$nowa_szerokosc = $rozmiary[0];
			$nowa_wysokosc = $rozmiary[1];
		}else{
			$nowa_wysokosc = $rozmiar;
			$nowa_szerokosc = abs($nowa_wysokosc/$proporcje);
		}
 
		$obrazek_docelowy=ImageCreateTrueColor($nowa_szerokosc,$nowa_wysokosc) or die('Blad podczas tworzenia obrazka');
		$obrazek_zrodlowy=ImageCreateFromJPEG($obrazek) or die('Problem z otwarciem zrodlowego obrazka');
		ImageCopyResized($obrazek_docelowy,$obrazek_zrodlowy,0,0,0,0,$nowa_szerokosc,$nowa_wysokosc,ImageSX($obrazek_zrodlowy),ImageSY($obrazek_zrodlowy)) or die('Problem podczas skalowania');
		ImageJPEG($obrazek_docelowy,$obrazek_min,$jakosc) or die('Problem podczas zapisywania, sprawdz CHMODY');
		imagedestroy($obrazek_docelowy);
	}
 
	// Wczytywanie pliku template/upload_done.tpl ktory zawiera szablon uploadu po wykonaniu.
 
		require_once("template/upload_done.tpl");
 
}else{
 
	// Wczytywanie pliku template/upload_start.tpl ktory zawiera szablon formularza uploadu.
 
		require_once("template/upload_start.tpl");
 
}
 
?>
[PHP] pobierz, plaintext 

Wszystko bylo by pieknie ale mam problem, nie wiem co z tym zrobic:

[PHP] pobierz, plaintext 
<?php
$obrazek_zrodlowy=ImageCreateFromJPEG($obrazek) or die('Problem z otwarciem zrodlowego obrazka');
?>
[PHP] pobierz, plaintext 

czyli jezeli plik do uploadu bedzie jpeg bedzie OK, ale jak gif lub png no to klapa :/ nie wiecie moze co z tym fantem zrobic? (jak sprawić aby skrypt rozpoznawał co mu sie podaje i automatycznie zmieniał ImageCreateFromJPEG na np ImageCreateFromGIF)

Pliki skryptu: http://www.sendspace.com/file/ywwpdf (skrypt poprawiony, obsluguje pliki jpg, png, gif)

Dzialanie skryptu mozecie sobie sprawdzic tutaj: http://smallville.pl/upload/

Pozdravki

Ten post edytował Nightwalk 13.03.2006, 01:17:02

[chomiczek]

hmm.. a nie łatwiej np:

[PHP] pobierz, plaintext 
<?php
if (strpos($_FILES['image']['name'],'.jpg')!='') {
	//tutaj wykonujesz wszystko to co potrzebujesz do zrobienia miniaturki
}
?>
[PHP] pobierz, plaintext 

po co takie coś?:

$ext = pathinfo( $uzyj_nazwy, PATHINFO_EXTENSION );

nie lepiej użyć takiej konstrukcji:

[PHP] pobierz, plaintext 
<?php
$ext = imagecreatefromjpeg("lokalizacjaKataloguZUplowanymiFotami/".$plik.".jpg");
?>
[PHP] pobierz, plaintext 

P.S. Proponuje użyć imagecopyresampled" title="Zobacz w manualu php" target="_manual zamiast imagecopyresized" title="Zobacz w manualu php" target="_manual

P.S. 2. Proponuje używać

[PHP] pobierz, plaintext 
<?php
 if (move_uploaded_file()) {
 //tutaj operacja na zdjęciu
}
 
?>
[PHP] pobierz, plaintext 

Jest to o tyle dobre, że dopiero kiedy plik został pomyślnie uploadowany możesz wykonywać na nim jakieś operacje, a póki co Ty używasz move_uploaded_file() ale i tak wykonujesz całą reszte.