从函数返回数据的最佳实践是什么?是返回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。它提供了一种快速而简单的机制来检测是否出现了错误，而不会抛出异常，也不会到处使用大量的try/catch。
其他回答
为了代码库的健康，我认为函数不应该返回null。我能想到几个原因:
将有大量的保护子句处理空引用if (f() != null)。
什么是空，它是一个公认的答案还是一个问题?null是特定对象的有效状态吗?(假设您是代码的客户端)。我的意思是所有引用类型都可以为空，但是它们应该为空吗?
当你的代码库不断增长时，使用null几乎总是会不时地产生一些意想不到的NullRef异常。
有一些解决方案，测试者-执行者模式或从函数式编程实现选项类型。
我通常返回null。它提供了一种快速而简单的机制来检测是否出现了错误，而不会抛出异常，也不会到处使用大量的try/catch。
如果返回类型是数组，则返回空数组，否则返回null。
请原谅我的伪php/代码。
我认为这真的取决于结果的预期用途。
如果你想编辑/修改返回值并保存它，那么返回一个空对象。这样，您就可以使用相同的函数在新对象或现有对象上填充数据。
假设我有一个函数，它接受一个主键和一个数据数组，用数据填充行，然后将结果记录保存到db。因为我打算用我的数据填充对象，所以从getter返回一个空对象可能是一个巨大的优势。这样，我可以在两种情况下执行相同的操作。无论如何都要使用getter函数的结果。
例子:
function saveTheRow($prim_key, $data) {
$row = getRowByPrimKey($prim_key);
// Populate the data here
$row->save();
}
在这里，我们可以看到相同的一系列操作操作了该类型的所有记录。
但是，如果返回值的最终目的是读取数据并对数据做一些事情，那么我将返回null。这样，我可以非常快速地确定是否没有返回数据，并向用户显示适当的消息。
通常，我将在检索数据的函数中捕获异常(因此我可以记录错误消息等)，然后从捕获中直接返回null。对于最终用户来说，问题是什么通常并不重要，因此我发现最好将错误记录/处理直接封装在获取数据的函数中。如果你在任何大公司维护一个共享的代码库，这是特别有益的，因为你可以强制适当的错误记录/处理，即使是最懒的程序员。
例子:
function displayData($row_id) {
// Logging of the error would happen in this function
$row = getRow($row_id);
if($row === null) {
// Handle the error here
}
// Do stuff here with data
}
function getRow($row_id) {
$row = null;
try{
if(!$db->connected()) {
throw excpetion("Couldn't Connect");
}
$result = $db->query($some_query_using_row_id);
if(count($result) == 0 ) {
throw new exception("Couldn't find a record!");
}
$row = $db->nextRow();
} catch (db_exception) {
//Log db conn error, alert admin, etc...
return null; // This way I know that null means an error occurred
}
return $row;
}
这是我的一般原则。到目前为止，它运行得很好。
在我们的业务对象中，我们有两个主要的Get方法:
为了让事情在语境中变得简单，或者你的问题是:
// Returns null if user does not exist
public UserEntity GetUserById(Guid userId)
{
}
// Returns a New User if user does not exist
public UserEntity GetNewOrExistingUserById(Guid userId)
{
}
第一种方法用于获取特定的实体，第二种方法用于在网页上添加或编辑实体。
这使我们能够在使用它们的上下文中两全其美。
