2

我有一个字符串，我必须在其中检查一些 if 条件并执行一些替换操作，如下所示，

for(int i =0; i<10;i++){
    if(name[i].contains("sometext")){
       name[i] = name[i].replaceAll("someregexpattern","replacementtexthere");
    }
}

而不是每次使用'if'条件检查'sometext'的存在，我可以简单地说如下，

for(int i =0; i<10;i++){
       name[i] = name[i].replaceAll("someregexpattern","replacementtexthere");
}

无论如何，如果字符串 'name' 不包含 'regexpattern' 的意思，它不会替换任何东西。那么，就性能而言，if 条件是否具有任何意义。

提前致谢。

4

3 回答 3

2

仅当sometext与 相同时才有效someregexpattern。否则，if条件确实很重要。

假设您someregexpattern确实存在于字符串中，但您的if条件评估为false，在这种情况下，您不应该进行替换。但是如果你删除if语句，那么替换就会发生，这并不完全相同。

例如：-帮助您更好地理解。

String testCondition = "dontReplace";
        String myString = "Hi, I'm SO";
        String regex = "SO";
        String replacement = "Not SO";

        if(myString.contains(testCondition)){
            myString = myString.replaceAll(regex, replacement); // stays "Hi, I'm SO"
        }

        myString = myString.replaceAll(regex, replacement); // Becomes "Hi, I'm Not SO"

如您所见，要求是String不应该发生替换，因为条件是dontReplace，但由于if被删除，替换发生了，即使它不应该发生。

于 2013-03-26T10:01:26.737 回答
0

“在性能方面的任何意义”还取决于整个事情在您的软件中执行的频率。

分析器运行（通过visualvm左右）将显示您的大部分执行时间是否花费在String#replaceAll(). 如果不是，您将无法通过优化获得显着改进。这听起来很明显，但它经常被遗忘。

如果优化不会带来显着的收益，那么选择更易读的代码变体几乎总是会更好。

于 2013-03-26T10:28:45.410 回答
0

“如果”本身不会影响性能。尽管现在函数调用非常便宜，但 String.contains() 会多做 4-5 次函数调用。

但是，前一种变体可读性更好，并且不易出错。

于 2013-03-26T10:07:51.923 回答