STRCON.LOOP

ループでの文字列連結の使用は時間とメモリを浪費します。代わりに、StringBuffer を使用してください。

例 1

コピー

9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
     public String test1(int n, String x) {
      String res = "";
      for (int i = 0; i < n; i++) {
        res+=x;
      }
      return res;
    }
    // fixed code
    public String test2(int n, String x) {
      StringBuffer res = new StringBuffer();
      for (int i = 0; i < n; i++) {
        res.append(x);
      }
      return res.toString();
    }

STRCON.LOOP が 12 行目に対して報告されています。ループ内で文字列 'res' に対して append が使用されています。