Replace Case Insensitive StringBuilder

Question

I have read many times over about the virtue of using Stringbuilder vs. string when multiple replacements are needed. I need to convert an HTML string (all very simple formatting) to WIKI notation and was going to roll my own from code I found on SO. I'm wondering if there is a simpler way to do this, and also how I can implement the benefits of a StringBuilder with it. All my attempts til now have failed miserably. Please advise and thank you in advance.

    public static string ConvertHTML2WIKInotation(string htmlstring)
        {
            StringBuilder sb = new StringBuilder(htmlstring);

            htmlstring = ReplaceInsensitive(htmlstring, "<hr>", "----");
            htmlstring = ReplaceInsensitive(htmlstring, "<hr />", "----");
            ... etc.

            return sb.ToString();
        }

        static public string ReplaceInsensitive(this string str, string from, string to)
        {
            str = Regex.Replace(str, from, to, RegexOptions.IgnoreCase);
            return str;
        }

You'd be better off using a html parser. – juharr Sep 14 '16 at 17:51 — juharr, Sep 14 '16 at 17:51

score 1 · Accepted Answer · answered Sep 14 '16 at 18:01

1

Since html tags case insensitive, I would suggest non StringBuilder since SB is not recommended for the scenarios where you need more replace string.

public static string ReplaceMultiple(
          string template, 
          IEnumerable<KeyValuePair<string, string>> replaceParameters)
{
   var result = template;

   foreach(var replace in replaceParameters)
   {
      var templateSplit = Regex.Split(result, 
                                    replace.Key, 
                                    RegexOptions.IgnoreCase);
      result = string.Join(replace.Value, templateSplit);
   }

   return result;
}

answered Sep 14 '16 at 18:01

RAJA.P.N

106
8

I'm sorry, I'm not well versed on IEnumerables, how do you go about creating the replaceParameters parameter? – Darkloki Sep 14 '16 at 18:07
Try this: replaceParameters = new [] { new KeyValuePair("
","----"), new KeyValuePair("
","----") }; – RAJA.P.N Sep 14 '16 at 18:11

score 0 · Answer 2 · edited May 23 '17 at 12:15

0

The benefit of stringbuilder is that you don't create a new string with every replacement. Your implementation doesn't benefit from this property. If you use a stringbuilder, actually do transformations on the stringbuilder, not on the input string parameter:

    public static string ConvertHTML2WIKInotation(string htmlstring)
    {
        StringBuilder sb = new StringBuilder(htmlstring);

        sb.Replace("<hr>", "----");
        sb.Replace("<hr />", "----");

        return sb.ToString();
    }

If it needs to be case insensitive, look at: How to do a multiple case insensitive replace using a StringBuilder

edited May 23 '17 at 12:15

Community

1
1

answered Sep 14 '16 at 17:49

Norbert Huurnink

1,326
10
18

1

This won't handle the case insensitive requirement. – juharr Sep 14 '16 at 17:52
thank you for the quick response. How does this help me with case-insensitivity though? – Darkloki Sep 14 '16 at 17:53
It doesn't. In your case, unless you do it 10000 times, I would use string.replace instead. – Norbert Huurnink Sep 14 '16 at 17:55
If you don't want that, look at the link I posted. – Norbert Huurnink Sep 14 '16 at 17:57

Replace Case Insensitive StringBuilder

2 Answers2