如何在实体框架中处理 System.InvalidOperationException？

Question

我是 asp.net Web API 的新手。我做了一个功能，应该验证前端发送数据的用户，然后我在数据库中搜索数据。但是当找不到帐户时，我总是遇到异常我应该如何处理该异常以发送到前端信息 当第一个 if 语句不正确时我应该返回什么，因为 null 不起作用。

public UserData ByPassword(string emailAddress, string password)
        {
            if (emailAddress != null && password != null)
            {
                    Account account = db.Accounts.Where(acc => acc.AccMail == emailAddress && acc.AccPassword == password.ToLower()).Single();
                    string token = OurAuthorizationAttribute.CreateTicket(account.AccID, false);
                    UserData data = new UserData();
                    data.Id = account.AccID;
                    data.Token = token;
                    return data;

            }

她我也添加了 try 和 catch 块，但仍然是同样的问题。

public UserData ByPassword(string emailAddress, string password)
        {
            if (emailAddress != null && password != null)
            {
                try
                {
                    Account account = db.Accounts.Where(acc => acc.AccMail == emailAddress && acc.AccPassword == password.ToLower()).Single();
                    string token = OurAuthorizationAttribute.CreateTicket(account.AccID, false);
                    UserData data = new UserData();
                    data.Id = account.AccID;
                    data.Token = token;
                    return data;
                }
                catch
                {
                    throw new OurException(OurExceptionType.InvalidCredentials);
                }
            }
             throw new OurException(OurExceptionType.InvalidCredentials);
        }

Answer 1

System.InvalidOperationException 表示编程错误。您可以通过修复代码来处理它。

在这种特殊情况下，错误出现在这一行：

Account account = db.Accounts.Where(acc => acc.AccMail == emailAddress && acc.AccPassword == password.ToLower()).Single();

您的代码假设Accounts 必须包含任何{emailAddress, password} 对的记录，这是不正确的。用SingleOrDefault 替换Single 将使异常消失。当然，您需要对结果进行空值检查以查看记录是否存在。

以下是更改代码的方法：

public UserData ByPassword(string emailAddress, string password) {
    // null-check your arguments to detect programming errors in the "upstream" code
    if (emailAddress == null) throw new ArgumentNullException("emailAddress");
    if (password == null) throw new ArgumentNullException("password");
    // Now that your arguments are "sanitized", fix the Single() call
    Account account = db.Accounts.Where(acc => acc.AccMail == emailAddress && acc.AccPassword == password.ToLower()).SingleOrDefault();
    // Missing credentials is not a programming error - throw your specific exception here:
    if (account == null) {
        throw new OurException(OurExceptionType.InvalidCredentials);
    }
    string token = OurAuthorizationAttribute.CreateTicket(account.AccID, false);
    UserData data = new UserData();
    data.Id = account.AccID;
    data.Token = token;
    return data;
}

注意：虽然上述更改将修复编码错误，但它不会解决以纯文本形式存储密码的主要设计缺陷。有关在数据库中存储密码的深入讨论，请参阅 this question。