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