異常代碼的處理

經常看到同事這樣寫代碼：

DataSet QueryDB()
{
   DataSet ds=null;

   try
    {
         //do something
    }
    catch (Exception ex)
    {
         //這裡要做日志記錄
    }

    return ds;
}
 

這裡有幾個問題：

1：很明顯，如果QueryDB方法發生了任何異常，客戶端無法得知，例如客戶端調用QueryDB方法，該方法返回了null，那這代表數據庫裡面沒有這個數據呢？還是拋出了異常？

2：注釋是不應該存在的，它應該被真實的日志記錄代碼給替代，例如Log.Write(ex);

3：該方法捕捉所有異常，這樣任何異常都被捕獲了，這對於開發很不方便，永遠不要捕獲你不能處理的異常。

4：為什麼這樣寫代碼？？解釋是：真實用戶不希望看到錯誤信息，初聽起來，好像有幾分道理，試想沒有哪個用戶會用你的軟件，然後老是拋出個異常什麼的，但是這是部署之後的事情啊，而不是開發的程序員不希望看到異常啊。在開發的時候，如果能夠看到詳細的異常信息，那麼可以很快的改正，修復Bug，何樂而不為之呢？？

於是修改為如下：

DataSet QueryDB()
{
     DataSet ds = null;

     try
     {
           //do something
     }
     catch (Exception ex)
     {
          Log.Write(ex);
          throw ex;
      }

      return ds;
} 好了，現在異常總算被捕獲了，並且也被成功了拋出來了。

這段代碼還是有問題？？

在catch語句塊中，throw ex; 最好修改為throw;

因為在.net中，異常都是不可修改的，每一次異常被拋出的時候，異常的堆棧跟蹤信息都會被重置，

Throw 不會重置堆棧跟蹤信息，但是throw ex;會重置。所以為了更方便的查找異常的拋出位置，最好使用throw 語句，而不是throw ex；否則clr會認為異常是在catch語句塊中拋出的。