Загрузка картинок

[Dimitry Wolotko]

Есть самописный скрипт, вс? вроде работает, но хочу спросить - мб посоветуете что - нибудь?

Из того что я буду дописывать - проверка на расширение, если только.

Задача скрипта - получить картинку, залить е?, положить е? имя в бд.

<?php
include('../config.php');
MYSQL_CONNECT($hostname,$username,$password) OR DIE("Не могу создать соединение ");
@mysql_select_db("$dbName") or die("Не могу выбрать базу данных ");
$pageget = $_GET['id'];
if($pageget != 'upload') {
echo '<form enctype="multipart/form-data" action="images.php?id=upload" method="post">
<input type="hidden" name="MAX_FILE_SIZE" value="1000000">
Send this file: <input name="userfile" type="file">

<input type="text" name="number">
<input type="submit" value="Send File">
</form>';
}
else {
$id = date("U");
$filename = $id + $id - $id / 2;
$filename = ceil($filename);
$nll = 0;
if (empty($_FILES['userfile']['name'])) {$_FILES['userfile']['name'] = $nll;$css = 'none';} else {$_FILES['userfile']['name'] = $filename;$css = 'block';}
$uploaddir = 'C:/www/home/test1.ru/www/images/';
if (move_uploaded_file($_FILES['userfile']['tmp_name'], $uploaddir.$_FILES['userfile']['name']))
{
 print "<span style='color:green;'><b>Все хорошо ... Загрузка удачна.</b></span>
";
}
else
{
 print "<span style='color:red;'><b>Ошибка!</b></span>
";
}

echo '
<img src="../../images/'. $_FILES['userfile']['name'] .'" style="display:'. $css .';padding:20;" />';
if($_FILES['userfile']['name'] != '0') 
$inputword = "INSERT INTO images VALUES('$number', '$filename')"; 
mysql_query($inputword);

//echo "Ошибка базы данных. MySQL пишет: ", mysql_error();

}
?>

[S ... (delete)]

1. Я бы не стал делать вс? в одном файле, MVC - не зря придумали!

2. Используй функцию is_uploaded_file

3. Если действительно нужно, чтобы была картинка используй функцию getimagesize или exif_imagetype

4. Не смог найти где объявляется переменная $number, она вообще нужна?

[Dimitry Wolotko]

1 - возможно;

2 - спасибо;

3 - спасибо;

4 -

[S ... (delete)]

4 -

Ну и?

А где

$number = isset($_POST['number']) ? (int)$_POST['number'] : 0;

И зачем нужна эта переменная?

[Dimitry Wolotko]

Она нужна, а за "интовку" спасибо, впишу.