Вопрос Есть небольшой калькулятор на PHP, помогите немного подкорректировать его.

[Robert]

Написал калькулятор, но есть проблема. Когда 1 из двух полей для ввода текста пустое, и выбранная операция, то он просто печатает на экран то число.
Вот код

<?php
$error = "";
if(isset($_POST["calculation"]))
{
if(empty($_POST["operation"])) $error = "Выберите операцию и заполните поля";
else if(!isset($_POST["operator_a"]) && !isset($_POST["operator_b"])) $error = "Заполните поля и выберите операцию заново";
else if(isset($_POST["operator_a"]) && isset($_POST["operator_b"])){
switch($_POST["operation"]){
case "+"; $summa = $_POST["operator_a"] + $_POST["operator_b"]; break;
case "-"; $summa = $_POST["operator_a"] - $_POST["operator_b"]; break;
case "/"; if($_POST["operator_a"] != 0) $summa = $_POST["operator_a"] / $_POST["operator_b"]; else $error = "Делить на 0 нельзя"; break;
case "*"; $summa = $_POST["operator_a"] * $_POST["operator_b"]; break;
default: $error = "Операция не найдена"; break;
}
echo $summa;
}
}
echo $error;
?>

Не выбрано
Сложение
Вычитание
Деление
Умножение

 

[Dashenok]

почему не спросишь у тех, кто этот код изначально сочинял?

 

[AlSlale]

else if (!isset($_POST["operator_a"]) && !isset($_POST["operator_b"])) а надо else if (!isset($_POST["operator_a"]) || !isset($_POST["operator_b"])) $error = "Заполните ВСЕ поля и выберите операцию заново";