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

Question

[KronosHD]

$pay = $_REQUEST['AMOUNT'];
$id = $row['id'];
exit($id+$pay);

Answer 1

[Александр Сухерев]

exit принимает только string, так что никаких плюсов.

Comments

[KronosHD]

Точняк

[Алексей Рытиков]

Александр Сухерев: это не так! KronosHD: Где точняк? Официальную документацию посмотрели бы php.net/manual/ru/function.exit.php.

[Александр Сухерев]

Алексей Рытиков: да, Вы правы, я не то сказал. exit выводит только string, так будет правильнее.

Answer 2

[Андрей]

Exit принимает как строки так и числа.Если статус числовой он его просто не выводит! Читайте документацию.

Answer 3

[Stalker_RED]

$pay = $_REQUEST['AMOUNT'];
$id = $row['id'];
$foo = '' . (intval($_REQUEST['AMOUNT']) + intval($id));
exit($foo);

Comments

[KronosHD]

$votes = $row['votes']+$_REQUEST['AMOUNT'];
exit($votes);
Не работает

[Stalker_RED]

KronosHD: Попробуйте так:
echo $votes;
exit();

Если всё равно не работает, добавьте в начале error_reporting(E_ALL); и посмотрите какие ошибки сыпятся.

Answer 4

[Алексей Рытиков]

С какой стороны не так? При определенных условиях этот код может даже заработать, но он совсем не обрабатывает различных ситуаций и не понятно, что тут должно быть. $pay и $id –это числа? Если числа, то их сумма не должна быть больше 254.
Возможно, будет так, если его преобразовать во что-то вроде:

$pay = isset($_REQUEST['AMOUNT']) ? (int) $_REQUEST['AMOUNT'] : 0;
$id = is_array($row) && isset($row['id']) ? (int) $row['id'] : 0;
$status = $id + $pay;
if ($status > 254)
{
    $status = 254;
}
exit($status);