---
title: "Feedback på kod och arkitektur"
type: "forum-thread"
url: "https://www.webforum.nu/amne/dotnet/173397-feedback-på-kod-och-arkitektur"
topic: ".NET"
topic_url: "https://www.webforum.nu/amne/dotnet"
author: "Lukaspojken"
published: "2008-08-17T15:28:27.000Z"
updated: "2008-08-18T21:44:01.000Z"
replies: 6
views: 499
page: 1
pages: 1
language: "sv-SE"
site: "webForum — webforum.nu"
rights: "Upphovsrätten till varje inlägg tillhör dess författare."
attribution: "Citera som: webForum, https://www.webforum.nu/amne/dotnet/173397-feedback-på-kod-och-arkitektur"
---

# Feedback på kod och arkitektur

## #1 — Lukaspojken, 2008-08-17T15:28Z

Under några dagar har jag testat att göra min första simpla Silverlight applikation. Jag har också försökt utgå från bland annat LINQ, Domain Driven Design och Test Driven Development. Det skulle vara intressant att få feedback på det jag gjort. Jag är ingen expert i området men vill gärna lära mig mer. 

Du finner koden och spelet på följande address: 
<http://kauppi.levasunt.nu/2008/08/13/minnesspel-i-silverlight>

Permalänk: https://www.webforum.nu/p/173397

## #2 — Nickemannen, 2008-08-18T12:38Z

I Game har du dessa fält som du kan bryta ut till ett eget om man skall följa DDD.  

```
 
        public int Difficulty { get; set; }
        [DataMember]
        public string DifficultyName { get; set; }

ex:
       public class Difficulty
       {
            public string Name { get; set; }
            public int Level { get; set; }
       }
```

Kanske skall heta ChangePlayer eftersom den egentligen bara byter spelare.
Egentligen borde det vara
ChangePlayer(Player player); och att du skapar upp Player någon annanstans.

```
        public void AddPlayer(string Name, string ip)
        {
            Player player = new Player();
            player.Name = Name;
            player.IP = ip;

            Player = player;
        }
```

Dina factories, gör hela klassen internal istället för bara metoden.

Kollade bara på BLL en snabbis, funderar lite på GameLevel och Game repositorierna och undrar om dom egentligen behövs?

En annan sak som kan vara en smaksak. Microsoft döper sina singleton properties till Current

Så istället för GetIntance(), gör en Property kallad Current. :)

Permalänk: https://www.webforum.nu/p/2133932

## #3 — Lukaspojken, 2008-08-18T14:24Z

Tack för din feedback! En difficulty klass låter som en bra idé. Jag ska göra lite ändringar utifrån det du tagit upp. Återkommer när detta är klart!

Permalänk: https://www.webforum.nu/p/2133942

## #4 — Lukaspojken, 2008-08-18T17:29Z

Nu har jag fixat till de ändringar du tog upp. Jag valde dock att kalla den nya klassen för GameType.

Tycker du det är ok att att returnera IEnumerable\<DataRow\> i MyMemoryDAL -\> GameResult.cs? Eller finns det något bättre sätt att göra detta på?

Permalänk: https://www.webforum.nu/p/2133982

## #5 — Nickemannen, 2008-08-18T18:32Z

Jag har bara kollat lite på BLL, men jag tycker alltid att det är bättre att skippa DataTable's.

Jag pushar alltid för NHibernate så jag får göra det här med :).
Alternativt nu när det är så få klasser så kan du ju implemmentera den själv med sql.

Permalänk: https://www.webforum.nu/p/2133997

## #6 — Lukaspojken, 2008-08-18T19:05Z

Jag gillar heller inte datatabeller. Att implementera NHibernate vore inte dumt. Ska se om jag kan få till det. Har aldrig jobbat med det tidigare.

Permalänk: https://www.webforum.nu/p/2134006

## #7 — Nickemannen, 2008-08-18T21:44Z

Använd inte Nhibernate.org där ligger bara den gamla, använd <http://sourceforge.net/project/showfiles.php?group_id=73818>.

Sedan finns det massor av information om det på nätet bara att söka, det finns en bok som beskriver riktigt bra också NHibernate in Action.

Permalänk: https://www.webforum.nu/p/2134034

---

Tråden på webben: https://www.webforum.nu/amne/dotnet/173397-feedback-på-kod-och-arkitektur