Ошибка в калькуляторе PHP?

Question

[makoto_msk]

Всем привет, учусь на первом курсе на программиста и пытаюсь учить PHP.
Написала на codewars функцию простого калькулятора(заранее прошу прощения за говнокод, но пока что умею только так), но unit_test выдает ошибку, хотя в sandbox все тесты вручную проходит, автотест codewars ругается. Почему-то перед unknown value появляются тесты, хотя в sandbox их нет.

Error

3-1153unknown value
Failed asserting that true matches expected 'unknown value'.

Code

function calculator($a, $b, $sign) {
		if(!is_numeric($b)) {
			return print_r('unknown value');
		}
		if(!is_numeric($a)) {
			return print_r('unknown value');
		}

		if($sign === '+') {
			$result = $a + $b;
			return print_r($result);
		} elseif ($sign === '-') {
			$result = $a - $b;
			return print_r($result);
		} elseif ($sign === '*') {
			$result = $a * $b;
			return print_r($result);
		} elseif ($sign === '/') {
			$result = $a / $b;
			return print_r($result);
		} elseif ($sign !== "*" && $sign !== '/' && $sign !== '+' && $sign !== '-') {
			return print_r("unknown value");
		}
	}

Comments

[FanatPHP]

Подсказка: для оформления кода есть специальная кнопочка, </>
Форматированный код можно читать не ломая глаза.

[Дмитрий]

Ну как минимум ещё надо проверку деления на 0 сделать, ну и можно сразу кастить к инту аргументы

[makoto_msk]

Дмитрий, Спасибо, доделаю, в целом ориентировалась на заданные параметры в unit test, поэтому он неполноценный.

[Сергей]

function calculator($a, $b, $sign) {
	if(!is_numeric($b) || !is_numeric($a)) {
		return 'unknown value';
	}

	switch ($sign) {
		case '+': $result = $a + $b; break;
		case '-': $result = $a - $b; break;
		case '/': $result = ($b > 0 ? $a / $b : 'unknown value'); break;
		case '*': $result = $a * $b; break;
		default: $result = 'unknown value';
	}
    return $result;
}

echo calculator(2, 0, '/');

Answer 1

[Александр Аксентьев]

Что за return print_r

Конечно ломается, вы же не возвращаете нифига нормально.

Comments

[makoto_msk]

Вывести в консоль строку "unknow value". Или просто выводить её как return?

[makoto_msk]

Спасибо, как только убрала print_r из вывода, всё заработало. Не могли бы тогда объяснить, как работает механизм? Если я хочу в консоль или в браузер вывести return 'какой либо текст", print_r не работает, или нужен echo?

[Денис Дерепко]

makoto_msk, return это возврат значения, дальше с ним можете делать что угодно, например вывести с помощью echo

[makoto_msk]

Денис Дерепко, Спасибо)

[Евгений]

makoto_msk, return возвращает то что вы указываете, когда вызываете функцию)
Чуть подправлю ваш код, а вы поиграйтесь с аргументами функции, тогда поймете как оно работает))

function calculator($a, $b, $sign) {

if(!is_numeric($b)) {
return 'второй аргумент некоректен';
}
if(!is_numeric($a)) {
return 'первый аргумент какая-то туфта';
}

if($sign === '+') {
$result = $a + $b;
return $result;
} elseif ($sign === '-') {
$result = $a - $b;
return $result;
} elseif ($sign === '*') {
$result = $a * $b;
return $result;
} elseif ($sign === '/') {
$result = $a / $b;
return $result;
} elseif ($sign !== "*" && $sign !== '/' && $sign !== '+' && $sign !== '-') {
return 'возводи в степень в уме, я отказываюсь это делать';
}
}

$a = calculator('дерево', 'кирпич', '4 раза');
echo $a;

[Just OK]

Евгений, print_r лишний. Он вообще используется для отладки или распечатки массивов, классов и т.д.
Там достаточно просто вернуть значение с помощь return.
return $result
Этого будет вполне достаточно.

[Евгений]

Олег Калимбах, не углядел, копипастом, то что правил - норм, остальное от автора) спасиб)

[Евгений]

Олег Калимбах, исправил, там вобще по сути былоб лучше через switch - case написать) просто я телефона не стал заморачиваться правкой всего кода)