From fd230b6d091bf07c0ef6ded30554c1e4f42fb621 Mon Sep 17 00:00:00 2001 From: Eduardo Novella Date: Sun, 11 Sep 2022 00:52:34 +0200 Subject: [PATCH] Refactor switch case with duplicated code --- .../scene/subbrute_scene_entrypoint.c | 26 +++++-------------- 1 file changed, 7 insertions(+), 19 deletions(-) diff --git a/applications/subbrute/scene/subbrute_scene_entrypoint.c b/applications/subbrute/scene/subbrute_scene_entrypoint.c index 96b7b4040..9720cf60d 100644 --- a/applications/subbrute/scene/subbrute_scene_entrypoint.c +++ b/applications/subbrute/scene/subbrute_scene_entrypoint.c @@ -14,27 +14,15 @@ void subbrute_scene_entrypoint_menu_callback(SubBruteState* context, uint32_t in context->current_scene = SceneSelectFile; break; case SubBruteAttackCAME12bit307: - context->frequency = 307800000; - context->bit = 12; - string_set_str(context->protocol, "CAME"); - string_set_str(context->preset, "FuriHalSubGhzPresetOok650Async"); - if(!subbrute_is_frequency_allowed(context)) { - return; - } - context->current_scene = SceneAttack; - break; case SubBruteAttackCAME12bit433: - context->frequency = 433920000; - context->bit = 12; - string_set_str(context->protocol, "CAME"); - string_set_str(context->preset, "FuriHalSubGhzPresetOok650Async"); - if(!subbrute_is_frequency_allowed(context)) { - return; - } - context->current_scene = SceneAttack; - break; case SubBruteAttackCAME12bit868: - context->frequency = 868350000; + if (index == SubBruteAttackCAME12bit307) { + context->frequency = 307800000; + } else if (index == SubBruteAttackCAME12bit433) { + context->frequency = 433920000; + } else if (index == SubBruteAttackCAME12bit868) { + context->frequency = 868350000; + } context->bit = 12; string_set_str(context->protocol, "CAME"); string_set_str(context->preset, "FuriHalSubGhzPresetOok650Async");