Где ошибка в switch-case?

Question

[Денис Давыденко]

Добрый вечер.
Подскажите где я ошибся пожалуйста. Изучаю switch-case.
Код:

$status = "";
$statusok = "Выполнен";
$statusbad = "Не выполнен";
switch ($mcg['klost'] && $mcg['slost']):
    case ($mcg['klost'] == 0 && $mcg['slost'] == 0): 
        $status = $statusok; break;
    default: 
        $status = $statusbad; break;
endswitch;
$smarty->assign('status', $status);

Как в моем понимании работает этот код.
Назначаем переменную $status - пустой строкой.
Объявляем переменные $statusok - "Выполнен" и $statusbad - "Не выполнен".
Выбираем для перебора значения из ассоциативного массива (объявлен ранее) $mcg столбец klost и $mcg столбец slost.
Проверяем на условие, что $mcg столбец klost равен 0 И $mcg столбец slost равен 0.
Присваем значение перемнной $status - $statusok.
В остальных случаях (т.е. если одно из значений не равно 0) переменной $status присваевается $statusbad.
Передаю переменную $status в Smarty.

Вывод в смарти осуществляется:
{$status}

Однако такой код присваивает ВСЕМ записям значение "Не выполнен", независимо от числа в $mcg['klost'] и $mcg['slost'].
Подозреваю, что неправильно написал условия для перебора.
Направьте на путь истинный пожалуйста.

Comments

[ThunderCat]

хоп, $statusok, не шей мне срок...

Answer 1

[Артём]

Интерпретатор подставит в switch результат выполнения $mcg['klost'] && $mcg['slost'], а он будет true или false. В итоге в рамках твоего примера это бессмысленно, если тебе нужно проверить две переменные, то используй if.

Код, аналогичный твоему, для понимания:

$status = '';
$statusok = 'Выполнен';
$statusbad = 'Не выполнен';
$something = $mcg['klost'] && $mcg['slost']; // в $something будет true или false, третьего не дано
$somethingElse = $mcg['klost'] == 0 && $mcg['slost'] == 0; // здесь то же самое, true или false

switch ($something):
    case ($somethingElse):
        $status = $statusok;
        break;
    default:
        $status = $statusbad;
        break;
endswitch;

$smarty->assign('status', $status);

Кстати, если у тебя в klost будет 1, а в slost будет 0, то в $something и $somethingElse будет false и в $status упадёт 'Выполнен'.

Comments

[Денис Давыденко]

Сначала по вашему примеру.
Мы объявляем переменную $something, которая содержит значения klost И slost.
Далее перебираем эту переменную.
Если в переменной $something значения klost = 0 И slost = 0, то возвращается true и выполняется кейс.
Если значения другие, то возвращается false и выполняется дефаулт.
Я всё правильно понял?

Переписал код под if, но ситуация такая же.

$status = "";
$statusok = "Выполнен";
$statusbad = "Не выполнен";
//switch ($mcg['klost'] && $mcg['slost']):
//    case ($mcg['klost'] == 0 && $mcg['slost'] == 0): 
//        $status = $statusok; break;
//    default: 
//        $status = $statusbad; break;
//endswitch;
if ($mcg['klost'] == 0 && $mcg['slost'] == 0): 
        $status = $statusok;
   else: 
        $status = $statusbad;
endif;
$smarty->assign('status', $status);

[Денис Давыденко]

С вашим кодом ситуация такая же. Везде "не выполнен".

[Артём]

Мы объявляем переменную $something, которая содержит значения klost И slost.

Нет, в $something упадёт результат вычисления
$mcg['klost'] && $mcg['slost']
Он будет true или false.

В твоём примере (то, что ты переделал на if) ты проверяешь, что klost равен нулю и slost равен нулю, и в таком случае заказ считается выполненным. Во всех остальных случаях он не выполнен.

Мой код эквивалентен твоему. Написанный мною код так же не является верным, он просто дополнен парой переменных, чтобы ты лучше понял, что происходит в твоём коде.

[Денис Давыденко]

Артём, Так и должно быть.
Т.е. если klost и slost = 0, то Выполнен, во всех остальных случаях он не выполнен.

Про switch понял вроде.

[Денис Давыденко]

Проблема оказалась в выводе Smarty. Он всем записям присваивал первое значение.