Walidacja uploadu obrazka Opcje

[japek]

Witam, czy da sie przed dodaniem obrazka na serwer sprawdzić jego szerokość i format aby przepuszczał tylko jpg bmp i gif'y? kombinowałem troche z getimagewidth ale coś nie idzie no i nie wiem jak sprawdzić rozszerzenie pliku
Po dodaniu na serwer to już coś tam działa ale chciałbym przez upload'em aby zapobiec ładowania byle czego. Pozdrawiam

[hwao]

co do rozszerzenia

[PHP] pobierz, plaintext 
<?php
$sFile = 'costam.costa.jpg';
$sEx = array_pop( explode( '.', $sFile ) );
echo $sEx;
?>
[PHP] pobierz, plaintext 

Co do wielkosci to

[PHP] pobierz, plaintext 
<?php
$aInfo = getimagesize( $_FILES['nameInputa']['tmp_name'] );
echo '<pre>'.print_r( $aInfo, true ).'</pre>';
?>
[PHP] pobierz, plaintext 

[NuLL]

Czlowieku - co ty za bzdety wklejasz Przeciez to co wkleiles skaluje obrazek

I zastosuj bbCode w swoim poscie.

[Bizard]

czlowieku x2 -wiesz , dopisalem tam ze ma to po swojemu przerobic , a wystarczylo by tam zmienic 4 linijki a byl by dobry skrypt o który mu chodzilo...

[NuLL]

czlowieku x2 -wiesz , dopisalem tam ze ma to po swojemu przerobic , a wystarczylo by tam zmienic 4 linijki a byl by dobry skrypt o który mu chodzilo...

Mam do Ciebie mala prosbe - nie pisz nic wiecej jak nie wiesz co piszesz.

[japek]

[PHP] pobierz, plaintext 
<?php
$obrazek=$_GET["upload"];
	$szerokosc=imagesx($obrazek);
	if ($szerokosc>400) {
		$obrazek=FALSE;
?>
[PHP] pobierz, plaintext 

Taką walidacje zrobiłem no i wywala błąd "supplied argument is not a valid Image resource" co jest źle? chce zablokować możliwość wstawienia większego obrazka niz 400 pix.

Upload jest z formularza

[HTML] pobierz, plaintext 
<input type="file" name="upload">
[HTML] pobierz, plaintext 

Ten post edytował japek 25.03.2006, 14:32:35

[Vengeance]

$obrazek=$_GET["upload"];

przy plikach stosuje się $_FILES a nie $_GET
poczytaj o tym na php.net/upload

[japek]

Ale mi chodzi o ten error "supplied argument is not a valid Image resource" a co do GET a FILES to wiem tylko z get chciałem walidacje przed wrzucaniem zdjęcia do bazy, zresztą ten sam error jest jak sprawdzam po dodaniu do bazy :/ juz tyle kombinowałem i dalej nic - oczywiście dodawać zdjęcia i wyświetlać to luz ale chce aby nie dodawał większych niż x pix.

[Balon]

blad Ci wywala bo sprawdza wymiary czegos co nie istnieje...

[japek]

dlatego pisze o tym ale juz sobie poradziłem bo tu widze o porade ciężko. Chciałem walidacje przez dodaniem na serwer, jednak chyba sie nie da więc zrobiłem po dodaniu.

EOT

Ten post edytował japek 25.03.2006, 22:26:38

[Prph]

[PHP] pobierz, plaintext 
<?php
 
 
if(isset($_POST['submit']))
{
  $boolError = false;
  $arrFile = $_FILES['photo_filename'];
 
  if(empty($arrFile['name']))
  {
	$strErrorMsg .= ERROR_GALLERY_PHOTO_FILE;
	$boolError = true;
  }
 
  if($arrFile['size'] > 102400)
  {
	$strErrorMsg .= ERROR_GALLERY_PHOTO_SIZE;
	$boolError = true;
  }
 
  if(!$boolError)
  {
	$strSummary = addslashes($_POST['photo_name']);
	$intAlbumId = $_POST['album_id'];
 
	if($arrFile['error'] > 0)
	{
	  switch($arrFile['error'])
	  {
		case 1:
		  $strErrorMsg .= ERROR_GALLERY_PHOTO_SIZE;
		  break;
 
		case 2:
		  $strErrorMsg .= ERROR_GALLERY_PHOTO_SIZE;
		  break;
 
		case 3:
		  $strErrorMsg .= ERROR_GALLERY_PHOTO_BROKEN;
		  break;
 
		case 4:
		  $strErrorMsg .= ERROR_GALLERY_PHOTO_EMPTY;
		  break;
	  }
	  app_redirect(PATH_SERVER.'?load_module=gallery&action=photo_add&album_id='.$_POST['album_id'].'&msg='.$strErrorMsg);
	}
 
	$arrFile['name'] = str_replace('ą', 'a', $arrFile['name']);
	$arrFile['name'] = str_replace('ę', 'e', $arrFile['name']);
	$arrFile['name'] = str_replace('ó', 'o', $arrFile['name']);
	$arrFile['name'] = str_replace('ł', 'l', $arrFile['name']);
	$arrFile['name'] = str_replace('ż', 'z', $arrFile['name']);
	$arrFile['name'] = str_replace('ź', 'z', $arrFile['name']);
	$arrFile['name'] = str_replace('ń', 'n', $arrFile['name']);
	$arrFile['name'] = str_replace('ć', 'c', $arrFile['name']);
	$arrFile['name'] = str_replace('ś', 's', $arrFile['name']);
 
	$strPath = PATH_GALLERY.$intAlbumId.'/';
 
	$strFileName = change_file_name($arrFile['name'], $strPath);
	$resDir = opendir($strPath);
 
	if(!$resDir)
	{
	  $strErrorMsg .= ERROR_GALLERY_PHOTO_OPEN_DIR;
	  app_redirect(PATH_SERVER.'?load_module=gallery&action=photo_add&album_id='.$_POST['album_id'].'&msg='.$strErrorMsg);
	}
 
	$strDest = $strPath.$strFileName;
 
	$strDestMin = $strPath.'min/'.$strFileName;
 
	if(is_uploaded_file($arrFile['tmp_name']))
	{
	  if(!move_uploaded_file($arrFile['tmp_name'], $strDest))
	  {
		$strErrorMsg .= ERROR_GALLERY_PHOTO_COPY;
		app_redirect(PATH_SERVER.'?load_module=gallery&action=photo_add&album_id='.$_POST['album_id'].'&msg='.$strErrorMsg);
	  }
 
?>
[PHP] pobierz, plaintext 

Framgemt mojego starego kodu. Na poczatku sprawdza czy przeslano plik. Nastepnie sprawdza Errory - te dodane przez php. Pozniej zamienia polskie na zwykle znaki. Na koncu przenosi do odpowiedniego katalogu.

[huntercs]

zamiast usywać polskie znaki, spacje i inne specjanle znaki wolę tworzyć unikalne nazwy np.:

[PHP] pobierz, plaintext 
<?php
$filename = md5(uniqid(rand(), true));
?>
[PHP] pobierz, plaintext 

czy coś w tym stylu

[Prph]

Przyznam, że naprawdę dobre
Mozna przyjąć, że nigdy nazwa nie będzie taka sama?

[huntercs]

jest takie prawdopobobieństwo więc najlepiej zorbić prostą rekurencyjną funkcje:

[PHP] pobierz, plaintext 
<?php
function GetUniqueName();
{
$filename = md5(uniqid(rand(), true));
if(file_exists($filename)) GetUniqueName();
else return $filename;
}
$name=GetUniqueName();
?>
[PHP] pobierz, plaintext 

i nazwy będą unikatowe

[Prph]

Czemu rekurencja? Nie lepiej na do-while?

[huntercs]

nie , generalnie chodzi o to że za pierwszym razem będzie już losował unikatową nazwę (małe prawdopodobieństwo że już taka istnieje), później sprawdza czy przypadkiem nie istnieje już plik o takiej nazwie i zwraca nazwę, nie potrzebne do while(), a pozatym można zapoznać się z rekurencją

Ten post edytował huntercs 27.03.2006, 18:12:16

[Prph]

Wlasnie dlatego do-while. w warunku dajesz czy plik istnieje a w ciele petli zmieniasz nazwe W idealnym przypadku wykona sie dokladnie raz.

Rekurencja moim zdaniem wprowadza troche zamieszania. Cale szczescie, ze to jest tak banalny przyklad jej wykorzystania, ze nie trtzeba wysilac tego glowy. Mialem w poprzednim semestrze wyklady z Ady i tam facet pokazywal nam rekurencje - tamte byly hardcorowe

Pozostaje przy do-while

Pozdrawiam, Adrian.