Конструкция if или switch?

Question

[Илья]

Вопрос простой и тупой. Два варианта одного и того же кода. Как бы вы написали и почему? Названия методов и переменных значения не имеют.

if (
	$this->inComplete($oldStatus, $newStatus) ||
	$this->outComplete($oldStatus, $newStatus)
) {
	$this->action();
	$this->save();
}

switch (true) {
	case ($this->inComplete($oldStatus, $newStatus)):
	case ($this->outComplete($oldStatus, $newStatus)):
		$this->action();
		$this->save();
}

Comments

[Alexey Hog]

Обычно приходится не только писать код, но и очень много читать его.
На мой взгляд, в данном случае, c двумя простыми условиями, конструкция с if легче читается чем switch.

Answer 1

[Saboteur]

в данном конкретном случае switch явно не нужен.
switch Нужен когда у вас есть переменная, которую вы хотите сравнить с разными значениями и в зависимости от этого делать разные действия. А вы в качестве условия switch вообще true пишете, и case без завершения - это с точки зрения читабельности, очень неявный OR.

Answer 2

[FanatPHP]

Второй вариант это хорошая попытка, но недостаточно полно реализованная. Писать надо хотя бы так

(function() { 
    foreach([true] as $condition) {
        $condition = ($condition) ? true : false; 
        while ($condition) {
            switch (true) {
                case ($this->inComplete(($oldStatus), ($newStatus))):
                case ($this->outComplete(($oldStatus), ($newStatus))):
                    ($this->action());
                    ($this->save());
            }
            break;
        }
    }
})();

В общем, перед тем как задавать вопрос на тостере, надо сесть и хорошенько сначала самому подумать, каких бессмысленных операторов можно добавить в код, кроме switch.

Comments

[Иван Родичев]

Дед, опять таблетки забыл выпить?

[FanatPHP]

А тебе, внучек, чтобы понять сарказм, надо обязательно чтобы закадровый смех был?

[Кирилл Несмеянов]

Попытался придумать "до чего бы доебаться" (с), но не смог. Пожалуй, это лучший вариант.

Разве что предлагаю:
1) В foreach добавить [true, 1, !false, 'yes, master', (0 | 1 << 3 ^ 2) / 10].
2) А break заменить на break 2, а то не слишком надёжно получается.

Что думаешь?

Answer 3

[Pavel Omelchenko]

If в котором бы вызывал один метод обертку возвращающий bool значение.