先列出正確的寫法,如果你只想馬上改錯就先複製吧,
protected void deleteDataRow(int RowID,DataTable dt)
{
for (int i = dt.Rows.Count - 1; i >= 0; i--)
{
if (Convert.ToInt32(dt.Rows[i]["RowID"]) == RowID)
dt.Rows.RemoveAt(i);
}
}
如果你有時間想學習一下就繼續看下面列出可能出錯的可能性吧。
1.如果只是想刪除datatable中的一行,可以用DataRow的delete,但是必須要刪除後讓DataTable知道,所以就要用到.AcceptChanges()方法,原因是這種刪除只是標識性刪除,就像我們通常在資料庫中用到的IsDelete欄位。
2.徹底刪除就要用到datatable的.Rows.Remove(DataRow dr)方法,同理也只是刪除一行可以,如果要迴圈刪除請繼續往下看。
3.迴圈徹底刪除就要用.Rows.RemoveAt(int index)方法,所以如果你是foreach的愛好者,在此請你換換口味,還有如果你是for的i++的忠實fans也希望你能換個思維。先看一下上面程式的正向寫法(錯誤的,不可用)
for (int i = 0, j = dt.Rows.Count; i < j; i++)
{
if (Convert.ToInt32(dt.Rows[i]["RowID"]) == RowID)
dt.Rows.RemoveAt(i);
}
這個的錯誤在於datatable的RemoveAt()會在刪除後更新dataTable的index,所以你要刪除的index可能已經不是你的符合Convert.ToInt32(dt.Rows[i]["RowID"]) == RowID的index了,甚者還會拋出異常,說你訪問的index不存在。
所以要從DataTable的下面網上尋找刪除,這樣即使這行符合條件被刪除了,上面的行依舊不受影響。
說了這麼多,不知道你明白了嗎?其實現在寫這種文章顯得有點“弱智”,技術學多了,越來越覺得自己的基礎不夠紮實,希望通過在此記錄一下可以督促一下自己,也希望能給初學者帶去絲絲協助。