Question

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

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）抱怨“更改此条件，以便它不会总是在检查实例是否为空的行处评估为”false“。它似乎假设try块中没有任何内容被执行，因此它将为null。但是，对于代码来说，将'instance'设置为null或值是有效的。

我该怎么做才能解决这个问题？这是Sonarlint的一个错误吗？我并不特别想要关闭这条规则 - 即使我可以也不知道如何。

编辑：我已将代码修改为可编译示例。它是从真实代码中删除的，所以抛出了许多其他的SonarLint错误，我不担心这里。特别是那个市场和提供者没有被使用！

由于

Answer 1

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

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

因此，您的代码无需修复，这非常好。问题出在分析仪方面。

如何修复Sonarlint显示的此错误？

1 个答案: