Комментарии к публикации
Обработка форм в PHP

Я вот так сделал

<?php

if (empty($_GET)) {
    return 'Ничего не передано';
}

if (empty($_GET['operation'])) {
    return 'Операция не передана';
}

$operations = $_GET['operation'];

$x1 = (float)filter_input(INPUT_GET, 'x1');
$x2 = (float)filter_input(INPUT_GET, 'x2');

switch ($operations) {
    case '+':
        $result = $x1 + $x2;
        break;
    case  '-':
        $result = $x1 - $x2;
        break;
    case  '/':
        $result = $x2 !== 0 ? ($x1 / $x2) : 'На ноль делить нельзя';
        break;
    case  '*':
        $result = $x1 * $x2;
        break;

    default:
        return 'Операция не поддерживается';
}

$expression = $x1 . ' ' . $operations . ' ' . $x2 . ' = ';

return $expression . $result;

единственное теперь при неправильном вводе не будет сообщений, а аргумент будет принимать значение 0.0

<?php

if (empty($_GET)) {
    return 'Ничего не передано';
}

if (empty($_GET['operation'])) {
    return 'Операция не передана';
}

$x1 = filter_input(INPUT_GET, 'x1', FILTER_VALIDATE_FLOAT);
$x2 = filter_input(INPUT_GET, 'x2', FILTER_VALIDATE_FLOAT);

if ($x1 === null || $x2 === null) {
    return 'Аргументы 1 или 2 не переданы';
}

if ($x1 === false || $x2 === false) {
    return 'Введите число!';
}

$operations = $_GET['operation'];

switch ($operations) {
    case '+':
        $result = $x1 + $x2;
        break;
    case  '-':
        $result = $x1 - $x2;
        break;
    case  '/':
        $result = $x2 !== 0 ? ($x1 / $x2) : 'На ноль делить нельзя';
        break;
    case  '*':
        $result = $x1 * $x2;
        break;

    default:
        return 'Операция не поддерживается';
}

$expression = $x1 . ' ' . $operations . ' ' . $x2 . ' = ';

return $expression . $result;

Если в любом аргументе ничего не передать и нажать на "Посчитать": Выведется сообщение: "Введите число!". Но логично было бы: "Аргументы 1 или 2 не переданы". Хотя мы ничего не передавали, но на самом деле мы передали пустую строку, а функция filter_input() обработала ее и вернула false. Тем самым выражение:

if ($x1 === null || $x2 === null) {
    return 'Аргументы 1 или 2 не переданы';
}

не сработает =*(

но можно сделать так:

<?php

if (empty($_GET)) {
    return 'Ничего не передано';
}

if (empty($_GET['operation'])) {
    return 'Операция не передана';
}

if ($_GET['x1'] === "" || $_GET['x2'] === "") {
    return 'Аргументы 1 или 2 не переданы';
}

$x1 = filter_input(INPUT_GET, 'x1', FILTER_VALIDATE_FLOAT);
$x2 = filter_input(INPUT_GET, 'x2', FILTER_VALIDATE_FLOAT);

if ($x1 === NULL || $x2 === null) {
    return 'Аргументы 1 или 2 не переданы';
}

if ($x1 === false || $x2 === false) {
    return 'Введите число!';
}

$operations = $_GET['operation'];

switch ($operations) {
    case '+':
        $result = $x1 + $x2;
        break;
    case  '-':
        $result = $x1 - $x2;
        break;
    case  '/':
        $result = $x2 !== 0 ? ($x1 / $x2) : 'На ноль делить нельзя';
        break;
    case  '*':
        $result = $x1 * $x2;
        break;

    default:
        return 'Операция не поддерживается';
}

$expression = $x1 . ' ' . $operations . ' ' . $x2 . ' = ';

return $expression . $result;

Хотя я уверен что есть способ поделикатнее, но это не точно =)

if (empty($_GET)) {
    return 'Ничего не передано';
}

if (empty($_GET['operation'])) {
    return 'Операция не передана';
}

$x1 = filter_input(INPUT_GET, 'x1', FILTER_VALIDATE_FLOAT);
$x2 = filter_input(INPUT_GET, 'x2', FILTER_VALIDATE_FLOAT);

$operations = $_GET['operation'];

if (!is_numeric($x1) || !is_numeric($x2)) {
    return 'Введите число';
}

switch ($operations) {
    case '+':
        $result = $x1 + $x2;
        break;
    case  '-':
        $result = $x1 - $x2;
        break;
    case  '/':
        $result = $x2 != 0 ? ($x1 / $x2) : 'На ноль делить нельзя';
        break;
    case  '*':
        $result = $x1 * $x2;
        break;

    default:
        return 'Операция не поддерживается';
}

$expression = $x1 . ' ' . $operations . ' ' . $x2 . ' = ';

return $expression . $result;

$x2 = $_GET['x2'] ?? null;

Если я правильно понял, то это читается так, что если $_GET['x2'] равно null, то ее значение будет null.
Зачем задавать null если значение и так null?

if ($_GET['x1'] === '' || $_GET['x2'] === '')

То есть в данном случае мы просто проверяем - не передано ли в полях что-либо, но не проверяем есть ли сами поля, верно?
В таком случае, если да, то, возможно, следует взять на вооружение и всегда использовать
либо empty, либо сравнение с null или isset, верно?

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

Получилось так. Немного не допер, как сделать аналогичный, но кастомный фильтр для $_GET['operation'], через FILTER_CALLBACK. Как-то там сильно наворочено. Или это еще рано?

<?php

if (empty($_GET)) {
    return 'Ничего не передано';
}

$x1 = filter_input(INPUT_GET, 'x1', FILTER_VALIDATE_FLOAT);
$x2 = filter_input(INPUT_GET, 'x2', FILTER_VALIDATE_FLOAT);
$operation = $_GET['operation'] ?? null;

if ($x1 === null || $x2 === null || $operation === null) {
    return 'Ничего не передано';
}

if ($x1 === false && $x2 === false && empty($operation)) {
    return 'Ничего не передано';
}

if ($x1 === false || $x2 === false) {
    return 'Введите число';
}

if (empty($operation)) {
    return 'Не передана операция';
}

$expression = $x1 . ' ' . $operation . ' ' . $x2 . ' = ';

switch ($operation) {
    case '+':
        $result = $x1 + $x2;
        break;
    case '-':
        $result = $x1 - $x2;
        break;
    case '*':
        $result = $x1 * $x2;
        break;
    case '/':
        if ($x2 == 0) {
            return 'Деление на ноль невозможно';
        }
        $result = $x1 / $x2;
        break;
    default:
        return 'Операция не поддерживается';
}

return $expression . $result;

Добрый день! Спасибо.
Освоил урок не без помощи комментариев. Пока возился, возник вопрос немного не в тему. Я добавил в калькуляторе, в файлах index.php и calc.php, две операции % и **. Всё работает. Но при вводе операции отображение вот такое.

Подскажите, что с этим можно сделать.

Да уж! Спасибо.

<?php

if (empty($_GET)) {
    return 'Ничего не передано';
}
if (empty($_GET['operation'])) {
    return 'Операция не передана';
}
$x1 = filter_input(INPUT_GET, 'x1', FILTER_VALIDATE_FLOAT);
$x2 = filter_input(INPUT_GET, 'x2', FILTER_VALIDATE_FLOAT);

$operations = $_GET['operation'];

$expression = $x1 . ' ' . $operations . ' ' . $x2 . ' ' . ' = ';

if (!is_numeric($x1) || !is_numeric($x2)) {
    return 'Аргументы х1 или х2 не переданы';
}
$x1 = (float)$x1;
$x2 = (float)$x2;

switch ($operation) {
    case '+':
        $result = $x1 + $x2;
        break;
    case '-':
        $result = $x1 - $x2;
        break;
    case '*':
        $result = $x1 * $x2;
        break;
    case '/':
        $result = $x2 != 0 ? ($x1 / $x2) : 'На ноль делить нельзя';
        break;
    case '/':
        $result = $x1 / $x2;
        break;
}

return $expression . $result;

<?php
if (empty($_GET)) {
    return 'Ничего не передано';
}

if (empty($_GET['operation'])) {
    return 'Операция не передана';
}

//проверяем существование в массиве значения по указанному ключу
$x1 = $_GET['x1'] ?? null;
$x2 = $_GET['x2'] ?? null;

if ($x1 === null || $x2 === null) {
    return 'Аргументы 1 или 2 не переданы';
}

$operations = $_GET['operation'];

//проверяем, что введенные данные являются числами
if (!is_numeric($x1) || !is_numeric($x2)) {
    return 'Введите число';
}

//фильтруем значения, полученные методом GET

$x1 = filter_input(INPUT_GET, 'x1', FILTER_VALIDATE_FLOAT);
$x2 = filter_input(INPUT_GET, 'x2', FILTER_VALIDATE_FLOAT);

switch ($operations) {
    case '+':
        $result = $x1 + $x2;
        break;
    case  '-':
        $result = $x1 - $x2;
        break;
    case  '/':
        $result = $x2 !== 0 ? ($x1 / $x2) : 'На ноль делить нельзя';
        break;
    case  '*':
        $result = $x1 * $x2;
        break;

    default:
        return 'Операция не поддерживается';
}

$expression = $x1 . ' ' . $operations . ' ' . $x2 . ' = ';

return $expression . $result;

Не очень понимаю, правильный ли кусок кода я заменила на filter_input?

<?php

if (empty($_GET)) {
    return 'Ничего не передано';
}

if (empty($_GET['operation'])) {
    return 'Операция не передана';
}

if (!isset($_GET['x1'], $_GET['x2'], $_GET['operation'])) {
    return 'Аргументы не переданы';
}

$x1 = filter_input(INPUT_GET, 'x1', FILTER_VALIDATE_FLOAT);
$x2 = filter_input(INPUT_GET, 'x2', FILTER_VALIDATE_FLOAT);

if ($x1 === null || $x2 === null) {
    return 'Аргументы 1 или 2 не переданы';
}

if ($x1 === false || $x2 === false) {
    return 'Введите число';
}

$operations = $_GET['operation'];

switch ($operations) {
    case '+':
        $result = $x1 + $x2;
        break;
    case  '-':
        $result = $x1 - $x2;
        break;
    case  '/':
        $result = $x2 !== 0 ? ($x1 / $x2) : 'На ноль делить нельзя';
        break;
    case  '*':
        $result = $x1 * $x2;
        break;

    default:
        return 'Операция не поддерживается';
}

$expression = $x1 . ' ' . $operations . ' ' . $x2 . ' = ';

return $expression . $result;

Добрый день. У меня получился такой код, но он не совсем корректно работает. А именно: если задать пустое поле, то оно распознается как false, а не как null. И в результате выдается сообщение: "Введите число".

Добрый день.

if (empty($_GET)) {
    return 'Ничего не передано';
}

if (empty($_GET['operation'])) {
    return 'Операция не передана';
}

$x1 = filter_input(INPUT_GET, 'x1', FILTER_VALIDATE_FLOAT);
$x2 = filter_input(INPUT_GET, 'x2', FILTER_VALIDATE_FLOAT);

$operations = $_GET['operation'];

if (($x1 === null) || ($x2 === null)) { 
    return 'Аргументы 1 или 2 не переданы';
}

if (($x2 === 0.0) && ($_GET['operation'] === '/')) {
    return 'На ноль делить нельзя';
}
if ($x1 === false || $x2 === false) { 
    return 'Аргументы должны быть числами';
}

$expression = $x1 . ' ' . $operations . ' ' . $x2 . ' = ';

switch ($operations) {
    case '+':
        $result = $x1 + $x2;
        break;
    case '-':
        $result = $x1 - $x2;
        break;
    case '/':                 
        $result = $x1 / $x2;
        break;
    case '*':
        $result = $x1 * $x2;
        break;
    default:
        return 'Операция не поддерживается';
}

return $expression . $result;

if (empty($_POST)) {
    return 'Nothing have been passed';
}

if (empty($_POST['option'])) {
    return 'No operator have been selected';
}
$operator = $_POST['option'];
$x = $_POST['var1'] ?? null;
$y = $_POST['var2'] ?? null;

if ($x === null) {
    return 'First variable has not been passed';
} elseif ($y === null) {
    return 'Second variable has not been passed';
}

$x = filter_var($x, FILTER_SANITIZE_SPECIAL_CHARS);
$y = filter_var($y, FILTER_SANITIZE_SPECIAL_CHARS);

if (filter_var($x, FILTER_VALIDATE_INT) === false) {
    return $x . ' is not a number';
}
if (filter_var($y, FILTER_VALIDATE_INT) === false) {
    return $y . ' is not a number';
}

switch ($operator) {
    case '+':
        $result = $x + $y;
        break;
    case '-':
        $result = $x - $y;
        break;
    case '*':
        $result = $x * $y;
        break;
    case '/':
        if ($y == 0) {
            return 'You cant divide by zero';
        }
        $result = $x / $y;

        break;
    default:
        return 'Операция не поддерживается';

}

return $x . ' ' . $operator . ' ' . $y . ' = ' . $result;

что скажешь? как бы усовершенствовал чтоб убрать возможные повторения кода?

$x1 = $_GET['x1'] ?? null;
$x2 = $_GET['x2'] ?? null;

if ($x1 === null || $x2 === null) {
    return 'Аргументы 1 или 2 не переданы';
}

Здравствуйте! Можете объяснить логику сего действа?
Фактически, это проверка на отсутствие полей вообще. Но в случае проверки x1/x2 на пустую строку(на что как бы намекает текст ошибки - что аргумент 1 или 2 не передан) это работать не будет, т.к. для строгого равенства пустая строка не null. Тут спасает проверка функцией is_numeric, которая выдаст ошибку(введите число) в случае пустой строки и без которой строгое сравнение работать не будет. Но корректно ли это? Это же вроде разные проверки - на пустоту и на число/не число. Вообще не совсем понятно, зачем тут строгое сравнение, как и во многих других случаях по курсу, где можно обойтись без него.
Насколько я понимаю, можно было бы просто в приведенном выше коде использовать нестрогое сравнение:

$x1 = $_GET['x1'] ?? null;
$x2 = $_GET['x2'] ?? null;

if ($x1 == null || $x2 == null) {
    return 'Аргументы 1 или 2 не переданы';
}

<?php

if (empty($_GET)) {
    return 'вы не ввели данные';
}

При отсутствии исходных данных в калькуляторе не возвращает это сообщение. Все остальное работает.
php-8

Добрый вечер!
Существует ли возможность функцией filter_input() обработать регулярное выражение, например "[+-/*//]" для контроля ввода операций калькулятора, в доках функции что-то не увидел про регулярки. И есть ли смысл это делать в данном коде, ведь операции обрабатываются switch'ем.

<?php
// Обработчик калькулятора для домашнего задания
if (empty($_GET)) {
    return 'Ничего не передано';
}
if (empty($_GET['operation'])) {
    return 'Операция не передана';
}

$x1 = $_GET['x1'] ?? null;
$x2 = $_GET['x2'] ?? null;
if (($x1 === null) && ($x2 === null)) {
    return 'Аргументы не переданы';
}
if ($x1 === null) {
    return 'Аргумент 1 не передан';
}
if ($x2 === null) {
    return 'Аргумент 2 не передан';
}

$operations = $_GET['operation'] ?? null;

$x1 = filter_input(INPUT_GET, 'x1', FILTER_VALIDATE_FLOAT);
$x2 = filter_input(INPUT_GET, 'x2', FILTER_VALIDATE_FLOAT);
if (($x1 === false) && ($x2 === false)) {
    return 'Аргументы не переданы или они "не-число"';
}
if ($x1 === false) {
    return 'Аргумент 1 не передан или он "не-число"';
}
if ($x2 === false) {
    return 'Аргумент 2 не передан или он "не-число"';
}

switch ($operations) {
    case '+':
        $result = $x1 + $x2;
        break;
    case  '-':
        $result = $x1 - $x2;
        break;
    case  '/':
        $result = $x2 !== 0.0 ? ($x1 / $x2) : 'На ноль делить нельзя';
        break;
    case  '*':
        $result = $x1 * $x2;
        break;
    default:
        return 'Операция не поддерживается или она не передана';
}
$expression = $x1 . ' ' . $operations . ' ' . $x2 . ' = ';
return $expression . $result;

<?php

if (empty($_GET)) {
    return 'Ничего не передано';
}

if (empty($_GET['operation'])) {
    return 'Операция не передана';
}

$operations = $_GET['operation'];

$x1 = filter_input(INPUT_GET, "x1", FILTER_VALIDATE_INT);
$x2 = filter_input(INPUT_GET, "x2", FILTER_VALIDATE_INT);
if ($x1 && $x2) {

    switch ($operations) {
        case '+':
            $result = $x1 + $x2;
            break;
        case  '-':
            $result = $x1 - $x2;
            break;
        case  '/':
            $result = $x2 !== 0 ? ($x1 / $x2) : 'На ноль делить нельзя';
            break;
        case  '*':
            $result = $x1 * $x2;
            break;

        default:
            return 'Операция не поддерживается';
    }

    $expression = $x1 . ' ' . $operations . ' ' . $x2 . ' = ';
    return $expression . $result;
} else echo 'Аргументы не переданны';

<?php
if (empty($_GET)) {
    return 'Ничего не передано!';
}

if (empty($_GET['operation'])) {
    return 'Не передана операция';
}

if (!isset($_GET['x1'], $_GET['x2'])) {
    return 'Аргументы 1 или 2 не переданы';
}

$x1 = filter_input(INPUT_GET, 'x1', FILTER_VALIDATE_FLOAT);
$x2 = filter_input(INPUT_GET, 'x2', FILTER_VALIDATE_FLOAT);

if (!is_numeric($x1) || !is_numeric($x2)) {
    return 'Обнаружено не числовое значение';
}

$expression = $x1 . ' ' . $_GET['operation'] . ' ' .$x2 . ' = ';

switch ($_GET['operation']) {
    case '+':
        $result = $x1 + $x2;
        break;
    case '-':
        $result = $x1 - $x2;
        break;
    case '*':
        $result = $x1 * $x2;
        break;
    case '/':
        $result = $x2 != 0 ? ($x1 / $x2) : 'Деление на ноль';
        break;
    default:
        return 'Операция не поддерживается';
}

return $expression . $result;

<?php

if (empty($_GET)){
    return 'Ничего не передано!';
}

if (empty($_GET['operation'])){
    return 'Не передан оператор!';
}

if ($_GET['x1'] === '' || $_GET['x2'] === ''){
    return 'Введите всё аргументы';
}

$x1 = filter_input(INPUT_GET, "x1", FILTER_VALIDATE_FLOAT);
$x2 = filter_input(INPUT_GET, "x2", FILTER_VALIDATE_FLOAT);

if ($x1 === false || $x2 === false){
    return 'Введите число';
}

$operation = $_GET['operation'];

$expression = $x1 . ' ' . $operation . ' ' . $x2 . ' = ';

switch ($operation) {
    case '+':
        $result = $x1 + $x2;
        break;
    case '-':
        $result = $x1 - $x2;
        break;
    case '*':
        $result = $x1 * $x2;
        break;
    case '/':
        $result = $x2 !== 0 ? ($x1 / $x2) : 'На ноль делить нельзя';
        break;
    default:
        $result = 'Операция не поддерживается';
}

return $expression . $result;

Здравствуйте, есть что дополнить?