2011-05-27 100 views
1

我有一个ListView。它有6列:c#ListView数据库保存问题

question_id | question_text | start_time | end_time | status | repeat 

分别。现在我可以显示数据库中的数据。这是我的代码:

private void Voting_Editor_Tool_Load(object sender, EventArgs e) 
{ 
    GetData(); 
} 

public void GetData() 
{ 
    try 
    { 
     now = DateTime.Now; 
     String time_date = now.ToString(); 
     myConnection = new SqlConnection(@"User ID=sa;Password=password123;Initial Catalog=dishtv;Persist Security Info=True;Data Source=ENMEDIA-EA6278E\ENMEDIA"); 
     //myConnection.Open(); 
     //SqlDataReader dr = new SqlCommand("SELECT question_text,question_id FROM otvtbl_question ", myConnection).ExecuteReader(); 

     // listView1.Columns.Clear(); 
     listView1.Items.Clear(); 

     myConnection.Open(); 
     String MyString1 = string.Format("SELECT question_id,question_text,start_time,end_time,status,repeat FROM otvtbl_question"); 

     com = myConnection.CreateCommand(); 
     com.CommandText = MyString1; 

     dr = com.ExecuteReader(); 
     ListViewItem itmX; 
     //Adding the Items To The Each Column 
     while (dr.Read()) 
     { 
      itmX = new ListViewItem(); 
      itmX.Text = dr.GetValue(0).ToString(); 
      var word = itmX.Text; 
      for (int i = 1; i < 6; i++) 
      { 
       itmX.SubItems.Add(dr.GetValue(i).ToString()); 
      } 
      if (dr.GetDateTime(2) < now && dr.GetDateTime(3) > now) 
      { 
       itmX.SubItems[4].Text = "Broadcasting"; 
      } 
      else if (dr.GetDateTime(3) < now) 
      { 
       string a=Convert.toString(dr.GetDateTime(3)); 
       itmX.SubItems[4].Text = "Expired"; 
       String broadcast = string.Format("UPDATE otvtbl_question SET    status='EXPIRED' where start_time='{6}'",a); 
       //Execute the SqlCommand 
       com = new SqlCommand(broadcast, myConnection); 
       com.ExecuteNonQuery(); 
      } 
      else 
      { 
       itmX.SubItems[4].Text = "Not Expired"; 
      } 
      listView1.Items.Add(itmX); 
     } 

     dr.Close(); 
     myConnection.Close(); 
    } 
    catch (Exception ex) 
    { 
     //Error Message While Fetching 
     MessageBox.Show("Error While Fetching the data From the DataBase" + ex); 
    } 
    finally 
    { 
     //Closing The Connection 
     if (dr != null) 
      dr.Close(); 

     if (myConnection.State == ConnectionState.Open) 
      myConnection.Close(); 

    } 
} 

在此代码中,状态列必须在用户每次加载表单时更新。在窗体加载时,它必须检查start_time是否大于当前时间。如果它大于状态栏必须显示NOT EXPIRED,否则它必须显示EXPIRED。问题是我可以通过比较时间来显示Status列中的EXPIREDNOT EXPIRED列,但我想将EXPIREDNOT EXPIRED值保存在数据库中,同时显示状态列中的值。我曾尝试使用以下命令来更新它:

String broadcast = string.Format("UPDATE otvtbl_question SET    status='EXPIRED' where start_time='{6}'",a); 
//Execute the SqlCommand 
com = new SqlCommand(broadcast, myConnection); 
com.ExecuteNonQuery(); 

但它说:

的DataReader已到更新数据之前被关闭。

我甚至试图关闭DataReader和尝试更新和它说不同的错误为:

索引(从零开始)必须大于该参数的大小大于或等于零且小于列表

有什么建议吗?

+0

不应该将您的更新查询作为参数字段{0}而不是{6}吗? – ChristiaanV 2011-05-27 06:16:02

回答

0

您应该实施using语句。这将解决问题。以下是应使用陈述应执行的区块。

  1. SQL连接
  2. DataReader的

而且我们应该使用参数化查询。以下是示例代码。

using (System.Data.SqlClient.SqlConnection con = new SqlConnection("YourConnection string")) { 
    con.Open(); 
    SqlCommand cmd = new SqlCommand(); 
    string expression = "Parameter value"; 
    cmd.CommandType = CommandType.StoredProcedure; 
    cmd.CommandText = "Your Stored Procedure"; 
    cmd.Parameters.Add("Your Parameter Name", 
       SqlDbType.VarChar).Value = expression;  
    cmd.Connection = con; 
    using (IDataReader dr = cmd.ExecuteReader()) 
    { 
     if (dr.Read()) 
     { 
     } 
    } 
} 

以下是您要求的IDisposable示例。 IDisposable

+0

to show Error1 \t名为'dr'的局部变量不能在此范围内声明,因为它会给'dr'赋予不同的含义,'dr'已在'父级或当前'范围内用于表示其他内容。 – bharathi 2011-05-27 06:25:26

+0

不显示我的身边。你可以在查询中分享更新的代码吗? – Pankaj 2011-05-27 06:28:05

+0

我认为你在using语句上面声明了两次。所以如果你可以请分享查询中的代码? – Pankaj 2011-05-27 06:29:36