不理解IEnumerable <T>

Question

我的代码遇到了很多麻烦.当我编译时,我收到以下错误:

---

'Ecommerce.DataHelpers.ProductNodeLoader'未实现接口成员'System.Collections.IEnumerable.GetEnumerator()'.'Ecommerce.DataHelpers.ProductNodeLoader.GetEnumerator()'无法实现'System.Collections.IEnumerable.GetEnumerator()',因为它没有匹配的返回类型'System.Collections.IEnumerator'.

---

我不知道如何解决这个问题现在我不得不问你们!

码:

namespace Ecommerce.DataHelpers
{
    public class ProductNodeLoader<T> : IEnumerable<T>
    {
        private ISqlHelper sqlHelper;
        private IRecordsReader nodeReader;

        public List<T> list = new List<T>();

        // load all products from given company
        public IEnumerator<T> GetEnumerator()
        {
            int companyId = 2;
            try
            {
                sqlHelper = DataLayerHelper.CreateSqlHelper(GlobalSettings.DbDSN);
                nodeReader = sqlHelper.ExecuteReader(@"
                    SELECT * FROM eCommerceNodes WHERE companyId = @companyId)
                    ", sqlHelper.CreateParameter("@companyId", companyId));

            }
            catch (Exception e)
            {
                Log.Add(LogTypes.Custom, -1, e.InnerException.ToString());
                yield break;
            }

            if (nodeReader.HasRecords)
            {
                while(nodeReader.Read())
                {
                    ProductNode node = new ProductNode();
                    node.id = nodeReader.Get<int>("id");
                    node.parentId = nodeReader.Get<int>("parentId");
                    node.companyId = nodeReader.Get<int>("companyId");
                    node.path = nodeReader.Get<string>("path");
                    node.sortOrder = nodeReader.Get<string>("sortOrder");
                    node.text = nodeReader.Get<string>("text");
                    node.nodeType = nodeReader.Get<int>("nodeType");

                    list.Add(node);

                }
                nodeReader.Close();

            }
            else
            {
                throw new ApplicationException("No products to load");
            }

            return list;
        }

    }
}

---

我为糟糕的编辑道歉!

Answer 1

您正在尝试实现非泛型IEnumerable类型,因为IEnumerable<T>扩展它.幸运的是,这很容易:

// Along with the existing using directives
using System.Collections;
...

// In the implementing class
IEnumerator IEnumerable.GetEnumerator()
{
    return GetEnumerator();
}

请注意,您必须对要实现的两种方法中的至少一种使用显式接口实现GetEnumerator(),因为它们具有相同的签名(相同名称,无参数)但返回类型不同.幸运的是,泛型版本的返回值可以用于非泛型版本,这就是通常使用上述模式的原因.

编辑:正如乔希在评论中正确指出的那样,你也有其他问题:

• return list;除非您删除yield break;之前的代码(并将其更改为return list.GetEnumerator();),否则您的代码末尾不应该有代码.如果要将代码保留为迭代器块,则应该使用它yield return来生成您创建的每个节点.
• 你应该T让你的实例- 而你正在构建的实例ProductNode.也许你应该实际实现IEnumerable<ProductNode>而不是IEnumerable<T>,并使你的类非通用？
• 只要调用者决定对其进行迭代,您的代码就会保持SQL连接处于打开状态.这可能是也可能不是问题 - 但值得记住.
• 你应该使用一个using声明来确保你的nodeReader错误处理(假设呼叫者处理IEnumerator<T>当然)
• 你的公共list领域是一个坏主意......为什么你让它成为一个实例变量？