Как обойтись в данной случае без goto?

Question

[Владимир Голубь]

Как сделать без оператора goto так, чтобы после вывода скриптом ошибка у меня полностью прекращалась работа всего. То есть переместиться сразу на exit; после echo 1;

if($_POST['operation'] == "confirmOrderPrint"){
    $items = $_POST['items'];
    foreach ($items as $value){

        $query = "UPDATE orders_print SET id_status = '2' WHERE id_order_print = '" . $value . "'";

        $result = db_connection($query);

        if($result != 'TRUE') {
            echo 2;
        }
    }
    echo 1;
    exit;
}

Comments

[Владимир Голубь]

Решил проблему так

if($_POST['operation'] == "confirmOrderPrint"){
    $items = $_POST['items'];
    $status = '';
    foreach ($items as $value){

        $query = "UPDATE orders_print SET id_status = '2' WHERE id_order_print = '" . $value . "'";

        $result = db_connection($query);

        if($result != 'TRUE') {
            $status = 'fail';
            break;
        }
    }
    if($status == 'fail'){
        echo 2;
    } else {
        echo 1;
    }
    exit;
}

Answer 1

[Андрей Мохов]

если у вас задача - выйти из цикла, то можно использовать break:

foreach ($items as $value) {
    if (условие) break;
}

Answer 2

[Алексей Волегов]

Не увидел тут goto, но вам лучше вообще забыть про этот оператор.

А кто мешает вам вызвать exit в любом месте?

php.net/manual/ru/control-structures.break.php

Comments

[Владимир Голубь]

Ну вот к примеру если я напишу тут exit
if($result != 'TRUE') {
echo 2;
exit;
}
То я выйду только из него, а как в случае ошибки выйти из foreach

[Алексей Яхненко]

Владимир Голубь: лучше почитай немного мануал.

[Алексей Волегов]

exit или die тебе приложение полностью убьет. Выйти из цикла можно через break

php.net/manual/ru/control-structures.break.php

Answer 3

[sbh]

Либо допишите ELSE и ваш вывод

Answer 4

[ustasby]

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

Comments

[Владимир Голубь]

напиши тогда более лучшее решение, все болтать умеют.

[ustasby]

Владимир Голубь: то есть прочитать книжку - это плохой совет?

[Владимир Голубь]

хороший, только в книге не взегда описаны все ситуации. И проходиться применять простые решения.

[ustasby]

Владимир Голубь: для начала в книге написано что такое true и false $result !== true, про элементарную безопасность (не записывать не отфильтрованные значения ) и приведение типов foreach ((array)$items as $value), да и вообще почитать все, знаний то нет.