小心StringBuilder.toString

最新推荐文章于 2022-07-09 09:55:32 发布

原创最新推荐文章于 2022-07-09 09:55:32 发布 · 1.2k 阅读

2 ·

CC 4.0 BY-SA版权

文章标签：

#java #toString

java 专栏收录该内容

9 篇文章

订阅专栏

本文探讨了在使用StringBuilder进行字符串拼接时出现性能瓶颈的原因，并提供了一种更高效的替代方案。

摘要生成于 C知道，由 DeepSeek-R1 满血版支持，前往体验 >

<span style="font-family: Arial, Helvetica, sans-serif; background-color: rgb(255, 255, 255);">最近看服务器运行的时候，发现CPU使用load特别高，排查了一下，问题出现在下面的代码里面</span>

 StringBuilder buf = new StringBuilder();
 String line = null;
 while ((line = br.readLine()) != null) {
	if (StringUtils.isNotEmpty(buf.toString())) {
		buf.append("\r\n");
	}
	buf.append(line);
}

dump服务器stack的时候，发现代码经常停留在if (StringUtils.isNotEmpty(buf.toString())) 这一行

这是为什么呢？

StringUtils.isNotEmpty是公共方法，应该没什么大问题。

问题就处在buf.toString()上面

public String toString() {
        // Create a copy, don't share the array
    return new String(value, 0, count);
    }

每次toString的时候会新建一个String对象

在看看String的构造方法

 public String(char value[], int offset, int count) {
        if (offset < 0) {
            throw new StringIndexOutOfBoundsException(offset);
        }
        if (count < 0) {
            throw new StringIndexOutOfBoundsException(count);
        }
        // Note: offset or count might be near -1>>>1.
        if (offset > value.length - count) {
            throw new StringIndexOutOfBoundsException(offset + count);
        }
        this.offset = 0;
        this.count = count;
        this.value = Arrays.copyOfRange(value, offset, offset+count);
    }

最后一句 Arrays.copyOfRange(value, offset, offset+count);
会不停的拷贝内存，所以内存不高才怪。
其实代码可以不用这么判断，直接判断

StringBuilder buf = new StringBuilder();
 String line = null;
 while ((line = br.readLine()) != null) {
	if (buf.length()>0) {
		buf.append("\r\n");
	}
	buf.append(line);
}

细节决定成败！理解底层实现原理对于编写高质量代码有很重要的作用！