How to concatenate several MvcHtmlString instances

M4N picture M4N · Aug 11, 2010 · Viewed 28.3k times · Source

I have some doubts about how to concatenate MvcHtmlString instances because of this information found in MSDN :

MvcHtmlString Class Represents an HTML-encoded string that should not be encoded again

Do I risk that strings are HTML-encoded twice when using code like this:

var label = Html.LabelFor(model => model.Email);
var textbox = Html.TextBoxFor(model => model.Email);
var validation = Html.ValidationMessageFor(model => model.Email);

var result = MvcHtmlString.Create(
  label.ToString() + textbox.ToString() + validation.ToString());

(note: this is supposed to go into an HtmlHelper extension method to reduce code-duplication in views).

Answer

Ryan picture Ryan · Aug 12, 2010

Too bad C# won't let us override the + operator here! How about using an extension method instead?

public static MvcHtmlString Concat(this MvcHtmlString first, params MvcHtmlString[] strings)
{
    return MvcHtmlString.Create(first.ToString() + string.Concat(strings.Select(s => s.ToString())));
}

This could probably be optimized, but you can run with it. It should be fairly trivial to prove that this doesn't double encode strings with a unit test.

Usage sample:

label.Concat(textbox, validation)   

And now a shameless plug for my blog: Use TagBuilder or HtmlTags to clean up your HTML