2012-11-22 74 views
1

我无法将表单数据插入到数据库中。我可以连接到数据库,因为它不会引发错误,但不会插入任何信息。以下是我的代码,任何帮助将不胜感激。未将表单数据插入数据库的PHP脚本

<?php 
// string checking 
function isValid($str) { 
    if(!preg_match('/[^A-Za-z0-9.-]/', $str)) { 
    return true; 
    } else { 
    return false; 
    } 
} 


// Check for form submission 
if(isset($_POST['submit'])){ 

// Get the POST data 
$agree = $_POST['agree']; 
$firstname = $_POST['firstname']; 
$surname = $_POST['surname']; 
$business = $_POST['business']; 
$state = $_POST['state']; 
$email = $_POST['email']; 

// If the T&C box has been ticked 
if($agree){ 

    // Validate the POST data 
    $validationError = ''; 

    // Name Validation 
    if($firstname == ''){ 
     $validationError .= "Please enter your first name.\n"; 
    } else { 
     if(is_numeric($firstname)){ 
     $validationError .= "The first name you have supplied cannot contain numbers.\n"; 
     } 
     if(strlen($firstname) > 50) { 
     $validationError .= "The first name you have supplied must be less than 50 characters.\n"; 
     } 
     if(strlen($firstname) < 3) { 
     $validationError .= "The first name you have supplied is too short.\n"; 
     } 
     if(isValid($firstname) == false){ 
     $validationError .= "The first name cannot contain special characters.\n"; 
    } 
    } 

if($surname == ""){ 
     $validationError .= "Please enter your Surname.\n"; 
    } else { 
     if(is_numeric($surname)){ 
     $validationError .= "The Surname you have supplied cannot contain numbers.\n"; 
     } 
     if(strlen($surname) > 50) { 
     $validationError .= "The Surname you have supplied must be less than 50 characters.\n"; 
     } 
     if(strlen($surname) < 3) { 
     $validationError .= "The Surname you have supplied is too short.\n"; 
     } 
     if(isValid($surname) == false){ 
     $validationError .= "The Surname cannot contain special characters.\n"; 
    } 
    } 


if($state == ''){ 
     $validationError .= "Please select your state.\n"; 
} 

    // Email validation 

    // Function to validate email addresses, taken from here: http://www.linuxjournal.com/article/9585 
    function check_email_address($email) { 

     // Set up regular expression strings to evaluate the value of email variable against 
     $regex1 = '/^[_a-z0-9-][^()<>@,;:\\"[] ]*@([a-z0-9-]+.)+[a-z]{2,4}$/i'; 

     // Run the preg_match function on regex 1 
     if (!preg_match("/([\w\-]+\@[\w\-]+\.[\w\-]+)/",$email)) { 
      return false; 
     } else { 
      return true;  
     } 
    } 

    if($email != ''){ 
     if(!check_email_address($email)) { 
     $validationError .= "The email address does not appear to be correct, please try again.\n"; 
     } 
    } else { 
     $validationError .= "Please enter your email address.\n"; 
    } 

    //Start the mySQL connection 

    if($validationError == ''){ 
    $link = mysql_connect('localhost', '--removed--', '--removed--'); 

    if (!$link) { 
     $validationError .= "There was an error connecting to the database. Please contact us so that we can fix the problem.\n"; 
    } else { 

     // Select the db 
     mysql_select_db("keyinv_seminar", $link); 

     // Check for an existing entry under that email address 
     $checkQuery = 'SELECT * FROM seminar WHERE email="' . mysql_real_escape_string($email) . '"'; 
     $result = mysql_query($checkQuery, $link); 

     if (mysql_num_rows($result) != 0) { 
     $validationError .= "There is already an entry in the competition, using that email address.\n"; 
     } else { 
     // There is no existing entry, update the db 
     $insertQuery = "INSERT INTO seminar (firstname, surname, business, state, email) VALUES ('" . mysql_real_escape_string($firstname) . "', '" . mysql_real_escape_string($surname) . "', '" . mysql_real_escape_string($business) . "', " . mysql_real_escape_string($state) . ", '" . mysql_real_escape_string($email) . "')"; 

     $result = mysql_query($insertQuery, $link); 

     } 

     // Close the connection 
     if($link){ 
     mysql_close($link); 

     if($validationError == ''){ 
      header('Location: thankyou.php'); 
     } 
     } 
    } 
    } 
} else { 
    $validationError = "You must accept the Terms and Conditions \nin order to enter this contest."; 
} 
} 
?> 
+0

您应该尝试捕获mysql错误而不是验证错误。请参阅mysql_error()php函数。 – rayfranco

+1

正如人们经常说的,你不应该再使用mysql_query了,但是我相信你的mysql_real_escape_string($ state)(在插入时)的输入需要在它周围标记。 - 检查F4r-20的帖子以获得更好的总结。 – Lucas

回答

5

查看您所在INSERT查询..

$insertQuery = "INSERT INTO seminar (firstname, surname, business, state, email) VALUES ('" 
    . mysql_real_escape_string($firstname) . "', '" 
    . mysql_real_escape_string($surname) . "', '" 
    . mysql_real_escape_string($business) . "', " 
    . mysql_real_escape_string($state) . ", '" 
    . mysql_real_escape_string($email) . "')"; 

你已经错过了周围的$state值的一些单引号。更改为:

$insertQuery = "INSERT INTO seminar (firstname, surname, business, state, email) VALUES ('" 
    . mysql_real_escape_string($firstname) . "', '" 
    . mysql_real_escape_string($surname) . "', '" 
    . mysql_real_escape_string($business) . "', '" 
    . mysql_real_escape_string($state) . "', '" 
    . mysql_real_escape_string($email) . "')"; 
+0

另外,他不会在任何地方回显验证错误。 –

+0

嗯,我们不知道这个剧本之后有很好的回应。 – George

+0

+1格式化无尽的线条也有助于赶上这种错别字 –