异常和消息框

Question

我遇到了异常处理问题。我正在做一个 Windows 窗体应用程序来检查社会安全号码的有效性。但是，当我对其进行测试时，系统会接管。 它没有在我的 MessageBox 中显示错误，而是直接显示在代码中（调试模式）。 下面是验证，我的错误检查方法和我调用它的按钮。 感谢您的帮助

private bool Validation(String s, out string message) 
{
    message = "";
    bool res = true;
    if (isNumeric(s) == false)
    {
        message = "Valeur non numérique";
        res = false;
    }
    else if (s.Length > 15)
    {
        message = "Vous avez entré trop de caractères";
        res = false;
    }

    if (s.Length < 15)
    {
        message = "Vous n'avez pas entré assez de caractères";
        res = false;
    }

    int genre = Convert.ToInt32(s.Substring(0, 1));
    if (genre != 1 || genre != 2)
    {
        message = "Genre inconnu";
        res = false;
    }

    int mois = Convert.ToInt32(s.Substring(3, 2));
    if (mois < 1 || mois > 12)
    {
        message = "Le mois entré est incorrect";
        res = false;
    }

    long r = Convert.ToInt64(s.Substring(0, 13));
    int clef = ((int)(97 - (r % 97)));
    int r1 = Convert.ToInt32(s.Substring(13, 2));
    if (clef != r1)
    {
        message = "La clef est incorrecte";
        res = false;
    }
    else
    {
        res = true;
        TxtNumSS.Clear();
    }

    return res;
}

private void btn_Click(object sender, EventArgs e)
{
    string message = string.Empty;
    if( Validation(TxtNumSS.Text,out message) == false)
    {
        MessageBox.Show("error", message, MessageBoxButtons.YesNo);
    }
    else
    {
        MessageBox.Show("Clef valide! ");
    }
}

Answer 1

我怀疑我们是否应该在这里使用任何异常，例行（而且有点乏味）检查就足够了。

让我们重新设计Validation 方法：我们将返回空字符串（无错误）或相应的错误消息：

using System.Linq;

...

private static string ValidationMessage(string s) {
  if (null == s)
    return "null string";
  if (s.Length > 15)
    return "Vous avez entré trop de caractères";
  if (s.Length < 15)
    return "Vous n'avez pas entré assez de caractères";
  if (!s.All(c => c >= '0' && c <= '9')) // all characters in s are [0..9] digits
    return "Valeur non numérique";
  if (s[0] != '1' && s[0] != '2')
    return "Genre inconnu";

  int mois = int.Parse(s.Substring(3, 2));
  
  if (mois < 1 || mois > 12)
    return "Le mois entré est incorrect"; 

  long clef = 97 - long.Parse(s.Substring(0, 13)) % 97;
  long r2 = long.Parse(s.Substring(13, 2));

  if (r2 != clef)
    return "La clef est incorrecte";

  return "";
}

那么我们就可以使用它了：

private void btn_Click(object sender, EventArgs e) {
  string errorMessage = ValidationMessage(TxtNumSS.Text);

  if (string.IsNullOrEmpty(errorMessage)) {
    //TODO: check: should we clear TxtNumSS here?
    TxtNumSS.Clear();
    MessageBox.Show("Clef valide! ");
  }
  else 
    MessageBox.Show("error", errorMessage, MessageBoxButtons.YesNo);  
}