将元素插入列表时索引超出范围异常

时间:2014-04-27 21:14:57

标签: c# .net arraylist

我一直收到以下错误: “索引超出范围。必须是非负数且小于集合的大小”,同时执行以下代码:

 try
        {
            string connectionstring = "server=localhost;user id=root;database=itemdb;password=root";
            MySqlConnection conn = new MySqlConnection(connectionstring);
            conn.Open();
            MySqlCommand cmd = new MySqlCommand("SELECT item.item_name,location.city,time.month,SUM(unit_sales) FROM sales_fact,time,item,location WHERE(sales_fact.item_key = item.item_key AND sales_fact.location_key = location.location_key AND sales_fact.time_key = time.time_key) GROUP BY item.item_name", conn);
            MySqlDataReader dr = cmd.ExecuteReader();
            int c = 0;
            List<Cube1> cubelist = new List<Cube1>();
            while (dr.Read())
            {
                //i++;
                cubelist[c].itemname = dr.GetValue(0).ToString();
                cubelist[c].city = dr.GetValue(1).ToString();
                cubelist[c].month = dr.GetValue(2).ToString();
                cubelist[c].totalsales = (int)dr.GetValue(3);
                MessageBox.Show(cubelist[0].totalsales.ToString());
                c++;
            }
            dr.Close();
            MySqlDataAdapter da = new MySqlDataAdapter(connectionstring, conn);
            DataSet dt = new DataSet();
            da.Fill(dt);
            dataGridView1.DataSource = dt;
            conn.Close();
        }
        catch (Exception ex)
        {
            MessageBox.Show(ex.Message);
        }

我哪里出错了?

2 个答案:

答案 0 :(得分:2)

您的列表为空,但您正在尝试访问它的元素。您应首先使用List<T>.Add方法。在您的代码中:

new List<Cube1>(100);

100仅指定Capacity,它不是列表的大小。它用于分配指定长度的数组,以避免每次添加新项目时重新分配你的清单。

cubelist.Add(new Cube());
cubelist[c].itemname = dr.GetValue(0).ToString();
cubelist[c].city = dr.GetValue(1).ToString();
...

答案 1 :(得分:2)

您创建一个新的(空)List<Cube1>,然后尝试分配不存在的元素的属性,因为列表为空。您需要执行cubelist[c]来实例化新的var cube = new Cube1(),然后将值分配给其属性,然后使用Cube1将其添加到列表中,而不是使用cubelist.Add(cube)

List<Cube1> cubelist = new List<Cube1>();
while (dr.Read())
{
    // Create a new Cube1 instance
    var cube = new Cube1();

    // Set its properties
    cube.itemname = dr.GetValue(0).ToString();
    cube.city = dr.GetValue(1).ToString();
    cube.month = dr.GetValue(2).ToString();
    cube.totalsales = (int)dr.GetValue(3);

    // Add it to the list
    cubelist.Add(cube);

    MessageBox.Show(cube.totalsales.ToString());
}

您根本不需要c变量。