Is there a way to append two string builders? And if so - does it perform better than appending a string to a StringBuilder ?
6 Answers
I know this is three years later, but the .NET 4 StringBuilder
behaves differently anyway.
Nevertheless, it does still come back to "what do you want to do?" Are you looking for simply the most performant way of appending two StringBuilders
and continuing on with just the latter result? Or are you expecting to continue working with the existing buffered value of the appended StringBuilder
?
For the former, and always in .NET 4,
frontStringBuilder.Append(backStringBuilder);
is best.
For the latter scenario in .NET 2/3.5,
frontStringBuilder.Append(backStringBuilder.ToString(0, backStringBuilder.Length));
is best (and won't hurt performance in .NET 4).
-
Very clear answer. I thought it could be worth adding that .AppendLine() does NOT provide the same flexibility, in case anyone wonders. Dec 22, 2015 at 0:14
-
@CharlesRobertoCanato I'm not sure what you mean:
.AppendLine
is implemented as.Append(string)
followed by.Append(NewLine)
for both .NET 2.0/3.5 and 4, which means both of the scenarios above are the same. Dec 22, 2015 at 0:52 -
1Sorry @MarkHurd I wasn't clear enough: what I meant is, although
.ApppendLine
is really implemented as you described, it only acceptsString
as a parameter in any .NET versions. So, even in .NET 4, you can't just do a.AppendLine( backStringBuilder )
- what I meant with "lacking flexibility". Not a problem, just a note - it's only an extra method call, anyway. Dec 23, 2015 at 3:23 -
7under the hood
frontStringBuilder.Append(backStringBuilder)
callsbackStringBuilder.toString()
inside. so it's basicallyfrontStringBuilder.Append(backStringBuilder.ToString())
.– rovsenJan 26, 2016 at 10:41 -
2@rovsen Yes, that's what I had until Nigrimmist pointed out that it would be better for future possibilities of a
StringBuilder
direct overload were to become available. Jan 28, 2016 at 0:34
Just like that....
StringBuilder sb = new StringBuilder();
StringBuilder sb1 = new StringBuilder();
sb.Append(sb1.ToString());
-
51That's appending a string to a sb. I thought to append to sb to avoid sealing sb to string Jun 23, 2011 at 13:10
This will do it without allocations
StringBuilder sb = new StringBuilder("aaaa");
StringBuilder second = new StringBuilder("bbbbb");
sb.EnsureCapacity(sb.Length + second.Length);
for (int i = 0; i < second.Length; i++)
{
sb.Append(second[i]);
}
-
3Unfortunately the indexer property in
StringBuilder
is non-trivial and actually iterates over each "chunk" inside the StringBuilder for every access, making this somewhat expensive to use in a loop. For my implementation I used a reusableChar[]
buffer if theStringBuilder
is more than ~4,000 characters long.– DaiJun 8, 2020 at 9:01
You don't need to call .ToString()
. You should simply append one to another. That's all. It will be better against direct .ToString()
call for next reason :
1) StringBuilder does not have constructor with StringBuilder
as a param, string, int/string etc. StringBuilder
overriding .ToString()
and as a result :
StringBuilder sb1 = new StringBuilder("1");
sb1.Append(new StringBuilder("2"));
Console.WriteLine(sb1);
that code will call overrided version of .ToString()
automatically. Output will be "12"
;
2)If StringBuilder
will be added as incoming param to StringBuilder
constructor's in next framework versions, your code will be clear and ready for correct appending without any refactoring.
Have a good day!
-
1
If the StringBuilder is large, then this will minimize string-allocations (especially if you can provide a reusable char-buffer):
public static void CopyTo(this StringBuilder source, StringBuilder dest)
{
char[] buffer = new char[Math.Min(source.Length, 1024)];
CopyTo(source, dest, buffer);
}
public static void CopyTo(this StringBuilder source, StringBuilder dest, char[] buffer)
{
dest.EnsureCapacity(dest.Length + source.Length);
for (int i = 0; i < source.Length; i += buffer.Length)
{
int charCount = Math.Min(source.Length - i, buffer.Length);
source.CopyTo(i, buffer, 0, charCount);
dest.Append(buffer, 0, charCount);
}
}