c# - ArgumentOutOfRangeException: Index was out of range - 2 Forms

Question

问题解决了。

原来的“private void buttonSave_Click”改为：

private void buttonSave_Click(object sender, EventArgs e)
{
    if (MusicCollection.FormMain.PublicVars.AlbumList.Count != 100)
    {
        MusicCollection.FormMain.PublicVars.AlbumList.Add(new Album(NameTextBox.Text));

        MessageBox.Show("New Album added: " + NameTextBox.Text);
        formMain.ListAlbums(formMain.AlbumsListBox.Items);
        this.Close();
    }
    else
    {
        MessageBox.Show("No room for new album.");
        this.Close();
    }
}

原帖：

我是使用 C# 的新手，因此对任何看似明显的错误或糟糕的编码表示歉意。

我正在尝试创建一个新专辑对象（从 FormAlbumAC 上的 NameTextBox.Text 获取其名称），并在用户单击 FormAlbumAC 上的保存按钮时将其添加到 List AlbumList 中。然后我想在 FormMain 的 ListBox 中列出所有 AlbumList。

当我运行程序并单击保存按钮时，我在以下行收到错误“ArgumentOutOfRangeException 未处理，索引超出范围”：

if (MusicCollection.FormMain.PublicVars.AlbumList[i] == null)
// line 8 on my excerpt from Form FormAblumAC

我不确定我做错了什么。任何帮助将不胜感激，谢谢。

FormFormMain:

public const int MAX_ALBUMS = 100;
public int totalAlbums = 0;

public FormMain()
    {
        InitializeComponent();
    }

public static class PublicVars
{
    public static List<Album> AlbumList { get; set; }

    static PublicVars()
    {
        AlbumList = new List<Album>(MAX_ALBUMS);
    }
}

public ListBox AlbumListBox
{
    get
    {
        return AlbumListBox;
    }
}

public void ListAlbums(IList list)
{
    list.Clear();
    foreach (var album in PublicVars.AlbumList)
    {
        if (album == null)
            continue;
        list.Add(album.Name);
    }
}

FormAlbumAC:

private FormMain formMain;

private void buttonSave_Click(object sender, EventArgs e)
{
    int index = -1;
    for (int i = 0; i < MusicCollection.FormMain.MAX_ALBUMS; ++i)
    {
        if (MusicCollection.FormMain.PublicVars.AlbumList[i] == null)
        {
            index = i;
            break;
        }
    }
    if (index != -1)
    {
        MusicCollection.FormMain.PublicVars.AlbumList[index] = new Album(NameTextBox.Text);
        ++formMain.totalAlbums;

        MessageBox.Show("New Album added: " + NameTextBox.Text);
        formMain.ListAlbums(formMain.AlbumsListBox.Items);
        this.Close();
    }
    else
    {
        MessageBox.Show("No room for new album.");
        this.Close();
    }
}

Answer 1

您的问题（来自您的 cmets）是您的 for 循环条件不正确。你的 for 循环是这样的：

for (int i = 0; i < MusicCollection.FormMain.MAX_ALBUMS; ++i)

这里有一个问题和一个潜在问题。首先，当这段代码实际运行时，它真的在运行：

for (int i = 0; i < 100; ++i)

因为MusicCollection.FormMain.MAX_ALBUMS 被声明为 100。当MusicCollection.FormMain.PublicVars.AlbumList 的长度小于 100 时，这会导致错误，因为您正在尝试获取一个不存在的索引。

相反，您需要从i=0 迭代到....PublicVars.AlbumList-1 的长度，或者最好是for(int i = 0; i < ....PublicVars.AlbumList.Count; i++)。

第二个潜在问题是您可能会跳过索引 0。数组从索引 0 开始并继续索引 length-1。因此，您可能想要i++，而不是++i。不过，这取决于您的实施。