2014-10-30 6 views
-1

Я построил форму для своего сайта, используя php. После отправки формы содержимое отправляется мне по электронной почте, и пользователь перенаправляется на другую страницу. Код отлично работает, когда я полностью заполняю форму. Однако одно из полей - это загрузка файла, и когда я ничего не загружаю, форма по-прежнему отправляется, но приводит к 500-страничной странице ошибок сервера. Как я могу сделать так, чтобы форма по-прежнему перенаправлялась на нужную страницу, когда файл не был загружен? Я включил мой код нижеКак я могу правильно обработать форму php?

<?php 
$email_to = "[email protected]"; 
$email_subject = "Completed Questionnaire"; 
$name = $_POST['name']; 
$gradyear = $_POST['gradyear']; 
$hschool = $_POST['hschool']; 
$hometown = $_POST['hometown']; 
$email_from = $_POST['email']; 
$DOB = $_POST['DOB']; 
$GPA = $_POST['GPA']; 
$height = $_POST['height']; 
$weight = $_POST['weight']; 
$about = $_POST['about']; 

$message = "Form details below.\n\n"; 
function clean_string($string) { 
$bad = array("content-type","bcc:","to:","cc:","href"); 
return str_replace($bad,"",$string);} 

$message .= "Name: ".clean_string($name)."\n"; 
$message .= "Graduation Year: ".clean_string($gradyear)."\n"; 
$message .= "High School: ".clean_string($hschool)."\n"; 
$message .= "Hometown: ".clean_string($hometown)."\n"; 
$message .= "Email: ".clean_string($email_from)."\n"; 
$message .= "Date of Birth: ".clean_string($DOB)."\n"; 
$message .= "GPA: ".clean_string($GPA)."\n"; 
$message .= "Height: ".clean_string($height)."\n"; 
$message .= "Weight: ".clean_string($weight)."\n"; 
$message .= "About: ".clean_string($about)."\n"; 


/* GET File Variables */ 
$tmpName = $_FILES['attachment']['tmp_name']; 
$fileType = $_FILES['attachment']['type']; 
$fileName = $_FILES['attachment']['name']; 

/* Start of headers */ 
$headers = "From: $email_from"; 

if (file($tmpName)) { 
/* Reading file ('rb' = read binary) */ 
$file = fopen($tmpName,'rb'); 
$data = fread($file,filesize($tmpName)); 
fclose($file); 

/* a boundary string */ 
$randomVal = md5(time()); 
$mimeBoundary = "==Multipart_Boundary_x{$randomVal}x"; 

/* Header for File Attachment */ 
$headers .= "\nMIME-Version: 1.0\n"; 
$headers .= "Content-Type: multipart/mixed;\n" ; 
$headers .= " boundary=\"{$mimeBoundary}\""; 

/* Multipart Boundary above message */ 
$message = "This is a multi-part message in MIME format.\n\n" . 
"--{$mimeBoundary}\n" . 
"Content-Type: text/plain; charset=\"iso-8859-1\"\n" . 
"Content-Transfer-Encoding: 7bit\n\n" . 
$message . "\n\n"; 

/* Encoding file data */ 
$data = chunk_split(base64_encode($data)); 

/* Adding attchment-file to message*/ 
$message .= "--{$mimeBoundary}\n" . 
"Content-Type: {$fileType};\n" . 
" name=\"{$fileName}\"\n" . 
"Content-Transfer-Encoding: base64\n\n" . 
$data . "\n\n" . 
"--{$mimeBoundary}--\n"; 
} 

@mail($email_to, $email_subject, $message, $headers); 
header("Location: http://myURL.com/contact.html"); 

?> 
+0

Вы не можете сужать его вниз больше, чем это? –

+2

Вы можете просмотреть этот пример на веб-сайте PHP: http://php.net/manual/en/features.file-upload.php. Вы - код, предполагающий наличие файла. Стоит использовать 'isset',' in_array', 'empty' или комбинацию этих функций для обеспечения отправки файла, а массив' $ _FILES' не пуст. Если он пуст (то есть файл не был отправлен, пропустите чтение файла ... – War10ck

+0

удалите '@' петь из '@mail ($ email_to, $ email_subject, $ message, $ headers);' и добавьте его в переменную : '$ success = mail ($ email_to, $ email_subject, $ message, $ headers);' и затем 'var_dump ($ success);' что он говорит? – vaso123

ответ

0

Вы должны проверить в первую очередь о том, что файл был действительно загружен, прежде чем манипулируя файл:

if (! empty($_FILES)) { 
//file manipulation... 
}