views:

24

answers:

1

The following code outputs:

http://www.google.com 
http://www.google.com&lang

What is the simplest way to change the code so it outputs:

http://www.google.com 
http://www.google.com&lang=en&param2=this&param3=that

CODE:

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;

namespace TestRegex9928228
{
    class Program
    {
        static void Main(string[] args)
        {
            string text1 = "try out [url=http://www.google.com]this site (http://www.google.com)[/url]";
            Console.WriteLine(text1.ExtractParameterFromBbcodeUrlElement());

            string text2 = "try out [url=http://www.google.com&lang=en&param1=this&param2=that]this site (http://www.google.com)[/url]";
            Console.WriteLine(text2.ExtractParameterFromBbcodeUrlElement());

            Console.ReadLine();
        }
    }

    public static class StringHelpers
    {
        public static string ExtractParameterFromBbcodeUrlElement(this string line)
        {
            if (line == null)
                return "";
            else
            {
                if (line.Contains("]"))
                {
                    List<string> parts = line.BreakIntoParts(']');
                    if (parts[0].Contains("="))
                    {
                        List<string> sides = parts[0].BreakIntoParts('=');
                        if (sides.Count > 1)
                            return sides[1];
                        else
                            return "";
                    }
                    else
                        return "";
                }
                else
                    return "";
            }
        }

        public static List<string> BreakIntoParts(this string line, char separator)
        {
            if (String.IsNullOrEmpty(line))
                return new List<string>();
            else
                return line.Split(separator).Select(p => p.Trim()).ToList();
        }
    }
}
+1  A: 

Simplest or most efficient? You're asking two different questions it seems. Simplest would be something like this:

Change:

List<string> sides = parts[0].BreakIntoParts('=');
if (sides.Count > 1)
   return sides[1];

To:

List<string> sides = parts[0].BreakIntoParts('=');
if (sides.Count > 1)
   return parts[0].Replace(sides[0], "");

Edit: Looks like you changed title to remove "most efficient". Here's the simplest change (fewest lines of code changed) that I see.

chrissr
thanks, that is very simple and works as well, I just had to add an equals sign: return parts[0].Replace(sides[0] + "=", "");
Edward Tanguay