diff --git a/backend/src/main/java/com/xly/erp/module/usr/service/impl/UserServiceImpl.java b/backend/src/main/java/com/xly/erp/module/usr/service/impl/UserServiceImpl.java
index cfd7f5a..48d4e1d 100644
--- a/backend/src/main/java/com/xly/erp/module/usr/service/impl/UserServiceImpl.java
+++ b/backend/src/main/java/com/xly/erp/module/usr/service/impl/UserServiceImpl.java
@@ -191,21 +191,19 @@ public class UserServiceImpl implements UserService {
             throw new BizException(40020, "用户号或用户名已存在");
         }
 
-        if (ids != null) {
-            userPermissionMapper.deleteByUserId(id);
-            if (!ids.isEmpty()) {
-                String createdBy = SecurityContextHelper.currentUserNo();
-                LocalDateTime now = LocalDateTime.now();
-                for (Integer cid : ids) {
-                    UserPermission rel = new UserPermission();
-                    rel.setSBrandsId(tenant.getBrandsId());
-                    rel.setSSubsidiaryId(tenant.getSubsidiaryId());
-                    rel.setTCreateDate(now);
-                    rel.setIUserId(id);
-                    rel.setICategoryId(cid);
-                    rel.setSCreatedBy(createdBy);
-                    userPermissionMapper.insert(rel);
-                }
+        userPermissionMapper.deleteByUserId(id);
+        if (ids != null && !ids.isEmpty()) {
+            String createdBy = SecurityContextHelper.currentUserNo();
+            LocalDateTime now = LocalDateTime.now();
+            for (Integer cid : ids) {
+                UserPermission rel = new UserPermission();
+                rel.setSBrandsId(tenant.getBrandsId());
+                rel.setSSubsidiaryId(tenant.getSubsidiaryId());
+                rel.setTCreateDate(now);
+                rel.setIUserId(id);
+                rel.setICategoryId(cid);
+                rel.setSCreatedBy(createdBy);
+                userPermissionMapper.insert(rel);
             }
         }
         return id;