бросания исключения несколько раз

Question

У меня есть следующий код:

static int GetLastAddedIDHelper(OleDbConnection connection) 
    { 
     try 
     { 
      // Init variables. 
      OleDbCommand command = null; 
      string cmdText = "SELECT @@IDENTITY"; 

      if (connection != null) 
       command = new OleDbCommand(cmdText, connection); 
      else 
       throw new ArgumentNullException("connection", "The connection was passed as null. Therefore the globally used connection is used but was never set."); 

      return (int)command.ExecuteScalar(); 
     } 
     catch (Exception ex) { throw ex; } 
    } 

    public static int GetLastAddedID() 
    { 
     try 
     { 
      return GetLastAddedIDHelper(_Connection); 
     } 
     catch (Exception ex) { throw ex; } 
    } 

    private void Button_Click_Action() 
    { 
     try 
     { 
      int i = AccessDbServiceBase.GetLastAddedID(); 
     } 
     catch (Exception ex) 
     { 
      MessageBox.Show(ExceptionHandler.GetExceptionMessage(ex)); 
     } 
    } 

Приведенные выше код будет получить последний вставленный ID из базы данных Access для меня. Теперь, чтобы сделать это, Button_Click_Action позвонит GetLastAddedID, и это позвонит GetLastAddedIDHelper. Когда exception происходит в GetLastAddedIDHelper, я исключаю исключение до основного метода Button_Click_Action.

Мне интересно, если я делаю это правильно, например, нужен ли GetLastAddedID, следует ли использовать throw вместо throw ex, ...?

Answer 1

Обработка исключений в GetLastAddedIDHelper и GetLastAddedID не служит никакой полезной цели в этом случае. Ваше намерение, казалось бы, состоит в том, чтобы просто поймать исключение, а затем ничего не делать с ним, кроме как восстановить его. Если это так, то зачем беспокоиться об этом? Просто пусть исключение распространяется на стек до тех пор, пока обработчик исключений, который может обрабатывать его, не получит его в методе Button_Click_Action в этом случае.

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

Answer 2

Возможно, вы захотите сделать throw;, потому что throw ex; сбросит трассировку стека.