diff --git a/cutejava/cutejava-system/src/main/java/cn/odboy/application/system/mapper/UserRoleMapper.java b/cutejava/cutejava-system/src/main/java/cn/odboy/application/system/mapper/UserRoleMapper.java
index 716dfc25..a56c2ab2 100644
--- a/cutejava/cutejava-system/src/main/java/cn/odboy/application/system/mapper/UserRoleMapper.java
+++ b/cutejava/cutejava-system/src/main/java/cn/odboy/application/system/mapper/UserRoleMapper.java
@@ -7,7 +7,7 @@ import java.util.Set;
 
 @Mapper
 public interface UserRoleMapper {
-    void insertData(@Param("userId") Long userId, @Param("roles") Set<Role> roles);
+    void insertBatchWithUserId(@Param("roles") Set<Role> roles, @Param("userId") Long userId);
 
     void deleteByUserId(@Param("userId") Long userId);
 
diff --git a/cutejava/cutejava-system/src/main/java/cn/odboy/application/system/service/impl/UserServiceImpl.java b/cutejava/cutejava-system/src/main/java/cn/odboy/application/system/service/impl/UserServiceImpl.java
index d0c01b6e..548de49d 100644
--- a/cutejava/cutejava-system/src/main/java/cn/odboy/application/system/service/impl/UserServiceImpl.java
+++ b/cutejava/cutejava-system/src/main/java/cn/odboy/application/system/service/impl/UserServiceImpl.java
@@ -94,7 +94,7 @@ public class UserServiceImpl extends ServiceImpl<UserMapper, User> implements Us
         // 保存用户岗位
         userJobMapper.insertBatchWithUserId(resources.getJobs(), resources.getId());
         // 保存用户角色
-        userRoleMapper.insertData(resources.getId(), resources.getRoles());
+        userRoleMapper.insertBatchWithUserId(resources.getRoles(), resources.getId());
     }
 
     @Override
@@ -146,7 +146,7 @@ public class UserServiceImpl extends ServiceImpl<UserMapper, User> implements Us
         userJobMapper.insertBatchWithUserId(resources.getJobs(), resources.getId());
         // 更新用户角色
         userRoleMapper.deleteByUserId(resources.getId());
-        userRoleMapper.insertData(resources.getId(), resources.getRoles());
+        userRoleMapper.insertBatchWithUserId(resources.getRoles(), resources.getId());
     }
 
     @Override
diff --git a/cutejava/cutejava-system/src/main/resources/mapper/UserRoleMapper.xml b/cutejava/cutejava-system/src/main/resources/mapper/UserRoleMapper.xml
index 5217445e..b73345ce 100644
--- a/cutejava/cutejava-system/src/main/resources/mapper/UserRoleMapper.xml
+++ b/cutejava/cutejava-system/src/main/resources/mapper/UserRoleMapper.xml
@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8" ?>
 <!DOCTYPE mapper PUBLIC "-//mybatis.org//DTD Mapper 3.0//EN" "http://mybatis.org/dtd/mybatis-3-mapper.dtd" >
 <mapper namespace="cn.odboy.application.system.mapper.UserRoleMapper">
-    <insert id="insertData">
+    <insert id="insertBatchWithUserId">
         insert into sys_users_roles (user_id, role_id)
         values
         <foreach collection="roles" item="item" open="(" separator="),(" close=")">