我和我的同事在就这段代码进行辩论：
var y = null;
if (x.parent != null)
y = x.parent.somefield;
我认为，在代码所在的地方，x.parent 不应该可能为 null。当它为 null 时，我们就会遇到严重的问题，我想知道发生了什么！因此，我们不应该加入空值检查，让下游异常发生。
我的同事说这是防御式编程。空值检查可以确保代码不会破坏应用程序。
我的问题是，这是防御式编程吗？还是一种糟糕的实践方式？
注：重点不在于谁是对的。我正在尝试从这个例子中学习。
看起来你的同事对“防御性编程”和/或异常有所误解。
防御性编程是关于保护免受某些类型的错误的影响。
在这种情况下，x.parent == null是一个错误，因为你的方法需要使用x.parent.SomeField。如果parent为null，则SomeField的值显然无效。任何使用无效值进行的计算或任务都会产生错误和不可预测的结果。
因此，你需要防范这种可能性。一种非常好的保护方式是，在发现x.parent == null时抛出NullPointerException。异常将阻止你从SomeField获取无效值。它将阻止你使用无效值进行任何计算或执行任何任务。并且它将中止所有当前工作，直到错误得到适当的解决。
请注意，异常不是错误；
parent中的无效值才是实际的错误。异常实际上是一种保护机制。异常是一种防御性编程技术，而不是应该被避免的东西。
由于C#已经抛出异常，你实际上不需要做任何事情。事实证明，在“以防御性编程之名”的同事的努力中，他正在撤销语言提供的内置防御性编程。
我注意到许多程序员过度担心异常。异常本身并不是错误，它们只是报告错误。
你的同事说：“空引用检查确保代码不会破坏应用程序”。这表明他认为异常会破坏应用程序。它们通常不会“破坏”整个应用程序。
如果糟糕的异常处理使应用程序处于不一致的状态，则异常可能会破坏应用程序。（但是如果隐藏错误，则更有可能发生这种情况。）如果异常“逃脱”线程，则它们也可能会破坏应用程序。（显然逃脱主线程意味着你的程序已经非常不雅观地终止了。但即使逃脱子线程也足够糟糕，以至于操作系统的最佳选择是GPF应用程序。）
异常将中止当前操作。这是它们必须要做的事情。因为如果你编写了一个名为DoSomething的方法，它调用DoStep1；DoStep1中的错误意味着DoSomething无法正常地完成其工作。继续调用DoStep2毫无意义。
问题可能是由于缺陷的因果分析引起的。抛出异常并不意味着您需要回滚代码。有许多其他原因会导致异常被抛出。当方法发生错误时，需要执行清理操作，因此需要回滚代码。换句话说，无论如何都需要异常处理代码。这表明，防止过度异常处理的最佳方法是以设计方式减少错误时的清理需求。
因此，不要“不抛出异常”以避免过度的异常处理。我同意，过度的异常处理是不好的（请参见上面的设计考虑）。但如果您甚至不知道有错误发生，那么不回滚是更糟糕的。
x.parent绝对不是一个参数。因此，你的建议更改实际上会使错误变得具有误导性。 - Disillusioned如何处理验证？
使用if（与你示范的代码一样）来跳过某些分配或使用可能对某些情况很好。例如，如果用户正在输入某些数据，并且这只是显示提示工具提示或其他次要信息，则可能可以安全地跳过。但是，如果代码片段执行某些重要操作，填充某些强制条件或执行某些其他进程，则不是正确的方法，因为它会导致下一个代码运行出现问题。问题在于，当您跳过某些代码时，务必安全执行此操作，没有任何副作用或意外后果，否则您将隐藏某个错误，而这在后期开发阶段非常难以调试。
优雅地中止当前进程是早期验证的好选择，当失败完全是预期的并且您确切地知道如何应对时。一个例子可能是缺少必需字段，该过程被中断并向用户显示一个消息，要求提供缺少的信息。简单忽略错误是不行的，但也不严重到需要抛出异常打断正常的程序流程。当然，根据您的架构，仍然可以使用异常，但无论如何都要捕获它并优雅地恢复。
当“不可能”真的发生时，抛出异常始终是一种选择。在这些情况下，您可能无法提供合理的响应以继续某些变化或仅取消当前进程，这可能是由于某个地方的错误或坏输入，但重要的是您想知道它并了解所有有关它的详细信息，因此最好的方式是让它尽可能大声地爆炸，以便异常上升并到达全局处理程序，中断所有内容，将其保存到日志文件/DB /任何位置，向您发送崩溃报告，并找到一种恢复执行的方法（如果可行或安全）。至少，如果您的应用程序崩溃，请以最优雅的方式崩溃，并留下痕迹以供进一步分析。
通常情况下，这要取决于具体情况。但是仅仅使用if语句避免编写异常处理程序肯定是一种不好的做法。异常处理程序必须始终存在，然后某些代码可能会依赖它 - 无论是否适当 - 如果失败对系统来说不是至关重要的话。
经过几年思考，我采用以下“防御性”编程风格——95% 的方法都返回字符串作为成功/失败的结果。
我会返回 string.Empty 表示成功，如果失败会返回相应的信息文本。在返回错误文本的同时，我也会将其写入日志中。
public string Divide(int numA, int numB, out double division)
{
division = 0;
if (numB == 0)
return Log.Write(Level.Error, "Divide - numB-[{0}] not valid.", numB);
division = numA / numB;
return string.Empty;
}
然后我使用它：
private string Process(int numA, int numB)
{
double division = 0;
string result = string.Empty;
if (!string.IsNullOrEmpty(result = Divide(numA, numB, out divide))
return result;
return SendResult(division);
}
当您拥有日志监控系统时，它可以让系统继续运行，但同时通知管理员。