Regex is not matching (for url)

Question

here we go

private static readonly string _regexPattern = @"[/]api[/]v[0-9].[0-9][/]Subscriber[/][0-9]{10}";

public override string GenerateRowKey(LogEvent logEvent, string suffix = null)
{
    var logResult = $"{logEvent.Properties["RequestPath"]}";

    var regex = new Regex(logResult);
    var fh = regex.IsMatch(_regexPattern);

    ---
}

logResult is "/api/v1.0/Subscriber/2727272727"

Have no idea why this doesn't work.

"Have no idea why this doesn't work" -- Well what is it supposed to do, we cant read your mind. We need a problem statement, sample input/output, etc — maccettura
– maccettura, Commented Sep 11, 2018 at 14:23
According to online regex tester like regexstorm.net/tester your regex works. — hakamairi
– hakamairi, Commented Sep 11, 2018 at 14:27
@maccettura regex is not match. If I had more info about what exactly isn't working I wouldn't ask for help — user9314289
– user9314289, Commented Sep 11, 2018 at 14:27

Alex K. · Accepted Answer · 2018-09-11 14:34:17Z

2

Your passing the pattern/text in the wrong order, the ctor wants a pattern for later testing with IsMatch():

var regex = new Regex(_regexPattern);
var fh = regex.IsMatch(logResult);

There is also a static IsMatch() thats less to type & uses an internal cache.

edited Sep 11, 2018 at 14:34

answered Sep 11, 2018 at 14:29

Alex K.

177k32 gold badges276 silver badges299 bronze badges

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

4 Comments

hakamairi Over a year ago

regex.IsMatch(logResult)?

user9314289 Over a year ago

THank you so much. Fully me

DavidG Over a year ago

@hakamairi Yup, I've fixed it

Alex K. Over a year ago

You may want to anchor your pattern with ^/$ to avoid substring matches too.

Collectives™ on Stack Overflow

Regex is not matching (for url)

1 Answer 1

4 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

4 Comments

Your Answer

Sign up or log in

Post as a guest

Related