IF 与/或（&&/||）逻辑结构失效问题求助

看起来你在SSN（社会安全号）的格式验证上踩了逻辑判断和正则写法的坑——毕竟两种格式（带短横线/不带短横线）的判断很容易搞混逻辑关系，我来帮你理清楚：

先修正你的正则问题 #

首先看你提供的代码片段，regexSSNNoDashes的正则后面多了三个多余的点，这会导致这个正则永远匹配不到正确的无短横线SSN，先把这个错误改掉：

// 带短横线的SSN正则（正确）
private static readonly Regex regexSSNDashes = new Regex(@"^\d{3}-\d{2}-\d{4}$");
// 去掉多余的三个点，也可以简化成^\d{9}$，效果是一样的
private static readonly Regex regexSSNNoDashes = new Regex(@"^\d{3}\d{2}\d{4}$");

逻辑判断的核心问题 #

你说只有当两个格式布尔值都为true时才能通过验证，这说明你可能用了&&（逻辑与）来判断，但这完全不符合需求——SSN是二选一的格式（要么带短横线，要么不带），所以应该用||（逻辑或）。

为什么之前用||会失效？大概率是上面那个正则的错误导致regexSSNNoDashes.IsMatch()永远返回false，所以只有带短横线的SSN能通过，无短横线的会一直触发错误。

正确的验证逻辑示例 #

public static bool IsValidSSN(string ssnInput)
{
    if (string.IsNullOrWhiteSpace(ssnInput))
        return false;

    bool matchesDashFormat = regexSSNDashes.IsMatch(ssnInput);
    bool matchesNoDashFormat = regexSSNNoDashes.IsMatch(ssnInput);

    // 只要符合两种格式中的任意一种，就判定为有效
    return matchesDashFormat || matchesNoDashFormat;
}

额外优化建议 #

其实你可以把两个正则合并成一个，用可选的短横线语法，这样代码更简洁，也避免了逻辑判断的麻烦：

// 一个正则搞定两种格式，-? 表示短横线可选
private static readonly Regex regexSSN = new Regex(@"^\d{3}-?\d{2}-?\d{4}$");

public static bool IsValidSSN(string ssnInput)
{
    return !string.IsNullOrWhiteSpace(ssnInput) && regexSSN.IsMatch(ssnInput);
}

如果需要更严格的SSN验证（比如排除全0、前三位为000/666等无效情况），还可以在正则里补充这些规则，比如：

private static readonly Regex regexStrictSSN = new Regex(@"^(?!000|666)[0-9]{3}-?(?!00)[0-9]{2}-?(?!0000)[0-9]{4}$");

内容的提问来源于stack exchange，提问作者user8716700