我正在解析一个不时包含 MalFormed 数据的文件。
它正在抛出异常，
我想从异常中恢复并忽略格式错误的数据。
最好的方法是什么？
try{
// parse file
}catch(Exception){
//eat it.
}
*编辑：*我想，我的问题没有被很好地理解。我想从异常中恢复，任何异常，我不希望我的程序停止。但要继续。
【问题讨论】：
标签： c# .net exception error-handling
try{
// parse file
}
catch(Exception){
//eat it.
}
finally
{
//cleanup must ... release system resources here (e.g. stream.Close() etc)
// code here always executes...even if the exception is unhandled
}
// code will resume execution from this point onwards...
【讨论】：
我想你要问的是：
逐行解析文件时，有时当前行包含格式错误的数据，这会导致代码中引发异常。也许您只需要构造您的代码，使 try/catch 仅围绕解析代码，而不是行读取代码。例如：
using System;
using System.IO;
using System.Windows.Forms;
public void ParseFile(string filepath)
{
TextReader reader = null;
try
{
reader = new StreamReader(filepath);
string line = reader.ReadLine();
while (!string.IsNullOrEmpty(line))
{
try
{
// parse line here; if line is malformed,
// general exception will be caught and ignored
// and we move on to the next line
}
catch (Exception)
{
// recommended to at least log the malformed
// line at this point
}
line = reader.ReadLine();
}
}
catch (Exception ex)
{
throw new Exception("Exception when trying to open or parse file", ex);
}
finally
{
if (reader != null)
{
reader.Close();
reader.Dispose();
}
reader = null;
}
}
外部的 try/catch 块用于在尝试打开或读取文件时正确关闭读取器对象。内部的 try/catch 块用于吞下如果读取的数据格式错误且无法正确解析时引发的异常。
不过，我几乎同意其他所有人的观点。仅仅吞下异常可能并不好。至少，我建议您将其记录在某个地方。
【讨论】：
throw; 而不是将原始异常包装在新异常中，以保留调用堆栈。此外，如果您使用using(reader = new StreamReader(filepath))，那么您无论如何都不需要那个外部异常块。
throw; 不会总是保留调用堆栈，throw ex; 更不用说。见：weblogs.asp.net/fmarguerie/…
我有一种感觉，你看到黑白之类的东西，感觉不对。是的，大多数时候，在验证输入时捕捉所有内容并马虎是不好的，但是不总是。这很可能是一个特例。我不知道要解析什么样的文件，但例外可能是正确的。
在我们决定什么是最好的之前，请告诉我们更多细节:)
【讨论】：
如果您希望它只为解析特定的错误抛出一种或两种特定的异常类型，那么捕获一般异常是一个坏主意。通常最好捕获每个特定的异常类型并独立处理它，这样您的代码就不会抑制任何其他（可能是意外的）错误（例如，如果文件丢失，或者网络连接超时，应该在就像文件包含损坏的数据一样？）
但是，如果您有意/想要捕获所有可能的错误并优雅地继续，那么捕获一般异常是一个好主意。如果对所有错误类型的处理相同，则可能会发生这种情况（例如，“如果出于任何原因，我无法读取此首选项值，我将返回默认值 5” - 这比您的程序崩溃要好得多，因为您没有t意识到它可能会由于网络超时而引发异常）。如果使用得当，这种方法可以使您的程序防弹 - 但如果使用不当，您可以抑制需要了解和修复的错误，这可能会非常痛苦。
在抑制任何异常时，您应该始终仔细考虑错误报告 - 您是否应该告诉用户您遇到了问题？您是否应该将其记录在跟踪文件中，以便当客户抱怨某些事情无法正常工作时，您可以追溯问题的根源？还是应该默默地忽略它？只是要小心，因为过分热心的压制可能会导致很难弄清楚为什么节目会表现得不可预测。
【讨论】：
一般来说是这样的：
try
{
// parse file
}
catch (FormatException)
{
// handle the exception
}
finally
{
// this block is always executed
}
您应该避免捕获一般的Exception 情况，而是捕获特定的异常，无论它可能是什么。
【讨论】：
这里有一些很好的背景信息： Is there any valid reason to ever ignore a caught exception
简短的回答：以这种方式使用异常的成本很高。如果可能的话，在发送它进行处理之前测试输入并在那里忽略而不是忽略异常。
同时确保你没有撒网，没有吃掉你可能想知道的合法例外。
【讨论】：
简而言之，您可能不应该为此使用异常。像 TryParse 这样返回 false 的东西效率更高，也更容易从中恢复。异常处理有点生硬。
【讨论】：
这行得通 - 如果您想查看导致异常发生的数据是什么并使用它来改进您的解析逻辑，也许您也可以记录错误。
【讨论】：