如果未指定要删除的项目，服务应该抛出异常或返回

我有一段代码可以表示为：

public class ItemService {

    public void DeleteItems(IEnumerable<Item> items)
    {
        // Save us from possible NullReferenceException below.
        if(items == null)
            return;

        foreach(var item in items)
        {
            // For the purpose of this example, lets say I have to iterate over them.
            // Go to database and delete them.
        }
    }
}

现在我想知道这是正确的方法还是应该抛出异常。我可以避免出现异常，因为返回将与遍历一个空集合相同，这意味着无论如何都不会执行任何重要的代码，但是另一方面，我可能会将问题隐藏在代码中的某个地方，因为有人为什么要调用DeleteItems有null参数吗？这可能表明代码中的其他地方存在问题。

这是我通常在服务中使用方法时遇到的一个问题，因为它们中的大多数都做某事并且不返回结果，因此，如果有人传递了无效信息，则该服务将无事可做，因此它将返回。

这是两个不同的问题。

你应该接受null吗？这取决于您关于null代码库的一般政策。我认为，禁止null在除明确记录的地方之外的任何地方都是一个很好的做法，但是最好还是遵循您的代码库已经存在的约定。

您应该接受空集合吗？我认为：是的，绝对如此。将所有调用者限制为非空集合比做正确的事情要付出更多的努力-即使这会让某些对零概念不满意的开发人员感到惊讶。

空值

正如@KilianFoth已经说过的那样，请遵循您的一般政策。如果将其null视为空列表的“简写”，请这样做。

如果您对null价值观没有一致的政策，建议您采取以下措施：

null应该保留以表示无法用“正常”类型表示的情况，例如null用于表示“我不知道”。这是一个不错的选择，因为每个尝试不小心使用此值的人都会遇到异常，这是正确的选择。

使用null作为一个空列表的简写没有资格这样，因为已经有一个完美的表现，是与零个元素的列表。从技术上来说，这是一个错误的选择，因为它会迫使您处理列表的代码的每个部分都要检查有效的简写形式null。

空清单

对于一种DeleteItems()方法，有效地传递一个空列表意味着什么也不做。我允许它作为参数，而不是引发异常，只是快速返回。

当然，DeleteItems()在这种情况下，调用方可以先检查零个元素，然后跳过该调用。如果我们在谈论Web API，出于效率方面的考虑，调用者实际上应该这样做，以避免不必要的流量和往返延迟。但是我不认为您的API应该执行该操作。

引发异常并处理调用代码中的null。

作为设计规则，请尝试避免将null作为参数值。通常，它将减少NullPointerExceptions，因为null确实是一个异常。

除此之外，请查看其余的代码。如果这是项目中的常见模式，请保持一致。

一般而言，抛出异常应保留用于例外情况，即，如果代码在当前上下文中没有采取合理的措施。

您可以将该思考过程应用于这种情况。鉴于这是带有公共方法的公共类，因此您具有公共API，并且从理论上讲，无法控制传递给它的内容。

您的代码没有关于调用它的上下文（因为它不应该这样做），并且您无法合理地使用null值。这肯定是一个候选人ArgumentNullException。

这个问题似乎与例外无关，而与null成为有效论点有关。

因此，首先，您必须确定null该方法的参数是否为允许的值。如果是，那么您不需要例外。如果不是，那么您确实需要一个例外。

nullGoogle 是否对此表示欢迎，这是否值得商，还是有争议的。意思是，您不会得到一个明确的答案，这在一定程度上取决于您工作所在的地方的观点和传统。

争论的另一个领域是一个库函数是否应该真正严格的这样的事情，或宽大越好，只要它是不是要“修复”错误的参数。将此与网络协议，邮件传输等（它们是接口协定，就像编程方法一样）进行比较。在那里，通常的策略是，发送方应尽可能严格地遵守协议，而接收方应尽其所能来处理即将发生的一切。

因此，您必须决定：实施相当大的策略（例如null处理）真的是一种库方法的工作吗？特别是如果您的图书馆也被其他人使用（他们可能有不同的政策）。

我可能会错在允许null值，定义和记录其语义（例如，null = empty array在这种情况下）方面，并且不会抛出异常，除非您的其他类似代码（“库”样式代码）中有99％这样做了'回合。