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]

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