Комментарии к публикации
Пишем калькулятор на PHP

1)

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

2)

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

3)

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

Проверка на строку :

if ((gettype($x1)=== "string") && (gettype($x2) === "string")){
    return 'Введите числа';
}
else{
    $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 = $x1 / $x2;
                break;
            default:
                return 'Операция не поддерживается';
        }

        return $expression . $result;
}

4 в индексе

 <label for="x1"></label><input type="number" name="x1" id="x1">

в calc.php

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

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

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

$x1 = intval($_GET['x1']);
$x2 = intval($_GET['x2']);

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

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

return $expression . $result;

Сделал, вроде все работает

<?php

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

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

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

$x1 = $_GET['x1'];
$x2 = $_GET['x2'];
$operations  = $_GET['operation'];

if (is_numeric($x1) && is_numeric($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 'Операция не поддерживается';
    }
} else {
    return 'Введите число';
}

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

return $expression . $result;

Так?

<?php
// проверяет через функцию пустой метод гет или нет
if (empty($_GET)){
    return 'Ничего не переданно';
}
// проверяем через функцию пустой метод гет с массивом операция 
if (empty($_GET['operation'])){
    return 'Операция не переданна';
}
// проверяем нахождения ключа в массиве метода гет
if (isset($_GET['x1']) && isset($_GET['x2'])){
    $x1 = $_GET['x1'];
    $x2 = $_GET['x2'];
} else return 'Ключу аргумента отсутсвоет';

    $operation = $_GET['operation'];
// проверка на то чтобы пользователь мог записовать только числа
if (is_numeric($x1) && is_numeric($x2)){
     switch ($operation){
        case '+':
            $result = $x1 + $x2;
            break;
        case  '-':
            $result = $x1 - $x2;
            break;
        case  '/':
            if($x2 != 0){
               $result = $x1 / $x2;
            } else {
                return 'На ноль нельзя делить';
            }
            break;
        case  '*':
            $result = $x1 * $x2;
            break;

        default:
            return 'Операция не поддерживается';
    }
} else {
    return 'Введите число';
}
// создаем переменную и записываем переменные в нужной последовательности
$expression = $x1 . ' ' . $operation . ' ' . $x2 . ' = ';
// вывод результата
return $expression . $result;

Все сделал, не знаю что еще добавить можно) все работает)

<?php

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

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

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

if ($x1 === null || $x2 === null){
    return 'Ключу аргумента отсутсвоет';
}

if (!file_exists(__DIR__ . '/calc.php')){
    return 'Ошибка файла';
}

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

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

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

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

return $expression . $result;

А в степень почему-то не получается с помощью pow, только ** - подскажите почему так?

<?php

if (empty($_GET)) {
    return "nothing";
}

if (empty($_GET['operation'])) {
    return "no operation";
}

if (($_GET['number1']=='') || ($_GET['number2']=='')) {
    return "nothing";
}

$x1 = $_GET['number1'];
$x2 = $_GET['number2'];
$operation = $_GET['operation'];

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

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

единственное. Проблему со string решил так:

<input type="number" name="number1">
<input type="number" name="number2">

Пока так и не получилось прибавить 0. Пишу позже, может кому поможет. Разобрался, почему не получалось, потому-что проверял x1 и x2 на empty(), и если вводил 0, то php считал, что поле пустое

<?php

if (empty($_GET)) {
    return "Ничего не передано, массив пустой";
}

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

//Теперь можно передать ноль
if (($_GET['x1'] == '') || ($_GET['x2'] == '')) {
    return 'Аргументы не переданы';
}

if (!is_numeric($x1) || !is_numeric($x2)) {
    return 'Нельзя вводить строковые значения';
}

//… задали переменные для массива гет
$x1 = $_GET['x1'];
$x2 = $_GET['x2'];

//Для наглядности выводим значения x1 и x2 =
$expression = $x1 . ' ' . $_GET['operation'] . ' ' . $x2 . ' = ';

// собственно сами вычисления
switch ($_GET['operation']) {
    case '+':
        $result = $x1 + $x2;
        break;
    case '-':
        $result = $x1 - $x2;
        break;
    case '*':
        $result = $x1 * $x2;
        break;
    case '/':
        if ($x2 != 0) {
            $result = $x1 / $x2;
        } else {
            return 'На ноль нельзя делить!';
        }
        break;

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

}

return $expression . $result;

Ты случаем не пропустил закрывающий тек </label> для id="operation"?
Проверка на пустоту $_GET и сопутствующее ему выполнение бессмысленны,т.к. селектор по умолчанию предлагает первый вариант и при нажатии кнопки посчитать, это значение передастся.
Для чего задавать label если мы оставляем их пустыми?
Нужно ли в логике в конструкции switch задавать default, если у нас селектор содержит только два варианта?

Мой топовый код :D

<?php

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

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

//Обработка нулевых аргументов
if (empty($_GET['x1']) && $_GET['x1'] !== '0' || empty($_GET['x2']) && $_GET['x2'] !== '0') {
    return 'Не переданы аргументы';
}

//Обработка деления на 0
if (($_GET['x1'] !== '0' && $_GET['x2'] == '0') && $_GET['operation'] == '/') {
    return 'Деление на ноль!';
}

//Обработа строковых аргументов

if (!is_numeric($_GET['x1']) || !is_numeric($_GET['x2'])) {
    return 'Переданные значения не являются числом';
}

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

$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 = $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 = $_GET['x1'];
$x2 = $_GET['x2'];

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

if (is_numeric($x1) && is_numeric($x2)) {

    if ($_GET['operation']) {

        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 'Операция не поддерживается';
        }

    }
} else {
    return 'Пожалуйста, введите число';
}

return $expression . $result;

if (empty($_GET)) {
    return 'Ничего не передано!';
}
if (empty($_GET['operation'])) {
    return 'Не передана операция';
}
if (!is_numeric($_GET['x1']) || !is_numeric($_GET['x2'])) {
    return 'Не переданы числовые аргументы';
}
$x1 = (float)$_GET['x1'];
$x2 = (float)$_GET['x2'];
$expression = $x1 . ' ' . $_GET['operation'] . ' ' . $x2 . ' = ';
switch ($_GET['operation']) {
    case '+':
        $result = $x1 + $x2;
        break;
    case '-':
        $result = $x1 - $x2;
        break;
    case '*':
        $result = $x1 * $x2;
        break;
    case '/':
        if ($x2 === (float)0) {
            $result = 'На ноль делить нельзя!';
        } else {
            $result = $x1 / $x2;
        }
        break;
    default:
        return 'Операция не поддерживается';
}
return $expression . $result;

if (empty($_GET)) {
    return 'Nothing has been transferred!';
}
if (empty($_GET['operation'])) {
    return 'Operation not transferred';
}
if (($_GET['x1' === '']) || ($_GET['x2'] === '')) {
    return 'Arguments are not passed';
}

$x1 = $_GET['x1'];
$x2 = $_GET['x2'];
if (!is_numeric($x1)) {
    return 'Input interger number';
}
if (!is_numeric($x2)) {
    return 'Input interger number';
}
if ($_GET['$x1'] = '0' || $_GET['$x2'] = '0') {
    return $x1 = 0 || $x2 = 0;
}

$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) : 'you can not divide it by zero';
        break;
    default:
        return 'Operation not supported';
}
return $expression . $result;

<?php

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

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

if (($_GET['x1']==='') || ($_GET['x2']==='')) {
    return 'Не переданны аргументы';
}

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

if (!is_numeric($x1) || !is_numeric($x2)) {
    return 'Введите числа!';
}else {
    $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 : 'Нельзя делить на 0';
        break;

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

return $expression . $result;

<?php
// Проверяем переданны ли данные вобще
if (empty($_POST)) {
    return 'Ничего не передано';
}
// проверяем передана ли операция

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

/* проверяем переданы ли аргументы, и проверяем числа это или нет.
проверяем можно ли передавать ноль */
if (($_POST['x1'] === '') || ($_POST['x2'] === '')) {
   return 'Не переданы аргументы';
}

if (is_numeric($_POST['x1']) || is_numeric($_POST['x2'])) {
} else {
    return 'Введите числовое значение';
}

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

$x1 = $_POST['x1'];
$x2 = $_POST['x2'];

$output = $x1 . '' . $_POST['operation'] . '' . $x2 . '=';

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

return $output . $result;

<?php

if (empty($_GET)) {
    return 'Ничего не передано!';
}
if (empty($_GET['operation'])) {
    return 'Не передана операция';
}
if (($_GET['x1'] === '') || ($_GET['x2'] === '')) {
    return 'Аргументы не переданы';
}
if (!is_numeric($_GET['x1']) ||  !is_numeric($_GET['x2'])) {
    return 'В поле можно ввести только цифры';
}

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

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

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

return $expression . $result;

Спасибо за урок! =)

Код из файла calc.php

<?php

if(empty($_GET))
{
    return 'Значений не передано!';
}
if(empty($_GET['operation']))
{
    return 'Операция не передана!';
}
if(empty($_GET['x1']) && $_GET['x1'] != 0 || empty($_GET['x2']) && $_GET['x2'] != 0)
{

}
if(gettype($_GET['x1']) === "string" || gettype($_GET['x2']) === "string" )
{
    return 'Введите числа!';
}

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

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

switch ($_GET['operation'])
{
    case '+':
        $result = $x1 + $x2;
        break;
    case '-':
        $result = $x1 - $x2;
        break;
    case '*':
        $result = $x1 * $x2;
        break;
    case '/':
        if($x1 == 0 || $x2 == 0)
        {
            $result = 'На ноль делить нельзя'; //На самом деле выйдет бесконечность =)
        } else {
            $result = $x1 / $x2;
        }
        break;
    default:
        return 'Операция не подерживается';
}

return $expression . $result;

На счет того что добавить/улучшить:
1)Когда я попробовал ввести 2 - -2 у меня выдало "Введите числа"
2)Степень, корень, факториал

• Попробуйте в качестве одного из аргументов передать 0. Какой получился результат? Почему так? Дополните код так, чтобы можно было передавать 0.

  if (empty(isset($_GET['x1'])) || empty(isset($_GET['x2']))) {
  return 'Не переданы аргументы';
  }

• Усовершенствуйте калькулятор так, чтобы он умножал и делил.

  case '*':
      $result = $x1 * $x2;
      break;
  case '/':
      if($x2 != 0)
          $result = $x1 / $x2 ;
      else
          return 'Простите, хорошие интерпритаторы на ноль не делят, а я очень хороший! =)';
      break;

• Что произойдёт, если поделить на ноль? Добавьте обработку такой ситуации.

  case '/':
      if($x2 != 0)
          $result = $x1 / $x2 ;
      else
          return 'Простите, хорошие интерпритаторы на ноль не делят, а я очень хороший! =)';
      break;

• Что произойдёт, если в качестве аргумента передать вместо числа строку? Сделайте так, чтобы в качестве аргументов можно было отправить только числа.

  if (!is_numeric(trim($_GET['x1'])) || !is_numeric(trim($_GET['x2']))) {
  return 'В параметры передана строка, а должно быть число, исправьте и повторите ввод снова!';
  }

• Какие ещё недостатки есть у этого кода? Как можно нарушить его работу? Что можно улучшить?

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

Весь код

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

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

if (empty(isset($_GET['x1'])) || empty(isset($_GET['x2']))) {
    return 'Не переданы аргументы';
}

if (!is_numeric(trim($_GET['x1'])) || !is_numeric(trim($_GET['x2']))) {
    return 'В параметры передана строка, а должно быть число, исправьте и повторите ввод снова!';
}

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

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

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

return $expression . $result;

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

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

if (!key_exists('x1', $_GET) || !key_exists('x2', $_GET)) {
    return 'Отсутсвуют аргументы';
}

if (!is_numeric($_GET['x1']) || !is_numeric($_GET['x2'])) {
    return 'Аргументы не могут быть строками';
}

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

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

switch ($_GET['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;

<?php
$x1 = $_GET['x1'];
$x2 = $_GET['x2'];

 if ($x1 === '' || $x2 === '') {   //1.Дополните код так, чтобы можно было передавать 0.
    return 'Не переданы аргументы';
}else if ($x2 == '0' && $_GET['operation'] == '/') {  //3.Что произойдёт, если поделить на ноль? Добавьте обработку такой ситуации.
    return 'На 0 делить нельзя';
}else if (!is_numeric($x1) || !is_numeric($x2)) {  //4. Сделайте так, чтобы в качестве аргументов можно было отправить только числа.
    return "Введите числа";
}

$expression = $x1 . ' ' . $_GET['operation'] . ' ' . $x2 . ' = ';
switch ($_GET['operation']) {
    case '+':
        $result = $x1 + $x2;
        break;
    case '-':
        $result = $x1 - $x2;
        break;
    case '/':   //2.Усовершенствуйте калькулятор так, чтобы он умножал и делил.
        $result = $x1 / $x2;
        break;
    case '*':   //2.Усовершенствуйте калькулятор так, чтобы он умножал и делил.
        $result = $x1 * $x2;
        break;
    default:
        return 'Операция не поддерживается';
}

return $expression . $result;

Почему бы просто не указать атрибут pattern для формы, чтобы только числа вводились? ))

<input type="text" name="x1" pattern="^[ 0-9]+$">

Добрый день!
Немного не понял 5 задание, но улучшил немного передавая переменную как htmlspecialchars:

<?php

if (empty($_GET)){
    return 'Ничего не передано!';
}
if (empty($_GET['operation'])){
    return 'Не передана операция';
}
if (($_GET['x1'] === ' ') || ($_GET['x2'] === ' ')){
    return 'Не переданы аргументы';
}
//Чтобы в строке невозможно было передать код:
$x1 = htmlspecialchars ($_GET['x1']);
$x2 = htmlspecialchars ($_GET['x2']);

$expression = $x1 . ' ' . $_GET['operation'] . ' ' . $x2 . ' = ';
if (is_numeric($x1) && is_numeric($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) : 'Деление на 0 невозможно';
            break;
        default:
            return 'Операция не поддерживается.';
    }
} else {
    return 'Введите число';
}
return $expression . $result;

if (empty($_GET)){
    return 'Ничего не передано!';
}
#Проверка на существование ключей в $_GET
$checkKeyInArray = function( array $listKeyInArray):bool
{  
    foreach($listKeyInArray as $keyInArray){
        if(!array_key_exists($keyInArray, $_GET) ){
            return false;
        }
    }
    return true;
};
$keyControl = ['x1', 'x2', 'operation'];
if($checkKeyInArray($keyControl)){
    if (empty($_GET['operation'])){
        return 'Не передана операция';
    }
    if (($_GET['x1'] === ' ') || ($_GET['x2'] === ' ')){
        return 'Не переданы аргументы';
    }
    //Чтобы в строке невозможно было передать код:
    $x1 = htmlspecialchars($_GET['x1']);
    $x2 = htmlspecialchars($_GET['x2']);

    $expression = $x1 . ' ' . $_GET['operation'] . ' ' . $x2 . ' = ';
    if (!is_numeric($x1) && !is_numeric($x2)) {
        return 'Введите число';
    }
    switch ($_GET['operation']) {
            case '+':
                $result = $x1 + $x2;
                break;
            case '-':
                $result = $x1 - $x2;
                break;
            case '*':
                $result = $x1 * $x2;
                break;
            case '/':
                $result = $x2 != 0 ? ($x1 / $x2) : 'Деление на 0 невозможно';
                break;
            default:
                return 'Операция не поддерживается.';
        }
    return $expression . $result;
}
return 'Техническая проблема, обратитесь в техническую поддержку!';

if (empty($_GET)){
        return 'Nichego net';
    }

    if (empty($_GET['operation'])) {
        return 'Nt peredna operacia';
    }
    if (($_GET['x1'] === '' ) || ($_GET['x2'] === '' )) {
        return 'Не переданы аргументы';
    }
        $x1 = $_GET['x1'];
        $x2 = $_GET['x2'];

    $expressionn = $x1 . ' ' . $_GET['operation'] . ' ' . $x2 . '=';
    //return $expressionn;

     if ((is_numeric($x1))  && (is_numeric($x2))){
            switch ($_GET['operation']) {
                case '/' :
                    $result = $x2 == 0 ? ' error' : ($x1 / $x2);
                    break;
                case  '+' :
                    $result = $x1 + $x2;
                    break;
                case  '-' :
                    $result = $x1 - $x2;
                    break;
                case  '/' :
                    $result = $x1 / $x2;
                    break;
                case  '*' :
                    $result = $x1 * $x2;
                    break;
                default :
                    return 'Operation failed';
            }
      } else {
    return 'Не верный тип';
    }

    return $expressionn . $result;

Привет! У меня вопрос по этому "Сделайте так, чтобы в качестве аргументов можно было отправить только числа." Я при создании своей формы установил четкий формат ввода, т.е. если пользователь накосячил в каких-то данных, окно все равно подсвечивается красным. В данном случае, при создании калькулятора, есть смысл устанавливать дополнительные ограничения на получаемые значения параметров из формы или достаточно того, что в форме они и так есть?

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

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

if ($_GET['x1'] === '' || $_GET['x2'] === '') {
    return 'Не переданы аргументы';
}

if ($_GET['operation'] === '/' && $_GET['x2'] === '0') {
    return 'Деление на ноль невозможно';
}

if (!is_numeric($_GET['x1']) || !is_numeric($_GET['x2'])) {
    return 'Один из аргументов имеет неверный тип';
}

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

$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 = $x1 / $x2;
        break;
    default:
        return 'Операция не поддерживается';
}

return $expression . $result;

if (empty($_GET)){
    return 'Ничего не переданно';
}
if (empty($_GET['operation'])){
    return ' Не заданно действие';
}
if (($_GET['x1']==='') || ($_GET['x2'] === '')){
    return 'Не заданны агрументы';
}
if (!is_numeric($_GET['x1']) || !is_numeric($_GET['x2'])){
    return 'Аргументы не являются числами';
}

$x1 = $_GET['x1'];
$x2 = $_GET['x2'];
$operation = $_GET['operation'];
$expression = $x1 . ' ' . $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) : ' На 0 дедить нельзя!';
        break;
    default:
        return 'Операция не поддерживается';
}

return $expression . $result;

v2

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

if (!isset($_GET['x1'], $_GET['x2'], $_GET['operation'])) {
    return 'Отсутствуют данные для вычесления';
}

$x1 = str_replace(',', '.', $_GET['x1']);
$x2 = str_replace(',', '.', $_GET['x2']);
$oper = $_GET['operation'];

if (empty($oper)) {
    return 'Не передана операция';
}
if (($x1 === '') || ($x2 === '')) {
    return 'Не переданы документы';
}
if ((is_numeric($x1) === false) || (is_numeric($x2) === false)) {
    return 'Данные не являются числом';
}
if (($x2 === '0') && ($oper == '/')) {
    return 'На ноль делить нельзя!';
}

$expr = $x1 . ' ' . $oper . ' ' . $x2 . '=';

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

return $expr . ' ' . $result;

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

if (is_null($_GET['x1']) || is_null( $_GET['x2']) )  {  // 1. Передача нуля.
    return 'Не переданы аргументы';
}

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

$expression = $x1 . ' ' . $_GET['operation'] . ' ' . $x2 . ' = ';
if(is_numeric($_GET['x1']) && is_numeric($_GET['x2'])) {              //4. Проверка на строку
    switch ($_GET['operation']) {
        case '+':
            $result = $x1 + $x2;
            break;
        case '-':
            $result = $x1 - $x2;
            break;
        case '^':
            $result = $x1 ** $x2;
            break;
        case '%':
            if ($_GET['x2'] != 0) {                                //3. если делить на 0.
                $result = $x1 % $x2;
            } else $result = 'На ноль делить нельзя!';
            break;
        case '/':
            if ($_GET['x2'] != 0) {                                 //3. если делить на 0.
                $result = $x1 / $x2;
            } else $result = 'На ноль делить нельзя!';
            break;
        case '*':
            $result = $x1 * $x2;
            break;
        default:
            return 'Операция не поддерживается';
    }

    if ($_GET['x2'] == 0 && $_GET['operation'] == '/' || $_GET['operation'] == '%') {            //3. если делить на 0.
        return $result;
    } else
        return $expression . $result;
}
else return 'Ведите число, а не строку!';

1)

if ($_GET['x1'] === '' || $_GET['x2'] === ''){
    return 'Не переданы аргументы';
}

2)

case '/':
        $result = $x1 / $x2;
        break;
    case '*':
        $result = $x1 * $x2;
        break;

3)

case '/':
        $result = $x2 != 0 ? $x1 / $x2 : 'На ноль делить нельзя!';
        break;

4)

<input type="number" name="x1">
<input type="number" name="x2">

5) Он прекрасен) Если улучшать то JS наверно.

<?php
    if(empty($_GET)) {
        return 'Ничего не передано!';
    }
    if(empty($_GET['operation'])) {
        return 'Не передана операция!';
    }
    if(!isset($_GET['x1']) || !isset($_GET['x2'])) {
        return 'Не переданы аргументы';
    }
    if(!is_numeric($_GET['x1']) || !is_numeric($_GET['x2'])) {
        return 'Вводить можно только цифры!';
    }

    $x1 = $_GET['x1'];
    $x2 = $_GET['x2'];
    $oper = $_GET['operation'];

    $exp = $x1 . ' ' . $oper . ' ' . $x2 . ' = ';

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

    return $exp . $result;

Не приходит в голову, как код можно нарушить, какие недостатки у кода и как его улучшить. Опыта еще не хватает.

<?php

if ($_GET['one'] === '' || $_GET['two'] === '') {
  return 'Не переданы аргументы';
}

if (($_GET['one'] == 0 || $_GET['two'] == 0) && ($_GET['operation'] == '/' || $_GET['operation'] == '*')) {
  return 'При умножении и делении на 0 результат всегда будет 0';
}

$one = $_GET['one'];
$two = $_GET['two'];

$expression = $one . ' ' . $_GET['operation'] . ' ' . $two . ' = ';

switch ($_GET['operation']) {
  case '+':
    $result = $one + $two;
    break;
  case '-':
    $result = $one - $two;
    break;
  case '*':
    $result = $one * $two;
    break;
  case '/':
    $result = $one / $two;
    break;
  default:
    return 'Операция не поддерживается';
}

return $expression . $result;
?>

По улучшениям:

1. При умножении та же песня что и при делении (всегда 0);
2. Первые две проверки можно убрать, так как operation всегда имеет какое-либо значение, по умолчанию.
3. Возможно можно использовать тернарный оператор, но это не точно)))

По 1 пункту задания: если использовать код

if (empty($_GET['x1']) || empty($_GET['x2'])) {
    return 'Не переданы аргументы';
}

то при передачи 0 в качестве одного из аргументов результат будет "Не переданы аргументы", потому что 0 принимается как пустое значение.

Доработка калькулятора:

<?php

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

if (empty($_GET)) {
    return 'Ошибка! Ничего не передано!';
}
if (empty($operation)){
    return 'Операция не переданна';
}
if ($x1 == null || $x2 == null) {
    return 'Ошибка! Аргумент 1 или аргумент 2 не передан!';
}
if (!is_numeric($x1) || !is_numeric($x2)) {
    return 'Ошибка! Введите число!';
}

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

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

return $expression . $result;

А как передать значение если мы используем checkbox или radio

<?php

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

if (!isset($_GET['x1']) || !isset($_GET['x2']) || !isset($_GET['operation'])){
    return 'Ключи не найдены';
}

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

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

if (!is_numeric($_GET['x1']) || !is_numeric($_GET['x2'])){
    return 'Аргументы могут содержать только числа!';
}

if (($_GET['operation'] == '/') && ($_GET['x2'] == 0)) {
    return 'Деление на 0';
}

$x1 = $_GET['x1'];
$x2 = $_GET['x2'];
$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 = $x1 / $x2;
        break;
    default :
        return 'Операция не поддерживается';
}

return $expression . $result;

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

Код:

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

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

if ((empty($_GET['x1'])) && (empty($_GET['x2']))){
    return 'Не переданы аргументы';
}

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

if ((is_numeric($x1)) && (is_numeric($x2))) {

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

    switch ($_GET['operation']) {
        case '+':
            $result = $x1 + $x2;
            break;
        case '-':
            $result = $x1 - $x2;
            break;
        case '/':
            if ($x2 == 0) {
                return 'На ноль делить нельзя';
            }
            $result = $x1 / $x2;
            break;
        case '*':
            $result = $x1 * $x2;
            break;
        default:
            return 'Операция не поддерживается';
    }
    return $expression . $result;
}
else return 'Переменные должны быть числами';

<?php
//Убрал проверку ввода операции так как ничего не ввести там невозможно
if (($_GET['x1'] == '') || ($_GET['x2'] == '')) {
    return 'Не переданы аргументы';
}

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

$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 (empty($_GET['x1']) || empty($_GET['x2'])) {
    return 'Не переданы аргументы';
}

if (($_GET['number1'] == '') || ($_GET['number2'] == '')) {
    return "Допустимо вводить только числа";
}
$x1 = $_GET['x1'];
$x2 = $_GET['x2'];
$expression = $x1 . ' ' . $_GET['operation'] . ' ' . $x2 . ' = ';

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

Сделал всё, кроме 0. Я думаю, что 0 не воспринимается как число, т.к. трактуется интерпретатором как 'false', но ничего не смог сделать. Подскажите мою ошибку

Добавил возведение в степень, на больше не хватило фантазии, в таком исполнении калькулятора=)

<?php
if(empty($_GET)){
    return 'Ничего не передано!';
}
if(empty($_GET['operation'])){
    return 'Не передана операция';
}
if (($_GET['x1']==='')|| ($_GET['x2']==='')){
    return 'Аргументы не переданы';
}
if (!is_numeric($_GET['x1'])|| !is_numeric($_GET['x2'])){
    return 'Переданные значения не являются числом';
}

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

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

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

if(empty($_GET)){
    echo 'Ничего не передано </br>';
}elseif(empty($_GET['operation'])){
    echo 'Не передана операция </br>';
}elseif(is_numeric($_GET['x1']) == false || is_numeric($_GET['x2']) == false){
    echo 'Нельзя передавать строку </br>';
}elseif(empty($_GET['x1']) != '' && empty($_GET['x2']) != ''){
    echo 'Не переданы аргументы </br>';
}elseif(empty($_GET['x1']) === ''){
    echo 'Не передан аргумент х1 </br>';
}elseif(empty($_GET['x2']) === ''){
    echo 'Не передан аргумент х2 </br>';
}else {
    $x1 = $_GET['x1'];
    $x2 = $_GET['x2'];
    $expression = $x1 . ' ' . $_GET['operation'] . ' ' . $x2 . ' = ';

switch($_GET['operation']) {
    case '+' :
        $result = $x1 + $x2;
        break;
    case '-' :
        $result = $x1 - $x2;
        break;
    case '*' :
        $result = $x1 * $x2;
        break;
    case '/' :
        if($x1 == 0){
            echo 'На ноль делить нельзя </br>';
        }else {
            $result = $x1 / $x2;
        }
        break;
    default:
        return 'Операция не может быть выполнена';
}

return $expression . $result;
}

пишите здесь ваш код
<?php
if (empty($_GET)) {
    return 'Ничего не передано!';
}

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

if (empty($_GET['int x1'] + 1) || empty($_GET['int x2'] + 1)) {
    return 'Не переданы аргументы';
}

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

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

switch ($_GET['operation']) {
    case '+':
        $result = $x1 + $x2;
        break;
    case '-':
        $result = $x1 - $x2;
        break;
    case '*':
        if ($x1 == 0 || $x2 == 0 || ($x1 == 0 && $x2 == 0)) {
            $result = 'при умножении числа на 0 или 0 на число или 0 на 0 всегда будет 0, не делайте так';
        } else {
            $result = $x1 * $x2;
        }
        break;
    case '/':
        if ($x1 == 0 || $x2 == 0 || ($x1 == 0 && $x2 == 0)) {
            $result = 'при делении 0 на число или 0 всегда  будет 0, а делить число на 0 нельзя, не делайте так';
        } else {
            $result = $x1 / $x2;
        }
        break;

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

return $expression . $result;

``

Вопросы:
1)Правильно ли я понял, что GET и POST преобразует все в строки?
2)Когда я заполняю форму то на странице результата я вижу такую ссылку http://calculate.loc/result.php?numOne=3&index=%2B&numTwo=4
а если убрать все данные и ввести вот так http://calculate.loc/result.php, мне выдаст кучу ошибок, ведь форму получается просто проигнорили)
Это норм или мой любимый калькулятор смогут взломать как картонную коробочку?)

3)Почему в условии if вы пишите return a не echo? И как вообще return выводит строки, когда он только должен возвращать?))

Домашнее задание
Чтобы калькулятор принимал только числа, я на страницу с html формой поменял атрибут на type="number";

<?php
    $numOne = $_GET['numOne'];
    $numTwo = $_GET['numTwo'];
    $index = $_GET['index'];

    if(empty($numOne) && empty($numTwo)) {
        return 'Вы ничего не вписали';
    }

    if(empty($numOne) && $numOne != 0) {
        return 'Не ввели первое число';
    }

    if(empty($numTwo) && $numTwo != 0) {
        return 'Не ввели второе число';
    }

    switch ($index) {
        case '+':
            $calculatorResult = $numOne + $numTwo;
            break;
        case '-':
            $calculatorResult = $numOne - $numTwo;
            break;
        case '/':
            if($numOne == 0) {
                $calculatorResult = 0;
                break;
            }
            if ($numTwo == 0) {
                $calculatorResult = 'Деление на ноль невозможно';
                break;
            }
            $calculatorResult = $numOne / $numTwo;
            break;
        case '*':
            if($numOne == 0 || $numTwo == 0){
                $calculatorResult = 0;
                break;
            }
            $calculatorResult = $numOne * $numTwo;
            break;
        default:
            echo 'Похоже ты сломал калькулятор, ты божественный хакер!';
    }
    $result =  $numOne . ' ' . $index . ' ' . $numTwo . '  = ' . $calculatorResult;
    return $result;

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

    if (empty($_GET['x1']) || empty($_GET['x2'])) {
        return "Не переданы аргументы";
    }

    if (empty($_GET['operation'])) {
        return "Не выбрано действие";
    }

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

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

    if (is_numeric($x1) && is_numeric($x2)) {
        switch ($_GET['operation']) {
            case '+':
                $res = $x1 + $x2;
                break;
            case '-':
                $res = $x1 - $x2;
                break;
            case '*':
                $res = $x1 * $x2;
                break;
            case '/':
                if ($x1 == 0 || $x2 == 0) {
                    return "На ноль делить нельзя";
                }
                $res = $x1 / $x2;
                break;
            default:
                return 'Неверная операция';
        }
        return $expression . $res;
    }
    return 'Введите числа';

3. • Добавил бы возврат на страницу с формой при ошибке, с выводом ошибки на эту страницу.
   • Вывод результата на страницу с формой

А как быть, если я хочу воспользоваться методом GET, а не POST. И пытаюсь сделать форму, в которую можно будет записать числа и получить результат на той же страничке? При таком написании все результаты равны нулю. Что я делаю не правильно?

$x1 = NULL;
$x2 = NULL;

if (isset($_GET['operation'])) { //если элемент массива operation существует
    if ((isset($_GET['x1']))&& isset($_GET['x2'])) {
        switch ($_GET['operation']) {
            case '+':
                $result = $x1 + $x2;
                break;
            case '-':
                $result = $x1 - $x2;
                break;
            case '*':
                $result = $x1 * $x2;
                break;
            case '/':
                if (0 == $x2) {
                    $result = 'На ноль делить нельзя!';
                } else {
                    $result = $x1 / $x2;
                }
                break;
        }
    } else {
        echo ('аргументы не заданы');
    }
    echo $_GET['x1'] . ' ' . $_GET['operation'] . ' ' . $_GET['x2'] . ' = ' . $result;
}
?>

<!doctype html>
<html lang="en">
<head>
    <meta charset="UTF-8">
    <meta name="viewport"
          content="width=device-width, user-scalable=no, initial-scale=1.0, maximum-scale=1.0, minimum-scale=1.0">
    <meta http-equiv="X-UA-Compatible" content="ie=edge">
    <title>Калькулятор</title>
</head>
<body>
    <form action="/index.php" method="get">
        <input type="number" name="x1" required>
        <select name="operation">
            <option value="+">+</option>
            <option value="-">-</option>
            <option value="*">*</option>
            <option value="/">/</option>
        </select>
        <input type="number" name="x2" required>
        <button type="submit">=</button>
    </form>
</body>
</html>

<?php
if (empty($_GET)) {
    return 'Ничего не переданно!';
}
if (empty($_GET['operation'])) {
    return 'Не переданна операция';
}
if (!isset($_GET['x1']) || !isset($_GET['x2'])) {     // Передача нуля
    return 'Не переданны аргументы';
}
if (!is_numeric($_GET['x1']) || !is_numeric($_GET['x2'])) {   // Проверка на строку
    return 'Ведите число!';
}
$x1 = $_GET['x1'];
$x2 = $_GET['x2'];

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

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

return  $expression .$result;

Функция empty() возвращает true, когда переменная не существует и имеет пустое или нулевое значение. Поэтому в данном примере следует использовать проверку только на null (Например функция is_null()).

При делении на 0 вызывается Warning: Division by zero. Следует отлавливать передачу "0" второму аргументу, когда выбрана операция деления.

При передаче в качестве аргумента строки будет выводится Warning: A non-numeric value при выполнении арифметических выражений. Чтобы отловить передачу нечисловой строки можно воспользоваться функцией is_numeric().

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

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

if (is_null($_GET['x1']) || is_null($_GET['x2'])){
    return 'Не переданы аргументы';
}

if (!is_numeric($_GET['x1']) || !is_numeric($_GET['x2'])) {
    return 'Используйте в качестве аргументов только числа';
}

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

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

5.
Улучшить.. Хорошо бы проверку на корректность ввода добавить еще на стороне клиента, дабы не обращаться к серверу, если пользователь накосячил. При этом проверку на серве тоже следует оставить на тот случай, если проверка в клиентской части косячная.
Еще можно использовать функцию number_format($result, 2), чтобы задать точность и избежать проблем при выполнении операций с дробными числами. Чтобы 36-35.99 равнялось 0.01, а не 0.009999999999998 :D

С вводом ноля и чисел разобрался, деление и умножение добавил, а вот как ещё сломать не придумал(

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

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

if (!is_numeric($_GET['x1']) || !is_numeric($_GET['x2'])) {
    return 'Не переданы аргументы';
}

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

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

switch ($_GET['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;

При отсутствии указания метода выбирается "Get" ?

Добрый день,такое решение задачи неверно? (пункт про ввод только цифр)

<html>
<head>
    <title>Калькулятор</title>
</head>
<body>
<form action="/result.php">
    <input onkeypress="return keyPress('0123456789')" type="number" name="x1">
    <select name="operation">
        <option value="+">+</option>
        <option value="-">-</option>
        <option value="*">*</option>
        <option value="/">/</option>
    </select>
    <input onkeypress="return keyPress('0123456789')" type="number" name="x2">
    <input type="submit" value="Посчитать">
</form>
</body>
</html>

<?php
if ($_GET['x2'] == 0 && $_GET['operation'] == "/"){
    return 'На ноль делить нельзя!';
}
if ($_GET['x1'] == '' || $_GET['x2'] == '' ){
    return 'Вы ввели пустой аргумент!';
}

$x1 = ($_GET['x1']);
$x2 = ($_GET['x2']);

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

if (!is_numeric($x1) || !is_numeric($x2)){
    return 'Один или несколько аргументов введены в неправильном формате. Аргумент должен быть числом!';
}

switch ($_GET['operation']){
    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($_GET['x1']) && empty($_GET['x2'])) {
    return 'Ничего не передано!';
}

if (empty($_GET['x1']) || empty($_GET['x2'])) {
    return 'Один из аргументов не передан.';
}

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

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

if (!is_numeric($_GET['x1']) || !is_numeric($_GET['x2'])) {
    return 'Только числа в качестве аргументов.';
}

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

$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;

Пришлось немного подсмотреть. Хочется, конечно, всё самостоятельно делать. И всё же очень крутые уроки, прям очень!!!) Спасибо большое!

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

if ($_GET['x1'] === '' && $_GET['x2'] === '') {
    return 'Ничего не передано.';
}

if ($_GET['x1'] === '' || $_GET['x2'] === '') {
    return 'Не передан один из аргументов.';
}

if (!is_numeric($_GET['x1']) || !is_numeric($_GET['x2'])) {
    return 'Только числа в качестве аргументов.';
}

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

$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;

Попробовал ещё один вариант, вроде работает нормально.

Здравствуйте, подскажите в чём причина такой проблемы?

В файле index.php такой код:

<html>
<head>
    <title>Калькулятор</title>
</head>
<body>
<form action="/result.php">
    <input type="text" name="x1">
    <select name="operation">
        <option value="+">+</option>
        <option value="-">-</option>
    </select>
    <input type="text" name="x2">
    <input type="submit" value="Посчитать">
</form>
</body>
</html>

И с ним выдаёт ошибку 404. Если сменить путь на просто <form action="result.php">, то всё работает, НО не почему-то в файле result.php работает только

$result = require __DIR__ . '/calc.php';

и если сменить на $result = require DIR . 'calc.php';, то выдаст ошибку:
Страница недоступнаСайт localhost пока не может обработать этот запрос.
HTTP ERROR 500

Все файлы находятся в 1ой директории, в одной папке

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

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

if(empty($_GET['x1']) || empty($_GET['x2'])) {
    return 'Не переданы аргументы';
}

if (!is_numeric($_GET['x1']) || !is_numeric($_GET['x2']) ) {
    return 'Аргументы не являются числом';
}
$x1 = $_GET['x1'];
$x2 = $_GET['x2'];

$expression = $x1 . ' ' . $_GET['operation'] . ' ' . $x2 . ' = ';
switch ($_GET['operation']) {
    case '+':
        $result = $x1 + $x2;
        break;
    case '-':
        $result = $x1 - $x2;
        break;
    case '*':
        if ($x1 == 0 || $x2 == 0 || ($x1 == 0 && $x2 == 0)) {
            $result = ' при умножении числа на 0 или 0 на число или 0 на 0 всегда будет 0, не делайте так';
        } else {
            $result = $x1 * $x2;
        }
        break;
    case '/':
        if ($x1 == 0 || $x2 == 0 || ($x1 == 0 && $x2 == 0)) {
            $result = 'при делении 0 на число или 0 всегда  будет 0, а делить число на 0 нельзя, не делайте так';
        } else {
            $result = $x1 / $x2;
        }
        break;
    default:
        return 'Операция не поддерживается';
}

return $expression . $result;

<?php

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

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

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

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

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

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

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

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

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

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

return $expression . $result;

4е задание:
поменять в index.php type на number.

Добрый вечер! и снова я с вами проверьте дз пож?

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

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

if (empty($_GET['x1']) === 'null' || empty($_GET['x2']) === 'null'){
    return 'Не переданы аргументы';
}
$x1 = $_GET['x1'];
$x2 = $_GET['x2'];

//проверка на число: int or string
//если условии истины то дальше идет на switch, если нет то возвращает return    
if (!is_numeric($x1) || !is_numeric($x2)){
    return 'x1 и x2 должны быть числами';
}

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

return $expression . $result;

<?php
//проверка на пустой запрос
if (empty($_GET)) {
    return 'Ничего не передано!';
}
//проверка на существование индексов х1, х2 и operation в запросе
if (isset($_GET['x1']) && isset($_GET['x2']) && isset($_GET['operation'])) {
    $x1 = $_GET['x1'];
    $x2 = $_GET['x2'];
} else {
    return 'Запрос некорректно сформирован. Обновите страницу и повторите.';
}
//проверка на пустой оператор
if (empty($_GET['operation'])) {
    return 'Не передана операция';
}
//проверка на пустые аргументы (и передача 0 в калькулятор)
if (is_null($_GET['x1']) || is_null($_GET['x2'])) {
    return 'Не переданы аргументы';
}
//проверка на строковые выражения
if (!is_numeric($_GET['x1']) || !is_numeric($_GET['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 '/':
        if ($x2 != 0) {
            $result = $x1 / $x2;
            break;
        }
        return 'Деление на ноль!';
        break;
    default:
        return 'Операция не поддерживается';
}

return $expression . $result;

<?php

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

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

if (isset($_GET['x1']) && isset($_GET['x2'])) { //Задание 5. Сам не додумался, подсмотрел у других
    $x1 = $_GET['x1'];
    $x2 = $_GET['x2'];
} else {
    return 'Ошибка в неполной передаче данных, должны быть переданы 2 числа';
}

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

if (!is_numeric($_GET['x1']) || !is_numeric($_GET['x2'])) { //Задание 4.
    return 'Введите корректные числа';
}

$operation = $_GET['operation'];

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

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

return $expression . $result;

Сперва полез в index.php сменить тип с text на number, но понял что это не то. Т.к. дробные числа не получиться написать.
Кстати, а есть ли необходимость в первых двух условиях, если и так все условия обработаны?

<?php
/**
 * Created by PhpStorm.
 * User: LUCKYPAS
 * Date: 15.12.2019
 * Time: 16:15
 */
if (empty($_GET)) {
    return 'Ничего не передано!';
}

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

if (empty($_GET['x1']) || empty($_GET['x2'])) {
    return 'Не переданы аргументы';
}
function MyString($text)
{
    $temp = trim($text); // обрубаем текст
    $temp = stripslashes($temp); // убираем слешы
    $temp = htmlspecialchars($temp, ENT_QUOTES);// преобразование в штмл сущности
    return $temp;
}

$x1 = $_GET['x1'];
$x2 = $_GET['x2'];
$x1 = MyString($x1);
$x1 = MyString($x2);

$expression = $x1 . ' ' . $_GET['operation'] . ' ' . $x2 . ' = ';
if (is_numeric($x1) && is_numeric($x2)) {// проверка на числа
    switch ($_GET['operation']) {
        case '+':
            $result = $x1 + $x2;
            break;
        case '-':
            $result = $x1 - $x2;
            break;
        case '/':
            if($x2 != 0){
                $result = $x1 / $x2;
            } else {
                return 'На ноль нельзя делить';
            }
            break;
        case '*':
            if($x1 != 0 || $x1 != 0){
                $result = $x1 * $x2;
            } else {
                return 'На ноль нельзя умножать';
            }
            break;
        case '%':
            $result = $x1 % $x2;
            break;

        default:
            return 'Операция не поддерживается';
    }
} else {
    echo "Введите цифры";
}
return $expression . $result;

if (is_numeric($x1) && is_numeric($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 'Такой операции еще нет';
}
    $expression = $x1 . ' ' . $_GET['operation'] . ' ' . $x2 . ' = ';
    return $expression . $result;}
    else {
        return 'Вводите числа, это же калькулятор';
    }

Правильно ли я понимаю, что при ф-и риквае, файл1 который вшиваю в файл2, имеет доступ к переменным файл2?
То есть в файле2 пишем риквае файл1

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

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

if (!is_numeric($_GET['x1'])) {
    return 'Значение x1 не является числом';
}

if (!is_numeric($_GET['x2'])) {
    return 'Значение x2 не является числом';
}

if (($_GET['x2'] == 0)  &&  ($_GET['operation'] == ':')) {
    return 'Не допустимое деление на ноль';
}

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

$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 = $x1 / $x2;
        break;
    default:
        return 'Операция не поддерживается';
}

return $expression . $result;

<?php
if (empty($_GET['x1']) || empty($_GET['x2'])) {
    return 'Не переданы аргументы';
}

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

if (!is_numeric($_GET['x1'])) {
    return 'Значение x1 не является числом';
}

if (!is_numeric($_GET['x2'])) {
    return 'Значение x2 не является числом';
}

if (($_GET['x2'] == 0)  &&  ($_GET['operation'] == ':')) {
    return 'Не допустимое деление на ноль';
}

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

$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 = $x1 / $x2;
        break;
    default:
        return 'Операция не поддерживается';
}

return $expression . $result;

Вот что получилось!

<?php

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

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

if ((empty($_GET['op1']) && !is_numeric($_GET['op1'])) || empty($_GET['op2']) && !is_numeric($_GET['op2'])) {
    return 'Не передан аргумент!';
}

if (!is_numeric($_GET['op1']) || !is_numeric($_GET['op2'])) {
    return 'Передано не число!';
}

$x1 = intval($_GET['op1']);
$x2 = $_GET['op2'];
$operator = $_GET['operation'];

switch ($operator) {
    case '+':
        $result = "$x1 + $x2 = " . ($x1 + $x2);
        break;
    case '-':
        $result = "$x1 - $x2 = " . ($x1 - $x2);
        break;
    case '*':
        $result = "$x1 * $x2 = " . ($x1 * $x2);
        break;
    case '/':
        if ($x2 === '0') {
            $result = 'Делить на 0 нельзя!';
        } else {
            $result = "$x1 / $x2 = " . ($x1 / $x2);
        }
        break;
    default:
        return 'Нет такой операции';
}

return $result;

ПО 5 заданию затрудняюсь ответить.

<?php
    $x1 = $_GET['x1'];
    $x2 = $_GET['x2'];
    $operation = $_GET['operation'];

if (empty($_GET)) {
    return 'Введите данные';
} 
if (!isset($x1) || !isset($x2)) {
    return 'Введите числа';
}
if ($x1 == 0 || $x2 == 0 && $operation == '/') {
    return 'На ноль делить нельзя';
}

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

switch ($operation) {
    case '+':
        $result = $x1 + $x2;
        break;
    case '-':
        $result =  $x1 - $x2;
        break;  
    case '*':
        $result =  $x1 * $x2;
        break;  
    case '/':
        $result =  $x1 / $x2;
        break;          
}

return $expression . $result;

1) Функция empty получила значение '0', а данное значение для нее является пустым, поэтому выдало сообщение 'Введите числа'. Поменял функцию empty на isset
2) Сделал. Добавил в два input'a в index.php и два case в calc.php
3) Вышла ошибка warning о делении на ноль. Добавил дополнительное условие
4) Вышла ошибка warning обнаружено нечисловое значение. Изменил тип для input'a, с text на number и добавил атрибут step="any", чтобы считать числа float
5) a) В option у нас нет пустого значения, поэтому условие operation можно убрать
b) В файле calc.php можно вначале добавить переменные для x1, x2 и operation, чтобы код легче читался
c) Из switch убрать default, т.к. операция будет выбрана в любом случае
d) Можно добавить еще операторы для вычислений

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

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

if (empty($_GET['x1']) && empty($_GET['x2'])) {
    return 'Не переданы аргументы';
}
if (!is_numeric($_GET['x1']) || !is_numeric($_GET['x2'])) {
    return 'Одно или более из переданных данных не является числом';
}
$x1 = ($_GET['x1']);
$x2 = ($_GET['x2']);

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

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

return $expression . $result;

Не пойму, почему операция "число" поделить на "не целое число" (например 1 / 0.2) приводит к "Операция не поддерживается"?

<?php
    if(empty($_GET)){
        return 'Нам нечего считать';
    }
    if(empty($_GET['operation'])){
        return 'Не передана операция';
    }
    if(empty((float)$_GET['x1'])|| empty((float)$_GET['x2'])){
        if($_GET['x1']==='0' || $_GET['x2']==='0'){
            if ($_GET['x1']==='0'){
                $_GET['x1']=0;
            };
            if ($_GET['x2']==='0'){
                $_GET['x2']=0;
            }
        } else {
            return 'Не переданы аргументы';
        }
    }
    $x1=$_GET['x1'];
    $x2=$_GET['x2'];

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

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

Так лучше?

    if(empty($_GET)){
        return 'Нам нечего считать';
    }
    if(empty($_GET['operation'])){
        return 'Не передана операция';
    }
    if(!isset($_GET['x1']) || !isset($_GET['x2'])){
        return 'Не переданы аргументы';
    }
    if (is_numeric($_GET['x1']) && is_numeric($_GET['x2'])){
        $x1=$_GET['x1'];
        $x2=$_GET['x2'];
    } else {
        return 'Переданы не числовые аргументы';
    }

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

    switch ($_GET['operation']){
        case '+':
            $result = $x1 + $x2;
            break;
        case '-':
            $result = $x1 - $x2;
            break;
        case '*':
            $result = $x1 * $x2;
            break;
        case '/':
            if($x2!=0){
                $result = $x1 / $x2;
            } else {
                return 'Это невозможно!!!';
            }
            break;
        default:
            return 'Операция не поддерживается';
    }
    return $expresion . $result;

<?php
if (isset($_GET['x1'], $_GET['x2'], $_GET['operation'])) {
    if (($_GET['x1'] == '') || ($_GET['x2'] == '')) {
        return 'Не переданы аргументы';
    }
    elseif (!is_numeric($_GET['x1']) || !is_numeric($_GET['x2'])) {
        return 'допустимы только целые числа';
    }
    if (empty($_GET)) {
        return 'Ничего не передано!';
    }
    if (empty($_GET['operation'])) {
        return 'Не передана операция';
    }
}
$x1 = (int) $_GET['x1'];
$x2 = (int) $_GET['x2'];
$expression = $x1 . ' ' . $_GET['operation'] . ' ' . $x2 . ' = ';
switch ($_GET['operation']) {
    case '+':
        $result = $x1 + $x2;
        break;
    case '-':
        $result = $x1 - $x2;
        break;
    case '*':
        $result = $x1 * $x2;
        break;
    case '/':
        if($x2 != 0) {
            $result = $x1 / $x2;
        }
        else {
            return 'На ноль делить нельзя';
        }
        break;
    default:
        return 'Операция не поддерживается';
}
return $expression . $result;

<?php

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

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

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

if (is_numeric($_GET['x1']) && (is_numeric($_GET['x2']))) {
} else {
    return 'Можно передавать только числа';
}

$x1 = $_GET['x1'];
$x2 = $_GET['x2'];
$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;

Подскажи пожалуйста, как сделать, чтоб результат отображался на этой же странице, где сама форма в index.php?
Т.е. ставим перед кнопкой Посчитать знак "=" и выводим результат тут же, при клике на кнопку Посчитать.

Я попробовал просто в index.php сделать подключение:

<?php
$result = require __DIR__ . '/1/calc.php';
?>

и в нужном месте вывести:

<?= $result ?>

Но потом меня сбило с толку, что делать с form action="" что тут писать тогда?!
Правильно ли вообще я думаю? В данном случае после подключения файла выводится ошибка:
https://cloud.mail.ru/public/EL5G/U99mFpcYx
и в результат сразу выводит информацию, даже не кликнув по кнопке Посчитать

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

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

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

if (!isset($x1)) {
    if (!isset($x2)) {
            return 'Аргументы не переданы';
    } else {
            return 'Не передан первый аргумент';
    }
}

if (!isset($x2)) {
        return 'Не передан второй аргумент';
}

if (!is_numeric($x1)) {
    if (!is_numeric($x2)) {
        return 'Аргументы не являются числами';
    } else {
        return 'Первый аргумент не является числом';
    }
}

if (!is_numeric($x2)) {
    return 'Второй аргумент не является числом';
}
$x1 += 0;
$x2 += 0;

$expression = $x1 . ' ' . $_GET['operation'] . ' ' . $x2 . ' = ';
switch ($_GET['operation']){
    case '+':
        $result = $x1 + $x2;
        break;
    case '-':
        $result = $x1 - $x2;
        break;
    case '*':
        $result = $x1 * $x2;
        break;
    case '/':
        if ($x2 == 0) {
            return 'Попытка деления на 0';
        }
        $result = $x1 / $x2;
        break;
    case '^':
        $result = pow($x1, $x2);
        break;
}
return $expression . $result;
?>

добрый день, сам сделал только 1 и 2, третье подсмотрел, всё как у всех, так что флудить не буду. Ломал голову пару вечеров по поводу третьего, но прежде чем писать код на 10 стр. решил подглядеть, оказалось, что всего то одна строка как тут case '/' :
$result = $x2 != 0 ? ($x1 / $x2) : 'На ноль делить нельзя';
break;

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

<?php
    //Request's
    $x1 = $_GET['x1'];
    $x2 = $_GET['x2'];

    //Block errors
    if(!is_numeric($x1) && !is_numeric($x2)) {
        return "Неверный синтаксис! - Калькулятор сломался! :,(";
    } elseif(!is_numeric($x1) || !is_numeric($x2)) {
        return "Неверный синтаксис! - Калькулятор сломался! :,(";
    }

    if(empty($_GET['operation'])) {
        return "operation не был передан! - Калькулятор сломался! :,(";
    }

    if(!empty($_GET['x1'] === '') || !empty($_GET['x2'] === '')) {
        return "x1 или x2 не были переданны! - Калькулятор сломался! :,(";
    }

    if($_GET['x2'] == 0 / true && $_GET['operation'] == ":") {
        return "На ноль делить нельзя! - Калькулятор сломался! :,(";
    }
    //Block errors

    $calculator = $_GET['x1'] . " " . $_GET['operation'] . " " . $_GET['x2'] . " = ";
    switch($_GET['operation']) {
        case "+":
            $result = $x1 + $x2;
            break;
        case "-":
            $result = $x1 - $x2;
            break;
        case "*":
            $result = $x1 * $x2;
            break;
        case ":":
            $result = $x1 / $x2;
            break;
        case "%":
            $result = $x1 % $x2;
            break;
    }
    return $calculator . $result;
    ?>

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

if (empty($_GET['operation'])) {
    return 'Не понимаю, что делать';
}

if (!isset($_GET['x1']) || !isset($_GET['x2'])) {
    return 'Как минимум один из аргументов пуст';
}

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

$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 = $x1 / $x2;
        break;
    default:
        return 'Операция не поддерживается';
}

#деление на 0:
if ($_GET['x2'] == 0 and $_GET['operation'] == '/') {
    return 'На ноль делить нельзя!';
}

#Вводим только числа:
if ($_GET['x1'] !== preg_match("[0-9]", '') or $_GET['x2'] !== preg_match("[0-9]", '')) {
    return 'С тем, что Вы ввели, в математику нельзя!';
}

return $expression . $result;

?>

Можно добавить возведение в степень и прочие более сложные операции.
Можно нарисовать кнопки, чтобы в них можно было тыкать, вместо того, чтобы вводить инпуты (ну, мало ли, у человека только мышь есть)
Сломать можно, думаю, слишком длинным вводом

Здравствуйте, форма передаёт GET запрос в result.php, почему вам удалось получить все аргументы GET запроса из другого файла(вызывая $_GET в calc.php), почему не возникло ошибки?

<?php
if (empty($_GET)) {
    return 'Ничего не передано!';
}
if (empty($_GET['operation'])) {
    return 'Не передана операция';
}
//Дополните код так, чтобы можно было передавать 0.
if (!isset($_GET['x1']) || !isset($_GET['x2']) || !isset($_GET['operation'])){
    return 'Не переданы ключи';
}
$x1 = $_GET['x1'];
$x2 = $_GET['x2'];
if ($x1 === '' || $x2 === '') {
    return 'Не переданы аргументы';
}
//Разделитель дробной части может быть ','. Заменим ',' на '.' (например, 0,6=0.6).
if (stristr($x1, ",") != false) {
    $x1 = str_replace( ",", ".", $x1);
}
if (stristr($x2, ",") != false) {
    $x2 = str_replace(",", ".", $x2);
}
//Чтобы в качестве аргументов можно было отправить только числа.
if (!is_numeric($x1)) {
    return 'Первое значение не число';
} else {
    $x1 = floatval($x1); //приводим к float
}
if (!is_numeric($x2)) {
    return 'Второе значение не число';
} else {
    $x2 = floatval($x2); //приводим к float
}
//Зачем приводим к float:
//Если вводим, например, 00000.3 или .3, то программа это воспринимает как 0.3 и выполняется
//Результат вычислений: 00000.3 + 4 = 4.3 или .3 + 4 = 4.3
//Т.е. при выводе нужно заменить 00000.3 и .3 на 0.3
$expression = $x1 . ' ' . $_GET['operation'] . ' ' . $x2 . ' = ';
switch ($_GET['operation']) {
    case '+':
        $result = $x1 + $x2;
        break;
    case '-':
        $result = $x1 - $x2;
        break;
    case '*':
        $result = $x1 * $x2;
        break;
    case '/':
        if ($x2 != 0) {
            $result = $x1 / $x2;
        } else {
            return 'На 0 делить нельзя!';
        }
        break;
    default:
        return 'Операция не поддерживается';
}
return $expression . $result;

Пыталась с помощью strpos проверить есть ли "," в строке.
strpos(',7', ',') != false будет false
strpos('0,7', ',') != false будет true
Почему так происходит?

<?php
//Проверка на пустой массив
if (empty($_GET)) {
    return 'Ничего не передано!';
}
// Получем данные из GET и обрезаем теги, далее пользуемся только переменными
$x1 = strip_tags($_GET['x1']);
$x2 = strip_tags($_GET['x2']);
$operation = strip_tags($_GET['operation']);
//Проверка заполнения операции
if (empty($operation)) {
    return 'Не передана операция';
}
//Проверка на числа
if (!is_numeric($x1) || !is_numeric($x2)) {
    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;
    case '^':
        $result = $x1 ** $x2;
        break;
    default:
        return 'Операция не поддерживается';
}

return $expression . $result;

<?php

$argumentOne = $_GET['argumentOne'];
$argumentTwo = $_GET['argumentTwo'];
$operation = $_GET['operation'];

if (empty($_GET)){
    return 'Nichego ne peredano';
}

if (empty($operation)){
    return 'Operaciya ne vibrana';
}

if ($argumentOne === '' || $argumentTwo === ''){
    return 'Argumenti ne peredani';
}

if (!is_numeric($argumentOne) || !is_numeric($argumentTwo)) {
    return 'Neverniy format';
}

$display = $argumentOne .' '. $operation .' '. $argumentTwo . ' = ';

switch ($operation) {
    case '+':
        $result = $argumentOne + $argumentTwo;
        break;

    case '-':
        $result = $argumentOne - $argumentTwo;
        break;

    case '*':
        $result = $argumentOne * $argumentTwo;
        break;
    case '/':
        if($argumentTwo == 0){
            return 'na 0 delit nelza';
        } else {
            $result = $argumentOne / $argumentTwo;
        }
        break;
}

return $display . $result;

Такой вот вопрос: В чем разница между проверкой на ввод числа на фронте и бэке ? Лучше ставить type="number" в инпуте, или всё же проверять с помощью php?

<?php

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

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

if (empty($_GET['x1']) || empty($_GET['x2'])) {
    if ($_GET['x1'] != 0 & $_GET['x2'] != 0) {
        return 'Не переданы аргументы';
    }
}
if (!is_numeric($_GET['x1']) || !is_numeric($_GET['x2'])) {
    return 'Вы вводите не числа, внимательнее!';
}

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

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

switch ($_GET['operation']) {
    case '+':
        $result = $x1 + $x2;
        break;
    case '-':
        $result = $x1 - $x2;
        break;
    case '*':
        $result = $x1 * $x2;
        break;
    case '/':
        if ($x2 != 0) {
            $result = $x1 / $x2;
            break;
        } else {
            $result = 'Деление на 0! Ужасно!';
            break;
        }
    default:
        return 'Operation is not defined';
}

return $expression . $result;

Я вот сделал так и решительно не понимаю, что можно улучшить, кроме уж совсем совершенствования калькулятора для подсчета, например, длинного выражения, но такое я пока не умею делать(

1. Заменить:

   if (empty($_GET['x1']) || empty($_GET['x2'])) {
   return 'Не переданы аргументы';

   на:

   if ($_GET['x1'] == '' || $_GET['x2'] == '') {
   return 'Не переданы аргументы';

2. В:

   switch ($_GET['operation'])

   добавить:

   case '*':
       $result = $x1 * $x2;
       break;
   case '/':
       $result = $x1 / $x2;
       break;

3. Изменить:

   case '/':
       $result = $x1 / $x2;
       break;

   на:

   case '/':
       if ($x2 != 0) {
           $result = $x1 / $x2;
       } else {
           $result = 'Деление на ноль запрещено';
       }
       break;

4. В файле index.php изменить:

   <input type="text" name="x1">

   и

   <input type="text" name="x2">

   на:

   <input type="number" name="x1" pattern="^[0-9]+$">

   и

   <input type="number" name="x2" pattern="^[0-9]+$">

   атрибут pattern с выражением "^[0-9]+$" разрешает отправлять форме только числа.

5.Можно добавить операции, например возведение в степень.
Я бы изменил:

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

на:

default:
        return 'Операция '.$_GET['operation'] .' не поддерживается';

для того, чтобы было понятно какая операция не выполняется.

<?php
if (empty($_GET)) {
    return 'Ничего не передано!';
}
if (empty($_GET['operation'])) {
    return 'Не передана операция';
}
if (!isset($_GET['x1']) || !isset($_GET['x2'])) {
    return 'Не переданы аргументы!';
}
if (!is_numeric($_GET['x1']) || !is_numeric($_GET['x2'])) {
    return 'Некорректный аргумент!';
}

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

switch ($_GET['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;

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

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

if (($_GET['x1'] === null) || ($_GET['x2'] === null))
    return 'Аргументы не переданы';

if (!is_numeric($_GET['x1']) || !is_numeric($_GET['x2']))
    return 'Введите числа';

if (isset($_GET['x1']) && isset($_GET['x2'])) {
    $x1 = intval($_GET['x1']);
    $x2 = intval($_GET['x2']);
}
$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;

1. "Не переданы аргументы". Это получилось из за того что выражение:

if (empty($_GET['x1']) || empty($_GET['x2'])) {
    return 'Не переданы аргументы';
}

выполнилось, потому что функция empty($_GET['x1'] вернула TRUE, а функция empty(): Возвращает FALSE, если переменная существует и содержит непустое ненулевое значение. В противном случае возвращает TRUE. Мы передали в нее 0, соответственно она вернула TRUE.

Исправить можно так:

if ((empty($_GET['x1']) && $_GET['x1'] != 0) || (empty($_GET['x2']) && $_GET['x2'] != 0)) {
    return 'Не переданы аргументы';
}

2.

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

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

if (empty($_GET['x1']) || empty($_GET['x2'])) {
    if ($_GET['x1'] != 0 || $_GET['x2'] != 0) {
        return 'Не переданы аргументы';
    }
}

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

$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 = $x1 - $x2;
        break;
    default:
        return 'Операция не поддерживается';
}

return $expression . $result;

3. Выдаст ошибку: На ноль делить нельзя.

Доработка:

switch ($_GET['operation']) {
    case '/':
        if ($x2 == 0) {
            return 'На ноль делить нельзя!!!';
        }
        $result = $x1 / $x2;
        break;

4.

<input type="number" name="x1" step="any">
<input type="number" name="x2" step="any">

5. Свой скромный функционал он выполняет на ура.

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

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

if (($_GET['x1'] === '') || ($_GET['x2'] === '')) {
    return 'Не переданы аргументы';
}

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

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

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

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

return $expression . $result;

Мой пример. Из проблем заметил, что не воспринимает запятую, только десятичную точку при вводе аргументов типа float.

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

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

if ((empty($_GET['x1']) and _GET['x1'] != 0) || (empty($_GET['x2']) and _GET['x2'] != 0)) {
    return 'Не переданы аргументы';
}
if (!is_numeric($_GET['x1']) || !is_numeric($_GET['x2'])) {
    return 'Введите корректные аргументы';
}

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

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

switch ($_GET['operation']) {
    case '+':
        $result = $x1 + $x2;
        break;
    case '-':
        $result = $x1 - $x2;
        break;
    case '*':
        $result = $x1 * $x2;
        break;
    case '/':
        if ($x2 == 0) {
            $result = "<a href='https://ru.wikipedia.org/wiki/%D0%94%D0%B5%D0%BB%D0%B5%D0%BD%D0%B8%D0%B5_%D0%BD%D0%B0_%D0%BD%D0%BE%D0%BB%D1%8C'>Деление на ноль недопустимо</a>";
            break;
        }
        $result = $x1 / $x2;
        break;
    default:
        return 'Операция не поддерживается';
}

return $expression . $result;

Самостоятельно получилось выполнить где-то только 50%, пришлось прибегнуть к помощи комментов.
Урок очень понравился. Из усовершенствований хотелось бы чтоб дробные числа вводились через "," как в стандартном калькуляторе.

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

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

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

if (!is_numeric($_GET['x1']) || !is_numeric($_GET['x2'])) {
    return 'Введите числа! (для ввода дробной части используйте знак ".")';
}

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

$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 (!isset($_GET['x1']) || !isset($_GET['x2'])) {
    return 'Аргументы не переданы';
}

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

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

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

Сам смог добавить умножение и деление. При возведении в степень в результате появлялся пустой квадратик. Поэтому не стал добавлять. Урок понравился очень. Форматирую с Ctrl+Alt+L. Я просто делаю copy-paste своего кода в комменты. Может из-за этого нарушается форматирование?

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

if (empty($_GET['operation'])) {
    return 'Пустой оператор';
}

if (!empty($_GET['x1']) || !empty($_GET['x2'])) {

$x1 = (int)$_GET['x1'];
$x2 = (int)$_GET['x2'];

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

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

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

return $expression . $result;

}

return 'Пустые все или один из аргументов';

Предыдущие задачи решены, есть в комментах, теперь новые:

https://codingzone.io/problems/9/solution/825
https://codingzone.io/problems/10/solution/842

Если передать 0, то пишет, что не переданы аргументы. Это происходит потому, что функцией empty cледующие значения воспринимаются как пустые:
"" (пустая строка), 0 (целое число), 0.0 (число с плавающей точкой), "0" (строка), NULL, FALSE, array() (пустой массив)(цитата из официальной документации). Решил проблему конструкцией !isset, предварительно почитав комменты :) Если поделить на ноль, то "результат вычислений:
10 / 0 = INF". Исправил. Если добавить строку, калькулятор не работает корректно. Исправил.

<?php

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

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

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

if ($_GET['x2'] == 0 && $_GET['operation'] == '/') {
    return 'На ноль делить нельзя!';
}

if (!is_numeric($_GET['x1']) || !is_numeric($_GET['x2'])) {
    return 'Можно вводить только числа!';
}

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

$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 = $x1 / $x2;
        break;
    default:
        return 'Операция не поддерживается';
}

return $expression . $result;

https://codingzone.io/problems/9/solution/980
https://codingzone.io/problems/10/solution/981

При передаче 0 функция empty воспринимает его как пустое значение, поэтому (уже после того как прочитал комменты и дополнительный урок) добавил isset.
При делении на ноль, то выводится 3 / 0 = INF, добавил для этого условие if
При вводе строки калькулятор работает не совсем корректно, добавил для этого условие if с функцией is_numeric
В качестве улучшения можно добавить распознавание запятой в десятичных дробях или добавление кнопок вместо самостоятельного ввода, больше никаких улучшений не надумал.

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

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

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

if (!is_numeric($_GET['x1']) || !is_numeric($_GET['x2'])) {
    return 'Необходимо ввести число';
}

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

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

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

return $expression . $result;

Доп. задание 1 -https://codingzone.io/ru/problems/9/solution/1624 или так https://codingzone.io/ru/problems/9/solution/1627 (не знаю, какой вариант лучше)
Доп. задание 2 - https://codingzone.io/ru/problems/10/solution/1630

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

1 задача
https://codingzone.io/problems/9/solution/1745
2 задача
https://codingzone.io/problems/10/solution/1747

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

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

if ($_GET['x1'] === '' || $_GET['x2'] === '') {
    return 'Не переданы аргументы';
}

if (!is_numeric($_GET['x1']) || !is_numeric($_GET['x2'])) {
    return 'Передано не число';
}

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

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

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

return $expression . $result;

Что можно улучшить, не знаю. Может сделать вывод результата на той же страничке, где вводим числа? Чтоб при нажатии кнопки "посчитать" справа выводился ответ?

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

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

if ($x1 === null || $x2 === null) {
    return 'Ключ аргумента отсутствует';
}
if (!is_numeric($x1) || !is_numeric($x2)) {
    return 'Введите число';
}

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

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

return $expression . $result;

Привет, решение задачи

1. https://codingzone.io/ru/problems/9

Привет, вот что получилось.

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

if (empty($_GET['operation'])) {
    return 'Не передана операция';
}
if ($_GET['x1'] == '' || $_GET['x2'] == '') {
    return 'Не переданы аргументы';
}
$x1 = $_GET['x1'];
$x2 = $_GET['x2'];
$expression = $x1 . ' ' . $_GET['operation'] . ' ' . $x2 . ' = ';
if (is_numeric($x2) && is_numeric($x1)) {
    switch ($_GET['operation']) {
        case '+':
            $result = $x1 + $x2;
            break;
        case '-':
            $result = $x1 - $x2;
            break;
        case '*':
            $result = $x1 * $x2;
            break;
        case '/':
            if ($x2 != 0) {
                $result = $x1 / $x2;
            } else {
                return 'На 0 делить нельзя';
            }
            break;
        default:
            return 'Операция не поддерживается';
    }
} else {
    return 'Введите числа';
}
return $expression . $result;

calc.php

<?php

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

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

if ((empty($_GET['x1']) && $_GET['x1'] != '0') || (empty($_GET['x2']) && $_GET['x2'] != '0')) {
    return 'Не переданы аргументы';
}

if (!is_numeric($_GET['x1']) || !is_numeric($_GET['x2'])) {
    return 'Допускаются только числа';
}

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

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

switch ($_GET['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;

А как теперь высылать ссылки на решения с задач? Или уже этого не нужно? Я смотрю вы уже codingzone под свой домен перенесли...

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

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

if (empty (isset($_GET['x1'])) || empty (isset($_GET['x2'] ))) {
    return 'Не переданы аргументы';
}
if (!is_numeric($_GET['x1']) || !is_numeric($_GET['x2']) ){
    return 'Бывает';
}

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

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

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

return $expression . $result;

<?php

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

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

// isset — Определяет, была ли установлена переменная значением отличным от NULL
if (!isset($_GET['x1']) || !isset($_GET['x2'])) {
    return 'Аргументы не переданы';
}

// is_numeric — Проверяет, является ли переменная числом или строкой, содержащей число
if (!is_numeric($_GET['x1']) || !is_numeric($_GET['x2'])) {
    return 'Только числа';
}

if (isset($_GET['x1']) && isset($_GET['x2'])) {
    $x1 = ($_GET['x1']);
    $x2 = ($_GET['x2']);
}

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

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

return $expression . $result;

<?php

if (empty($_GET)){
    return 'Ничего не передано';
}
if (empty($_GET['operation'])) {
    return 'Операция не передана';
}
if (($_GET['x1'] === '') || ($_GET['x2'] === '')) {
    return 'Аргументы не переданы';
}

$x1 = $_GET['x1'];
$x2 = $_GET['x2'];
$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 = $x1 / $x2;
        break;
}

return $expression. $result;

У меня почему то

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

При пустых формах всё равно возвращает 1.

<?php
    $x1 = $_GET['x1'];
    $x2 = $_GET['x2'];
    $operation = $_GET['operation'];
    $result = '';
    if (is_numeric($x1) && is_numeric($x2)) {
        switch ($operation) {
            case '+':
                $result = $x1 + $x2;
                break;
            case '-':
                $result = $x1 - $x2;
                break;
            case '/':
                if ($x2 == 0) {
                    $result = 'Делить на 0 нельзя!';
                    return $result;
                } else {
                    $result = $x1 / $x2;
                    break;
                }
            case '*':
                $result = $x1 * $x2;
                break;
            default:
                $result = 'Введена неизвестная операция';
                return $result;
        }
    } else {
        $result = 'Введены не числовые переменные';
        return $result;
    }
    return $x1 . ' ' . $operation . ' ' . $x2 . ' = ' . $result;

<?php
$line = '7 0 39 0 282 2 4 0 45';
    $array = explode(' ', $line);

    $countZero = 0;
    $resultArray = [];
    foreach ($array as $item) {
        if ($item == 0) {
            $countZero++;
            continue;
        }
        $resultArray[] = $item;
    }
    while ($countZero != 0) {
        $resultArray[] = 0;
        $countZero--;
    }
    echo '<pre>';
    var_dump($resultArray);
    echo '</pre>';

Код файла Calc:

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

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

if ($_GET['operation'] == '/' && $_GET['x2'] == '0') {
    return 'На ноль делить нельзя';
}

if (!isset($_GET['x1']) || !isset($_GET['x2']) || $_GET['x1'] == "" || $_GET['x2'] == "") {
    return 'Не переданы аргументы';
}

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

$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 = $x1 * $x2;
        break;
    default:
        return 'Операция не поддерживается';
}

return $expression . $result;

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

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

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

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

if (!is_numeric($_GET['x1']) || !is_numeric($_GET['x2'])) {
    return 'Можно вводить только числовые значения';
}

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

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

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

return $expression . $result;

<?php

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

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

if (empty($_GET['x1']) && $_GET['x1']!=0) {
    return 'Не переданы первый аргумент';
}

if (empty($_GET['x2']) && $_GET['x2']!=0) {
    return 'Не переданы второй аргумент';
}

if ($_GET['operation']=="/" && $_GET['x2']==0) {
    return 'Делить на ноль нельзя';
}

if (!is_numeric($_GET['x1']) || !is_numeric($_GET['x2'])) {
    return 'Некорретные данные';
}

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

$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 = $x1 / $x2;
        break;
    default:
        return 'Операция не поддерживается';
}

return $expression . $result;

<?php

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

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

if (is_null($_GET['x1']) || is_null($_GET['x2'])) {
    return 'Не переданы аргументы';
}

if (!is_numeric($_GET['x1']) || !is_numeric($_GET['x2'])) {
    return 'Не является числом';
}

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

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

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

    return $expression . $result;

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

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

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

// проверка на число
if(!is_numeric($x1) && !is_numeric($x2)) {
    return 'Введите число';
}

// если передали ноль или пустоту
if (empty($x1) || empty($x2)) {
    // при сложении или вычитании в первый аргумент
    if ($x1 === '0' && $x2 !== '0') {
        return $x2;
    }

    // при сложении или вычитании во второй аргумент
    if ($x1 !== '0' && $x2 === '0') {
        return $x1;
    }

    // если ноль поделили на число
    if ($x1 === '0' && $_GET['operation'] === '/') {
        return 0;
    }

    // если число поделили на ноль
    if ($x2 === '0' && $_GET['operation'] === '/') {
        return 'На ноль делить нельзя';
    }

    // если передали ноль и знак умножения
    if ($x1 === '0' || $x2 === '0' && $_GET['operation'] === '*') {
        return 0;
    }

    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 = $x1 / $x2;
        break;
    default:
        return 'Операция не поддерживается';
}

return $expression . $result;

if ((isset($_GET['x1']) && $_GET['x1'] == '') || (isset($_GET['x2']) && $_GET['x2'] == '')) {
    return 'Не переданы аргументы';
}

/*$x1 = filter_input(INPUT_GET, $_GET['x1'], FILTER_VALIDATE_INT);вот тут не получилось
$x2 = filter_input(INPUT_GET, $_GET['x2'], FILTER_VALIDATE_INT);совладать с этой функцией
var_dump( $x1);
if ($x1 === null || $x2 === null) {
    return 'Аргументы 1 или 2 не переданы';
}*/

$x1 = $_GET['x1'];// можно привести к (int), но тогда и строка будет приравниваться к нулю
$x2 = $_GET['x2'];// а уже не то что нужно???
$operation = $_GET['operation'];

$expression = $x1 . ' ' . $_GET['operation'] . ' ' . $x2 . ' = ';
if (!is_numeric($x1) || !is_numeric($x2)) {
    return 'Необходимо ввести число';
}

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:
        return 'Операция не поддерживается';
}
return $expression . $result;