Convert Extension methods to generic methods

Question

I was looking at some Linq queries with multiple join statements. I was thinking that they could be cleaned up using extension methods that hide away the join / where statements.

For example:

(from myObject in _context.MyObject
join myObjectGroupName in _context.MyObjectGroupNames on myObject.TaskGroupNameKey equals myObjectGroupName.ReportGroupNameKey
join myObjectResourceType in _context.MyObjectResourceTypes on myObject.ResourceKey equals myObjectResourceType.ResourceKey
where myObjectGroupName.ReportGroupName.Trim().ToUpper().Equals(groupName.Trim().ToUpper()) && fileTypes.Contains(myObjectResourceType.ResourceType)
select myObject.PermissionKey);

Now becomes:

_context.MyObject.JoinWhere(_context.MyObjectGroupNames, groupName)
                 .JoinWhere(_context.MyObjectResourceTypes, fileTypes)
                 .Select(x => x.PermissionKey);

Using:

public static IQueryable<MyObject> JoinWhere(this IQueryable<MyObject> initialObject, IQueryable<MyObjectGroupName> customerReportGroupNames, string groupName)
{
    return (from myObject in initialObject
        join myObjectGroupName in customerReportGroupNames on myObject.TaskGroupNameKey equals myObjectGroupName.ReportGroupNameKey
        where myObjectGroupName.ReportGroupName.Trim().ToUpper().Equals(groupName.Trim().ToUpper())
        select myObject);
}

public static IQueryable<MyObject> JoinWhere(this IQueryable<MyObject> initialObject, IQueryable<MyObjectResourceType> myObjectResourceTypes, IEnumerable<string> types)
{
    return (from myObject in initialObject
        join myObjectResourceType in myObjectResourceTypes on myObject.ResourceKey equals myObjectResourceType.ResourceKey
        where types.Contains(myObjectResourceType.ResourceType)
        select myObject);
}

After more consideration I was wondering if it would be possible to make these methods generic:

public static IQueryable<T> JoinWhere(this IQueryable<T> initialObject, IQueryable<TJoinOn> joinOn, IEnumerable<string> types)
{
    return (from myObject in initialObject
                                               //Here                      //Here
        join myObjectResourceType in joinOn on myObject.ResourceKey equals myObjectResourceType.ResourceKey
                             //Here
        where types.Contains(myObjectResourceType.ResourceType)
        select myObject);
}

Where I have put my comments are specific to specific objects. Could those be used generically or would I have to create an extension method for each dataObject making this WAY more work than it's really worth?

Not sure if your ORM supports this, but in general you can remove the need for a lot of joins altogther by fixing your entity mappings. ie instead of joining MyObjectGroupNames, you can go "MyObject.GroupNames.Where(x ==x...." for example. — Mitchell Lee
– Mitchell Lee, Commented Jul 31, 2014 at 15:07
I have been considering doing that. EF allows you to create partial classes in tandem with the generated objects it gives you. I was considering adding relationships to all of the objects. There are over 50 so I was hoping for a simpler solution — Robert
– Robert, Commented Jul 31, 2014 at 16:06
I've been using the Database First, Code Second approach (use Code First with an existing database) for all of my projects, and it works really well. I like how that allows for the abstraction of the POCO classes out of the Repository layer so that you can freely swap in and out any database storage layers and still use the same classes and not have to worry about partials. My worst one was only about 25 though, and it takes MAYBE 45 minutes per table/class to get everything done and running. — krillgar
– krillgar, Commented Jul 31, 2014 at 17:39
@MitchellLee I have decided that's what I am going to go with. Could you put your comment as an answer and I will accept it. — Robert
– Robert, Commented Aug 1, 2014 at 12:49

Mitchell Lee · Accepted Answer · 2014-10-12 19:25:27Z

1

Not sure if your ORM supports this, but in general you can remove the need for a lot of joins altogther by fixing your entity mappings.

ie instead of joining MyObjectGroupNames, you can go:

MyObject.GroupNames.Where(x == x....

answered Oct 12, 2014 at 19:25

Mitchell Lee

5601 gold badge3 silver badges7 bronze badges

Add a comment |

NPSF3000 · Accepted Answer · 2014-07-31 14:58:49Z

1

Have you considered using in Interface on MyObject?

public static IQueryable<T> JoinWhere(this IQueryable<T> initialObject, IQueryable<TJoinOn> joinOn, IEnumerable<string> types) 
where T : IMyObject
{
    return (from myObject in initialObject
                                               //Here                      //Here
        join myObjectResourceType in joinOn on myObject.ResourceKey equals myObjectResourceType.ResourceKey
                             //Here
        where types.Contains(myObjectResourceType.ResourceType)
        select myObject);
}

answered Jul 31, 2014 at 14:58

NPSF3000

7153 silver badges9 bronze badges

\$\begingroup\$ This is good except I would have to do that for all of my different objects from the database. The problem with that is, in its current structure the DB has 50+ tables \$\endgroup\$

Robert
– Robert

2014-07-31 16:05:13 +00:00
Commented Jul 31, 2014 at 16:05

Add a comment |

Stack Exchange Network

Convert Extension methods to generic methods

2 Answers 2

You must log in to answer this question.

Hot Network Questions

Convert Extension methods to generic methods

2 Answers 2

You must log in to answer this question.

Related

Hot Network Questions