代码优化

稚屿

2022-02-09 c99eb98001e22f671ce514268231edcd27fc73ef

代码优化

 ruoyi-common/src/main/java/com/ruoyi/common/utils/DictUtils.java

@@ -41,8 +41,7 @@
        Object cacheObj = SpringUtils.getBean(RedisCache.class).getCacheObject(getCacheKey(key));

        if (StringUtils.isNotNull(cacheObj))

        {

            List<SysDictData> dictDatas = StringUtils.cast(cacheObj);

            return dictDatas;

            return StringUtils.cast(cacheObj);

        }

        return null;

    }


 ruoyi-common/src/main/java/com/ruoyi/common/utils/ExceptionUtil.java

@@ -18,8 +18,7 @@
    {

        StringWriter sw = new StringWriter();

        e.printStackTrace(new PrintWriter(sw, true));

        String str = sw.toString();

        return str;

        return sw.toString();

    }



    public static String getRootErrorMessage(Exception e)


 ruoyi-common/src/main/java/com/ruoyi/common/utils/ServletUtils.java

@@ -99,9 +99,8 @@
     * 

     * @param response 渲染对象

     * @param string 待渲染的字符串

     * @return null

     */

    public static String renderString(HttpServletResponse response, String string)

    public static void renderString(HttpServletResponse response, String string)

    {

        try

        {

@@ -114,7 +113,6 @@
        {

            e.printStackTrace();

        }

        return null;

    }



    /**

@@ -125,13 +123,13 @@
    public static boolean isAjaxRequest(HttpServletRequest request)

    {

        String accept = request.getHeader("accept");

        if (accept != null && accept.indexOf("application/json") != -1)

        if (accept != null && accept.contains("application/json"))

        {

            return true;

        }



        String xRequestedWith = request.getHeader("X-Requested-With");

        if (xRequestedWith != null && xRequestedWith.indexOf("XMLHttpRequest") != -1)

        if (xRequestedWith != null && xRequestedWith.contains("XMLHttpRequest"))

        {

            return true;

        }

@@ -143,10 +141,6 @@
        }



        String ajax = request.getParameter("__ajax");

        if (StringUtils.inStringIgnoreCase(ajax, "json", "xml"))

        {

            return true;

        }

        return false;

        return StringUtils.inStringIgnoreCase(ajax, "json", "xml");

    }

}


 ruoyi-common/src/main/java/com/ruoyi/common/utils/VerifyCodeUtils.java

@@ -208,7 +208,6 @@


        int period = random.nextInt(40) + 10; // 50;



        boolean borderGap = true;

        int frames = 20;

        int phase = 7;

        for (int i = 0; i < w1; i++)

@@ -216,13 +215,9 @@
            double d = (double) (period >> 1)

                    * Math.sin((double) i / (double) period + (6.2831853071795862D * (double) phase) / (double) frames);

            g.copyArea(i, 0, 1, h1, 0, (int) d);

            if (borderGap)

            {

                g.setColor(color);

                g.drawLine(i, (int) d, i, 0);

                g.drawLine(i, (int) d + h1, i, h1);

            }



            g.setColor(color);

            g.drawLine(i, (int) d, i, 0);

            g.drawLine(i, (int) d + h1, i, h1);

        }

    }

}

 ruoyi-common/src/main/java/com/ruoyi/common/utils/file/FileUploadUtils.java

@@ -2,6 +2,8 @@


import java.io.File;

import java.io.IOException;

import java.util.Objects;



import org.apache.commons.io.FilenameUtils;

import org.springframework.web.multipart.MultipartFile;

import com.ruoyi.common.config.RuoYiConfig;

@@ -100,7 +102,7 @@
            throws FileSizeLimitExceededException, IOException, FileNameLengthLimitExceededException,

            InvalidExtensionException

    {

        int fileNamelength = file.getOriginalFilename().length();

        int fileNamelength = Objects.requireNonNull(file.getOriginalFilename()).length();

        if (fileNamelength > FileUploadUtils.DEFAULT_FILE_NAME_LENGTH)

        {

            throw new FileNameLengthLimitExceededException(FileUploadUtils.DEFAULT_FILE_NAME_LENGTH);

@@ -112,8 +114,7 @@


        File desc = getAbsoluteFile(baseDir, fileName);

        file.transferTo(desc);

        String pathFileName = getPathFileName(baseDir, fileName);

        return pathFileName;

        return getPathFileName(baseDir, fileName);

    }



    /**

@@ -145,8 +146,7 @@
    {

        int dirLastIndex = RuoYiConfig.getProfile().length() + 1;

        String currentDir = StringUtils.substring(uploadDir, dirLastIndex);

        String pathFileName = Constants.RESOURCE_PREFIX + "/" + currentDir + "/" + fileName;

        return pathFileName;

        return Constants.RESOURCE_PREFIX + "/" + currentDir + "/" + fileName;

    }



    /**

@@ -161,7 +161,7 @@
            throws FileSizeLimitExceededException, InvalidExtensionException

    {

        long size = file.getSize();

        if (DEFAULT_MAX_SIZE != -1 && size > DEFAULT_MAX_SIZE)

        if (size > DEFAULT_MAX_SIZE)

        {

            throw new FileSizeLimitExceededException(DEFAULT_MAX_SIZE / 1024 / 1024);

        }

@@ -228,7 +228,7 @@
        String extension = FilenameUtils.getExtension(file.getOriginalFilename());

        if (StringUtils.isEmpty(extension))

        {

            extension = MimeTypeUtils.getExtension(file.getContentType());

            extension = MimeTypeUtils.getExtension(Objects.requireNonNull(file.getContentType()));

        }

        return extension;

    }


 ruoyi-common/src/main/java/com/ruoyi/common/utils/file/ImageUtils.java

@@ -59,7 +59,7 @@
    /**

     * 读取文件为字节数据

     * 

     * @param key 地址

     * @param url 地址

     * @return 字节数据

     */

    public static byte[] readFile(String url)


 ruoyi-common/src/main/java/com/ruoyi/common/utils/http/HttpHelper.java

@@ -5,6 +5,7 @@
import java.io.InputStream;

import java.io.InputStreamReader;

import java.nio.charset.Charset;

import java.nio.charset.StandardCharsets;

import javax.servlet.ServletRequest;

import org.apache.commons.lang3.exception.ExceptionUtils;

import org.slf4j.Logger;

@@ -25,7 +26,7 @@
        BufferedReader reader = null;

        try (InputStream inputStream = request.getInputStream())

        {

            reader = new BufferedReader(new InputStreamReader(inputStream, Charset.forName("UTF-8")));

            reader = new BufferedReader(new InputStreamReader(inputStream, StandardCharsets.UTF_8));

            String line = "";

            while ((line = reader.readLine()) != null)

            {


 ruoyi-common/src/main/java/com/ruoyi/common/utils/http/HttpUtils.java

@@ -9,6 +9,7 @@
import java.net.SocketTimeoutException;

import java.net.URL;

import java.net.URLConnection;

import java.nio.charset.StandardCharsets;

import java.security.cert.X509Certificate;

import javax.net.ssl.HostnameVerifier;

import javax.net.ssl.HttpsURLConnection;

@@ -130,9 +131,8 @@
        StringBuilder result = new StringBuilder();

        try

        {

            String urlNameString = url;

            log.info("sendPost - {}", urlNameString);

            URL realUrl = new URL(urlNameString);

            log.info("sendPost - {}", url);

            URL realUrl = new URL(url);

            URLConnection conn = realUrl.openConnection();

            conn.setRequestProperty("accept", "*/*");

            conn.setRequestProperty("connection", "Keep-Alive");

@@ -144,7 +144,7 @@
            out = new PrintWriter(conn.getOutputStream());

            out.print(param);

            out.flush();

            in = new BufferedReader(new InputStreamReader(conn.getInputStream(), "utf-8"));

            in = new BufferedReader(new InputStreamReader(conn.getInputStream(), StandardCharsets.UTF_8));

            String line;

            while ((line = in.readLine()) != null)

            {

@@ -218,7 +218,7 @@
            {

                if (ret != null && !"".equals(ret.trim()))

                {

                    result.append(new String(ret.getBytes("ISO-8859-1"), "utf-8"));

                    result.append(new String(ret.getBytes(StandardCharsets.ISO_8859_1), StandardCharsets.UTF_8));

                }

            }

            log.info("recv - {}", result);


 ruoyi-common/src/main/java/com/ruoyi/common/utils/ip/AddressUtils.java

@@ -25,7 +25,6 @@


    public static String getRealAddressByIP(String ip)

    {

        String address = UNKNOWN;

        // 内网不查询

        if (IpUtils.internalIp(ip))

        {

@@ -51,6 +50,6 @@
                log.error("获取地理位置异常 {}", ip);

            }

        }

        return address;

        return UNKNOWN;

    }

}


 ruoyi-common/src/main/java/com/ruoyi/common/utils/poi/ExcelUtil.java

@@ -1122,7 +1122,7 @@
        if (StringUtils.isNotEmpty(excel.targetAttr()))

        {

            String target = excel.targetAttr();

            if (target.indexOf(".") > -1)

            if (target.contains("."))

            {

                String[] targets = target.split("[.]");

                for (String name : targets)

@@ -1217,7 +1217,7 @@
        for (Object[] os : this.fields)

        {

            Excel excel = (Excel) os[1];

            maxHeight = maxHeight > excel.height() ? maxHeight : excel.height();

            maxHeight = Math.max(maxHeight, excel.height());

        }

        return (short) (maxHeight * 20);

    }


 ruoyi-common/src/main/java/com/ruoyi/common/utils/sign/Md5Utils.java

@@ -1,5 +1,6 @@
package com.ruoyi.common.utils.sign;



import java.nio.charset.StandardCharsets;

import java.security.MessageDigest;

import org.slf4j.Logger;

import org.slf4j.LoggerFactory;

@@ -55,7 +56,7 @@
    {

        try

        {

            return new String(toHex(md5(s)).getBytes("UTF-8"), "UTF-8");

            return new String(toHex(md5(s)).getBytes(StandardCharsets.UTF_8), StandardCharsets.UTF_8);

        }

        catch (Exception e)

        {


 ruoyi-common/src/main/java/com/ruoyi/common/utils/sql/SqlUtil.java

@@ -50,10 +50,8 @@
            return;

        }

        String[] sqlKeywords = StringUtils.split(SQL_REGEX, "\\|");

        for (int i = 0; i < sqlKeywords.length; i++)

        {

            if (StringUtils.indexOfIgnoreCase(value, sqlKeywords[i]) > -1)

            {

        for (String sqlKeyword : sqlKeywords) {

            if (StringUtils.indexOfIgnoreCase(value, sqlKeyword) > -1) {

                throw new UtilException("参数存在SQL注入风险");

            }

        }


 ruoyi-generator/src/main/java/com/ruoyi/generator/util/GenUtils.java

@@ -151,8 +151,7 @@
    {

        int lastIndex = packageName.lastIndexOf(".");

        int nameLength = packageName.length();

        String moduleName = StringUtils.substring(packageName, lastIndex + 1, nameLength);

        return moduleName;

        return StringUtils.substring(packageName, lastIndex + 1, nameLength);

    }



    /**

@@ -165,8 +164,7 @@
    {

        int lastIndex = tableName.lastIndexOf("_");

        int nameLength = tableName.length();

        String businessName = StringUtils.substring(tableName, lastIndex + 1, nameLength);

        return businessName;

        return StringUtils.substring(tableName, lastIndex + 1, nameLength);

    }



    /**


 ruoyi-generator/src/main/java/com/ruoyi/generator/util/VelocityUtils.java

@@ -228,8 +228,7 @@
    public static String getPackagePrefix(String packageName)

    {

        int lastIndex = packageName.lastIndexOf(".");

        String basePackage = StringUtils.substring(packageName, 0, lastIndex);

        return basePackage;

        return StringUtils.substring(packageName, 0, lastIndex);

    }



    /**


 ruoyi-system/src/main/java/com/ruoyi/system/service/impl/SysDeptServiceImpl.java

@@ -63,12 +63,9 @@
        {

            tempList.add(dept.getDeptId());

        }

        for (Iterator<SysDept> iterator = depts.iterator(); iterator.hasNext();)

        {

            SysDept dept = (SysDept) iterator.next();

        for (SysDept dept : depts) {

            // 如果是顶级节点, 遍历该父节点的所有子节点

            if (!tempList.contains(dept.getParentId()))

            {

            if (!tempList.contains(dept.getParentId())) {

                recursionFn(depts, dept);

                returnList.add(dept);

            }


 ruoyi-system/src/main/java/com/ruoyi/system/service/impl/SysMenuServiceImpl.java

@@ -498,7 +498,7 @@
     */
    private boolean hasChild(List<SysMenu> list, SysMenu t)
    {
        return getChildList(list, t).size() > 0 ? true : false;
        return getChildList(list, t).size() > 0;
    }

    /**