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

[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