这是我的代码：
public void ReadSomeFile(string filePath)
{
if (!File.Exists(filePath))
throw new FileNotFoundException();
var stream = new FileStream(filePath, ....)
.....
}
我是否应该自己抛出异常(请看File.Exists条件判断)? 如果文件不存在，FileStream已经会抛出FileNotFoundException了。在这里什么是良好的编程实践呢？代码分析说我们应该验证参数。但如果我直接将参数传递给另一个方法（我的或其他人的代码），并且该方法本身会抛出异常，那么在我的代码中验证参数的优点是什么呢？
if (File.Exists(f)) { DoSomething(f) }（或其否定形式）是一个反模式。这个文件可能在这两个语句之间被删除或创建，因此这样检查它的存在意义不大。
除此之外，正如评论中指出的那样，虽然File.Exists()可能会返回true，但实际上打开文件仍然可能因各种原因失败。因此，在打开文件时需要重复错误检查和抛出。
由于您不想重复自己而是保持代码DRY，只需尝试打开文件并让new FileStream()抛出异常。然后，您可以捕获异常，并且如果您希望，重新抛出原始异常或引发特定应用程序异常。
当然，调用File.Exists()可能是有道理的，但不在这种模式下。
File.Open(f) 抛出异常和你捕获它之间，其他一些进程可能会创建该文件。问题不是情况在处理之前会发生改变（因为在失败的情况下几乎无法避免），而是你应该进行一次错误检查以避免重复（并且你必须在打开时进行错误检查）。 （还要注意，当与某些文件系统交互时，成功打开并不意味着文件不会被删除：因此即使在打开似乎有效的情况下也可能发生故障。） - Yakk - Adam Nevraumont你的方法被称为ReadSomeFile，并以filename作为输入，因此抛出FileNotFoundException是合理的。由于你不能通过捕获异常再自己抛出来添加任何价值，所以就让.NET抛出它。
但是如果你的方法是LoadData（databaseName），并且需要访问许多文件，则捕获异常并抛出自定义异常可能是有价值的，因为你可以将databaseName和其他有用的信息添加到异常中。
FileNotFoundException没有意义，实际上会引发竞争条件问题。你可以处理异常、让它传播或将其包装在自己的异常中。这分别对应于"我知道该怎么做"、"我不知道该怎么做"和"我想在堆栈更高的地方处理此异常"。 - LuaanfilePath是否有效（即绝对路径，或者至少不包含Path.GetInvalidFileNameChars()）。 - Alexei Levenkov