SereBen
Да. Это код - пример того как писать ненадо. В шести строчках твоего кода я нашел аж 7 недостатков:
1. Информация о загруженных файлах лежит в мвссиве $_FILES
Его можно увидеть с помощью функции [m]phpinfo[/m] либо вывев функциями [m]print_r[/m] и [m]var_dump[/m]
2. Для копирования загруженных файлов существует функция [m]move_uploaded_file[/m]
3. Перед е1 применением нужно проверять наличие файла функцией [m]is_uploaded_file[/m]
4. @ тут не нужно. Забудь вообще о существовании такого оператора. Если хочешь отключить вывод ошибок это делается установкой параметра конфигурации
display_errors
Например в .htaccess так
PHP:
php_value display_errors 0 #либо 1 в процессе отладки.
php_value error_reporting 2047
php_value log_errors 1
либо в скриптах так:
PHP:
ini_set('display_errors',0);
error_reporting(E_ALL);
ini_set('log_errors',1);
5. Элементы массива
всегда заключаются в кавычки
6. Для прверки "пустоты" строк существует функция [m]empty[/m]
7. Кусок кода:
PHP:
"$image" , "cars/$image[name]"
следует переписать как (акцент на работе со строками):
PHP:
$image , 'cars/'.$image[name]
От сюда следует что:
7.1 массивы нужно всегда выносить за приделы строк.
7.2 Переменные незачем заключать в двойные кавычки.
7.3 Строки в которых нет переменных нужно заключать в одинарные кавычки.
