[c#]switch .OrderBy() efficienter?

dinsdag 20 december 2011 08:23

Acties:

0 Henk 'm!

mmniet

De beetje weter

Topicstarter

Hallo,

Ik heb de volgende code geschreven

code:

List<HotelService.DataAccess.Reservation> reservationList;
            switch (orderBy)
            { 
                case "ID":
                    if (sortOrder == "ascending") {reservationList = query_total.OrderBy(query => query.ID).Skip(skipCount).Take(countPerPage).ToList();}
                    else {reservationList = query_total.OrderByDescending(query => query.ID).Skip(skipCount).Take(countPerPage).ToList();}
                    break;
                case "name":
                    if (sortOrder == "ascending") {reservationList = query_total.OrderBy(query => query.Name).Skip(skipCount).Take(countPerPage).ToList(); }
                    else {reservationList = query_total.OrderByDescending(query => query.Name).Skip(skipCount).Take(countPerPage).ToList(); }
                    break;
                case "email":
                    if (sortOrder == "ascending") {reservationList = query_total.OrderBy(query => query.Email).Skip(skipCount).Take(countPerPage).ToList(); }
                    else {reservationList = query_total.OrderByDescending(query => query.Email).Skip(skipCount).Take(countPerPage).ToList(); }
                    break;
                case "hotelname":
                    if (sortOrder == "ascending") {reservationList = query_total.OrderBy(query => query.HotelID).Skip(skipCount).Take(countPerPage).ToList(); }
                    else {reservationList = query_total.OrderByDescending(query => query.HotelID).Skip(skipCount).Take(countPerPage).ToList(); }
                    break;
                case "roomtype":
                    if (sortOrder == "ascending") {reservationList = query_total.OrderBy(query => query.RoomTypeID).Skip(skipCount).Take(countPerPage).ToList(); }
                    else {reservationList = query_total.OrderByDescending(query => query.RoomTypeID).Skip(skipCount).Take(countPerPage).ToList(); }
                    break;
                case "checkin":
                    if (sortOrder == "ascending") {reservationList = query_total.OrderBy(query => query.CheckinTime).Skip(skipCount).Take(countPerPage).ToList(); }
                    else {reservationList = query_total.OrderByDescending(query => query.CheckinTime).Skip(skipCount).Take(countPerPage).ToList(); }
                    break;
                case "checkout":
                    if (sortOrder == "ascending") {reservationList = query_total.OrderBy(query => query.CheckoutTime).Skip(skipCount).Take(countPerPage).ToList(); }
                    else {reservationList = query_total.OrderByDescending(query => query.CheckoutTime).Skip(skipCount).Take(countPerPage).ToList(); }
                    break;
                case "numberofadult":
                    if (sortOrder == "ascending") {reservationList = query_total.OrderBy(query => query.NumberOfAdults).Skip(skipCount).Take(countPerPage).ToList(); }
                    else {reservationList = query_total.OrderByDescending(query => query.NumberOfAdults).Skip(skipCount).Take(countPerPage).ToList(); }
                    break;
                case "channel":
                    if (sortOrder == "ascending") {reservationList = query_total.OrderBy(query => query.Channel).Skip(skipCount).Take(countPerPage).ToList(); }
                    else {reservationList = query_total.OrderByDescending(query => query.Channel).Skip(skipCount).Take(countPerPage).ToList(); }
                    break;
                case "clientreference":
                    if (sortOrder == "ascending") {reservationList = query_total.OrderBy(query => query.ClientReference).Skip(skipCount).Take(countPerPage).ToList(); }
                    else {reservationList = query_total.OrderByDescending(query => query.ClientReference).Skip(skipCount).Take(countPerPage).ToList(); }
                    break;
                case "status":
                    if (sortOrder == "ascending") {reservationList = query_total.OrderBy(query => query.Status).Skip(skipCount).Take(countPerPage).ToList(); }
                    else {reservationList = query_total.OrderByDescending(query => query.Status).Skip(skipCount).Take(countPerPage).ToList(); }
                    break;
                default:
                    if (sortOrder == "ascending") {reservationList = query_total.OrderBy(query => query.Created).Skip(skipCount).Take(countPerPage).ToList(); }
                    else {reservationList = query_total.OrderByDescending(query => query.Created).Skip(skipCount).Take(countPerPage).ToList(); }
                    break;
            }

maar vind dit een grote lap tekst. Kan dit efficienter? Dus in de zin van

code:

1	query_total.OrderBy(query => <mijn-variabele-met-naam-kolom-als-waarde>)......

Heb gezocht op google, maar kwam vrijwel altijd de switch optie tegen, maar vind die niet zo heel prettig omdat dit veel code herhaling bevat.

It's me Mario

dinsdag 20 december 2011 08:32

Acties:

0 Henk 'm!

evolution536

besh besh

Kun je me vertellen wat je met deze code wilt? Dat maakt het voor ons wat makkelijker om mee te denken.

dinsdag 20 december 2011 08:34

Acties:

0 Henk 'm!

asfaloth_arwen

Zelf heb ik wel eens iets gebruikt op basis van: http://aonnull.blogspot.c...nq-orderby-extension.html

Specs

dinsdag 20 december 2011 08:36

Acties:

0 Henk 'm!

mmniet

De beetje weter

Topicstarter

het gaat hier om het sorteren van een tabel na het klikken op een bepaalde kolom

It's me Mario

dinsdag 20 december 2011 08:40

Acties:

0 Henk 'm!

Pin0

Als je je orderBy die je dus meegeeft hetzelfde noemt als je kolomnaam kan je dat dus als variabele in je query gebruiken (wel valideren!)
Kan dat niet dan zou je ook nog een array met een mapping oid. kunnen bijhouden met daarin de orderBy en de kolomnaam.

Mijn Lego Mocs - LEGO idea: The Motorcycle Garage

dinsdag 20 december 2011 08:49

Acties:

0 Henk 'm!

mmniet

De beetje weter

Topicstarter

maar hoe gebruik je dan die variabele naam? ik krijg het niet voor elkaar

It's me Mario

dinsdag 20 december 2011 09:10

Acties:

0 Henk 'm!

D-Raven

Je zou een dictionary kunnen gebruiken als mapping, waarin je de key als column naam hebt en een Action<T> als value.

Dan kun je dus zoiets doen:

C#:

mappingDict["status"](reservationList);

//waar de value dus iets als dit is
var statusActionVoorbeeld = new Action<List<vaniets>>(l => l.Orderby(x => x.status));

dinsdag 20 december 2011 09:22

Acties:

0 Henk 'm!

RayNbow

Kirika <3

D-Raven schreef op dinsdag 20 december 2011 @ 09:10:
Je zou een dictionary kunnen gebruiken als mapping, waarin je de key als column naam hebt en een Action<T> als value.

In plaats van Action<T> kun je er ook Func<Reservation, object> instoppen:

C#:

class Reservation
{
    public int ID;
    public string Name;
    public string Email;

    public override string ToString()
    {
        return string.Format("Reservation({0}, {1}, {2})", ID, Name, Email);
    }
}

class Program
{
    static void Main(string[] args)
    {
        var reservationList = new List<Reservation> {
            new Reservation() {ID = 3, Name = "Three", Email = "foo@bar.org"},
            new Reservation() {ID = 4, Name = "Four",  Email = "bar@123.org"},
            new Reservation() {ID = 5, Name = "Five",  Email = "rab@oof.org"},
        };

        var sortOrder = "descending";
        var orderBy = "name";
        var skipCount = 0;
        var countPerPage = 100;

        // deze dict vervangt in feite de switch
        var columns = new Dictionary<string, Func<Reservation, object>>() {
            {"ID",    x => x.ID},
            {"name",  x => x.Name},
            {"email", x => x.Email},
        };

        reservationList =
            (sortOrder == "ascending"
                ? reservationList.OrderBy(columns[orderBy])
                : reservationList.OrderByDescending(columns[orderBy])
            )
            .Skip(skipCount)
            .Take(countPerPage)
            .ToList();

        PrintIEnumerable(reservationList);
        Console.ReadKey();
    }

    static void PrintIEnumerable<T>(IEnumerable<T> xs)
    {
        var s = String.Join(",\n", xs.ToArray());
        Console.WriteLine(s);
    }
}

Ipsa Scientia Potestas Est
NNID: ShinNoNoir

dinsdag 20 december 2011 09:54

Acties:

0 Henk 'm!

D-Raven

RayNbow schreef op dinsdag 20 december 2011 @ 09:22:
[...]

In plaats van Action<T> kun je er ook Func<Reservation, object> instoppen:
...
// deze dict vervangt in feite de switch
var columns = new Dictionary<string, Func<Reservation, object>>() {
{"ID", x => x.ID},
{"name", x => x.Name},
{"email", x => x.Email},
};

..

Dit gaat voor problemen zorgen zodra je op deze manier een DateTime wilt gaan sorten.

dinsdag 20 december 2011 10:40

Acties:

0 Henk 'm!

RayNbow

Kirika <3

D-Raven schreef op dinsdag 20 december 2011 @ 09:54:
[...]

Dit gaat voor problemen zorgen zodra je op deze manier een DateTime wilt gaan sorten.

Op welke manier gaat dat fout dan?

C#:

class Reservation
{
    public int ID;
    public string Name;
    public string Email;

    public DateTime CreationDate;

    public override string ToString()
    {
        return string.Format("Reservation({0}, {1}, {2}, {3})", ID, Name, Email, CreationDate);
    }
}

class Program
{
    static void Main(string[] args)
    {
        var reservationList = new List<Reservation> {
            new Reservation() {ID = 3, Name = "Three", Email = "foo@bar.org", 
                                CreationDate = DateTime.Now},
            new Reservation() {ID = 4, Name = "Four",  Email = "bar@123.org",
                                CreationDate = DateTime.Parse("01-01-2010")},
            new Reservation() {ID = 5, Name = "Five",  Email = "rab@oof.org",
                                CreationDate = DateTime.Parse("12-12-2009")},
        };

        var sortOrder = "descending";
        var orderBy = "";
        var skipCount = 0;
        var countPerPage = 100;


        // switch:
        var defaultColumn = "date";
        var columns = new Dictionary<string, Func<Reservation, object>>() {
            {"ID",    x => x.ID},
            {"name",  x => x.Name},
            {"email", x => x.Email},
            {"date", x => x.CreationDate},
        };
        var orderByFunc = columns[defaultColumn];
        if (columns.ContainsKey(orderBy))
            orderByFunc = columns[orderBy];
            
        reservationList =
            (sortOrder == "ascending"
                ? reservationList.OrderBy(orderByFunc)
                : reservationList.OrderByDescending(orderByFunc)
            )
            .Skip(skipCount)
            .Take(countPerPage)
            .ToList();

        PrintIEnumerable(reservationList);
        Console.ReadKey();
    }

    static void PrintIEnumerable<T>(IEnumerable<T> xs)
    {
        var s = String.Join(",\n", xs.ToArray());
        Console.WriteLine(s);
    }
}

Ipsa Scientia Potestas Est
NNID: ShinNoNoir

dinsdag 20 december 2011 10:45

Acties:

0 Henk 'm!

mmniet

De beetje weter

Topicstarter

ik heb de manier van RayNbow toegepast en werkt perfect! ook op datetimes

.

toppie en bedankt! Helder en goed voorbeeld! Kudo's!

It's me Mario

dinsdag 20 december 2011 11:29

Acties:

0 Henk 'm!

Sebazzz

3dp

Ik zou een Expression<Func<Reservation, object>> gebruiken. Waarom? Omdat je met de Func<Reservation, object> Enumerable.OrderBy aanroept en met Expression<Func<Reservation, object>> Queryable.OrderBy. In de praktijk betekent dit dat het ene via code werkt, en het andere via SQL/Database.

[Te koop: 3D printers] [Website] Agile tools: [Return: retrospectives] [Pokertime: planning poker]

dinsdag 20 december 2011 12:49

Acties:

0 Henk 'm!

D-Raven

RayNbow schreef op dinsdag 20 december 2011 @ 10:40:
[...]

Op welke manier gaat dat fout dan?

C#:

class Reservation
{
    public int ID;
    public string Name;
    public string Email;

    public DateTime CreationDate;

    public override string ToString()
    {
        return string.Format("Reservation({0}, {1}, {2}, {3})", ID, Name, Email, CreationDate);
    }
}

class Program
{
    static void Main(string[] args)
    {
        var reservationList = new List<Reservation> {
            new Reservation() {ID = 3, Name = "Three", Email = "foo@bar.org", 
                                CreationDate = DateTime.Now},
            new Reservation() {ID = 4, Name = "Four",  Email = "bar@123.org",
                                CreationDate = DateTime.Parse("01-01-2010")},
            new Reservation() {ID = 5, Name = "Five",  Email = "rab@oof.org",
                                CreationDate = DateTime.Parse("12-12-2009")},
        };

        var sortOrder = "descending";
        var orderBy = "";
        var skipCount = 0;
        var countPerPage = 100;


        // switch:
        var defaultColumn = "date";
        var columns = new Dictionary<string, Func<Reservation, object>>() {
            {"ID",    x => x.ID},
            {"name",  x => x.Name},
            {"email", x => x.Email},
            {"date", x => x.CreationDate},
        };
        var orderByFunc = columns[defaultColumn];
        if (columns.ContainsKey(orderBy))
            orderByFunc = columns[orderBy];
            
        reservationList =
            (sortOrder == "ascending"
                ? reservationList.OrderBy(orderByFunc)
                : reservationList.OrderByDescending(orderByFunc)
            )
            .Skip(skipCount)
            .Take(countPerPage)
            .ToList();

        PrintIEnumerable(reservationList);
        Console.ReadKey();
    }

    static void PrintIEnumerable<T>(IEnumerable<T> xs)
    {
        var s = String.Join(",\n", xs.ToArray());
        Console.WriteLine(s);
    }
}

Ik zou verwachten dat datums nu alfanumeriek vergeleken worden. Maar ik hebt mis. Spoke to soon

Ook al is het return type Object, blijkbaar word er nog steeds gekeken naar IComparable interface e.d.

dinsdag 20 december 2011 13:17

Acties:

0 Henk 'm!

RayNbow

Kirika <3

D-Raven schreef op dinsdag 20 december 2011 @ 12:49:
[...]

Ik zou verwachten dat datums nu alfanumeriek vergeleken worden. Maar ik hebt mis. Spoke to soon

Ook al is het return type Object, blijkbaar word er nog steeds gekeken naar IComparable interface e.d.

De OrderBy extension method legt geen restricties op het returntype van de meegegeven functie:

C#:

public static IOrderedEnumerable<TSource> OrderBy<TSource, TKey>(
    this IEnumerable<TSource> source,
    Func<TSource, TKey> keySelector
)

Zoals hierboven te zien wordt TKey verder nergens gebruikt. De documentatie vertelt trouwens dat de comparer wordt opgevraagd via de Comparer<TKey>.Default property.

Ipsa Scientia Potestas Est
NNID: ShinNoNoir

Onderwerpen