Упрощение сложного метода

Question

Упрощение сложного метода

0

У меня есть метод, который в основном представляет собой длинный список if else с переменной глубиной (иногда довольно глубокий), поскольку он интерпретирует массив байтов и в зависимости от каждого байтового значения он делает что-то другое, например "чтение" большего от массива и снова проверяя их для некоторых значений, или он имеет прямое действие. Что-то вроде этого:

char current = array[current_index ++];
if(current == 1)
{
    char something = array[current_index ++];
    if(something == 2)
    {
         // open the fridge
    }
    else
    {
         char something_else = array[current_index ++];
         if(something_else == 3)
         {
             // make me a cake
         }
         else
         {
             // bring me a beer
         }
    }
}
else
if(current == check_this(another_variable))
{
     // so something else
}

Я пытаюсь упростить весь этот беспорядок. Итак, чтобы избавиться от первого уровня, if я создал массив указателей на функции, извлечение тел if в конкретные функции и в основном увидеть, имеет ли массив элемент для данного current и выполняет функцию (в качестве побочного эффекта это несколько снизило скорость выполнения и в значительной степени снизило читаемость).

Краткий пример

список команд:

1 - питание 2 - напиток 3 - домой 4 - теплый 5 - холодный 6 - мясо 7 - помидор

так что: 1 4 6 - доставит мне мясо на гриле (или, по крайней мере, по заявке), а 2 7 - даст мне томатный сок... и 3 - выход.

Edit: switch - case конструкция не всегда использовать в этом случае, так как иногда current сверяется с возвращаемым значением функции.

Итак, вопрос: я любопытно, какие другие методы вы использовали бы для атаки на эту проблему.

fritzone 03 фев. 2014, в 12:57

Источник

2

switch-case может быть?
devnull 03 фев. 2014, в 11:04
0

Этот вопрос кажется не по теме, потому что он принадлежит codereview.SE
Shoe 03 фев. 2014, в 11:05
1

array[current_index] ++ ? Или это должен быть array[current_index++] ?
Shahbaz 03 фев. 2014, в 11:06
0

@Shahbaz :) массив [current_index ++] определенно ... Спасибо, исправлено!
Ferenc Deak 03 фев. 2014, в 11:06
1

@devnull, пожалуйста, смотрите редактирование. Иногда случай переключения не применим, так как здесь также проверяются возвращаемые значения функции
Ferenc Deak 03 фев. 2014, в 11:09
2

Я не знаю проблемы, поэтому этот комментарий, вероятно, будет спамом, но рассматривали ли вы рекурсию?
IssamTP 03 фев. 2014, в 11:14
3

Трудно сказать, не зная реальной проблемы, которую вы решаете. Учитывая тот факт, что вы проверяете символы в массиве (или строке), вы, вероятно, хотите рекурсию или FSM.
Brave Sir Robin 03 фев. 2014, в 11:21
0

чистый код В принципе, вы можете сохранить ваш поток таким, какой он есть сейчас, просто убедитесь, что ваши значения глубины не превышают 2-го уровня, и в этом случае вы должны извлечь код для какого-либо другого метода.
A4L 03 фев. 2014, в 11:24
0

Ваша проблема, как уже говорилось, имеет некоторые сложности: действие, которое нужно предпринять, зависит не только от значения в массиве, но и от позиции, которую занимает значение. Это правильно? Есть ли семантика / отношения, связанные с каждой позицией массива? Другими словами, если я правильно понимаю: 1 4 6 не будет генерировать такой же ввод, как 1 6 или 4 1 6.
Picarus 03 фев. 2014, в 12:06
0

@Picarus Вы правы: 1 6 даст вам сырое мясо, а 4 1 6 не будет определено для этого контекста. Таким образом, всегда есть действие (1, 2 или 3), за которым следует один или несколько спецификаторов (4 5), а затем есть объект действия (6 7). Очевидно, этот пример очень упрощен :)
Ferenc Deak 03 фев. 2014, в 12:59

Показать ещё 8 комментариев

Теги:

c++

c

simplify

3 ответа

Ещё вопросы

Этот вопрос кажется не по теме, потому что он принадлежит codereview.SE
array[current_index] ++ ? Или это должен быть array[current_index++] ?
@Shahbaz :) массив [current_index ++] определенно ... Спасибо, исправлено!
@devnull, пожалуйста, смотрите редактирование. Иногда случай переключения не применим, так как здесь также проверяются возвращаемые значения функции
Я не знаю проблемы, поэтому этот комментарий, вероятно, будет спамом, но рассматривали ли вы рекурсию?
Трудно сказать, не зная реальной проблемы, которую вы решаете. Учитывая тот факт, что вы проверяете символы в массиве (или строке), вы, вероятно, хотите рекурсию или FSM.
чистый код В принципе, вы можете сохранить ваш поток таким, какой он есть сейчас, просто убедитесь, что ваши значения глубины не превышают 2-го уровня, и в этом случае вы должны извлечь код для какого-либо другого метода.
Ваша проблема, как уже говорилось, имеет некоторые сложности: действие, которое нужно предпринять, зависит не только от значения в массиве, но и от позиции, которую занимает значение. Это правильно? Есть ли семантика / отношения, связанные с каждой позицией массива? Другими словами, если я правильно понимаю: 1 4 6 не будет генерировать такой же ввод, как 1 6 или 4 1 6.
@Picarus Вы правы: 1 6 даст вам сырое мясо, а 4 1 6 не будет определено для этого контекста. Таким образом, всегда есть действие (1, 2 или 3), за которым следует один или несколько спецификаторов (4 5), а затем есть объект действия (6 7). Очевидно, этот пример очень упрощен :)

igoris · Answer 1 · 2014-02-03T09-51-00.000Z

Вы можете использовать что - то подобное Command шаблон. Создайте словарь, где ключи будут возможными значениями элементов в array а значения будут указателями на соответствующие функции, например:

{
1 = (*open_the_fridge),
2 = (*make_me_a_cake),
...
}

Затем вы можете перебрать array и выполнить соответствующие функции.

псевдокод:

func_map{1=(*open_the_fridge), 2=(*make_me_a_cake),...}
for (i = 0; i < length(array); i++) {
    (*get_value_from_dict(func_map, array[i]))();
}

Picarus · Answer 2 · 2014-02-03T11-37-00.000Z

Основываясь на вашем ответе на комментарии, я думаю, вы должны изменить свою "парадигму" и преобразовать структуру массива в кортеж или объект со значимыми атрибутами. Дело в том, что, похоже, не все "пути выполнения" потребуют одинакового количества параметров, но инкапсуляция OO должна помочь вам справиться с этим. Итак, вы должны решить проблему в источнике, где генерируется массив. Если это невозможно, потому что это библиотека или код, которые вы не можете изменить по какой-либо причине, вам следует написать адаптер, чтобы вы могли правильно обрабатывать вещи.

Затем вы указываете в своих тегах C и C++. В зависимости от подхода, который вы хотите предпринять, сообщество может предоставить более подробную информацию о конкретных решениях.

Шаблон Factory (или фабричный) может помочь вам создать соответствующие объекты или кортежи. Более простой подход может заключаться в изменении подписи вашего метода, чтобы принять 3 параметра: действие, параметры, тип объекта, как вы указали.

AlexT · Answer 3 · 2014-02-03T09-00-00.000Z

Это похоже на дерево решений - вы можете инициализировать это дерево со всей возможной комбинацией с результатами, а затем написать один цикл, который ищет комбинацию ввода и возвращает результат. Результатом может быть строка или команда (как предложено в другом ответе)