Какие проблемы вы видите при обработке исключений для одного метода?

Question

В настоящее время я работаю с некоторым кодом, который используется в связи с клиентским сервером.

Я очень много (около 50) из следующих блоков try-catch

try 
{ 
    return GetLogFiles(date); 
} 
catch (TimeoutException ex) 
{ 
    this.GetLogger("GetAllLogs").Error("C is not answering!", ex); 
} 
catch (ConnectionInterruptedException ex) 
{ 
    this.GetLogger("GetAllLogs").Error("Connection interrupted", ex); 
} 
catch (ActionNotSupportedException ex) 
{ 
    this.GetLogger().Error("Software-version not comaptible!", ex); 
    throw new VersionNotCompatibleException(); 
} 
catch (EndpointNotFoundException ex) 
{ 
    this.GetLogger().Error("Problem with network, connection to core is lost!", ex); 
} 
catch (CommunicationException ex) 
{ 
    this.GetLogger().Error("Not expected communication-exception was thrown:", ex); 
} 

Это делает много кода, который более или менее всегда то же самое.

Теперь я подумал о реорганизации всех блоков catch в метод и просто позвонил ему. Как

try 
{ 
    return GetLogFiles(date); 
} 
catch(Exception ex) 
{ 
    ExceptionHandling(string operation, Exception ex) 
} 


private void ExceptionHandling(string operation, Exception ex) 
{ 
    if (ex is TimeoutException) 
    { 
     this.GetLogger(operation).Error("C is not answering!", ex); 
    } 
    else if (ex is ConnectionInterruptedException) 
    { 
     this.GetLogger(operation).Error("Connection interrupted", ex); 
    } 
    else if (ex is CommunicationObjectFaultedException) 
    { 
     this.GetLogger(operation).Error("Core is not answering!", ex); 
    } 
    else if (ex is FaultException) 
    { 
     this.GetLogger(operation).Error("C is not answering!", ex); 
    } 
    else if (ex is ActionNotSupportedException) 
    { 
     this.GetLogger().Error("Software-version not comaptible!", ex); 
     throw new VersionNotCompatibleException(); 
    } 
    else if (ex is EndpointNotFoundException) 
    { 
     this.GetLogger().Error("Problem with network, connection is lost!", ex); 
    } 
    else if (ex is CommunicationException) 
    { 
     this.GetLogger().Error("Not expected communication-exception was thrown:", ex); 
    } 
    else 
    { 
     this.GetLogger().Error("Unknown exception was thrown:", ex); 
     throw new Exception("Unknown exception occured during request handling", ex); 
    } 
} 

Ни один из codeparts не делают любое обычное обращение в случае исключения, то есть они получают вход и это все.

Какие проблемы вы видите при извлечении части исключения в свой собственный метод?

Answer 1

Основная проблема, которую я вижу, заключается в том, что вы перерабатываете исключение, если оно не вы ожидаете, заставляя вас потерять реальную трассировку стека, плюс использование Exception в вашем улове - это очень широкая вещь.

Линии кода не являются реальной проблемой, и да, делая вещи DRY - это хорошо, но более вероятно, что вещь, которая нуждается в сухом виде, - это функция GetLogFiles, чтобы включить обработку ошибок внутри этого, а не обработчик исключений.

Answer 2

это лучше сделать это так:

public class MyException : Exception 
{ 
    public MyException(Exception ex) 
    { 
     // handle exception 
    } 
} 

и

try 
{ 
} 
catch(Exception ex) 
{ 
    throw new MyException(ex) 
}