Передача параметров PHP Post при пустом значении - PullRequest
Новая
       4

Передача параметров PHP Post при пустом значении

3 голосов
/ 16 октября 2011

У меня проблема при публикации значения выбора для проверки на стороне сервера.Когда пользователь не делает выбор, поэтому он все еще выбирается на «Выбрать ...».Он по-прежнему проходит метод empty (), хотя значение пустое. 

<?php

// Email Properties
$email_to = "user@domain.com";
$email_subject = "3DMark3t Contact:";
$first_name = $_POST['first_name'];
$last_name = $_POST['last_name'];
$email = $_POST['email'];
$telephone = $_POST['telephone'];
$comments = $_POST['comments'];
$select = $_POST['select'];

function formError($error) {
    // your error code can go here
    echo "We are very sorry, but there were error(s) found with the form you submitted. ";
    echo "These errors appear below.<br /><br />";
    echo $error."<br /><br />";
    echo "Please go back and fix these errors.<br /><br />";
    die();
}

// Make sure all fields are not empty
if(empty($first_name) || empty($last_name) || empty($email) ||
    empty(telephone) || empty($comments) || empty($select)) {
    // Return error
    formError('Please fill in all of the fields.');       
}

// Regex Tests
$error_message = "";

// Expressions
$email_exp = '/^[A-Za-z0-9._%-]+@[A-Za-z0-9.-]+\.[A-Za-z]{2,4}$/';
$string_exp = "/^[A-Za-z .'-]+$/";

if(!preg_match($email_exp, $email_from)) {
    $error_message .= 'The Email Address you entered does not appear to be valid.<br />';
}
if(!preg_match($string_exp, $first_name)) {
    $error_message .= 'The First Name you entered does not appear to be valid.<br />';
}
if(!preg_match($string_exp, $last_name)) {
    $error_message .= 'The Last Name you entered does not appear to be valid.<br />';
}
if(!empty($error_message)) {
    formError($error_message);
}

$email_message = "3DMark3t Contact: \n";
$email_message .= "First Name: {$first_name} \n";
$email_message .= "Last Name: {$last_name} \n";
$email_message .= "Email: {$email} \n";
$email_message .= "Telephone: {$telephone} \n";
$email_message .= "Select One: {$select} \n";
$email_message .= "Comments: {$comments} \n";

// Create email headers
$headers = 'From: '.$email_from."\r\n".
'Reply-To: '.$email_from."\r\n" .
'X-Mailer: PHP/' . phpversion();
@mail($email_to, $email_subject, $email_message, $headers);  

header("Location /");

HTML: 

<form name="htmlform" method="post" action="php/html_form_send.php">
    <table width="450px">
        <tr>
            <td valign="top">
                <label for="first_name">First Name *</label>
            </td>
            <td valign="top">
                <input  type="text" name="first_name" maxlength="30" size="30">
            </td>
        </tr>
        <tr>
            <td valign="top">
                <label for="last_name">Last Name *</label>
            </td>
            <td valign="top">
                <input  type="text" name="last_name" maxlength="30" size="30">
            </td>
        </tr>
        <tr>
            <td valign="top">
                <label for="email">Email Address *</label>
            </td>
            <td valign="top">
                <input  type="text" name="email" maxlength="30" size="30">
            </td>
        </tr>
        <tr>
            <td valign="top">
                <label for="telephone">Telephone Number</label>
            </td>
            <td valign="top">
                <input  type="text" name="telephone" maxlength="30" size="30">
            </td>
        </tr>
        <tr>
            <td valign="top">
                <label for="select">Select One *</label>
            </td>
            <td valign="top">
                <select name="select">
                    <option value="">Select...</option>
                    <option value="3DModels">3DModels</option>
                    <option value="Graphic Design">Graphic Design</option>
                    <option value="Web Design">Web Design</option>
                    <option value="Tutorials">Tutorials</option>
                    <option value="Report">Report</option>
                    <option value="Requests">Requests</option>
                </select>
            </td>
        </tr>
        <tr>
            <td valign="top">
                <label for="comments">Comments *</label>
            </td>
            <td valign="top">
                <textarea  name="comments" maxlength="1000" cols="32.5" rows="6"></textarea>
            </td>
        </tr>
        <tr>
            <td colspan="2" style="text-align:center">
             <input type="submit" value="Submit">
            </td>
        </tr>
    </table>
</form>

Ответы [ 3 ]

2 голосов
/ 16 октября 2011

Вы тестируете, если установлено $_POST['formGender'], но PHP вернет TRUE для isset() в пустой строке. Отсутствие выбора в вашей форме означает, что $_POST['formGender'] == "", и это считается «установленным» в проверке isset().

Вместо этого проверьте, является ли оно empty(), которое одновременно проверит, определена ли переменная, как isset(), и имеет ли она непустое значение. 

if(empty($_POST['formGender']))
{
  $errorMessage .= "<li>You forgot to select your Gender!</li>";
}

Также проверьте здесь empty() вместо !isset()

 if(!isset($_POST['first_name']) ||
    !isset($_POST['last_name']) ||
    !isset($_POST['email']) ||
    !isset($_POST['telephone']) ||
    !isset($_POST['comments'])) {
 if(empty($_POST['select']) )

Обновление

Я не совсем уверен, что вы пытаетесь сделать с этим. Но 3DModels, Graphic Design, etc не будет частью массива $_POST. Они являются потенциальными значениями $_POST['select']. Я не понимаю, каково ваше намерение здесь. 

if(empty($_POST['select']) )
{
  $var3dmodels = $_POST['3DModels'];
  $vargraphic_design = $_POST['Graphic Design'];
  $varweb_design = $_POST['Web Design'];
  $vartutorials = $_POST['Tutorials'];
  $varreport = $_POST['Report'];
  $varrequests = $_POST['Requests'];
}

Источник вашей проблемы:

Я наконец вижу, где ваша ошибка. Вместо $error_message вы используете $errorMessage для $_POST['select']: 

if(empty($_POST['select']))
{
  // Oops...
  $errorMessage .= 'The selection you made does not appear to be valid.<br />';

  // Should be
  $error_message .= 'The selection you made does not appear to be valid.<br />';
}

Вы должны включить error_reporting и display_errors в PHP.ini только для разработки и тестирования. Неинициализированная переменная, такая как $errorMessage .= ..., выдаст уведомление, и вы увидите ошибку на экране. Выключите это для производственного кода. 

// Report all errors and notices
error_reporting(E_ALL);
ini_set('display_errors', 1);
1 голос
/ 16 октября 2011

Попробуйте это: 

   if(!isset($_POST['first_name']) ||
    !isset($_POST['last_name']) ||
    !isset($_POST['email']) ||
    !isset($_POST['telephone']) ||
    !isset($_POST['comments'])) ||
     $_POST['formGender'] == '')
0 голосов
/ 16 октября 2011

Как насчет проверки значения полей ввода с использованием javascript / jquery? Тогда вы сможете вернуть false при отправке, если необходимые поля ввода не заполнены.

...