1

我正在尝试从组合框中检索一个值并使用列表将其放入另一个组合框中。

我成功地从列表中删除了该值，并将其实施到另一个列表中。但是当我这样做时，我的列表容量出现故障，并且出现索引超出范围的错误。

设想：

列表字母选择：B

'按下左 Shift 按钮'

'B' 从字母列表中删除

'B' 添加到号码列表

打印值

错误：“索引超出范围。必须为非负数且小于集合大小

第 66 行：Console.WriteLine("Numbers:" + numbers[i] + "\tIteration:" + i);

数字列表容量：8，字母列表容量：4。

数字列表：1、2、3、4、B。字母列表：A、C、D

代码：

public partial class Form1 : Form
{
    public List<string> letters = new List<string>();
    public List<string> numbers = new List<string>();

    public Form1()
    {
        InitializeComponent();
    }

    private void Form1_Load(object sender, EventArgs e)
    {
        InitialiseLists();
        LoadListsIntoCombo();
    }

    public void InitialiseLists()
    {
        /*
         * Add Letter Data
         */
        letters.Add("A");
        letters.Add("B");
        letters.Add("C");
        letters.Add("D");

        /*
         * Add Number Data
         */
        numbers.Add("1");
        numbers.Add("2");
        numbers.Add("3");
        numbers.Add("4");

    }

    public void LoadListsIntoCombo()
    {

        comboLetter.DataSource = letters;
        comboNumber.DataSource = numbers;

    }

    public void PrintList()
    {
        /*
         * Print Lists To Console
         */
        for (int i = 0; i < numbers.Capacity; i++)
        {
            Console.WriteLine("NUM CAPACITY: " + numbers.Capacity);
            Console.WriteLine("Numbers: " + numbers[i] + "\tIteration: " + i);
        }

        for (int i = 0; i < letters.Capacity; i++)
        {
            Console.WriteLine("LET CAPACITY: " + letters.Capacity);
            Console.WriteLine("Letters : " + letters[i] + "\tIteration: " + i);

        }

    }


    private void cmdLeft_Click(object sender, EventArgs e)
    {

        AddLetterToNumber(GetLetter());
        RemoveLetter(GetLetter());

    }

    public void RemoveLetter(string value)
    {
        letters.Remove(value);
    }

    public void AddLetterToNumber(string value)
    {
        numbers.Add(value);
    }

    public string GetLetter()
    {
        string letter = comboLetter.SelectedItem.ToString();
        return letter;
    }

    public int GetLetterIndex()
    {
        int letterIndex = comboLetter.SelectedIndex;

        return letterIndex;
    }

    private void cmdRight_Click(object sender, EventArgs e)
    {
        PrintList();
    }

}

    }

非常感谢您提供的任何帮助。

4

2 回答 2

5

使用Count属性而不是Capacity. 因为第一个返回列表中包含的项目数。第二个只是说在调整列表大小之前可以添加多少项目。

public void PrintList()
{
   Console.WriteLine("Numbers count: " + numbers.Count);    
   for (int i = 0; i < numbers.Count; i++)   
       Console.WriteLine("Numbers: {0}\tIteration: {1}", numbers[i], i);       

   Console.WriteLine("Letters count: " + letters.Count);    
   for (int i = 0; i < letters.Count; i++)       
       Console.WriteLine("Letters : {0}\tIteration: {1}", letters[i], i);   
}
于 2012-10-21T00:59:51.750 回答
1

您正在使用容量而不是计数。容量不是列表中元素的数量。

    Capacity: Gets or sets the total number of elements the internal data structure can hold without resizing.
    Count: Gets the number of elements actually contained in the List<T>.

从这里 http://msdn.microsoft.com/en-us/library/6sh2ey19.aspx

于 2012-10-21T01:01:53.160 回答