从函数返回数据的最佳实践是什么?是返回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的用户信息。我认为在这种情况下抛出异常是不合适的??另外，我的印象是异常处理会损害性能。
当前回答
如果返回的对象是可以迭代的对象，我将返回一个空对象，这样我就不必首先测试null。
例子:
bool IsAdministrator(User user)
{
var groupsOfUser = GetGroupsOfUser(user);
// This foreach would cause a run time exception if groupsOfUser is null.
foreach (var groupOfUser in groupsOfUser)
{
if (groupOfUser.Name == "Administrators")
{
return true;
}
}
return false;
}
其他回答
这取决于什么对你的案子最有意义。
返回null是否有意义?“不存在这样的用户”?
或者创建一个默认用户有意义吗?当您可以安全地假设如果用户不存在，则调用代码在请求时希望用户存在时，这是最有意义的。
或者，如果调用代码要求使用无效ID的用户，抛出异常(如“FileNotFound”)是否有意义?
然而，从分离关注点/SRP的角度来看，前两点更为正确。从技术上讲，第一个是最正确的(但只差一点点)——GetUserById应该只负责一件事——获取用户。通过返回其他内容来处理自己的“用户不存在”情况可能违反SRP。如果您确实选择抛出异常，则分隔为不同的check - bool DoesUserExist(id)是合适的。
根据下面大量的评论:如果这是一个api级别的设计问题，这个方法可以类似于“OpenFile”或“readwholfile”。我们正在从某个存储库中“打开”一个用户，并从结果数据中补充对象。在这种情况下，一个例外可能是合适的。也许不是，但也有可能。
所有的方法都是可以接受的——这取决于API/应用程序的上下文。
如果您打算表示没有可用的数据，返回null通常是最好的主意。
空对象表示已返回数据，而返回null则明确表示没有返回任何数据。
此外，如果试图访问对象中的成员，返回null将导致null异常，这对于突出显示有bug的代码很有用——试图访问没有任何成员的成员是没有意义的。访问空对象的成员不会失败，这意味着bug不会被发现。
就我个人而言，我使用NULL。它清楚地表明没有数据要返回。但在某些情况下，空对象可能是有用的。
在这种情况下，如果没有这样的用户，则最好返回“null”。还要使你的方法是静态的。
编辑:
通常这样的方法是一些“User”类的成员，不能访问它的实例成员。在这种情况下，方法应该是静态的，否则你必须创建一个“User”的实例，然后调用GetUserById方法，该方法将返回另一个“User”实例。我同意这很令人困惑。但是如果GetUserById方法是某个“DatabaseFactory”类的成员，那么将它作为实例成员是没有问题的。
它将根据上下文而变化，但如果我正在寻找一个特定的对象(如您的示例)，我通常会返回null，如果我正在寻找一组对象，但没有对象，则返回空集合。
如果你在代码中犯了一个错误，返回null会导致空指针异常，那么越早发现越好。如果您返回一个空对象，最初使用它可能会工作，但稍后可能会得到错误。
