Czy ten kod jest dobry i optymalny? Opcje

[Balus]

[PHP] pobierz, plaintext 
<?php
$haslo = 'tajnehaslo';
$f_haslo = $_POST['haselko'];
$p_tmp = $_FILES['pliczek']['tmp_name'];
$p_nazwa = $_FILES['pliczek']['name'];
$p_rozmiar = $_FILES['pliczek']['size'];
// $p_typ = $_FILES['pliczek']['type'];
// $p_blad = $_FILES['pliczek']['error'];
 
if($f_haslo != $haslo)
{
	echo 'Niepoprawne haslo!';
}
else
{
	if(is_uploaded_file($f_tmp))
	{
		move_uploaded_file($p_tmp, '/pliczki/'$p_nazwa);
		echo 'Plik <strong>'.$p_nazwa.'</strong> ('.$p_rozmiar.') zostal wrzucony na serwer!';
	}
	else
	{
		echo 'Blad podczas wrzucania pliku!';
	}
}
?>		
[PHP] pobierz, plaintext 

Tak jak w temacie (IMG:style_emoticons/default/winksmiley.jpg)
Czy ten kod jest dobry i optymalny?
Szczególnie chodzi mi o move_uploaded_files

Ten post edytował Balus 30.11.2009, 15:30:25

[MateuszS]

Tzn jeżeli jest to do jakiegoś Hostingu czy coś to trochę źle, bo ci ktoś może plik PHP wrzucić na serwer a potem rzeźnia - po stronie ;D Właściwie nawet jeżeli tylko ty z tego korzystasz powinieneś jakoś to zabezpieczyć. No i skąd wiesz że plik zostanie wrzucony?

[PHP] pobierz, plaintext 
@ $a=move_uploaded_file($p_tmp, '/pliczki/'$p_nazwa);
if($a==true) 
echo 'Plik <strong>'.$p_nazwa.'</strong> ('.$p_rozmiar.') zostal wrzucony na serwer!';
else
echo 'Blad';
[PHP] pobierz, plaintext 

[phpion]

O jaką optymalność Ci chodzi? W takim krótkim kodzie ciężko cokolwiek spaprać. A jednak:
- niepotrzebnie tworzysz tyle zmiennych dla $_FILES,
- komunikat błędu nie sprawdza czy upload się powiódł, tylko czy jest plik do uploadu (dodaj analogiczny warunek dla funkcji move_uploaded_file).

[Balus]

[PHP] pobierz, plaintext 
<?php
$haslo = 'tajnehaslo';
$f_haslo = $_POST['haselko'];
$p_tmp = $_FILES['pliczek']['tmp_name'];
$p_nazwa = $_FILES['pliczek']['name'];
$p_rozmiar = $_FILES['pliczek']['size'];
 
if($f_haslo != $haslo)
{
	echo 'Niepoprawne haslo!';
}
else
{
	if(is_uploaded_file($f_tmp))
	{
		$przenies = move_uploaded_file($p_tmp, $p_nazwa);
		if($przenies == true)
		{
			echo 'Plik <strong>'.$p_nazwa.'</strong> ('.$p_rozmiar.') zostal wrzucony na serwer!';
		}
		else
		{
			echo 'Blad podczas przenoszenia pliku!';
		}
	}
	else
	{
		echo 'Blad podczas wrzucania pliku!';
	}
}
?>
[PHP] pobierz, plaintext 

A teraz co powiecie? :]

[darko]

[PHP] pobierz, plaintext 
$przenies = move_uploaded_file($p_tmp, $p_nazwa);
if($przenies == true)
{
  echo 'Plik <strong>'.$p_nazwa.'</strong> ('.$p_rozmiar.') zostal wrzucony na serwer!';
}
else
{
  echo 'Blad podczas przenoszenia pliku!';
}
[PHP] pobierz, plaintext 

zamieniłbym na

[PHP] pobierz, plaintext 
if(move_uploaded_file($p_tmp, $p_nazwa)) {
  echo 'Plik <strong>'.$p_nazwa.'</strong> ('.$p_rozmiar.') zostal wrzucony na serwer!';
}
else {
  echo 'Blad podczas przenoszenia pliku!';
}
[PHP] pobierz, plaintext 

[Balus]

[PHP] pobierz, plaintext 
if(move_uploaded_file($p_tmp, $p_nazwa)) {
  echo 'Plik <strong>'.$p_nazwa.'</strong> ('.$p_rozmiar.') zostal wrzucony na serwer!';
}
else {
  echo 'Blad podczas przenoszenia pliku!';
}
[PHP] pobierz, plaintext 

if(move_uploaded_file($p_tmp, $p_nazwa)) {
pierw chyba trzeba php rozkazac przeniesc ten plik zeby potem sprawdzic czy zostal przeniesiony (IMG:style_emoticons/default/smile.gif)
poprawcie mnie jesli zle mowie (IMG:style_emoticons/default/tongue.gif)

[darko]

Nie mogę coś edytować, więc napiszę nowego posta. Popracowałbym na tym
1. $f_haslo = $_POST['haselko']; - w ogóle nie filtrujesz danych z formularza
2. przed move_uploaded_file sprawdzałbym za pomocą ini_get czy $p_rozmiar jest mniejszy/równy maksymalnemu dopuszczalnemu i jesli jest, dopiero move_uploaded_file

poprawcie mnie jesli zle mowie

Poprawiam: źle mówisz, sam sprawdź

Ten post edytował darko 30.11.2009, 15:52:22

[Balus]

1.

[PHP] pobierz, plaintext 
$f_haslo = strip_tags(HTMLSpecialChars($_POST['haselko']));
[PHP] pobierz, plaintext 

Tyle wystarczy czy jeszcze jkaieś metody filtrowania polecacie? (IMG:style_emoticons/default/smile.gif)
2

[PHP] pobierz, plaintext 
if($p_rozmiar >= 666)
{
...
}
[PHP] pobierz, plaintext 

cos takiego? (IMG:style_emoticons/default/smile.gif)

Ten post edytował Balus 30.11.2009, 15:55:28

[darko]

Z wartości ini_get("upload_max_filesize") usuń literę "M" na końcu, przelicz sobie na bajty tę wartość, bo z tego, co pamiętam $_FILES["jakis_name"]["size"] zwraca rozmiar w bajtach i potem:
sprawdź czy $_FILES["jakis_name"]["error"] == 0 (czyli wszystko OK) i:

if($p_rozmiar <= $przeliczone_na_bajty && $_FILES["jakis_name"]["error"] == 0) {
move_uploaded_file...

Ten post edytował darko 30.11.2009, 16:05:57

[wookieb]

przelicz sobie na bajty wartość ini_get("upload_max_filesize"), bo z tego, co pamiętam $_FILES["jakis_name"]["size"] zwraca rozmiar w bajtach i potem:
if($p_rozmiar <= $przeliczone_na_bajty) {

maksymalny rozmiar wysyłanego pliku

[Balus]

Ok dla wszystkich pomógł (IMG:style_emoticons/default/winksmiley.jpg)
troszkę pokombinuję i odezwę się (IMG:style_emoticons/default/smile.gif)
Proszę o nie zamykanie tematu (IMG:style_emoticons/default/smile.gif)