Try/Catch/Наконец, используйте исключение из Catch in finally?

Question

Я посмотрел на несколько других вопросов try catch finally, но я не уверен, что на этот ответ был дан ответ. Есть ли запах плохо делать что-то вроде:

Exception? ex = null; 
try { //something } 
catch (Exception e) { ex = e ;} 
finally { 
    DoSomething();   
} 
... //more code 

//end of method 
if (ex !=null) { throw ex; } 

В принципе, я пытаюсь обеспечить определенный код (за пределами TRY/улова/наконец) выполняется и что исключение, если один случается, но до тех пор, пока указанный код не запустится. Я не могу поместить весь код в блок finally, потому что он вне некоторых условных выражений.

Если это действительно так, плохо пахнет (я подозреваю, что это так), как бы добиться этого?

Answer 1

Это, безусловно, запах кода. Повторное бросание исключения в конце метода перезапишет стек вызовов исключения, поэтому появится, что все исключения произошли в конце метода, а не там, где они действительно произошли.

Если вы не можете поместить дополнительный код в существующих, наконец, вы создаете вложенные try..finally блоков:

try { 
    try { //something } 
    finally { 
     DoSomething();   
    } 
    ... //more code 
} 
finally { 
    // cleanup code 
} 

Примечание Из OP: см this code для того, что автор этот ответ привел меня правильно вывести.

Answer 2

Поскольку вы хотите запустить «больше кода» даже в случае исключения, почему бы не добавить «больше кода» в конце? Это по сути то, что, наконец, для - это guarenteed выполнить, даже если исключение происходит

так:

try { 
    // something 
} catch (Exception e) { 

    ***// more code here*** 

    DoSomething(); 
    throw; 
} 

Хранение ех и бросить ех; проглотит внутренние ожидания. Вы хотите бросить; чтобы ничто в стеке не проглотило.