Sonar抱怨:使用StringBuilder而不是StringBuffer

时间:2016-08-10 15:21:32

标签: java append sonarqube stringbuilder stringbuffer

我目前正在做一段必须能够匹配某些正则表达式的代码,并在与正则表达式匹配的字符串中进行一些替换。

为了做到这一点,我在java中使用matcher对象。但 正如你可以在互联网上看到的那样,所有的例子都使用StringBuffer来使用appendreplacement和appendtail方法(oracleDoc)进行替换。

但是当我推送我的代码时,Sonar抱怨使用stringbuffer而不是stringbuilder。

  • 在这种情况下,Sonar发出的警告是错误吗?
  • 是否有任何库使用stringbuilder执行相同操作?

显然有些开发人员抱怨here

我找到了一种不使用StringBuffer并使用StringBuilder的方法,但我确信这不如使用StringBuffer有效(并且可能是一种不好的做法)。您可以在以下示例代码的junit测试中复制粘贴:

    String entry = "Actual 4.11-6 and 13-5";
    String expectedReturn = "Actual 4*11^(-6) and 13^(-5)";

    String number = "(^|\\s)-?\\d+((\\.||,)\\d+){0,1}(.\\d+){0,1}-\\d+";
    Pattern pattern = Pattern.compile(number);
    Matcher matcher = pattern.matcher(entry);

    //USING STRING BUFFER
    StringBuffer stringBuffer = new StringBuffer();
    String substring;
    while(matcher.find()){
        substring = matcher.group(0);
        matcher.appendReplacement(stringBuffer,substring.replace(".","*").replace("-","^(-")+")");
    }
    matcher.appendTail(stringBuffer);

    //USING STRING BUILDER
    matcher = pattern.matcher(entry);
    int lastIndex = 0;
    StringBuilder stringBuilder = new StringBuilder();
    while(matcher.find()){
        stringBuilder.append(entry.substring(lastIndex,matcher.start()));
        substring = matcher.group(0);
        stringBuilder.append(substring.replace(".","*").replace("-","^(-")+")");
        lastIndex = matcher.end();
    }
    stringBuilder.append(entry.substring(lastIndex,entry.length()));

    Assert.assertEquals(expectedReturn,stringBuffer.toString());
    Assert.assertEquals(expectedReturn,stringBuilder.toString());

信息:管理stringbuilder for append的匹配器将在jdk 9 source codecode review

1 个答案:

答案 0 :(得分:5)

这是一个警告,而不是错误。声纳的工作是警告你StringBuilder更可取because it is faster。如果API强制您使用StringBuffer,那么我会使用它并使警告静音。

我认为Pattern和两个缓冲区的真正潜在问题是StringBufferStringBuilder不共享用于构建字符串的特定接口(Appendable更通用接口,我认为他们应该共享一个更具体的接口来构造字符串,比如StringConstructor),这将允许你通过零努力(通常)切换实现。