如何在保存到數據庫時跳過列表視圖行？

Question

這是我的代碼保存到MySQL數據庫的列表視圖項目在C＃...我怎麼能跳過列表視圖行，如果它沒有插入數據呢？我有點失去了我要插入If語句的位置..請任何人嗎？

for (int cnt = 0; cnt <= lv1.Items.Count - 1; cnt++) 
{ 
    string query = "insert into results(sid,c_id)values('" + _studid + "','" + lv1.Items[cnt].SubItems[2].Text + "')"; 

    conn.Open(); 
    MySqlCommand cmd = new MySqlCommand(query, conn); 
    cmd.ExecuteNonQuery(); 
    conn.Close(); 


} 

Answer 1

從循環中提取所有代碼以構建MySqlCommand並初始化兩個參數，第一個參數始終具有相同的值，而第二個參數不同。因此，在循環內檢查listview子項目中的空值，如果不爲空，則設置第二個參數的值。現在你可以執行它，後面的循環只需要改變第二個參數的值。

string query = "insert into results(sid,c_id)values(@id, @cid))"; 
MySqlCommand cmd = new MySqlCommand(query, conn); 
cmd.Parameters.Add("@id", MySqlDbType.Int32).Value = _stuid; 
cmd.Parameters.Add("@cid", MySqlDbType.Int32); 

conn.Open(); 

for (int cnt = 0; cnt < lv1.Items.Count; cnt++) 
{ 
    string subItem = lv1.Items[cnt].SubItems[2].Text; 
    if(!string.IsNullOrWhiteSpace(subItem)) 
    { 
     cmd.Parameters["@cid"].Value = Convert.ToInt32(subItem); 
     cmd.ExecuteNonQuery(); 
    } 

} 
conn.Close(); 

以這種方式，您只需打開一次連接，然後在執行循環前準備好命令。該命令使用參數化查詢，因此從Sql Injection安全。
另請參閱如何不使用引號和字符串連接的sql字符串更具可讀性。

最後說明：
我假設c_id字段是一個數字字段。
我已將for ...循環更改爲更簡單的形式。
這種類型的代碼導致使用一個事務來保持一切都是原子的（意思是說，如果一個插入失敗，所有的東西都應該回滾，而不需要對數據庫進行任何修改）。欲瞭解更多信息，請看this question

Answer 2

你可以簡單地檢查你的項目的值是空

我推薦你讀參數化的SQL querys http://msdn.microsoft.com/en-us/library/system.data.sqlclient.sqlcommand.parameters.aspx

if(lv1.Items[cnt].SubItems[2].Text== string.Empty) 
{ 
    continue; 
} 

Answer 3

此代碼跳過插入到數據庫中，如果沒有數據添加，現在查詢使用參數： 你可以打開和關閉連接以外的連接，它肯定會提高性能

conn.Open(); 
for (int cnt = 0; cnt <= lv1.Items.Count - 1; cnt++) 
{ 
    if(lv1.Items[cnt].SubItems[2].Text=="") 
     continue; 
MySqlCommand m = new MySqlCommand(readCommand); 
m.Parameters.Add(new MySqlParameter("@id", _studid)); 
m.Parameters.Add(new MySqlParameter("@val", lv1.Items[cnt].SubItems[2].Text)); 

    string query = "insert into results(sid,c_id)values(@id, @val)"; 

    MySqlCommand cmd = new MySqlCommand(query, conn); 
    cmd.ExecuteNonQuery(); 
} 
conn.Close();