可以处理错误的组件的异常或返回状态

Question

我目前正在设计一个处理不同类型文件的系统。我定义了以下接口

public interface IFileProcessor
{
    bool ProcessFile(string fileContents)
}

，目的是创建许多具体的实现来处理不同的文件类型。控制器类将负责：

• 监视文件夹中新添加的文件
• 读取文件内容
• 获取这些 IFileProcessor 具体实现的集合
• 逐个调用每个组件上的 ProcessFile()，传递文件内容
• 如果组件无法处理该文件文件它返回 false，否则它将处理内容并返回 true
• 如果没有 IFileProcessor 实现可以处理该文件，则控制器会将其移动到“未处理”文件夹
• 如果组件成功处理该文件，则会将其移动到“已处理”文件夹
• 如果一个组件抛出异常，它被移动到“失败”文件夹

我正在创建 IFileProcessor 的实现，它将首先检查它是否可以根据类型（即 csv）处理文件，然后执行一些顶级验证（即检查文件头）。如果这些检查中的任何一个失败，都会向控制器抛出异常，因为整个文件被视为无效。

然而，一旦顶级验证成功，组件将处理文件中的每一行。从此时开始，单个行处理（即验证）失败是可以接受的，而其余的过程继续进行。

这就是问题所在，我想知道是否最好记录已发生的验证错误，然后在进程结束时抛出异常，或者更改 ProcessFile() 签名以返回枚举（已处理、未处理之一） ，已处理错误）？

从我所读到的内容来看，异常似乎是优于状态代码的首选途径，但是在流程可以继续的特殊情况下，在最后使用人为异常来声明流程未 100% 完成似乎是错误的。

我真的很想知道人们对此的想法。

Answer 1

一个建议：

不要返回 bool 或 enum，而是返回一个调用者可以检查的对象。也许称之为 FileProcessorResult。在该对象中，您可以存储各种信息，例如总体成功​​或失败、验证状态、处理状态等。

在严重失败的情况下，我会返回异常，以便调用者可以采取正确的操作。您可以派生一个异常类，以便 try catch 是干净的。

示例：

try
{
    FileProcessorResult fpr = ProcessFile(Contents);
    //Do something with fpr

}
catch (FileProcessorException fpe)
{
   //Something unexpected occured during file processing, handle it
}

我可能无法给出绝对正确的答案，因此请将此作为建议。

Answer 2

我不认为这是一个绝对的问题，因为我没有看到明确的做这个/做那个答案......

不过我同意你的观点，在函数末尾抛出异常确实看起来很奇怪为了发出部分处理信号，我还会使用返回代码，并在函数完全失败时抛出异常（即无法从文件读取等）。

这样做的一个很好的理由是，调用函数中可能不会处理异常，并且您可能不希望在文件被部分处理时停止执行，对吧？

无论您选择哪条路，您都需要彻底记录抛出的返回码/异常，其他人会为此感谢您。