I recently encountered an idiom I haven't seen before: string assembly by StringWriter
and PrintWriter
. I mean, I know how to use them, but I've always used StringBuilder
. Is there a concrete reason for preferring one over the other? The StringBuilder
method seems much more natural to me, but is it just style?
I've looked at several questions here (including this one which comes closest: StringWriter or StringBuilder ), but none in which the answers actually address the question of whether there's a reason to prefer one over the other for simple string assembly.
This is the idiom I've seen and used many many times: string assembly by StringBuilder:
public static String newline = System.getProperty("line.separator");
public String viaStringBuilder () {
StringBuilder builder = new StringBuilder();
builder.append("first thing").append(newline); // NOTE: avoid doing builder.append("first thing" + newline);
builder.append("second thing").append(newline);
// ... several things
builder.append("last thing").append(newline);
return builder.toString();
}
And this is the new idiom: string assembly by StringWriter and PrintWriter:
public String viaWriters() {
StringWriter stringWriter = new StringWriter();
PrintWriter printWriter = new PrintWriter(stringWriter);
printWriter.println("first thing");
printWriter.println("second thing");
// ... several things
printWriter.println("last thing");
printWriter.flush();
printWriter.close();
return stringWriter.toString();
}
Edit It appears that there is no concrete reason to prefer one over the other, so I've accepted the answer which best matches my understanding, and +1ed all the other answers. In addition, I posted an answer of my own, giving the results of the benchmarking I ran, in response to one of the answers. Thanks to all.
Edit again It turns out that there is a concrete reason to prefer one (specifically the StringBuilder) over the other. What I missed the first time was the addition of the newline. When you add a newline (as above, as a separate append), it's slightly faster - not hugely, but coupled with the clarity of intent, it's definitely better. See my answer below for the improved timings.
builder.append("first thing" + newline)
. The problem is that there's an extra String allocation which is not needed (it first creates "first thing\n" and then copies the string data to the builder). Instead, do it like this:builder.append("first thing").append(newline)
. This directly copies"first thing"
to the builder and thennewline
. Of couse, StringBuilder should only be used if the number of things is not constant (e.g. it's in a loop), javac already optimizes"a" + "b" + "c"
intonew StringBuilder().append("a").append("b").append("c").toString()
.Throwable.printStackTrace()
requires a writer.