程式設計時犯錯是必然的,我們來解讀一下程式設計中最容出現的錯誤
1、拼接字串
在C#程式設計中,字串型別的處理是比較容易出錯的地方,在.NET Framework中,字串是一個不可變的型別,當一個字串被修改後,總是建立一個新的副本,不會改變源字串,大多數開發人員總是喜歡使用下面這樣的方法格式化字串:
string updateSqlText = "UPDATE Table SET Name='" + name+ "' WHERE Id=" + id;
這裡它使用了多重串聯拼接,因此會在記憶體中建立三個不必要的字串垃圾副本,這種方式是最容易忽略的,最好的辦法是使用string.Format,因為它內部使用的是可變的StringBuilder,也為淨化程式碼鋪平了道路,如下:
string updateSqlText = string.Format("UPDATE Table SET Name='{0}' WHERE Id={1}", name, id);
2、巢狀異常處理
在方法中新增異常處理模組try-cathc是必然的,但是沒有必要在一個方法裡面多次加上異常處理的巢狀方法,如下:
public class Class1 { public void MainMethod() { try { //some implementation ChildMethod1(); } catch (Exception exception) { //Handle exception } } private void ChildMethod1() { try { //some implementation ChildMethod2(); } catch (Exception exception) { //Handle exception throw; } } private void ChildMethod2() { try { //some implementation } catch (Exception exception) { //Handle exception throw; } } }
如果相同的異常被處理多次,整個專案都這樣寫?毫無疑問,效能開銷將會劇增。
解決辦法是讓異常處理方法獨立出來(一個大的方法只需要一個異常處理即可,特殊複雜場景可酌情多次使用),如:
public void MainMethod() { try { //some implementation ChildMethod1(); } catch (Exception exception) { //Handle exception } } private void ChildMethod1() { //some implementation ChildMethod2(); } private void ChildMethod2() { //some implementation }
3、for和foreach的選擇
大部分開發人員更喜歡使用for迴圈,而無視foreach迴圈,因為for更容易使用,但操作大型資料集時,使用foreach無疑是最快的,
根據廣大網友實驗證明(分別對記錄數為10000,100000,1000000條記錄的時候進行取樣分析),
foreach的平均花費時間只有for20%-30%左右。所以,我也要根據實際請求選擇使用而不是一直使用某一種。
C#中foreach在處理集合和陣列相對於for存在以下幾個優勢和劣勢:
- foreach語句簡潔
- 效率比for要高(C#是強型別檢查,for迴圈對於陣列訪問的時候,要對索引的有效值進行檢查)
- 不用關心陣列的起始索引是幾(因為有很多開發者是從其他語言轉到C#的,有些語言的起始索引可能是1或者是0)
- 處理多維陣列(不包括鋸齒陣列)更加的方便
- 在型別轉換方面foreach不用顯示地進行型別轉換
- 當集合元素如List<T>等在使用foreach進行迴圈時,每迴圈完一個元素,就會釋放對應的資源
- 上面說了foreach迴圈的時候會釋放使用完的資源,所以會造成額外的gc開銷,所以使用的時候,請酌情考慮
- foreach也稱為只讀迴圈,所以再迴圈陣列/集合的時候,無法對陣列/集合進行修改
- 陣列中的每一項必須與其他的項型別相等
4、驗證簡單的原始資料型別
很多人員都忽略內建的驗證原始資料型別的方法,如System.Int32(其他型別亦然),因此都是自己實現的方法,下面就是一個自己實現的驗證一個字串是否是數值的程式碼:
public bool CheckIfNumeric(string value) { bool isNumeric = true; try { int i = Convert.ToInt32(value); } catch (FormatException exception) { isNumeric = false; } return isNumeric; }
它使用了try catch語句進行捕捉判斷,因此不是最佳的做法,更好的辦法是象下面這樣使用int.TryParse:
int output = 0; bool isNumeric = int.TryParse(value, out output);
5、處理物件實現IDisposable介面
物件的處理和使用一樣重要,理想的辦法是在類中實現IDisposable介面的dispose方法,在使用這個類的物件後,可以透過呼叫dispose方法進行處理。
下面的程式碼顯示了一個SqlConnection物件的建立,使用和處理:
public void DALMethod() { SqlConnection connection = null; try { connection = new SqlConnection("XXXXXXXXXX"); connection.Open(); //implement the data access } catch (Exception exception) { //handle exception } finally { connection.Close(); connection.Dispose(); } }
在上面的方法中,連線處理在最後一個程式碼塊中被明確呼叫,如果發生一個異常,catch程式碼塊就會執行,然後再執行最後一個程式碼塊處理連線,
因此在最後一個程式碼塊執行之前,連線將一直留在記憶體中,.NET Framework的一個基本原則就是當物件不被使用時就應該釋放資源。
下面是呼叫dispose更好的辦法:
public void DALMethod() { using (SqlConnection connection = new SqlConnection("XXXXXXXXXX")) { connection.Open(); //implement the data access } }
當你使用using程式碼塊時,物件上的dispose方法將在執行退出程式碼塊時呼叫,這樣可以保證SqlConnection的資源被處理和儘早釋放,
你也應該注意到這個辦法也適用於實現IDisposable介面的類。
6、宣告公共變數
聽起來可能有點簡單,但我們經常看到濫用公共變數宣告的情況,先來看一個例子:
static void Main(string[] args) { MyAccount account = new MyAccount(); //The caller is able to set the value which is unexpected account.AccountNumber = "YYYYYYYYYYYYYY"; Console.ReadKey(); } public class MyAccount { public string AccountNumber; public MyAccount() { AccountNumber = "XXXXXXXXXXXXX"; } }
在上面的MyAccount類中宣告瞭一個AccountNumber公共變數,理想情況下,AccountNumber應該是隻讀的,但MyAccount類卻沒有對它實施任何控制。
宣告公共變數正確的做法應該是使用屬性,如:
public class MyAccount { private string _accountNumber; public string AccountNumber { get { return _accountNumber; } } public MyAccount() { _accountNumber = "XXXXXXXXXXXXX"; } }
這裡MyAccount類對AccountNumber公共變數實施了很好的控制,它變成只讀,不能由呼叫者類修改。
7、利用System.Data.DataTable訪問資料
人多人經常使用列索引從資料庫訪問資料,如:
public void MyMethod() { //GetData fetches data from the database using a SQL query DataTable dt = DataAccess.GetData(); foreach (DataRow row in dt.Rows) { //Accessing data through column index int empId = Convert.ToInt32(row[0]); } }
按照這種寫法,如果列順序在SQL查詢匹配資料時發生了變化,你的應用程式將會受到影響,正確的做法應該是使用列名訪問資料。
private const string COL_EMP_ID = "EmpId"; public void MyMethod() { //GetData fetches data from the database using a SQL query DataTable dt = DataAccess.GetData(); foreach (DataRow row in dt.Rows) { //Accessing data through column name int empId = Convert.ToInt32(row[COL_EMP_ID]); } }
這樣的程式碼更加穩固,列順序發生變化不會給應用程式造成任何影響,
如果在一個地方使用區域性變數儲存列名更好,即使將來你的列名發生了變化,也不用修改應用程式程式碼。
喜歡就點贊加關注。
歡迎關注訂閱微信公眾號【熊澤有話說】,更多好玩易學知識等你來取
作者:熊澤-學習中的苦與樂
公眾號:熊澤有話說
QQ群:711838388
出處:https://www.cnblogs.com/xiongze520/p/17164309.html
您可以隨意轉載、摘錄,但請在文章內註明作者和原文連結。