refactor(system): 优化通知保存逻辑，移除冗余依赖

chenhj

2026-04-22 1a6c518be93901a41bc2b7b7a1475c58e9b1d2c0

 src/main/java/com/ruoyi/project/system/service/impl/SysRoleServiceImpl.java

@@ -1,27 +1,22 @@
package com.ruoyi.project.system.service.impl;



import java.util.ArrayList;

import java.util.Arrays;

import java.util.HashSet;

import java.util.List;

import java.util.Set;

import org.springframework.beans.factory.annotation.Autowired;

import org.springframework.stereotype.Service;

import org.springframework.transaction.annotation.Transactional;

import com.ruoyi.common.constant.UserConstants;

import com.ruoyi.common.exception.CustomException;

import com.ruoyi.common.exception.ServiceException;

import com.ruoyi.common.utils.SecurityUtils;

import com.ruoyi.common.utils.StringUtils;

import com.ruoyi.common.utils.spring.SpringUtils;

import com.ruoyi.framework.aspectj.lang.annotation.DataScope;

import com.ruoyi.project.system.domain.SysRole;

import com.ruoyi.project.system.domain.SysRoleDept;

import com.ruoyi.project.system.domain.SysRoleMenu;

import com.ruoyi.project.system.domain.SysUserRole;

import com.ruoyi.project.system.domain.*;

import com.ruoyi.project.system.mapper.SysRoleDeptMapper;

import com.ruoyi.project.system.mapper.SysRoleMapper;

import com.ruoyi.project.system.mapper.SysRoleMenuMapper;

import com.ruoyi.project.system.mapper.SysUserRoleMapper;

import com.ruoyi.project.system.service.ISysRoleService;

import lombok.RequiredArgsConstructor;

import org.springframework.stereotype.Service;

import org.springframework.transaction.annotation.Transactional;



import java.util.*;



/**

 * 角色 业务层处理

@@ -29,19 +24,13 @@
 * @author ruoyi

 */

@Service

@RequiredArgsConstructor

public class SysRoleServiceImpl implements ISysRoleService

{

    @Autowired

    private SysRoleMapper roleMapper;



    @Autowired

    private SysRoleMenuMapper roleMenuMapper;



    @Autowired

    private SysUserRoleMapper userRoleMapper;



    @Autowired

    private SysRoleDeptMapper roleDeptMapper;

    private final SysRoleMapper roleMapper;

    private final SysRoleMenuMapper roleMenuMapper;

    private final SysUserRoleMapper userRoleMapper;

    private final SysRoleDeptMapper roleDeptMapper;



    /**

     * 根据条件分页查询角色数据

@@ -120,7 +109,7 @@
     * @return 选中角色ID列表

     */

    @Override

    public List<Integer> selectRoleListByUserId(Long userId)

    public List<Long> selectRoleListByUserId(Long userId)

    {

        return roleMapper.selectRoleListByUserId(userId);

    }

@@ -144,7 +133,7 @@
     * @return 结果

     */

    @Override

    public String checkRoleNameUnique(SysRole role)

    public boolean checkRoleNameUnique(SysRole role)

    {

        Long roleId = StringUtils.isNull(role.getRoleId()) ? -1L : role.getRoleId();

        SysRole info = roleMapper.checkRoleNameUnique(role.getRoleName());

@@ -162,7 +151,7 @@
     * @return 结果

     */

    @Override

    public String checkRoleKeyUnique(SysRole role)

    public boolean checkRoleKeyUnique(SysRole role)

    {

        Long roleId = StringUtils.isNull(role.getRoleId()) ? -1L : role.getRoleId();

        SysRole info = roleMapper.checkRoleKeyUnique(role.getRoleKey());

@@ -183,7 +172,30 @@
    {

        if (StringUtils.isNotNull(role.getRoleId()) && role.isAdmin())

        {

            throw new CustomException("不允许操作超级管理员角色");

            throw new ServiceException("不允许操作超级管理员角色");

        }

    }



    /**

     * 校验角色是否有数据权限

     * 
     * @param roleIds 角色id

     */

    @Override

    public void checkRoleDataScope(Long... roleIds)

    {

        if (!SysUser.isAdmin(SecurityUtils.getUserId()))

        {

            for (Long roleId : roleIds)

            {

                SysRole role = new SysRole();

                role.setRoleId(roleId);

                List<SysRole> roles = SpringUtils.getAopProxy(this).selectRoleList(role);

                if (StringUtils.isEmpty(roles))

                {

                    throw new ServiceException("没有权限访问角色数据！");

                }

            }

        }

    }



@@ -339,10 +351,11 @@
        for (Long roleId : roleIds)

        {

            checkRoleAllowed(new SysRole(roleId));

            checkRoleDataScope(roleId);

            SysRole role = selectRoleById(roleId);

            if (countUserRoleByRoleId(roleId) > 0)

            {

                throw new CustomException(String.format("%1$s已分配,不能删除", role.getRoleName()));

                throw new ServiceException(String.format("%1$s已分配,不能删除", role.getRoleName()));

            }

        }

        // 删除角色与菜单关联

@@ -381,7 +394,7 @@
     * 批量选择授权用户角色

     * 

     * @param roleId 角色ID

     * @param userIds 需要删除的用户数据ID

     * @param userIds 需要授权的用户数据ID

     * @return 结果

     */

    @Override