Enities & Datatable

Jeg vil mene at du enten skal bruge DataSet/DataTable eller Entity Framework - du skal ikke forsøge at blande dem.

Problemet er jeg ik aner hvordan pokker jeg så henter alle Categories ud vha. Entities :)

public IList<ForumModel.Category> GetCategories()
    {
        IList<ForumModel.Category> Categories = _entities.Categories.ToList();
        return Categories;
   
    }

Måske noget ala over stående ... uden at være sikker.

Fik klaret den selv :)

svaret var følgende :)

    public List<Category> GetCategories()
    {
        return _entities.Categories.ToList<Category>();
    }

Ja, næsten som jeg også skrev.

En bedre løsning ville klart være at bruge en iterator , dvs. IEnumerable<Category>, frem for at bruge en liste.

Problemet er overordnet at _entities.Categories er et query object, som når du henter data, bliver transformeret direkte til SQL, frem for at blive transformeret i memory.

Ved at bruge en IEnumerable opnår du generelt bedre kode, og sikre at fremtigt brug af dit repository er mere fornuftigt og konsistent.

Samt du muligvis undgå dobbelt memoryforbrug.

Der er sådan set heller ingen grund til at pakke din data ind i en DataTabel, da LINQ2SQL/EntityFramework netop er til for at undgå dette. Du kan sagtens databinde direkte til en IEnumerable<T>

så altså:

Tror ikke han er helt henne i de tanker endnu, da der tydeligt var andre problemer der kom først :-) ... men fight bad code er altid godt.

Hvem siger ikke det er kald fra BS ... hvor alle categories skal bruges. Uden at kende context hvor det bruges, synes jeg ikke man kan kan sige hvad der skal bruges. Husk der er både pros/cons i at bruge IQueryable etc.

Men også hvis alle kategorierne skal bruges, vil jeg stadigvæk bruge en IEnumerable.

Og ja, det kræver at man holder tungen lige i munden at bruge IQueryable. Men generelt er det fornuftigt at gennemtænke logikken.

Modsat så tror jeg ikke at mjdigital helt er klar over den dybere mening. Så hvis vi bare kan sikre os at han ikke bruger DataTable og DataReaders, er det fint fint :)

Scott Hanselman skrev en fint post om det for et stykke tid siden, http://www.hanselman.com/blog/ExtendingNerdDinnerExploringDifferentDatabaseOptions.aspx

Ja, det skal gennemtænkes og du skal sikre dig kloge programmører i den anden ende :-)

Da den stadig ikke er transformed, kan du jo bygge videre på den query hvilket kan gå hen og give perf issues. Men ja, ved alle hvad der sker, så er det intet problem.

Vil da lige kigge forbi Scott.

Tak for svarene :-) og bare rolig ingen datatable... vil da klart lige kigge på det anden... og kalder pt. Min liste ind som et objekt og binder den..