0

我的代码的精简版本看起来像这样

package com.funbluebits.test;

import static java.io.File.separator;
import static java.nio.file.Files.lines;
import static java.nio.file.Paths.get;

import java.io.*;
import org.apache.commons.io.FileUtils;
import org.apache.commons.lang.StringUtils;

public class SonarLintExample {

    public static void main(String[] args) {

        SonarLintExample sle = new SonarLintExample();
        try {
            sle.handleInputFile("hello.txt");
            sle.handleInputFile("");
            sle.handleInputFile("myfile.pgp");
        }
        catch (Exception e) {

        }
    }

    public String handleInputFile(final String filePath) throws Exception {
        MarketAndProvider marketAndProvider = null;
        try {
            marketAndProvider = extractMarketAndProvider(filePath);

            if (filePath.contains(".pgp") || filePath.contains(".gpg"))
                return "encrypted";
            else {
                throw new Exception("Can only process encrypted files ");
            }

        } catch (Exception e) {
            if (marketAndProvider == null) {
                marketAndProvider = new MarketAndProvider("defaultMarket", "defaultProvider");
            }
            // real code does other stuff here too.
            throw new Exception(e);
        }
    }

    private MarketAndProvider extractMarketAndProvider(String filePath) {
        if (StringUtils.isEmpty(filePath))
        {
            return null;
        }
        int pos = filePath.indexOf(',');
        String market = filePath.substring(0,  pos);
        String provider = filePath.substring(pos+1);
        return new MarketAndProvider(market, provider);

    }

    public class MarketAndProvider {
        private final String market;
        private final String provider;

        public MarketAndProvider(String market, String provider) {
            this.market = market;
            this.provider = provider;
        }

        public String getMarket() {
            return market;
        }

        public String getProvider() {
            return provider;
        }
    }
}

Sonarlint (v1.3.1) 在检查实例是否为空的行中抱怨“更改此条件，使其不总是评估为“假””。似乎假设 try 块中没有任何内容被执行，因此它将为空。但是代码达到这一点是有效的，'instance' 被设置为 null 或一个值。

我应该怎么做才能解决这个问题？它是 Sonarlint 中的错误吗？我并不是特别想关闭规则——即使可以，我也不知道该怎么做。

编辑：我已将代码修改为可编译的示例。它是从真实代码中删除的，因此会引发许多其他我不担心的 SonarLint 错误。特别是没有使用marketAndProvider！

谢谢

4

1 回答 1

1

这是影响 Java 插件 3.10 版的符号执行引擎的一个已知错误，SonarLint 1.3.1 本身使用该插件。该问题与以下 JIRA 票证有关：SONARJAVA-1538

该问题将在即将发布的 Java 插件 3.12 版中修复，并将包含在下一版本的 SonarLint 中。

因此，您的代码中没有什么需要修复的，这非常好。问题出在分析仪方面。

于 2016-03-16T09:57:43.577 回答