Forum PHP.pl > [php] Upload zdjęć na server

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [php] Upload zdjęć na server

ArczeX

2.01.2008, 18:12:30

Witam...

Mam taki zajebiś... mały problem lecz dla mnie jest on jednak zbyt duży

A mianowicie znalazłem sobie skrypcik do wgrywania plików na serwer działa bardzo dobrze nie ma z nim problemów lecz chciałem go udoskonalić o funkcję która by pozwalała jedynie na wgrywanie zdjęć jpg i gif i żadnego innego pliku poza tymi dwoma formatami ;]

po godzinnych poszukiwaniach dowiedziałem sie że trzeba cos zmotać z $_FILES['plik']['type'] ale nie wiem jak to zmodyfikować znalazłem pare wzorów w necie ale nie działały mimo tego że wyskakiwało w przeglądarce "niewłaściwy plik" to i tak znajdował się on na servie ;/

Prosze o pomoc !

oto skrypcik który wypadało by przerobić ;]

KO DO UMIESZCZENIA NA STRONIE

[HTML] pobierz, plaintext 
<html>
<body>
 <form action="plik2.php" method="POST" ENCTYPE="multipart/form-data">
  <input type="file" name="plik"/>
  <input type="submit" value="Wyślij plik"/>
 </form>
</body>
</html>
[HTML] pobierz, plaintext

PLIK PHP

[PHP] pobierz, plaintext 
<?php
 
$max_rozmiar = 1024*1024;
if (is_uploaded_file($_FILES['plik']['tmp_name'])) {
  if ($_FILES['plik']['size'] > $max_rozmiar) {
  echo 'Błąd! Plik jest za duży!';
  } else {
		echo 'Odebrano plik. Początkowa nazwa:   '.$_FILES['plik']['name'];
		echo '<br/>';
		if (isset($_FILES['plik']['type'])) {
		  echo 'Typ: '.$_FILES['plik']['type'].'<br/>';
		 }
  move_uploaded_file($_FILES['plik']['tmp_name'],
$_SERVER['DOCUMENT_ROOT'].'/foto/'.$_FILES['plik']['name']);
  }
} else {
	 echo 'Błąd przy przesyłaniu danych!';
}
 
?>
[PHP] pobierz, plaintext

Hazel

2.01.2008, 18:22:06

[PHP] pobierz, plaintext 
<?php
if ($_FILES['plik']['type'] != 'jpeg' && $_FILES['plik']['type'] != 'pjpeg' && $_FILES['plik']['type'] != 'gif')
{
  echo 'Nieprawidłowy format pliku!';
  exit;
}
?>
[PHP] pobierz, plaintext

Na początku skryptu.

ArczeX

2.01.2008, 18:35:58

Dzięki wielkie ;] wszystko ładnie śmiga ;]

A mam jeszcze takie pytanie chciałbym aby po wysłaniu pliku lub jego nie wysłaniu ( Niewłaściwy plik ) pojawiał się przycisk lub link powrotny do strony

Hazel

2.01.2008, 18:38:48

[PHP] pobierz, plaintext 
<?php
if ($_FILES['plik']['type'] != 'jpeg' && $_FILES['plik']['type'] != 'pjpeg' && $_FILES['plik']['type'] != 'gif')
{
  echo 'Nieprawidłowy format pliku!';
}
else
{
$max_rozmiar = 1024*1024;
if (is_uploaded_file($_FILES['plik']['tmp_name'])) {
  if ($_FILES['plik']['size'] > $max_rozmiar) {
  echo 'Błąd! Plik jest za duży!';
  } else {
		echo 'Odebrano plik. Początkowa nazwa:   '.$_FILES['plik']['name'];
		echo '<br/>';
		if (isset($_FILES['plik']['type'])) {
		  echo 'Typ: '.$_FILES['plik']['type'].'<br/>';
		 }
  move_uploaded_file($_FILES['plik']['tmp_name'],
$_SERVER['DOCUMENT_ROOT'].'/foto/'.$_FILES['plik']['name']);
  }
} else {
	 echo 'Błąd przy przesyłaniu danych!';
}
}
echo '<a href="'.$_SERVER['HTTP_REFERER'].'">Powrót</a>';
?>
[PHP] pobierz, plaintext

lub zamiast ostatniej linijki:

[PHP] pobierz, plaintext 
<?php
echo '<form action="'.$_SERVER['HTTP_REFERER'].'" method="post"><input type="submit" value="Powrót"></form>';
?>
[PHP] pobierz, plaintext

Tylko zamiast tego index.php musisz wpisać adres strony (da się to zrobić w JS za pomocą history.back(), ale zazwyczaj jak można nie stosować JS to się tego nie robi).
Pozdrawiam.

edit: poprawiłem kod, trochę źle się wczytałem - nie o to chodziło, sorry.

ArczeX

2.01.2008, 18:43:44

Upss... za wcześnie sie pochwaliłem jest błąd ;/

po dpoisaniu

[PHP] pobierz, plaintext 
<?php
if ($_FILES['plik']['type'] != 'jpeg' && $_FILES['plik']['type'] != 'pjpeg' && $_FILES['plik']['type'] != 'gif')
{
  echo 'Nieprawidłowy format pliku!';
  exit;
}
?>
[PHP] pobierz, plaintext

Jest taki problem że nie wgrywa żadnego pliku nawet JPG czy GIF cały czas wyskakuje "Nieprawidłowy format pliku!"

Hazel

2.01.2008, 18:48:41

na początku skryptu napisz:

[PHP] pobierz, plaintext 
<?php
print_r($_FILES);
?>
[PHP] pobierz, plaintext

i zwróć wynik.

ArczeX

2.01.2008, 18:53:01

zwrot wygląda następująco

[PHP] pobierz, plaintext 
<?php
Array ( [plik] => Array ( [name] => Golf I 043.jpg [type] => image/pjpeg [tmp_name] => /tmp/phpTMukVZ [error] => 0 [size] => 34336 ) ) Nieprawidłowy format pliku!
?>
[PHP] pobierz, plaintext

Hazel

2.01.2008, 18:54:56

Kurde, sorry, ciężki dzień miałem ;/
Ośmieszyłem się trochę

Ale to już będzie działać.

[PHP] pobierz, plaintext 
<?php
if ($_FILES['plik']['type'] != 'image/jpeg' && $_FILES['plik']['type'] != 'image/pjpeg' && $_FILES['plik']['type'] != 'image/gif')
{
  echo 'Nieprawidłowy format pliku!';
}
else
{
$max_rozmiar = 1024*1024;
if (is_uploaded_file($_FILES['plik']['tmp_name'])) {
  if ($_FILES['plik']['size'] > $max_rozmiar) {
  echo 'Błąd! Plik jest za duży!';
  } else {
		echo 'Odebrano plik. Początkowa nazwa:   '.$_FILES['plik']['name'];
		echo '<br/>';
		if (isset($_FILES['plik']['type'])) {
		  echo 'Typ: '.$_FILES['plik']['type'].'<br/>';
		 }
  move_uploaded_file($_FILES['plik']['tmp_name'],
$_SERVER['DOCUMENT_ROOT'].'/foto/'.$_FILES['plik']['name']);
  }
} else {
	 echo 'Błąd przy przesyłaniu danych!';
}
}
echo '<a href="'.$_SERVER['HTTP_REFERER'].'">Powrót</a>';
?>
[PHP] pobierz, plaintext

I przy okazji Twój drugi problem też chyba rozwiązuje.
P.S. Internet Explorer nie jest najlepszą przeglądarką

ArczeX

2.01.2008, 19:16:34

a teraz wywala mi cos takiego ;/

[PHP] pobierz, plaintext 
<?php
Odebrano plik. Pocz&#353;tkowa nazwa: Golf IV 398.jpg
Typ: image/jpeg
 
Warning: move_uploaded_file(/home.14/m/o/t/motodwus/www/foto/Golf IV 398.jpg) [function.move-uploaded-file]: failed to open stream: No such file or directory in /home.14/m/o/t/motodwus/www/crossy/plik2.php on line 19
 
Warning: move_uploaded_file() [function.move-uploaded-file]: Unable to move '/tmp/php4fzZvM' to '/home.14/m/o/t/motodwus/www/foto/Golf IV 398.jpg' in /home.14/m/o/t/motodwus/www/crossy/plik2.php on line 19
Powrót
?>
[PHP] pobierz, plaintext

....Nie no sorki juz chyba wiem co jest nie tak ;]

franki01

2.01.2008, 19:26:15

Katalog gdzie jest uploadowany plik ma chmod 0777?

ArczeX

2.01.2008, 19:33:50

Cytat(franki01 @ 2.01.2008, 19:26:15 )

Katalog gdzie jest uploadowany plik ma chmod 0777?

Nie nie Folder do wgrywania zdjęć źle umieściłem ;]

A mam jeszcze takie pytanko jak można

Dało by radę dopisać jakiegoś skrypta który zmieniał by nazwy dodawany plików... tzn.

jakiś koleś dodaje plik o nazwie blebleble.jpg a skrypt zmienia jego nazwe dajmy na to na 1.jpg następny koleś dodaje plik blebleble3456.jpg a skrypt zmienia jego nazwe na 2.jpg

Czy była by taka możliwość ?

franki01

2.01.2008, 19:45:51

[PHP] pobierz, plaintext 
<?php
$i = 1;
$e = explode('.', $_FILES['plik']['name']);
$ext = $e[(count($e) - 1)];
 
while(file_exists('katalog/'.$i.'.'.$ext)) {
  $i++;
}
 
$file = 'katalog/'.$i.'.'.$ext;
?>
[PHP] pobierz, plaintext

Nazwa pliku bedzie teraz w $file. Mam nadzieje ze dalej wiesz co z tym zrobic

ArczeX

2.01.2008, 19:59:18

Cytat(franki01 @ 2.01.2008, 19:45:51 )

Nazwa pliku bedzie teraz w $file. Mam nadzieje ze dalej wiesz co z tym zrobic

yyy... troszke kombinowałem ale coś mi nie wychodzi i sorki ale ja dość zielony jestem

mógłbyś tak krok po kroku napisać o co biega

franki01

2.01.2008, 20:02:42

[PHP] pobierz, plaintext 
<?php
$i = 1;
$dir = $_SERVER['DOCUMENT_ROOT'].'/foto/';
$e = explode('.', $_FILES['plik']['name']);
$ext = strtolower($e[(count($e) - 1)]);
 
while(file_exists($dir .$i.'.'.$ext)) {
  $i++;
}
 
$file = $dir .$i.'.'.$ext;
?>
[PHP] pobierz, plaintext

Ten kod wstaw nad move_uploaded_file() a sama funkcja move_uploaded_file() niech wyglada tak:

[PHP] pobierz, plaintext 
<?php
move_uploaded_file($_FILES['plik']['tmp_name'], $file);
?>
[PHP] pobierz, plaintext

ArczeX

2.01.2008, 20:29:38

No to tak...

Zmodyfikowałem koda następująco (mam nadzieje dobrze

)

[PHP] pobierz, plaintext 
<?php
if ($_FILES['plik']['type'] != 'image/jpeg' && $_FILES['plik']['type'] != 'image/pjpeg' && $_FILES['plik']['type'] != 'image/gif')
{
  echo 'Nieprawidłowy format pliku!';
}
else
{
$max_rozmiar = 1024*1024;
if (is_uploaded_file($_FILES['plik']['tmp_name'])) {
  if ($_FILES['plik']['size'] > $max_rozmiar) {
  echo 'Błąd! Plik jest za duży!';
  } else {
		echo 'Odebrano plik. Początkowa nazwa:   '.$_FILES['plik']['name'];
		echo '<br/>';
		if (isset($_FILES['plik']['type'])) {
		  echo 'Typ: '.$_FILES['plik']['type'].'<br/>';
		 }
		 $i = 1;
$dir = $_SERVER['DOCUMENT_ROOT'].'/crossy/foto/';
$e = explode('.', $_FILES['plik']['name']);
$ext = strtolower($e[(count($e) - 1)]);
 
while(file_exists($dir .$i.'.'.$ext)) {
  $i++;
}
 
$file = $dir .$i.'.'.$ext;
  move_uploaded_file($_FILES['plik']['tmp_name'], $file);
$_SERVER['DOCUMENT_ROOT'].'/crossy/foto/'.$_FILES['plik']['name']);
  }
} else {
	 echo 'Błąd przy przesyłaniu danych!';
}
}
echo '<a href="'.$_SERVER['HTTP_REFERER'].'">Powrót</a>';
?>
[PHP] pobierz, plaintext

i niestety wywala mi błąd

[HTML] pobierz, plaintext 
Parse error: syntax error, unexpected ')' in /home.14/m/o/t/motodwus/www/crossy/plik2.php on line 29
[HTML] pobierz, plaintext

Jak znam życie pewnie coś zmaściłem... tylko teraz pytanie co ?

Hazel

2.01.2008, 20:32:36

[PHP] pobierz, plaintext 
<?php
if ($_FILES['plik']['type'] != 'image/jpeg' && $_FILES['plik']['type'] != 'image/pjpeg' && $_FILES['plik']['type'] != 'image/gif')
{
  echo 'Nieprawidłowy format pliku!';
}
else
{
$max_rozmiar = 1024*1024;
if (is_uploaded_file($_FILES['plik']['tmp_name'])) {
  if ($_FILES['plik']['size'] > $max_rozmiar) {
  echo 'Błąd! Plik jest za duży!';
  } else {
		echo 'Odebrano plik. Początkowa nazwa:   '.$_FILES['plik']['name'];
		echo '<br/>';
		if (isset($_FILES['plik']['type'])) {
		  echo 'Typ: '.$_FILES['plik']['type'].'<br/>';
		 }
		 $i = 1;
$dir = $_SERVER['DOCUMENT_ROOT'].'/crossy/foto/';
$e = explode('.', $_FILES['plik']['name']);
$ext = strtolower($e[(count($e) - 1)]);
 
while(file_exists($dir .$i.'.'.$ext)) {
  $i++;
}
 
$file = $dir .$i.'.'.$ext;
  move_uploaded_file($_FILES['plik']['tmp_name'], $file);
  }
} else {
	 echo 'Błąd przy przesyłaniu danych!';
}
}
echo '<a href="'.$_SERVER['HTTP_REFERER'].'">Powrót</a>';
?>
[PHP] pobierz, plaintext

Przed zabieraniem się za takie rzeczy dobrze jest zapoznać się jednak trochę ze składnią języka, manualem i takimi tam głupotami

Cytat(ArczeX)

Zmodyfikowałem koda następująco (mam nadzieje dobrze

)

Pomyśl - jakby było dobrze, to by nie rzucało błędem... Ot tak, na logikę...

franki01

2.01.2008, 20:33:58

Bo ta linia to ciag dalszy funkcji move_uploaded_file(). Usun cala linie 29.

EDIT:
@Hazel byl szybszy

ArczeX

2.01.2008, 21:02:21

Dzięki wam za wszystko

normalnie zajebiaszczo to działa

jak chcecie zobaczyć moją próbną galerie to zapraszam

http://www.motodwusuwy.info/crossy/galeria.php sprawdźcie jak śmiga

brzuchalski

3.01.2008, 00:21:03

try this one:

[PHP] pobierz, plaintext 
<?php
if(!in_array($_FILES['plik']['type'], array('image/jpeg', 'image/pjpeg', 'image/gif'))) {
	  echo 'Nieprawidłowy format pliku!';
 
  } else {
	  $max_rozmiar = 1024*1024;
 
	  if(is_uploaded_file($_FILES['plik']['tmp_name']) && $_FILES['plik']['size'] < $max_rozmiar) {
 
			  echo 'Odebrano plik. Początkowa nazwa: '. $_FILES['plik']['name'] .'
';
			  if (isset($_FILES['plik']['type']))
				  echo 'Typ: '.$_FILES['plik']['type'].'
';
 
			  $d = dir($_SERVER['DOCUMENT_ROOT'] .'/crossy/foto/');
			  $e = (strpos($_FILES['plik']['name'], '.') ? array_pop(explode('.', $_FILES['plik']['name'])) : 'jpg');
			  $i = 1;
			  while($file = $d->read())
				preg_replace('/([0-9]*).'. $e .'/ie', '$i = (1 > $i ? 1 : $i);', $file);
 
			  move_uploaded_file($_FILES['plik']['tmp_name'], $d->path . $i .'.'. $e);
	  } elseif($_FILES['plik']['size'] > $max_rozmiar) {
 
		  echo 'Błąd! Plik jest za duży!';
	  } else {
 
		  echo 'Błąd przy przesyłaniu danych!';
	  }
  }
 
  echo '<a href=\"'.$_server['http_referer'].'\">Powrót';
 
  ?>
[PHP] pobierz, plaintext

troche estetyki kodu ;) </a href=\"'.$_SERVER['http_referer'].'\"> </a href=\"'.$_server['http_referer'].'\">

Hazel

3.01.2008, 08:12:22

Nie dość, że problem rozwiązany, to jeszcze taki kod:
1. Nie jest ani trochę estetyczny (mój też nie jest, ale sam go nie pisałem, jedynie przerabiałem kod ArczeX).
2. Nie działa, bo jest różnica między $_SERVER a $_server.

hacking jak trzeba...

ArczeX

3.01.2008, 10:04:25

Cytat(Hazel @ 3.01.2008, 08:12:22 )

hacking jak trzeba...

To tak jak dla mnie liczy sie sprawność a nie wygląd nie jestem jakimś mistrzem php wiec takie drobiazgi mi nie wadzą ;]
a co do drugiego to nie kapuje... mi wszystko śmiga... zdjęcia się poprwnie wgrywają pod firefoxem jak IE wiec nie czaje ?

Hazel

3.01.2008, 16:26:09

Nie do Ciebie, pisałem do bliżej niezidentyfikowanego osobnika, który celowo wprowadza ludzi w błąd.

brzuchalski

3.01.2008, 19:35:05

to nie było celowe wprowadzanie w błąd tylko efekt szybkiego pisania i wcisnięcia caps looka a co do estetyki kodu to uważasz że pisanie miliona zmiennych w każdej linijce inacze nawiasów używanie nadmiarowych instrukcji jest bardziej estetyczne a co najmniej efektywne ?? bo chyba NIE

Hazel

3.01.2008, 21:00:34

Skoro problem jest rozwiązany, to nie widzę problemu. A jak zaczynałeś, to raczej nie pisałes kodu, w którym w jednej linijce używasz 4 funkcji z różnymi argumentami. Pisząc każdą zmienną w osobnej linijce i dokładając do funkcji, chciałem uczynić kod łatwiejszym do zrozumienia (choć faktcyznie masz rację, że niekoniecznie czytelniejszym). Taka uroda Przedszkola, to ludzie naprawdę niewiele wiedzą o PHP i najlepiej jednak pomóc im zrozumieć, a nie dać regexpa (co by nie mówić, zbędnego w tym przypadku) i jaikes funkcje tablicowe, do niczego niepotrzebne, i do tego jeszcze napisać małymi literami coś, co powinno być napisane wielkimi. Może chciałes dobrze, i może nawet Twój kod jest lepszy, trzeba się było wypowiedzieć przed rozwiązaniem problemu. Proponuję zakończyć.

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.