10

Проблема

Есть задача при использовании switch у разных case выполнять еще и код из default (или например final реализовать), что бы не дублировать одинаковый код в разных кейсах.

switch ("b") {
    case 'a':
    case 'z':
    case 'q':
        echo "a\n"; // not need run default action
        break;
    case 'b':
        echo "b\n"; // need run default case without case 'c'
        break;
    case 'c':
        echo "c\n";
    default:
        echo "default"; // need run for "b" & "c" cases
}

Выводит:

b

А хотелось бы:

b
default

Если убрать break у варианта b, то будет выполнен лишний код

b
c
default

Есть решение через goto, что является плохим стилем кодирования:

switch ("b") {
    case 'a':
        echo "a\n";
        break;
    case 'b':
        echo "b\n";
        goto caseFinal;
        break;
    case 'c':
        echo "c\n";
        goto caseFinal;
        break;
    default:
        caseFinal:
        echo "default";
}

Полная задача

Есть массив полей разных сущностей и нужно разные действия у разных полей сделать.

switch ($fieldName) {
    case 'id': // не нужно default
        unset($properties['actions']['create']);
        unset($properties['actions']['edit']);
        break;
    case 'content':
        // одинаковый код нужно вынести в defaut
        unset($properties['actions']['dashboard']);
        // ...
        break;
    case 'isDeleted':
        // одинаковый код нужно вынести в defaut
        unset($properties['actions']['dashboard']);
        // ...
        break;
    default:
        // код нужне для всех остальных полей + кейсы выше
        unset($properties['actions']['dashboard']);
}

Вопрос

Как это сделать без использования goto? Может есть вариант с другим подходом в архитектуре.

P.S. Если goto здесь уместен (лично я не боюсь динозавров), то почему это плохой стиль кодирования?

15
  • тело default вынесите из case и будет вам счастье. goto не плохой, просто им мало кто умеет пользоваться, поэтому лучше забыть про него.
    – BOPOH
    23 дек 2015 в 11:32
  • 1
    вы может не ту задачу решаете? Делаете ваши а, бэ и прочее классами и там уже реализуете такое поведение. Ну может и не классами их делать, а просто вынести обработку этих значений в отдельные классы. Если у вас будет куча таких значений то как-то не совсем удобно с этим работать будет и без всяких там goto
    – BOPOH
    23 дек 2015 в 11:39
  • 1
    сделайте свитч в свитче. но сама архитектура уже сильно напрягает.
    – etki
    23 дек 2015 в 11:46
  • 2
    можно ввести переменную run_add_code= false, и в нужных случаях задавать ей true. Тогда будет достаточно в одном месте поправить, чтоб этот код выполнился
    – splash58
    23 дек 2015 в 12:07
  • 1
    Goto ненавидят только полные нубы. Причём ненавидят демонстративно, чтобы казаться умнее. Так они считают. Но мало кто знает, что и switch тоже считается плохим стилем. Пользуйтесь всеми инструментами языка, исходя из здравого смысла, а не потому, что какой-то крикун что-то там кричит о каком-то там стиле.
    – Sergey
    24 дек 2015 в 6:00

5 ответов 5

9

Зачем усложнять? Можно вынести повторяющийся код в функцию/метод класса и вызывать их из каждого кейса. Кода получится ровно столько же, зато goto мы не используем и код остаётся читаемый и легко поддерживаемый:

function someAction() {
    //some code
}

switch ($fieldName) {
    case 'id':
        unset($properties['actions']['create']);
        unset($properties['actions']['edit']);
        break;
    case 'content':
        someAction();
        unset($properties['actions']['dashboard']);
        // ...
        break;
    case 'isDeleted':
        someAction();
        unset($properties['actions']['dashboard']);
        // ...
        break;
    default:
        someAction();
        unset($properties['actions']['dashboard']);
}
6
  • Да, но хотелось всю обработку собрать в одном методе и не использовать декомпозицию. Может потребоваться много параметров передавать (по ссылкам в том числе), или сразу доп. классы реализовывать и кол-во кода увеличится соответственно
    – korytoff
    23 дек 2015 в 12:52
  • Кстати я именно так чаще всего поступаю, и со временем очень много методов образуется маленьких
    – korytoff
    23 дек 2015 в 17:42
  • ООП и стайл гайды ООПшных языков обычно и предполагают небольшие легко читабельные методы. Не вижу тут проблемы
    – n4nn31355
    23 дек 2015 в 17:49
  • Проблема в том, что придётся тогда для этого отдельный класс делать для обработки полей, а много классов усложняет чтение программы, согласитесь, проще линейный листинг в 50 строк прочесть чем 10 с кучей вхождений
    – korytoff
    23 дек 2015 в 17:55
  • Класс один раз написал в отдельном файле, нормально методы назвал, структурировал, отладил и забыл про реализацию и эти 50 строк в процедурном стиле не маячат постоянно перед глазами.
    – n4nn31355
    23 дек 2015 в 18:05
1

Если только одно действие не должно быть продолжено default, почему бы не попробовать вынести default действие за switch и огородить if ($fieldName != 'id') { unset($properties['actions']['dashboard']); }

5
  • да, но если таких кейсов будет много, то придется много "городить" и + дублирование названия полей
    – korytoff
    23 дек 2015 в 12:04
  • 2
    Всё, что делает default, завернуть в функцию. В нужных кейсах добавляется всего-то один вызов функции.
    – Sergey
    23 дек 2015 в 12:24
  • @banme да, вариант, но не хотелось быть сильно декомпозицию и так не сложного метода делать. Хочется всю обработку полей в одном методе сделать
    – korytoff
    23 дек 2015 в 12:26
  • goto на самом деле не всегда так плох, как говорят. В switch можно сделать goto default?
    – Sergey
    23 дек 2015 в 12:31
  • @Sergey нет, в switch default зарезервированное слово. Можно только свою метку
    – korytoff
    23 дек 2015 в 17:40
0

Можно оформить условия как отдельные функции (методы).

Ключевая функция. Получает действие на входе и выполняет соответствующую функцию:

function make($action) {
    if ($action()) 
        $default();
    return;
}

Собственно действия. Возвращают true или false как маркеры необходимости выполнять действие по умолчанию.

function a() {
    echo "a\n";
    return false; // дефолтное действие выполнять не нужно
}

function b() {
    echo "b\n";
    return true; // дефолтное действие выполнить нужно
}

Функция (метод) для действия по умолчанию:

function default() {
    echo 'default';
    return;
}

Используем:

make('a');
4
  • вариант конечно интересный, но во-первых кода стало больше, во-вторых при ООП походе кода в таком решении кода будет еще больше и в третьих усложняет чтение кода
    – korytoff
    23 дек 2015 в 12:29
  • Не думаю, что код станет менее читаемым. Здесь каждая функция выполняет ОДНУ И ТОЛЬКО ОДНУ задачу. В методологии SOLID речь идет о классах, но сути это не меняет. 23 дек 2015 в 12:40
  • 1
    Здесь каждая функция выполняет две задачи: эхнуть что-то свое и сообщить вызывающему о том, нужно ли вызывать еще что-то. Прозрачность такого кода стремится к нулю.
    – etki
    23 дек 2015 в 12:42
  • Хм. С тем, что выполняются две задачи - согласен. 23 дек 2015 в 12:56
-1

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

Вывод

В данном случае использование goto допустимо, пока не будет добавлена подобная конструкция в синтаксис switch и если соблюдаются условия:

  1. Ты уже не Junior
  2. Переходить можно только вперёд
  3. Заходить в блоки и выходить категорически нельзя

Любой while и вложенные if так же опасны как goto. Для выхода из множественных циклов есть break N.

Плюсы которые я вижу

  1. Краткость и простота кода при соблюдении вышеописанных условиях
  2. Меньше занимаемой памяти и скорее всего быстрота выполнения
  3. Лучше "плохой тон", чем много избыточного кода, который сложнее поддерживать

Для себя я решил использовать только в группах исключающих условий:

switch ("b") {
    case 'a':
        echo "a\n";
        break;
    case 'b':
        echo "b\n";
        goto caseFinal;
        break;
    case 'c':
        echo "c\n";
        goto caseFinal;
        break;
    default:
        caseFinal:
        echo "default";
}

Конечно при усложнении логики и увеличении строк кода, лучше всего воспользоваться ответом n4nn31355 и выносить в классы и отдельные методы, но внутри у них все равно наверно будет что-то похожее ;)

Интересные статьи на эту тему:

  1. Запретный плод GOTO сладок
  2. GOTO or not GOTO вот в чём вопрос
4
  • Что за бред? Вам дали нормальный ответ, а вы накидали какой-то треш и еще пометили как правильный... 24 авг 2017 в 6:58
  • @Artur возможно вы просто не в состоянии воспринять этот код, равно как и писать аккуратные комментарии
    – korytoff
    25 авг 2017 в 15:14
  • @Artur и хотелось бы аргументы, а не просто "треш"
    – korytoff
    25 авг 2017 в 18:55
  • По комментарию, скажите что подкорректировать чтобы он был более аккуратным. По коду - действительно не могу воспринять. Мы тут на ревью избавляемся от лишнего if/else в пользу состояний, стратегий и полиморфизма для того чтобы сделать код более очевидным и читаемым, а вы предлагаете вернуться к методам, которые и 15 лет назад считались гавнокодом. Хорошо хоть, что в конце написали примечание про усложнение и увеличение строк кода. 28 авг 2017 в 12:54
-2

Усугубим goto

...
if (b == 'a') {
    echo "a\n";
    goto done;
} else if (b == 'b') {
    echo "b\n";
    goto default;
} else if (b == 'c') {
    echo "c\n";
    goto default;
}
default:
    echo "default";
done:
...

И кто скажет что это отвратительно - тот просто ничего не сечёт в программировании:)

2
  • default в goto не прокатит, это зарезервированное слово
    – korytoff
    23 дек 2015 в 12:42
  • @korytoff Используйте что прокатит. Что за вас всю программу и без ошибок составить? Это-то же можно додумать, правильно?
    – Sergey
    23 дек 2015 в 12:44

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.