2013-07-02 75 views
-2

我是新来的.net开发,当循环获取下面的代码中的错误,如c# - 索引超出范围例外

索引超出范围。必须是非负数且小于集合的大小。

public void postM() 
    { 
     for (int i = 0; i < listCustomer.Count; i++) 
     { 

      var grt = listCustomer[i]; 
      id = grt.UserId; 

      //When the loops come second time 

      for (int j = 0; j < listPost.Count; j++) 
      { 
       //Here I'm getting the above error 
       var grt1 = listPost[i]; 
       postId = listPost[i].PostId; 
       posts1 = listPost[i].Posts; 
       postTime = listPost[i].PostTimeStamp; 

       DbConnection.Open(); 
       DbCommand = new OleDbCommand("select count(*) from mw_post where post_id = '" + postId + "'", DbConnection); 
       OleDbDataReader DbReader1 = DbCommand.ExecuteReader(); 
       while (DbReader1.Read()) 
       { 
        count = DbReader1[0].ToString(); 
        cnt = Convert.ToInt32(count); 
        if ((cnt == 0) && (posts != "")) 
        { 
         DbCommand = new OleDbCommand("insert into mw_post(post_id,customer_id,post,post_date,community) values('" + postId + "','" + id + "','" + posts1 + "', '" + postTime + "','LinkedIn')", DbConnection); 
         DbCommand.ExecuteNonQuery(); 

         if (posts.ToUpper().Contains("Personal Loan".ToUpper())) 
         { 
          DbCommand = new OleDbCommand("UPDATE mw_post set prod_id = '2',customer_id='" + id + "' where post = '" + posts + "'", DbConnection); 
          DbCommand.ExecuteNonQuery(); 
         } 
         else if (posts.ToUpper().Contains("Credit Card".ToUpper())) 
         { 
          DbCommand = new OleDbCommand("UPDATE mw_post set prod_id = '1',customer_id='" + user_id + "' where post = '" + posts + "'", DbConnection); 
          DbCommand.ExecuteNonQuery(); 
         } 
         else if (posts.ToUpper().Contains("Home Loan".ToUpper())) 
         { 
          DbCommand = new OleDbCommand("UPDATE mw_post set prod_id = '3',customer_id='" + user_id + "' where post = '" + posts + "'", DbConnection); 
          DbCommand.ExecuteNonQuery(); 
         } 
         else if (posts.ToUpper().Contains("Car Loan".ToUpper())) 
         { 
          DbCommand = new OleDbCommand("UPDATE mw_post set prod_id = '4',customer_id='" + user_id + "' where post = '" + posts + "'", DbConnection); 
          DbCommand.ExecuteNonQuery(); 
         } 
         else if (posts.ToUpper().Contains("Deposit".ToUpper())) 
         { 
          DbCommand = new OleDbCommand("UPDATE mw_post set prod_id = '5',customer_id='" + user_id + "' where post = '" + posts + "'", DbConnection); 
          DbCommand.ExecuteNonQuery(); 
         } 
         else if (posts.ToUpper().Contains("Debit Card".ToUpper())) 
         { 
          DbCommand = new OleDbCommand("UPDATE mw_post set prod_id = '7',customer_id='" + user_id + "' where post = '" + posts + "'", DbConnection); 
          DbCommand.ExecuteNonQuery(); 
         } 
         else 
         { 
          DbCommand = new OleDbCommand("UPDATE mw_post set prod_id = '6',customer_id='" + user_id + "' where post = '" + posts + "'", DbConnection); 
          DbCommand.ExecuteNonQuery(); 
         } 
        } 
       } 
       DbReader1.Close(); 
       DbConnection.Close(); 
      } 
     } 
    } 

如何设置循环?请给我建议。

任何想法?提前致谢。

+0

我们需要更多信息。什么类型的对象是listPost? mw_post表如何连接到程序? – called2voyage

+0

我建议您的计数变量的更多描述性名称,比如'customerCounter'或'itrCustomer',而不是'i'和'postCounter'或'itrPost'而不是'j',或者其他类似的东西。这样,你可以避免混淆'i'和'j'(或任何其他计数器)。使错误的代码看起来不对。 www.joelonsoftware.com/articles/Wrong.html – Tory

回答

5

变化ij

var grt1 = listPost[j]; 
postId = listPost[j].PostId; 
posts1 = listPost[j].Posts; 
postTime = listPost[j].PostTimeStamp; 

i被用作迭代器listCustomerjlistPost,这就是为什么你出界了。

0

应该这样:

var grt1 = listPost[i]; 
postId = listPost[i].PostId; 
posts1 = listPost[i].Posts; 
postTime = listPost[i].PostTimeStamp; 

是不是这样的:

var grt1 = listPost[j]; 
postId = listPost[j].PostId; 
posts1 = listPost[j].Posts; 
postTime = listPost[j].PostTimeStamp; 
0

您的问题似乎是,当你通过listPosts循环,你正在使用“J”循环,但随后引用与“我”的项目。如果“我”增长到大(大于listPost.Count),您将会看到BOOM。