Question

我使用索引的get / set部分来获取堆栈溢出。我已经尝试将值放入列表而不是get set中但结果不好。

class TrackList : CollectionBase
{
    public Tracks this[int i]
    {
        get
        {
            return this[i];
        }
        set
        {
            this[i] = value;
        }
    }
}

主要

class Program
{
    public static void Main(string[] args)
    {
        TrackList l1 = new TrackList();
        l1[0] = new Tracks(1, "Random");
        l1[1] = new Tracks(2, "Random");
        l1[2] = new Tracks(3, "Random");
    }
}

答案： 我在我的get / set中循环。这是工作代码：

class TrackList : CollectionBase
{
    public Tracks this[int i]
    {
        get
        {
            return (Tracks) List[i];
        }
        set
        {
            List[i] = value;
        }
    }
}

主要

class Program
{
    public static void Main(string[] args)
    {
        TrackList l1 = new TrackList();
        l1.Add(new Tracks(1, "random"));
        l1.Add(new Tracks(2, "random"));
        l1.Add(new Tracks(3, "random"));
    }
}

Answer 1

您递归地调用属性索引器无数次。我想你想要这个：

public Tracks this[int i]
{
    get
    {
        return (Tracks) List[i];
    }
    set
    {
        List[i] = value;
    }
}

然后，添加您的项目：

TrackList l1 = new TrackList();
l1.List.Add(new Tracks(1, "Random"));
l1.List.Add(new Tracks(2, "Random"));
l1.List.Add(new Tracks(3, "Random"));

然后使用索引器替换项目：

l1[1] = new Tracks(2, "Replacement");

或使用索引器检索项目：

var firstItem = l1[0];

尝试使用l1[i]，其中i <= l1.Count会产生ArgumentOutOfRangeException，如您所料。

Answer 2

首先，您的stackoverflow异常是由您自己调用索引器引起的：

    return this[i];

你调用你的索引器，它会调用你的索引器，然后调用你的索引器等等。

您的主程序应如下所示：

class Program
{
    public static void Main(string[] args)
    {
        TrackList l1 = new TrackList();
        l1.Add(new Tracks(1, "Random"));
        l1.Add(new Tracks(2, "Random"));
        l1.Add(new Tracks(3, "Random"));
    }
}

您不能只访问列表中尚不存在的索引。

添加ok后从中获取某个元素：

Tracks t = l1[2]; // to get the third element in your list

请勿在索引器设置器中使用list.Add。这违反了least astonishment principle。

否则，唯一的可能性是在构造函数中创建具有预定义数量的null元素的列表。但我不推荐这个。

TrackList l1 = new TrackList(3); // in constructor create a list of 3 null elements

整个方法的正确方法是使用List<Tracks>，你就完成了。

C＃get / set导致Stack Overflow

2 个答案: