How to sort XML in LINQ C# by an attribute value? Also MVC

Question

(Using the latest MVC 2 RC 2) I'm trying to sort some XML in LINQ (C#) by an element's attribute's value...

var sites = from s in xDoc.Element("sites").Elements("site")
orderby s.Attribute("name")
select s;

But when I pass this to my View I get the exception:

Exception Details: System.ArgumentException: At least one object must implement IComparable.
Source Error: 
Line 37:                </th>
Line 38:            </tr>
Line 39:            <% foreach (var item in Model)
Line 40:               { %>
Line 41:            <tr>

Can someone tell me how to sort XML using LINQ and have it render properly?

Jon Skeet · Accepted Answer · 2010-02-05 17:34:39Z

12

EDIT: Okay, I think you just want:

var sites = from s in xDoc.Element("sites").Elements("site")
            orderby (string) s.Attribute("name")
            select s;

Which could also be written as:

var sites = xDoc.Element("sites")
                .Elements("site")
                .OrderBy(s => (string) s.Attribute("name"));

edited Feb 5, 2010 at 17:34

answered Feb 5, 2010 at 17:04

Jon Skeet

1.5m893 gold badges9.3k silver badges9.3k bronze badges

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

4 Comments

Matt W Over a year ago

Sorry, yes, you're correct- I had copied the wrong piece of code into my post! Argh. I've corrected it now.

Matt W Over a year ago

The 'site' variable is the value I'm passing into my view, eg: return View(site);

Vitali Climenco Over a year ago

You may want to replace orderby (string) s.Attribute("name") by orderby s.Attribute("name").Value

Jon Skeet Over a year ago

@VitaliClimenco: That depends on what you want to happen if the element doesn't have a name attribute. Using .Value it will throw an exception; using the cast it will just be ordered at the start (as nulls are).

Collectives™ on Stack Overflow

How to sort XML in LINQ C# by an attribute value? Also MVC

1 Answer 1

4 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

4 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related