57

Is there a way to append two string builders? And if so - does it perform better than appending a string to a StringBuilder ?

2

6 Answers 6

40

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).

6
  • 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.
    – Mark Hurd
    Dec 22, 2015 at 0:52
  • 1
    Sorry @MarkHurd I wasn't clear enough: what I meant is, although .ApppendLine is really implemented as you described, it only accepts String 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
  • 7
    under the hood frontStringBuilder.Append(backStringBuilder) calls backStringBuilder.toString() inside. so it's basically frontStringBuilder.Append(backStringBuilder.ToString()) .
    – rovsen
    Jan 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.
    – Mark Hurd
    Jan 28, 2016 at 0:34
24

Just like that....

StringBuilder sb = new StringBuilder();
StringBuilder sb1 = new StringBuilder();
sb.Append(sb1.ToString());
1
  • 51
    That's appending a string to a sb. I thought to append to sb to avoid sealing sb to string
    – Elad Benda
    Jun 23, 2011 at 13:10
19

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]);
}
1
  • 3
    Unfortunately 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 reusable Char[] buffer if the StringBuilder is more than ~4,000 characters long.
    – Dai
    Jun 8, 2020 at 9:01
14

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
  • 1
    Great suggestion and clean understandible coding Jun 10, 2015 at 14:51
5

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);
        }
    }
4

Simply as that:

firstStringBuilder.Append(secondStringBuilder.ToString());
0

Your Answer

Reminder: Answers generated by Artificial Intelligence tools are not allowed on Stack Overflow. Learn more

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Not the answer you're looking for? Browse other questions tagged or ask your own question.