Code inspection: Redundant catch clause
Consider the following piece of code:
try
{
string s = File.ReadAllText("test.txt");
}
catch (Exception ex)
{
throw;
}
The catch
statement may appear to be doing something, but it really isn't: all it's doing is throwing the exception (with the same stack information), which is exactly what would happen if the catch statement weren't written at all. Note that if we wrote throw ex
instead of just throw
, we can no longer argue that the catch clause is redundant.
Last modified: 11 February 2024