Trying to order LINQ results

Question

To one of my tables I added a sortOrder. I now need to update this query to respect the sortOrder authority!

Here is my current query:

var HTMLdocs = db.ProcedureDocs.Where(m => m.Procedures.Processes.processID == id && m.Document.html != null && m.Document.approvedBy != null).Select(m => m.Document);

The table that has sortOrder is Procedures.

How can I get the above query to return results that are ordered by sortOrder.

Here is the table structure.

Processes can have 0 or many Procedures. Document can have 0 or many ProcedureDocs. ProcedureDocs has a foreign key to Procedures on procedureid.

I somehow need to grab a collection of Document that somehow respects the order found in the Procedures table.

Let me know if you need any other information.

p.s.w.g · Accepted Answer · 2013-09-06 19:56:23Z

3

Try something like this:

var HTMLdocs = db.ProcedureDocs .Where(m => m.Procedures.Processes.processID == id && m.Document.html != null && m.Document.approvedBy != null) .OrderBy(m => m.Procedures.sortOrder) .Select(m => m.Document);

Or in query syntax:

var HTMLdocs = from m in db.ProcedureDocs where m.Procedures.Processes.processID == id && m.Document.html != null && m.Document.approvedBy != null orderby m.Procedures.sortOrder select m.Document;

answered Sep 6, 2013 at 19:56

p.s.w.g

150k31 gold badges307 silver badges339 bronze badges

Sign up to request clarification or add additional context in comments.

9 Comments

MarcinJuraszek Over a year ago

I think m.Procudures is a collection (because of 0 or more)

p.s.w.g Over a year ago

@MarcinJuraszek I thought that initially (because it's plural), but OP's current query uses m.Procedures.Processes.processID == id. If the current code compiles at all, it probably isn't a collection.

MarcinJuraszek Over a year ago

Didn't notice that. But that's why correct naming is a key in writing good code...

James Wilson Over a year ago

@MarcinJuraszek How should it have been named? The Procedures table houses a collection of Procedures. Should I have named it Procedure?

MarcinJuraszek Over a year ago

If it's just a single Procedure from Procedures table then yes, the property should be called Procedure.

|

Dan Hunex · Accepted Answer · 2013-09-06 20:03:56Z

var HTMLdocs = db.ProcedureDocs .Where(m => m.Procedures.Processes.processID == id && m.Document.html != null && m.Document.approvedBy != null) .OrderBy(x=>x.Procedures.sortOrder) .Select(m => m.Document)

Collectives™ on Stack Overflow

Trying to order LINQ results

2 Answers 2

9 Comments

Comments

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

9 Comments

Comments

Related