10

根据工具 PMD,以下是一种不好的做法:

String s = "" + 123; // bad 
String t = Integer.toString(456); // ok 


This is an inefficient way to convert any type to a `String`.

为什么这是一件坏事?

4

4 回答 4

22

它效率低下,因为它涉及不需要的字符串连接,因此会创建一个或两个额外的String对象——尽管我相信 JIT 可以将其优化掉。

对我来说,更大的问题是代码不太清楚。调用toString是一个标准的习惯用法,每个 Java 开发人员都可以理解(希望 :-),所以你应该更喜欢这个。

于 2010-09-02T12:13:39.257 回答
14
String s = "" + 123; // bad     
String t = Integer.toString(456);

将编译为:

String s = "123";
String t = Integer.toString(456);

所以:""+123 明显略好!通过JAD检查

public static void main(String args[])
{
//    0    0:ldc1            #16  <String "123">
//    1    2:astore_1
//    2    3:sipush          456
//    3    6:invokestatic    #18  <Method String Integer.toString(int)>
//    4    9:astore_2
//    5   10:getstatic       #24  <Field PrintStream System.out>
//    6   13:new             #30  <Class StringBuilder>
//    7   16:dup
//    8   17:aload_1
//    9   18:invokestatic    #32  <Method String String.valueOf(Object)>
//   10   21:invokespecial   #38  <Method void StringBuilder(String)>
//   11   24:aload_2
//   12   25:invokevirtual   #41  <Method StringBuilder StringBuilder.append(String)>
//   13   28:invokevirtual   #45  <Method String StringBuilder.toString()>
//   14   31:invokevirtual   #48  <Method void PrintStream.println(String)>
//   15   34:return
}

编辑:

对于非常量值:

int i = 123;
String s = (new StringBuilder()).append(i).toString();
String t = Integer.toString(i);
System.out.println((new StringBuilder(String.valueOf(s))).append(t).toString());

    public static void main(String args[])
    {
    //    0    0:bipush          123
    //    1    2:istore_1
    //    2    3:new             #16  <Class StringBuilder>
    //    3    6:dup
    //    4    7:invokespecial   #18  <Method void StringBuilder()>
    //    5   10:iload_1
    //    6   11:invokevirtual   #19  <Method StringBuilder StringBuilder.append(int)>
    //    7   14:invokevirtual   #23  <Method String StringBuilder.toString()>
    //    8   17:astore_2
    //    9   18:iload_1
    //   10   19:invokestatic    #27  <Method String Integer.toString(int)>
    //   11   22:astore_3
    //   12   23:getstatic       #32  <Field PrintStream System.out>
    //   13   26:new             #16  <Class StringBuilder>
    //   14   29:dup
    //   15   30:aload_2
    //   16   31:invokestatic    #38  <Method String String.valueOf(Object)>
    //   17   34:invokespecial   #44  <Method void StringBuilder(String)>
    //   18   37:aload_3
    //   19   38:invokevirtual   #47  <Method StringBuilder StringBuilder.append(String)>
    //   20   41:invokevirtual   #23  <Method String StringBuilder.toString()>
    //   21   44:invokevirtual   #50  <Method void PrintStream.println(String)>
    //   22   47:return
    }
于 2010-09-02T12:23:52.570 回答
6

它扩展为 "" + String.valueOf(yourObject) 并因此进行了不需要的连接。连接涉及分配一个额外的字符串并额外复制字符串的值。

于 2010-09-02T12:11:17.260 回答
0
String s = "" + 123; // bad

上面的代码创建了一个临时字符串,将 "" 和 123 结合起来

于 2010-09-02T12:49:51.947 回答