diff --git a/src/MaaCore/Task/Roguelike/RoguelikeBattleTaskPlugin.cpp b/src/MaaCore/Task/Roguelike/RoguelikeBattleTaskPlugin.cpp
index 4c73e8edc4..f1c49df7ef 100644
--- a/src/MaaCore/Task/Roguelike/RoguelikeBattleTaskPlugin.cpp
+++ b/src/MaaCore/Task/Roguelike/RoguelikeBattleTaskPlugin.cpp
@@ -491,6 +491,8 @@ void asst::RoguelikeBattleTaskPlugin::postproc_of_deployment_conditions(const ba
             }
         }
         break;
+    default:
+        break;
     }
 
     if (m_force_air_defense.has_finished_deploy_air_defense && position != OperPosition::AirDefense) {
@@ -577,6 +579,8 @@ std::optional<asst::battle::DeploymentOper> asst::RoguelikeBattleTaskPlugin::cal
         case Role::Medic:
             has_medic = true;
             break;
+        default:
+            break;
         }
 
         const auto& position = get_role_position(role);
@@ -587,6 +591,8 @@ std::optional<asst::battle::DeploymentOper> asst::RoguelikeBattleTaskPlugin::cal
         case OperPosition::AirDefense:
             has_air_defense = true;
             break;
+        default:
+            break;
         }
 
         // found all
diff --git a/src/MaaCore/Task/Roguelike/RoguelikeRecruitTaskPlugin.cpp b/src/MaaCore/Task/Roguelike/RoguelikeRecruitTaskPlugin.cpp
index e17937880a..787d359b5c 100644
--- a/src/MaaCore/Task/Roguelike/RoguelikeRecruitTaskPlugin.cpp
+++ b/src/MaaCore/Task/Roguelike/RoguelikeRecruitTaskPlugin.cpp
@@ -118,7 +118,7 @@ bool asst::RoguelikeRecruitTaskPlugin::_run()
         const auto& team_complete_condition = RoguelikeRecruit.get_team_complete_info(rogue_theme);
         for (const auto& condition : team_complete_condition) {
             int count = 0;
-            for (const std::string group_name : condition.groups)
+            for (const std::string& group_name : condition.groups)
                 count += group_count[group_name];
             if (count < condition.threshold) {
                 complete = false;