将产量线提取到通用方法中

Question

我们在代码中使用自动生成的代码（带有存储库模式的 SubSonic3），并且有很多这样的行。

public IEnumerable<MyModels.StatusLookup> GetAll()
    {
        var results = Database.Current.pStatusLookupLoadAll()
            .ExecuteTypedList<MyModels.StatusLookup>();
        if (results.IsNull()) yield break;
        foreach (var m in results)
        {
            ..Common logic lines...
            ..Common logic lines...
            yield return m;
        }
    }

我想做的是将产量线重构为通用方法。但我不知道我是否可以，因为产量的工作方式。

然后，当我们有自定义代码调用存储库自动生成的代码之外的数据库时，我可以在加载的模型对象上调用这个通用方法。

public IEnumerable<Books> GetByFancy(int anInteger)
{
    DB db = Database.Current;
    var r = from b in db.Books
            join a in db.Authors on b.AuthorId equals a.AuthorId
            where a.AuthorId == anInteger
            select b;

    if (r.IsNull()) yield break;
    foreach (var m in r)
    {
        m.AcceptChanges();
        yield return m;
    }
}

因此，上面的示例中有公共重复行，我想在其中进行公共方法调用来删除公共重复代码行。

---

这是我得到的例外。

System.InvalidCastException : Unable to cast object of type '<AcceptChangesAndYield>d__6' to type 'System.Collections.Generic.IEnumerable`1[MyModels.StatusLookup]'.

public IEnumerable<MyModels.StatusLookup> GetAll()
{
    var results = Database.Current.pStatusLookupLoadAll()
        .ExecuteTypedList<MyModels.StatusLookup>();
    return (IEnumerable<MyModels.StatusLookup>)results.AcceptChangesAndYield();
}

这是我尝试过的扩展方法。

public static class BaseModelExtensions
{
    public static IEnumerable<MyModels.BaseModel> AcceptChanges(this IEnumerable<MyModels.BaseModel> obj)
    {
        if (obj.IsNull()) yield break;
        foreach (var m in obj)
        {
            m.AcceptChanges();
            yield return m;
        }
    }

    public static IEnumerable<MyModels.Interfaces.ILookup> AcceptChangesAndYield(this IEnumerable<MyModels.Interfaces.ILookup> obj)
    {
        if (obj.IsNull()) yield break;
        foreach (var m in obj)
        {
            yield return m;
        }
    }
}

Answer 1

更新：您的问题与 yield 关键字无关。它与类型差异有关。

您的 AcceptChangesAndYield 方法返回一个实现 IEnumerable类型的对象（事实上，它是编译器生成的类型，但这并不重要）。在您的方法调用中，您尝试将其向下转换为IEnumerable，这是更具体的。

IEnumerable 接口是协变的，它允许您向上转换为不太具体的类型；例如，您可以从 List 转换为 IEnumerable

幸运的是，解决方案非常简单。重新定义您的 AcceptChangesAndYield 方法，如下所示：

// Note: We are using a generic type constraint on T.
public static IEnumerable<T> AcceptChangesAndYield<T>(this IEnumerable<T> obj)
    where T : MyModels.Interfaces.ILookup
{
    if (obj.IsNull()) yield break;
    foreach (var m in obj)
    {
        // Did you mean to put m.AcceptChanges() here?
        yield return m;
    }
}

这将依次允许您的 GetAll 方法实现如下：

public IEnumerable<MyModels.StatusLookup> GetAll()
{
    var results = Database.Current.pStatusLookupLoadAll()
        .ExecuteTypedList<MyModels.StatusLookup>();
    // Note: no need for a cast, as the return value is now
    // already strongly typed as IEnumerable<MyModels.StatusLookup>.
    return results.AcceptChangesAndYield();
}

---

原始答案：看来您只是想要这个吗？

IEnumerable<T> EnumerateResults<T>(IEnumerable<T> results)
{
    if (results.IsNull()) yield break;
    foreach (T result in results)
    {
        // ..Common logic lines...
        yield return result;
    }
}

然后在您想要删除重复的代码中，您只需：

// Specific stuff
var results = BlahBlahBlah();

// Common stuff
return EnumerateResults(results);

对吗？还是我误解了你的问题？