从函数返回数据的最佳实践是什么?是返回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的用户信息。我认为在这种情况下抛出异常是不合适的??另外，我的印象是异常处理会损害性能。
当前回答
这是一个业务问题，取决于具有特定Guid Id的用户的存在是否是该函数的预期正常用例，还是会阻止应用程序成功完成该方法向用户对象提供的任何功能的异常情况……
如果它是一个“异常”，因为缺少具有该Id的用户将阻止应用程序成功完成它正在做的任何功能，(假设我们正在为我们已向其发货的客户创建发票……)，那么这种情况应该抛出ArgumentException(或其他一些自定义异常)。
如果没有用户(调用此函数的潜在正常结果之一)，则返回空值....
编辑:(处理亚当在另一个回答中的评论)
If the application contains multiple business processes, one or more of which require a User in order to complete successfully, and one or more of which can complete successfully without a user, then the exception should be thrown further up the call stack, closer to where the business processes which require a User are calling this thread of execution. Methods between this method and that point (where the exception is being thrown) should just communicate that no user exists (null, boolean, whatever - this is an implementation detail).
但是如果应用程序中的所有进程都需要一个用户，我仍然会在这个方法中抛出异常……
其他回答
我个人会返回null，因为这是我所期望的DAL/Repository层的操作方式。
如果它不存在，不要返回任何可以被解释为成功获取对象的东西，null在这里工作得很好。
最重要的是在你的DAL/Repos层保持一致，这样你就不会对如何使用它感到困惑。
(仅)在特定契约被破坏时抛出异常。 在您的特定示例中，请求基于已知Id的UserEntity，这将取决于丢失(删除)用户是否是预期情况。如果是，则返回null，但如果不是预期的情况，则抛出异常。 注意，如果函数名为UserEntity GetUserByName(字符串名)，它可能不会抛出而是返回null。在这两种情况下，返回空的UserEntity都没有帮助。
对于字符串、数组和集合，情况通常是不同的。我记得MS的一些指导原则，方法应该接受null作为一个“空”列表，但返回零长度的集合而不是null。字符串也是一样。注意，你可以声明空数组:int[] arr = new int[0];
我更喜欢null，因为它与空合并操作符(??)兼容。
我会说返回null而不是空对象。
但你在这里提到的具体例子， 您正在通过用户id搜索用户，即排序 那个用户的键，在这种情况下，我可能想要 如果没有用户实例，则抛出异常 发现。
这是我通常遵循的规则:
如果通过主键查找操作没有找到结果， 把ObjectNotFoundException。 如按任何其他标准没有发现结果， 返回null。 如果通过非关键条件查找未找到结果，则可能返回多个对象 返回一个空集合。
有趣的问题，我认为没有“正确”的答案，因为它总是取决于你的代码的职责。您的方法是否知道没有找到的数据是否存在问题?在大多数情况下，答案是“不”，这就是为什么返回null并让调用者处理他的情况是完美的。
也许区分抛出方法和返回null方法的一个好方法是在您的团队中找到一个约定:如果没有得到任何东西，表示它们“得到”某些东西的方法应该抛出异常。可能返回null的方法可以以不同的方式命名，可能是“Find…”。
