! is_null () не работает должным образом

«dispatch_address_postcode не является обязательным, и он все равно будет работать, даже если он пуст…»

Просто посмотрите на это предложение еще раз. Если поле не является обязательным, это нормально, если код запускается, если поле пустое. Если поле не является обязательным, не проверяйте его как обязательное.

Однако настоящая проблема в том, что is_null проверяет только, равна ли переменная null. POST-значения никогда не будут null, если они пустые, они будут '' (пустая строка). Все ваши !is_null тесты всегда будут true, и вы получите предупреждение, если переменная не будет установлена ​​(чего вы не хотите). Более подходящим тестом будет !empty.

Еще более подходящие тесты могли бы включать проверку того, является ли значение допустимым (выглядит ли электронная почта как адрес электронной почты, есть ли в телефоне хотя бы x цифр?). Вы также должны прокручивать поля в цикле, чтобы сделать ваш код более читабельным, бесконечные вложенные и связанные if условия не доставляют удовольствия.

$mandatoryFields = array('foo' => 'email', 'bar' => 'telephone');

foreach ($mandatoryFields as $field => $rule) {
    if (empty($_POST[$field]) || !validateByRule($_POST[$field], $rule)) {
        raiseHell();
    }
}

Похоже, вы пытаетесь убедиться, что отправлены все переменные сообщения. Вам нужна помощь с этим?

Использование! Empty () не может быть ответом на ваш конкретный вопрос, но определенно поможет с тем, что, похоже, вы пытаетесь сделать.

empty () возвращает TRUE, если ключ $ _POST не установлен, если это пустой массив или даже если это пустая строка, поэтому использование! empty () - хороший способ убедиться, что пользователь заполнил информацию .

Попробуйте написать свою собственную is_valid функцию и использовать ее, а не is_null.

Например (и это отнюдь не исчерпывающе):

function is_valid(&$array, $key, $required=false) {
    if(!array_key_exists($array))
        return false;
    $value = trim($array[$key]);
    if(empty($value) && $required)
        return false;
    return true;
}

Используйте так:

if(is_valid($_POST, 'personal_info_first_name', true) && ...)

используйте 1_

Изменить: пожалуйста, примите это во внимание, прежде чем голосовать против. Я оставляю это здесь, чтобы показать, "чего не следует делать". Я бы удалил его, потому что он плохой, но тогда никто не будет учиться на моих ошибках.

НЕ ДЕЛАЙТЕ ЭТО - прочтите комментарии, чтобы узнать, почему это плохо.

Мой ответ будет совершенно другим, но я совершенно другой парень ...

Я ПРОСТО обнаружил, что это сработает. Вместо всего этого, просто назначьте переменные программно! Думаю, мне нужно провести рефакторинг ... ну, знаешь, весь мой код ...

if (!is_array($_POST)){exit "$_POST isn't an array";}
foreach ($_POST as $param => $value){
    ${$param} = secure($value);
}

//now you have a set of variables that are named exactly as the posted param
//for example, $_POST['personal_info_first_name'] == $personal_info_first_name

if ($payment_method == "Pay by credit card."){
    //do stuff that you were gonna do anyways
} else if ($payment_method == "Pay by new credit card.") {
    if ($card_number && $expiration_date && $security_code){
        //do stuff that you were gonna do anyways
    } else {
        exit("info missing for credit card transaction");
    }
} else {
    exit("unknown payment method")
}

function secure($input){
    //sanitize user input
}

Если вы используете этот код, то не имеет значения, что имеет значение null, а что нет в foreach, потому что ничего, что равно null, просто не будет сделано. Затем вы можете использовать более красивый код (и, возможно, более быстрый код), чтобы проверить все, что требуется.