从Java到C＃的这段代码正确吗？

Question

i'm trying to port some Java stuff to C#. I'm just wondering if the following C# code is the equivalent to the original Java source.

Source: Java Code

private static final Pattern SIMPLE_IDENTIFIER_NAME_PATTERN = 
    Pattern.compile("^[a-zA-Z_][a-zA-Z0-9_]*$");

    private static boolean isValidIdentifier(String s) {
        Matcher m = SIMPLE_IDENTIFIER_NAME_PATTERN.matcher(s);
        return (m.matches() && !reserved.contains(s));
    }

Destination: C# Code

private static readonly Regex SIMPLE_IDENTIFIER_NAME_PATTERN = 
    new Regex("^[a-zA-Z_][a-zA-Z0-9_]*$", RegexOptions.Compiled);

private static bool IsValidIdentifier(string s)
{
    Match match = SIMPLE_IDENTIFIER_NAME_PATTERN.Match(s);
    return (match.Success && !Reserved.Contains(s));
}

Cheers :)

Answer 1

根据我的评论，我认为您应该编写一个或多个单元测试以验证端口是否按预期工作。

Answer 2

看起来不错，但是为什么不从移植单元测试开始呢？

Answer 3

Your use of the Caret and Dollar indicates that you want to match embedded newlines in the subject string, as opposed to the beginning and end of the entire string. If so, then you should definitely set the RegexOptions.Multiline option for your Regex. If you do not set that option, your Caret and Dollar will have no special implication.

private static readonly Regex SIMPLE_IDENTIFIER_NAME_PATTERN = new Regex("^[a-zA-Z_][a-zA-Z0-9_]*$", RegexOptions.Compiled | RegexOptions.Multiline);

It may also be worthwhile to evaluate the need for compiling this Regex. Does it need to be used repeatedly (such as, in a loop) ? If not, then your Regex will in fact have lower performance.

Besides this point, your conversion appears to be valid. As some of the others have suggested, the only way to be reasonably sure is to unit test it.

Answer 4

Beware that a readonly type is not immutable. That means that you cannot change which reference of regex you point to, but you can change the regex object itself. (Luckily the contract on the regex won't let you change the expression though)

Beware that .Net regex syntax is not the same as *nix regex syntax, so you may get bitten there. Confirm as per the MSDN docs what you need your string to do:

MSDN Regex Syntax