# SonarLint 检查提示和解决方法

SonarLint 检查提示和解决方法

代码质量控制和代码质量相关工具说明: https://blog.csdn.net/qq_37248504/article/details/125383986

---

SonarLint: Use the built-in formatting to construct this argument.

• 翻译：将连接字符串传递给日志记录方法也可能导致不必要的性能损失，因为每次调用该方法时都会执行连接，无论日志级别是否低到足以显示消息。应该使用内置的字符串格式而不是字符串连接，如果消息是方法调用的结果，则应该完全跳过前置条件，而应该有条件地抛出相关异常。

• 反例

String message = "Hellow World!";
logger.info("Test Info message："+message);

• 正例

String message = "Hellow World!";
String format = String.format("Test Info message：%s", message);
logger.info(format);

---

SonarLint: Replace this use of System.out or System.err by a logger.

• 翻译：使用专门的日志框架操作日志
• 反例

System.out.printf("Hellow");

• 正例

private static final Logger logger = Logger.getLogger(String.valueOf(Demo.class));
logger.info("Hellow");

---

SonarLint: Invoke method(s) only conditionally.

• 翻译：构建代码，将静态或预先计算的值传递给前置条件、条件检查和记录调用。方法中参数直接是计算之后的结果，而不是计算并得到结果。
• 反例

logger.info(String.valueOf((char)c));

• 正例

String s2 = String.valueOf((char) c);
logger.info(s2);

---

SonarLint: Define and throw a dedicated exception instead of using a generic one.

• 翻译：使用自己定义的异常类进行抛出异常错误等，不要使用原始的Exception等。
• 反例

throw new Exception("Test");

• 正例

public class MyException extends RuntimeException{

    public MyException() {
    }

    public MyException(String message) {
        super(message);
    }
}

throw new MyException("Hellow");

---

SonarLint: String contains no format specifiers.

• 翻译：字符串不包含格式说明符。
• 反例

String message = "Hellow Test!";
logger.info(String.format("method error: {}",message));

• 正例

String message = "Hellow Test!";
logger.info(String.format("method error: %s",message));

---

SonarLint: Either re-interrupt this method or rethrow the “InterruptedException”.

• 翻译：绝不应该在代码中忽略interruptedexception，在这种情况下，简单地将异常计数记录为“忽略”。InterruptedException的抛出将清除线程的中断状态，因此，如果异常没有得到正确的处理，那么线程被中断的事实就会丢失。相反，interruptedexception应该被重新抛出——立即或在清除方法的状态之后——或者应该通过调用thread .interrupt()重新中断线程，即使这是一个单线程的应用程序。任何其他操作都有延迟线程关闭的风险，并丢失线程被中断的信息——可能没有完成其任务。
• 反例：

private  void dealAddNum() {
    try {
        num++;
        Thread.sleep(100);
    } catch (InterruptedException e) {
        log.info(e.getMessage());
    }
}

• 正例：

private  void dealAddNum() {
    try {
        num++;
        Thread.sleep(100);
    } catch (InterruptedException e) {
        log.info(e.getMessage());
        Thread.currentThread().interrupt();
    }
}

---

SonarLint: Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.

• 翻译：方法没有方法主体有以下几个原因:这是一个无意的疏忽，应该加以修正，以防止生产中出现意外的行为。它还没有得到支持，或者永远不会得到支持。在这种情况下，应该抛出UnsupportedOperationException。该方法是一个故意为空的覆盖。在这种情况下，一个嵌套的注释应该解释空白覆盖的原因。

• 反例：

@Override
public void setProperties(Properties properties) {
    
}

• 正例：

@Override
public void setProperties(Properties properties) {
    throw new UnsupportedOperationException();
}

---

SonarLint: “ThreadLocal” variables should be cleaned up when no longer used

• 翻译：定义的线程局部变量在使用完之后，需要销毁。一旦持有线程不再活着，ThreadLocal变量就会被垃圾收集。在重用持有的线程时(在使用线程池的应用服务器上就是这种情况)可能会发生内存泄漏。
• 反例：使用后没有销毁。

private static final ThreadLocal<SimpleDateFormat> DATETIME_FORMATTER = ThreadLocal.withInitial(() -> new SimpleDateFormat("yyyy-MM-dd HH:mm:ss"));

• 正例：

private static final ThreadLocal<SimpleDateFormat> DATETIME_FORMATTER = ThreadLocal.withInitial(() -> new SimpleDateFormat("yyyy-MM-dd HH:mm:ss"));

public void test1(){
    try{
        // do something
    }catch(Exception e){
        // do something
    }finally{
		DATETIME_FORMATTER.remove();
    }
}

---

SonarLint: Extract this nested try block into a separate method.

• 翻译：嵌套的try/catch块严重影响了源代码的可读性，因为它使理解哪个块捕获哪个异常变得非常困难。
• 反例：

@Test
public void test1(){
    try {

        try{

        }catch (NullPointerException e){
            logger.info(e.getMessage());
        }
    }catch (Exception e){
        logger.info(e.getMessage());
    }
}

• 正例：

@Test
public void test1(){
    try {
        logger.info("test messages!");
    }catch (Exception e){
        logger.info(e.getMessage());
    }
}

---

SonarLint: Replace the usage of the “instanceof” operator by a catch block.

• 翻译：应该使用适当类型的多个catch块，而不是捕获一般异常，然后对类型进行测试。
• 反例：

try {
    // do somethings
} catch (Exception e) {
    if (e instanceof InvocationTargetException) {
        logger.error(LogConstant.LOG_FAILURE_PREFIX + (((InvocationTargetException) e).getTargetException()).getMessage());
    } else {
        logger.error(LogConstant.LOG_FAILURE_PREFIX + e.getMessage());
    }
    return result;
} 

• 正例：

try{
    logger.info("test!");
}catch (NullPointerException nullPointerException){
    logger.info(nullPointerException.getMessage());
}catch (IllegalArgumentException illegalArgumentException){
    logger.info(illegalArgumentException.getMessage());
}

---

SonarLint: Use “java.util.Random.nextInt()” instead.

• 翻译：没有必要将Random的nextDouble方法的输出乘以一个随机整数。使用nextt方法代替。当随机的任何返回浮点值的方法的返回值被转换为整数时，该规则会引发一个问题。
• 反例：

user.setGender(((int)(10 * Math.random())) % 2  );

• 正例：

Random random = new Random();
user.setGender(random.nextInt(10) % 2  );