从函数返回数据的最佳实践是什么?是返回Null对象好还是返回空对象好?为什么要选择一种而不是另一种呢?
考虑一下:
public UserEntity GetUserById(Guid userId)
{
//Imagine some code here to access database.....
//Check if data was returned and return a null if none found
if (!DataExists)
return null;
//Should I be doing this here instead?
//return new UserEntity();
else
return existingUserEntity;
}
让我们假设在这个程序中存在有效的情况，即数据库中没有具有该GUID的用户信息。我认为在这种情况下抛出异常是不合适的??另外，我的印象是异常处理会损害性能。
当前回答
我同意这里的大多数帖子，它们趋向于零。
我的理由是，生成一个具有非空属性的空对象可能会导致错误。例如，具有int ID属性的实体的初始值为ID = 0，这是一个完全有效的值。如果这个对象，在某些情况下，被保存到数据库中，这将是一件坏事。
对于任何带有迭代器的东西，我总是使用空集合。类似的
foreach (var eachValue in collection ?? new List<Type>(0))
在我看来是代码的味道。集合属性永远不应该为空。
An edge case is String. Many people say, String.IsNullOrEmpty isn't really necessary, but you cannot always distinguish between an empty string and null. Furthermore, some database systems (Oracle) won't distinguish between them at all ('' gets stored as DBNULL), so you're forced to handle them equally. The reason for that is, most string values either come from user input or from external systems, while neither textboxes nor most exchange formats have different representations for '' and null. So even if the user wants to remove a value, he cannot do anything more than clearing the input control. Also the distinction of nullable and non-nullable nvarchar database fields is more than questionable, if your DBMS is not oracle - a mandatory field that allows '' is weird, your UI would never allow this, so your constraints do not map. So the answer here, in my opinion is, handle them equally, always.
Concerning your question regarding exceptions and performance: If you throw an exception which you cannot handle completely in your program logic, you have to abort, at some point, whatever your program is doing, and ask the user to redo whatever he just did. In that case, the performance penalty of a catch is really the least of your worries - having to ask the user is the elephant in the room (which means re-rendering the whole UI, or sending some HTML through the internet). So if you don't follow the anti-pattern of "Program Flow with Exceptions", don't bother, just throw one if it makes sense. Even in borderline cases, such as "Validation Exception", performance is really not an issue, since you have to ask the user again, in any case.
其他回答
如果返回类型是数组，则返回空数组，否则返回null。
如果您打算表示没有可用的数据，返回null通常是最好的主意。
空对象表示已返回数据，而返回null则明确表示没有返回任何数据。
此外，如果试图访问对象中的成员，返回null将导致null异常，这对于突出显示有bug的代码很有用——试图访问没有任何成员的成员是没有意义的。访问空对象的成员不会失败，这意味着bug不会被发现。
还有一种方法涉及传入一个回调对象或委托，它将对值进行操作。如果没有找到值，则不调用回调。
public void GetUserById(Guid id, UserCallback callback)
{
// Lookup user
if (userFound)
callback(userEntity); // or callback.Call(userEntity);
}
当您希望避免在整个代码中进行空检查时，以及当找不到值并不是错误时，这种方法非常有效。如果需要任何特殊处理，还可以在没有找到对象时提供回调。
public void GetUserById(Guid id, UserCallback callback, NotFoundCallback notFound)
{
// Lookup user
if (userFound)
callback(userEntity); // or callback.Call(userEntity);
else
notFound(); // or notFound.Call();
}
使用单个对象的相同方法如下所示:
public void GetUserById(Guid id, UserCallback callback)
{
// Lookup user
if (userFound)
callback.Found(userEntity);
else
callback.NotFound();
}
从设计的角度来看，我真的很喜欢这种方法，但是它的缺点是在不支持第一类函数的语言中使调用站点变得更庞大。
我们使用CSLA。NET，并且它认为失败的数据获取应该返回一个“空”对象。这实际上是相当烦人的，因为它要求检查obj。IsNew而不是obj == null。
正如前面提到的，null返回值将导致代码立即失败，从而降低了由空对象引起的隐形问题的可能性。
就我个人而言，我认为null更优雅。
这是一种非常常见的情况，我很惊讶这里的人似乎对此感到惊讶:在任何web应用程序中，数据通常是使用querystring参数获取的，这显然会被破坏，因此要求开发人员处理“未找到”的情况。
你可以这样处理:
if (User.Exists(id)) {
this.User = User.Fetch(id);
} else {
Response.Redirect("~/notfound.aspx");
}
.．.但是这每次都是对数据库的额外调用，这在高流量页面上可能是一个问题。而:
this.User = User.Fetch(id);
if (this.User == null) {
Response.Redirect("~/notfound.aspx");
}
.．.只需要一个呼叫。
我个人会返回null，因为这是我所期望的DAL/Repository层的操作方式。
如果它不存在，不要返回任何可以被解释为成功获取对象的东西，null在这里工作得很好。
最重要的是在你的DAL/Repos层保持一致，这样你就不会对如何使用它感到困惑。
