Где ошибка в php-коде?

Question

[0ldn0mad]

Уважаемые Гуру!
Делаю первые шаги в освоении PHP - пишу карулятор.
Первую задачу себе поставил - сделать проверку существования данных, пришедших из формы. Т.е. если какое-то поле осталось пустым, вывести сообщение под формой - "Что-то не передано!"
Скрипт написал, но сообщение не появляется. Где я ошибся?

<form action='calc.php' method="post">
      <label>Число 1:</label>
      <br />
      <input name='num1' type='text' />
      <br />
      <label>Оператор: </label>
      <br />
	  <label for="operator">
		    <select name="operator" id="operator">
			    <option value="+">+</option>
			    <option value="-">-</option>
		    </select>
      <br />
      <label>Число 2: </label>
      <br />
      <input name='num2' type='text' />
      <br />
      <br />
      <input type='submit' value='Считать'>
    </form>
    <br />
    <br />
<?php

$num1 = int($_POST['num1']);
$num2 = int($_POST['num2']);

if (empty($num1 or $num2)) {
	echo "Что-то не передано!";
}

?>

Answer 1

[Anton Mashletov]

if (isset($_POST['num1']) && isset($_POST['num2']) && isset($_POST['operator'])) {
  $num1 = intval($_POST['num1']);
  $num2 = intval($_POST['num2']);
  $operator = $_POST['operator'];

  switch ($operator) {
     case '+':
         echo $num1 + $num2;
         break;
     case '-':
         echo $num1 - $num2;
         break;
  }
   
} else {
  echo "Что-то не передано!";
}

Comments

[Yan-s]

всё обучение человеку обломали

[0ldn0mad]

Yan-s,
Ну с чего Вы взяли что обломали? Все очень даже превосходно! =)

[0ldn0mad]

При открытии формы ( без отправки данных через форму) уже появляется сообщение что "Что-то не передано!", хотя это сообщение должно появиться когда я нажал педаль сабмит и отослал только одно число из двух.
Где может быть проблема?

[Anton Mashletov]

0ldn0mad,
потому что, когда мы просто выводим форму мы проверяем $_POST, он пустой, вот и выводится сообщение.
Оберните весь if else в дополнительный

if ($_SERVER['REQUEST_METHOD'] === 'POST') {
    // …
}

Тем самым, мы будем проверять $_POST и вычислять, только если пользователь нажал на Отправить, а не всегда.

Answer 2

[Максим Тимофеев]

1. Начиная с php 5.5 можно так как Вы сделали, но для более ранних версий в empty должна быть переменная, а не выражение, так что в зависимости от версии php, возможно стоит попробовать
if (empty($num1) || empty($num2)) {
2. Вы не верно приводите данные к integer

$num1 = int($_POST['num1']); // не верно
$num1 = (int) $_POST['num1'];  //верно

Comments

[0ldn0mad]

При открытии формы ( без отправки данных через форму) уже появляется сообщение что "Что-то не передано!", хотя это сообщение должно появиться когда я нажал педаль сабмит и отослал только одно число из двух.
Где может быть проблема?

[Максим Тимофеев]

0ldn0mad, проблемы нет, так и должно быть, а если Вам не надо выводить, то стоит сделать проверку на существование post запроса.

if(isset($_POST)){
   $num1 = int() $_POST['num1'];
   // и так далее

Кстати если бы Вы включили вывод всех ошибок, что я крайне рекомендую сделать, особенно пока изучаете php. Вы бы уже увидели эту проблем в виде ошибки. И вообще при error_all меньше "почему так".

Answer 3

[Roman Terekhin]

Это конкретно под ваш кусок кода. В идеале нужно смотреть все и переписывать с нуля.

<?php

$num1 = intval(isset($_POST['num1']) ? $_POST['num1'] : 0);
$num2 = intval(isset($_POST['num2']) ? $_POST['num2'] : 0);

if (empty($num1) || empty($num2)) {
  echo "Что-то не передано!";
}

?>

Comments

[0ldn0mad]

При открытии формы ( без отправки данных через форму) уже появляется сообщение что "Что-то не передано!", хотя это сообщение должно появиться когда я нажал педаль сабмит и отослал только одно число из двух.
Где может быть проблема?

Answer 4

[Александр Торопов]

Сделал вариант с выводом сообщения под полем, которое не заполнили:

<?php
if($_SERVER['REQUEST_METHOD'] == 'POST') {
    $require = ['num1', 'num2']; //поля которые надо заполнить.
    $errors[] = ''; //массив в котором сохраним ошибки
    
    foreach($require as $key) {
        if(empty($_POST[$key])) {
            $errors[$key] = 'Это поле надо заполнить!';
        }
    }
}
?>

<form action='calc.php' method="post">
      <label>Число 1:</label>
      <br />
      <input name='num1' type='text' />
      <?php
        $msg = isset($errors['num1']) ? "Введите первое число" : "";
        echo $msg;
      ?>
      <br />
      <label>Оператор: </label>
      <br />
    <label for="operator">
        <select name="operator" id="operator">
          <option value="+">+</option>
          <option value="-">-</option>
        </select>
      <br />
      <label>Число 2: </label>
      <br />
      <input name='num2' type='text' />
      <?php
        $msg = isset($errors['num2']) ? "Введите первое число" : "";
        echo $msg;
      ?>
      <br />
      <br />
      <input type='submit' value='Считать'>
    </form>
    <br />
    <br />
<?php
    if (count($errors) > 0) {
        echo 'Заполните все поля!';
    }
?>

Сначала проверим, была ли отправлена форма, а потом проверяем заполнены ли все обязательные поля из формы.