C# remove empty url parameters regex

Question

I am trying to remove empty url type parameters from a string using C#. My code sample is here.

    public static string test ()
    {
        string parameters = "one=aa&two=&three=aaa&four=";
        string pattern = "&[a-zA-Z][a-zA-Z]*=&";
        string replacement = "";
        Regex rgx = new Regex(pattern);
        string result = rgx.Replace(parameters, replacement);

        return parameters;
    }

    public static void Main(string[] args)
    {
        Console.WriteLine(test());
    }

I tried the code in rextester

output: one=aa&two=&three=aaa&four=

expected output: one=aa&three=aaa

Why use Regex? You could just use HttpUtility.ParseQueryString — maccettura
– maccettura, Commented Jan 25, 2018 at 14:56

maccettura · Accepted Answer · 2018-01-25 15:06:02Z

3

You absolutely do not need to roll your own Regex for this, try using HttpUtility.ParseQueryString():

public static string RemoveEmptyUrlParameters(string input)
{
    var results = HttpUtility.ParseQueryString(input);

    Dictionary<string, string> nonEmpty = new Dictionary<string, string>();
    foreach(var k in results.AllKeys)
    {
        if(!string.IsNullOrWhiteSpace(results[k]))
        {
            nonEmpty.Add(k, results[k]);
        }
    }

    return string.Join("&", nonEmpty.Select(kvp => $"{kvp.Key}={kvp.Value}"));
}

Fiddle here

answered Jan 25, 2018 at 15:06

maccettura

10.9k3 gold badges31 silver badges37 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

SamWhan · Accepted Answer · 2018-01-25 15:11:28Z

1

Regex:

(?:^|&)[a-zA-Z]+=(?=&|$)

This matches start of string or an ampersand ((?:^|&)) followed by at least one (english) letter ([a-zA-Z]+), an equal sign (=) and then nothing, made sure by the positive look-ahead ((?=&|$)) which matches end of string or a new parameter (started by &).

Code:

public static string test ()
{
    string parameters = "one=aa&two=&three=aaa&four=";
    string pattern = "(?:^|&)[a-zA-Z]+=(?=&|$)";
    string replacement = "";
    Regex rgx = new Regex(pattern);
    string result = rgx.Replace(parameters, replacement);

    return result;
}

public static void Main(string[] args)
{
    Console.WriteLine(test());
}

Note that this also returns the correct variable (as pointed out by Joel Anderson)

See it live here at ideone.

edited Jan 25, 2018 at 15:11

answered Jan 25, 2018 at 15:02

SamWhan

8,3621 gold badge21 silver badges46 bronze badges

1 Comment

Dracke Over a year ago

thank you that worked, will accept your answer when it will be available

Joel Anderson · Accepted Answer · 2018-01-25 15:00:47Z

0

The results of the Regex replace is not returned by the function. The function returns the variable "parameters", which is never updated or changed.

string parameters = "one=aa&two=&three=aaa&four=";
...
string result = rgx.Replace(parameters, replacement);
return parameters;
....

Perhaps you meant

return results;

answered Jan 25, 2018 at 15:00

Joel Anderson

5358 silver badges22 bronze badges

Collectives™ on Stack Overflow

C# remove empty url parameters regex

3 Answers 3

Comments

1 Comment

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related