如何使我的if陈述较少矛盾？

Question

我有一个正在处理密码强度检查器的积分系统。

当前，规则像以下内容一样工作：+5如果用户条目的案例字母较低，则+10如果其具有较低和上限的字母，数字或特殊字符等。

我遇到的问题是，我的if语句是矛盾的，如果我可以使用任何替代方案，我正在徘徊。

下面的陈述是我试图获得它，因此，如果他们放置了一定类型的角色，他们将获得积分，并且它将有助于分数，即如果有数字，则得分为+5。

if (userPassword.Any(char.IsDigit))
{
    points += 5;
}

if (userPassword.Any(char.IsLower))
{
    points += 5;
}

if (userPassword.Any(char.IsUpper))
{
    points += 5;
}

if (userPassword.Any(c => specialCharacters.Contains(c)))
{
    points += 5;
}

为了摆脱矛盾，我试图在可能扣除用户点的语句上执行所有功能，但似乎不足，我认为这对于该程序并不适合。

如果用户仅使用一种类型的字符，我基本上是在尝试将其扣除点，即如果他们只使用数字而没有其他东西会失去积分。

if (userPassword.All(char.IsLower) || userPassword.All(char.IsUpper)) 
{
    points -= 5;
}

if (userPassword.All(char.IsDigit)) 
{
    points -= 5;
}

if (userPassword.All(c => specialCharacters.Contains(c))) 
{
    points -= 5;
}

解决这个问题是什么选择？

编辑： 好的，谢谢您澄清我的if语句没有任何问题。现在，我想添加一个IF语句，如果其中包含大型情况，较低的情况，数字和特殊字符，则添加10点。我试图使用.contain语句尝试并将其与其他添加if语句区分开用于绕过它。

if (userPassword.Contains(char.IsUpper) &&
    userPassword.Contains(char.IsLower) &&
    userPassword.Contains(char.IsDigit) &&
    userPassword.Contains(c => specialCharacters.Contains(c)))
{
    points += 10;
}

Answer 1

如果我正确理解，则需要适合“所有大写”等有条件的密码来减去点，同时也没有添加单个大写的积分。像目前一样，在您的示例中，“ ABC”将获得+5，因为一个小写，然后将所有小写的小写损失为5，而晚上为0，而不是-5。

我会在开始时将您的点递减条件放入单个IF语句中，然后将您的所有积分添加条件都放在if语句中，以在其他语句中：

if (userPassword.All(char.IsLower) ||
  userPassword.All(char.IsUpper) ||
  userPassword.All(char.IsDigit) ||
  userPassword.All(c => specialCharacters.Contains(c)) {
    points -= 5;
  } else {
      if (userPassword.Any(char.IsDigit)) {
        points += 5;
      }
      if (userPassword.Any(char.IsLower)) {
        points += 5;
      }
      if (userPassword.Any(char.IsUpper)) {
        points += 5;
      }
      if (userPassword.Any(c => specialCharacters.Contains(c))) {
        points += 5;
      }
    }

另外，如果您需要分开这些点赋予的条件以允许不同条件下的点扣除值的不同值，请使用更多的if if语句对不良情况使用：

if (userPassword.All(char.IsLower) || userPassword.All(char.IsUpper)) {
  points -= 5;
} else if (userPassword.All(char.IsDigit)) {
  points -= 5;
} else if (userPassword.All(c => specialCharacters.Contains(c))) {
  points -= 5;
} else {
  if (userPassword.Any(char.IsDigit)) {
    points += 5;
  }
  if (userPassword.Any(char.IsLower)) {
    points += 5;
  }
  if (userPassword.Any(char.IsUpper)) {
    points += 5;
  }
  if (userPassword.Any(c => specialCharacters.Contains(c))) {
    points += 5;
  }
}

Answer 2

将您的“矛盾”检查与相关的“确认”检查结合在一起。例如，在测试包含数字的测试时，还要确保它们不是全部数字：

if( userPassword.Any(char.isDigit)
    && !userPassword.All(char.isDigit) )
{
    points += 5;
}
else /*if(userPassword.All(char.isDigit)*/ // if desired
{
    points -= 5; // if appropriate
}

现在让我们讨论重组您的算法。我们可以创建一个可以用来定义检查的类，而不是定义或的长链，而语句，

public class PasswordEvaluator
{
    public Func<string, bool> TestCondition { get; init set; }
    public int AffirmativeResultValue { get; init set; }
    public int NegativeResultValue { get; init set; }

    public int Evaluate(string password) =>
        TestCondition(password)
            ? AffirmativeResultValue 
            : NegativeResultValue ;
}

则可以定义测试。使用上述解决方案：

var hasDigitEvaluator = new PasswordEvaluator
{
    Condition = (string password) => password.Any(char.isDigit)
        && !password.All(char.isDigit),
    AffirmativeResultValue = 5,
    NegativeResultValue = -5, // or 0, whatever
};

然后，您可以使用此对象执行操作：

points += hasDigitEvaluator.Evalutate(userPassword);

现在，要处理多个评估器，您可以组合iEnumerable＆lt; passwordEvaluator＆gt;，然后在其上调用单个操作以获取您的聚合结果：

var passwordEvaluators = new[]
{
    new PasswordEvaluator { ... },
    new PasswordEvaluator { ... },
    ...
};

var points = passwordEvaluators
    .Select(test => test.Evaluate(userPassword))
    .Sum();