Что не так в коде?

Question

[apaicer]

Нужно написать функцию, в которую приходит 2 числа и оператор. и выполнить операцию между ними. Если оператор не +-*/ вывести ошибку; если одно из чисел не является числом то нулл.

$arr = ['+','-','*','/'];
$operator = $arr;

$operandOne = 6;
$operandTwo = 4;

function calculate($operandOne, $operandTwo, $operator) {
	if ($operandOne > 0){
		return null;
	}
	else if (!isset($operator)){
        return error;
	}
	else {
	$result = "$operandOne $operator[1] $operandTwo";
	}
}
 echo $result;

Я понимаю, что код не правильный, но в каком моменте? Видимо не могу уловить синтаксис.

Answer 1

[Sanovskiy]

Всё не так.

function calculate($operandOne, $operandTwo, $operator)
{

    if (!is_numeric($operandOne) || !is_numeric($operandTwo)) {
        return null;
    }

    switch ($operator) {
        case '+':
            return $operandOne + $operandTwo;
        case '-':
            return $operandOne - $operandTwo;
        case '/':
            if ($operandTwo <= 0) {
                return null;
            }
            return $operandOne / $operandTwo;
        case '*':
            return $operandOne * $operandTwo;
        default:
            echo 'Допускаются только операторы + - / *';
            return null;
    }
}

$operandOne = 6;
$operandTwo = 4;

echo calculate($operandOne, $operandTwo, '+');

Вот рабочий код.
Вы, похоже, не понимаете что такое функции.

Comments

[Anton Mashletov]

if ($operandTwo <= 0) — делить на отрицательное число можно

[Sanovskiy]

Можно, но не нужно.

[Anton Mashletov]

Sanovskiy, к чему скриншот? Ноль — не отрицательное число. А на отрицательное всё делится легко: 4 / -2 = -2. Нужно поменять <= 0 на == 0.

[Sanovskiy]

Anton Mashletov, а, вот вы о чем. Согласен. Это я что-то к вечеру затупил после часов ковыряния в подобном легаси :)

[Stalker_RED]

Sanovskiy, это не легаси. Это новьë, хоть и кривое.

[Sanovskiy]

Stalker_RED, Я имел в виду, что помимо тостера у тут есть легаси проект. Местами на 4 PHP. После шести часов разгребания оно подчиняет твой мозг.