是否有任何工具可以帮助我们将 IEnumerator 属性重构为 IList<T> 或类似的？

Question

我们有一个非常古老的代码库（实际上质量并不差）。它可以追溯到 .Net 预发布的时候，我怀疑这是造成这些奇怪约定的原因。

无论如何，我们刚刚开始放弃对 .Net 1.1 的支持，并且正在将事物转换为泛型并使用 Linq 和所有有趣的东西。我们代码库中最烦人的模式之一是我们会有类似

private ArrayList mylist;
public IEnumerator MyList
{
  get
  {
    if(mylist==null)
      return new EmptyEnumerator.Enumerator;
    return mylist.GetEnumerator();
  }
}

这种模式特别可怕，因为它阻止我们简单地执行foreach(var item in MyList)，因为 IEnumerator 没有实现 IEnumerable。相反，我们必须这样做：

IEnumerator enumerator=MyList;
while(enumerator.MoveNext())
{
    object item=enumerator.Current;
}

所以，对于重构，我们当然希望使用 ReadOnlyCollection<T> 或 IList<T> 或类似的东西。但是，要做到这一点，我们必须更新对MyList 的每个引用：

IEnumerator enumerator=MyList;

到

IEnumerator enumerator=MyList.GetEnumerator();

在某些情况下，我们可以对一个属性有超过一百个引用。是否有任何工具可以使这更容易？我们最近得到了 Resharper（不是为了这个问题，只是为了一般用途），但它似乎没有涵盖这种类型的场景。

Answer 1

听起来你需要做的是返回一个实现IEnumerator和IEnumerable<T>的类

制作你自己的类型实际上并不难：

public class MessedUpIterator<T> : IEnumerable<T>, IEnumerator
{
    private IEnumerable<T> source;
    private IEnumerator enumerator;

    private IEnumerator MyEnumerator
    {
        get
        {
            return enumerator ?? source.GetEnumerator();
        }
    }

    public MessedUpIterator(IEnumerable<T> source)
    {
        this.source = source;
    }
    public IEnumerator<T> GetEnumerator()
    {
        return source.GetEnumerator();
    }

    IEnumerator IEnumerable.GetEnumerator()
    {
        return source.GetEnumerator();
    }

    object IEnumerator.Current
    {
        get { return MyEnumerator.Current; }
    }

    bool IEnumerator.MoveNext()
    {
        return MyEnumerator.MoveNext();
    }

    void IEnumerator.Reset()
    {
        MyEnumerator.Reset();
    }
}

现在，您可以返回 both 的东西，而不是返回 任一 IEnumerator 或 IEnumerable<T>。

请注意，IEnumerator 是显式实现的，而 IEnumerable<T> 是隐式实现的，因此它鼓励将其用作 IEnumerable，同时仍将其用作 IEnumerator 可能。

是的，它很丑，但肯定会更糟。