一些私有方法公开iQuery<t>和所有公共方法公开IEnumerable<t>有什么问题吗?

我想知道是否有更好的方法来解决这个问题。目的是重用代码。

假设我有一个linq-to-sql数据上下文,我编写了一个“知识库风格”类,它包含了我需要的许多方法,并公开了iqueryables。(到目前为止,没问题)。

现在,我正在构建一个服务层来放置在这个存储库的顶部,其中许多服务方法将是1<->1和存储库方法,但有些方法不会。我认为代码示例比文字更能说明这一点。

public class ServiceLayer 
{
    MyClassDataContext context;
    IMyRepository rpo;

    public ServiceLayer(MyClassDataContext ctx) 
    { 
        context = ctx; 
        rpo = new MyRepository(context);   
    }

    private IQueryable<MyClass> ReadAllMyClass()
    {
        // pretend there is some complex business logic here
        // and maybe some filtering of the current users access to "all"
        // that I don't want to repeat in all of the public methods that access
        // MyClass objects.
        return rpo.ReadAllMyClass();
    }

    public IEnumerable<MyClass> GetAllMyClass()
    {
        // call private IQueryable so we can do attional "in-database" processing
        return this.ReadAllMyClass();
    }

    public IEnumerable<MyClass> GetActiveMyClass()
    {
        // call private IQueryable so we can do attional "in-database" processing
        // in this case a .Where() clause
        return this.ReadAllMyClass().Where(mc => mc.IsActive.Equals(true));
    }

    #region "Something my class MAY need to do in the future"
    private IQueryable<MyOtherTable> ReadAllMyOtherTable()
    {
        // there could be additional constrains which define
        // "all" for the current user
        return context.MyOtherTable;
    }

    public IEnumerable<MyOtherTable> GetAllMyOtherTable()
    {
        return this.ReadAllMyOtherTable();
    }

    public IEnumerable<MyOtherTable> GetInactiveOtherTable()
    {
        return this.ReadAllMyOtherTable.Where(ot => ot.IsActive.Equals(false));
    }
    #endregion

}

这个特殊的例子并不是最好的例子,因为我可以直接在getActiveMyclass方法中调用存储库,但是我们假设我的私有iqueryable执行了一些额外的处理和业务逻辑,我不想在两个公共方法中复制它们。

这是攻击这样一个问题的坏方法吗?我不认为它如此复杂,以至于它真的需要构建一个第三类来介于存储库和服务类之间,但我想听听您的想法。

为了争论,让我们假设另外两件事。

1. 此服务将通过WCF公开,并且这些公共IEnumerable方法中的每一个都将调用 .Select(m => m.ToViewModel()) 在每个返回的集合上,将其转换为POCO进行序列化。
2. 服务最终需要公开一些 context.SomeOtherTable 不会被打包到存储库中。