diff --git a/game-server/app/servers/battle/handler/ladderHandler.ts b/game-server/app/servers/battle/handler/ladderHandler.ts
index 414378048..f1e5f0f5f 100644
--- a/game-server/app/servers/battle/handler/ladderHandler.ts
+++ b/game-server/app/servers/battle/handler/ladderHandler.ts
@@ -235,7 +235,7 @@ export class LadderHandler {
 
         await checkTaskInLadderEnd(serverId, roleId, sid, isSuccess, historyRank, atkLadderMatch.rank);
 
-        if (isSuccess && !rec.defenseInfo?.isRobot) await pushLadderIconShow(rec.roleId2, true);
+        if (isSuccess && (!rec.defenseInfo?.isRobot)) await pushLadderIconShow(rec.roleId2, true);
         return resResult(STATUS.SUCCESS, {...pick(result, ['rank', 'historyRank', 'challengeCnt','status', 'time', 'oppPlayers']), battleGoods, breakGoods});
     }
 
diff --git a/game-server/app/servers/guild/handler/gvgFightHandler.ts b/game-server/app/servers/guild/handler/gvgFightHandler.ts
index 22a3d76d0..714100025 100644
--- a/game-server/app/servers/guild/handler/gvgFightHandler.ts
+++ b/game-server/app/servers/guild/handler/gvgFightHandler.ts
@@ -378,7 +378,7 @@ export class GVGProduceHandler {
 
         await saveScoreToRank(rec);
         addVestigeBattleEndRec(rec);
-        if (isSuccess && !rec.defenseInfo?.isRobot) await pushGvgFightIconShow(rec.defenseRoleId, battleCode);
+        if (isSuccess && (!rec.defenseInfo?.isRobot)) await pushGvgFightIconShow(rec.defenseRoleId, battleCode);
 
         return resResult(STATUS.SUCCESS, {
             vestigeId: rec.vestigeId, 
diff --git a/shared/db/LadderMatchRec.ts b/shared/db/LadderMatchRec.ts
index 8d01f6476..ab07f5421 100644
--- a/shared/db/LadderMatchRec.ts
+++ b/shared/db/LadderMatchRec.ts
@@ -141,7 +141,7 @@ export default class LadderMatchRec extends BaseModel {
         ];
         const recs: LadderMatchRecType[] = await LadderMatchRecModel.aggregate(pipeline);
         for (let obj of recs) {
-            if (obj.roleId2 === roleId && !obj.defenseInfo.isSuccess) return true;
+            if (obj.roleId2 === roleId && (!obj.defenseInfo.isSuccess)) return true;
         }
         return false;
     }