2

我正在开发一个程序，其中每个项目都可以包含一组项目（我正在制作一个菜单，它具有树状结构）

目前我将项目作为列表而不是数组，但我觉得我没有充分利用它来简化代码。我选择了一个列表而不是标准数组，因为接口（.add、.remove 等）很有意义。

我有代码来搜索结构并返回名称的路径（即 Item.subitem.subsubitem.subsubsubitem）。下面是我的代码：

public class Item
{
                                                                //public Item[] subitem; <-- Array of Items
    public List<Item> subitem;                                  // <-- List of Items

    public Color itemColor = Color.FromArgb(50,50,200);
    public Rectangle itemSize = new Rectangle(0,0,64,64);
    public Bitmap itemBitmap = null;
    public string itemName;


    public string LocateItem(string searchName)
    {
        string tItemName = null;

        //if the item name matches the search parameter, send it up)
        if (itemName == searchName)
        {
            return itemName;
        }

        if (subitem != null)
        {

            //spiral down a level
            foreach (Item tSearchItem in subitem)
            {
                tItemName = tSearchItem.LocateItem(searchName);

                if (tItemName != null)
                    break;  //exit for if item was found
            }
        }


        //do name logic (use index numbers)
        //if LocateItem of the subitems returned nothing and the current item is not a match, return null (not found)
        if (tItemName == null && itemName != searchName)
        {
            return null;
        }

        //if it's not the item being searched for and the search item was found, change the string and return it up
        if (tItemName != null && itemName != searchName)
        {
            tItemName.Insert(0, itemName + ".");  //insert the parent name on the left -->  TopItem.SubItem.SubSubItem.SubSubSubItem
            return tItemName;
        }

        //default not found
        return null;
    }


}

我的问题是，是否有更简单的方法来处理列表？关于应该使用列表还是只使用数组，我一直在脑海中反复思考。我有一个列表的唯一原因是我不必在每次添加或删除项目时编写代码来调整数组的大小。

4

3 回答 3

2

列表听起来很棒。不过，我建议对您的定义进行一些修改。尝试像这样创建你的类：

public class Item : List<Item>
{
    public string Name;
}

如果您Item从您那里继承，List<Item>则自动使其成为树而不需要该subitem字段。

这是我的完整版您的课程：

public class Item : List<Item>
{
    public string Name;

    private List<Item> LocateItems(string searchName)
    {
        if (this.Name == searchName)
            return (new [] { this }).ToList();

        var result =
            this
                .Select(s => s.LocateItems(searchName))
                .Where(x => x !=null && x.Count > 0)
                .FirstOrDefault();

        if (result != null)
            result.Add(this);

        return result;
    }

    public string LocateItem(string searchName)
    {
        var items = this.LocateItems(searchName);
        if (items == null)
            return null;
        else
            return String.Join(".", items.Select(i => i.Name).Reverse());
    }
}

该方法LocateItems返回从匹配Item开始的列表Item，然后是所有父Item实例，直到并包括根。

我用这段代码测试过：

var foos = new Item() { Name = "Foo" };
var bars = new Item() { Name = "Bar" };
var qazs = new Item() { Name = "Qaz" };
var wees = new Item() { Name = "Wee" };

foos.Add(bars);
bars.Add(qazs);
foos.Add(wees);

Console.WriteLine(foos.LocateItem("Wee"));
Console.WriteLine(foos.LocateItem("Qaz"));
Console.WriteLine(foos.LocateItem("Bar"));
Console.WriteLine(foos.LocateItem("Foo"));

我得到了这些结果：

Foo.Wee
Foo.Bar.Qaz
Foo.Bar
Foo
于 2012-09-03T04:53:01.427 回答
2

在这种情况下使用列表是完全可以接受的。如果性能是一个问题，那么阵列将是一个更好的选择 - 如果是这样，阵列会稍微快一些，但正如您所发现的那样，灵活性要低得多。

人们谈论得不够多的一件事是简单性是构建代码的重要基础。如果使用列表比数组更容易编写和维护，那么（在其他条件相同的情况下）使用列表是完全正确的。

于 2012-09-03T03:14:37.473 回答
1

我会建议列表。由于向数组添加/删除项目会重新分配内存，因此对于项目的动态集合（我假设是您的情况）列表通常总体上具有更好的性能。你可能想看看：

Array 与 List<T>：何时使用哪个？

于 2012-09-03T03:35:26.020 回答