Jak dobrze używać Data Contract w WCF

0

Wałkuje od pewnego czasu WCF, a teraz zaczynam pisać aplikację opartą o SOA i nie mogę dojść jak to jest z atrybutem [DataContract] - jak tego używać.
Załóżmy że mam rozdzielony projekt na różne komponenty. Na przykład stworzyłem sobie bibliotekę klas powiedzmy MyApp.Contracts w której mam między innymi DataContract.Mam też bibliotekę klas z Encjami na podstawie których EF utworzy sobie bazę danych. Powiedzmy że mam encję Person oraz encję Car. Te dwie encje połączone są ze sobą relacją jeden do wielu.

 
public class Person
{
	public int PersonId {get; set;}
	public string FirstName {get; set;}
	public string LastName {get; set;}

	public virtual ICollection<Car> Cars {get; set;}
}

Mój Data Contract

 
public class PersonData
{
	public int PersonId {get; set;}
	public string FirstName {get; set;}
	public string LastName {get; set;}

	public virtual ICollection<CarData> Cars {get; set;} //czy ta linijka jest poprawna ? chodzi mi o to CarData ?
}

To jest mój serwis

 

public class DataService : IDataService
{
	PersonRepository repository = new PersonRepository();

	public IEnumerable<PersonData> GetAll()
	{
		IEnumerable<Person> result = repository.GetAll();

		//TUTAJ MAPUJE ENCJE PERSON NA PERSONDATA (DataContract)
		
		IList<PersonData> resultData = new List<PersonData>();

            	foreach (var p in result)
            	{
                	resultData .Add(new PersonData { FirstName = p.FirstName, LastName = p.LastName, Cars = p.Cars });
            	};

            return resultData;
	}
}		

Kontroler w aplikacji Klienta -ASP.NET MVC

 

public ActionResult DisplayAll()
        {
            IEnumerable<PersonData> list= proxy.GetAll();

            List<DisplayAllViewModel> viewModel = new List<DisplayAllViewModel>();

            foreach(var x in list)
            {
                viewModel.Add(new DisplayAllViewModel { FirstName = x.FirstName, LastName = x.LastName });
            }

            return View(viewModel);
        }

Czego tu nie rozumiem? Czy mam oznaczyć sobie moje Encje, czyli klasę Person i Car jako [DataContract] i [DataMember], czy mapować ją w relacji 1:1 na PersonData, a encje pozostawić bez żadnych atrybutów związanych z WCF?

Jeżeli powinienem dać atrybuty [DataContract] i [DataMember] na encje to teraz w repozytorium mam mieć powiedzmy dwie różne metody? np:

 
public class DataService : IDataService
{
	PersonRepository repository = new PersonRepository();

	public IEnumerable<PersonData> GetSelected()
	{
		IEnumerable<Person> result = repository.GetAll();

		//TUTAJ MAPUJE ENCJE PERSON NA PERSONDATA (DataContract)
		
		IList<PersonData> resultData = new List<PersonData>();

            	foreach (var p in result)
            	{
                	resultData .Add(new PersonData { FirstName = p.FirstName });
            	};

            return resultData;
	}

	public IEnumerable<Person> GetAll()
	{
		IEnumerable<Person> result = repository.GetAll();

		return result;
	}
}	

Proszę o jakieś wytłumaczenie jak postępować z data kontraktami. Dzięki!

5
  • Repozytorium powinno operować wyłącznie na encjach.
  • Encje biznesowe nie powinny mieć żadnych zależności od infrastruktury, czyli też nie posiadać żadnych atrybutów wiążących je z WCF, EF, WTF, ani innymi takimi.
  • Jeśli nie masz encji biznesowych tylko obiekty z ORMa, to taką klasę do operowania na nich prawidłowo należałoby nazywać DAO, a nie repozytorium.
  • Masz klasę PersonData, to ona jest Twoim DataContract, i to jest to, co zwracasz z WCFa, nie encje biznesowe, ani nie obiekty mapowane przez EF na bazę danych.

Uwagi dodatkowe:

  • Nie ma czegoś takiego "relacja jeden do wielu". Jeden do wielu to może być krotność związku encji na diagramie ERD. w przypadku powiązań między obiektami mówi się raczej o agregacjach.
  • Jaki jest sens istnienia metody GetAll w repozytorium? Czy jeśli w tabeli będzie milion rekordów, to też będziesz używał takiej metody?
  • Takie coś:
IList<PersonData> resultData = new List<PersonData>(); 
                foreach (var p in result)
                {
                    resultData .Add(new PersonData { FirstName = p.FirstName, LastName = p.LastName, Cars = p.Cars });
                };

można zapisać prościej:

var resultData = result.Select(p => new PersonData{ FirstName = p.FirstName, LastName = p.LastName, Cars = p.Cars });
1

I koniecznie zainteresuj się AutoMapper http://automapper.org/
W Twoim przykładzie mapujesz Person do PersonData, są tylko 3 property, więc powiedzmy, że zapiszesz krótko, ale jak będziesz miał klasy po 50 propert to będzie katorga.
Nie mówiąc już o tym, że możesz mapować w wielu miejscach aplikacji i będziesz kod powtarzał.

Z AutoMapper robisz definicje mapowania i potem jak używasz to masz coś takiego:

var personData = mapper.Map<PersonData>(person);
0

W takim razie po co i do czego używa się encji biznesowych skoro do ORMa z tego co rozumiem muszę tworzyć własne osobne obiekty ? Jak to powinno wyglądać w strukturze aplikacji w sensie gdzie takie obiekty powinny być?

Skoro w repozytorium operuje się na encjach a w EF używa się obiektów ORM to ja mam mapować w repo obiekty ORM na encje ?

 
DbSet<ObiektORM> ObiektORM { get; set; }

to w repozytorium nie napiszę sobie

Encja GetEncja(int id)
{
	return context.ObiektORM.First(x => x.id == id);
}
 

Jak używam EF i mam encje biznesowe, to w przypadku jak użyje Data Annotations przestają one być już encjami tylko stają się obiektami ORMa, tak samo nie zaznaczę sobie powiązań między encjami, bo wtedy też już to przestanie być encją z tego co rozumiem. Natomiast jeżeli do powyższych zadań użyje Fluent API w DataContext, to cały czas moje encje pozostaną encjami biznesowymi ?

Czyli Data Contracty przważnie będą odzwierciedleniem obiektów mapowanych przez EF na bazę danych, no chyba że np. nie potrzebuje wszystkich właśiwości to wybieram sobie te potrzebne ?

Dzięki za uwagi ;)

2
RideorDie napisał(a):

W takim razie po co i do czego używa się encji biznesowych

Do stworzenia modelu biznesowego tego, do czego służy aplikacja. Encjami są np.: Klient, Faktura, Towar.

skoro do ORMa z tego co rozumiem muszę tworzyć własne osobne obiekty

Nie musisz - możesz mieć encje biznesowe zmapowane na relacje, możesz mieć oddzielny zestaw klas modelujących warstwę składowania danych. Wszystko zależy od konkretnego przypadku.

Skoro w repozytorium operuje się na encjach a w EF używa się obiektów ORM to ja mam mapować w repo obiekty ORM na encje ?

Jeśli to będą inne obiekty, to tak, musisz je jakoś zmapować.

Jak używam EF i mam encje biznesowe, to w przypadku jak użyje Data Annotations przestają one być już encjami tylko stają się obiektami ORMa, tak samo nie zaznaczę sobie powiązań między encjami, bo wtedy też już to przestanie być encją z tego co rozumiem. Natomiast jeżeli do powyższych zadań użyje Fluent API w DataContext, to cały czas moje encje pozostaną encjami biznesowymi ?

Ogólnie tak - model biznesowy powinien nie mieć związku z infrastrukturą, taką jak ORM. Mieszanie tych odpowiedzialności utrudnia testowanie, refaktoryzację i zaciemnia kod. Warto trzymać się zasady "persistence ignorance".

Czyli Data Contracty przważnie będą odzwierciedleniem obiektów mapowanych przez EF na bazę danych, no chyba że np. nie potrzebuje wszystkich właśiwości to wybieram sobie te potrzebne ?

Data contracty maja odzwierciedlać to, co ma trafić do klienckiej aplikacji przez web serwis. Mogą mieć strukturę identyczną z tabela mi bazy danych, a mogą mieć zupełnie inną. To wszystko zależy od aplikacji.

0

Jest jeszcze jedna rzecz która do końca nie wiem jak powinna być zrobiona. Chodzi o powiązania pomiędzy obiektami Data Contract. Chciałbym żeby Data Contracty zachowywały się jak połączone między sobą obiekty ORM, czyli serwis zwraca mi PersonData do klienta i mogę sobie wypisać wszystkie samochody które należą do tej osoby bez kolejnego kontaktu z serwisem i wyciągnięcia listy samochodów (CarData) po id usera.
Mało na ten temat jest w internecie, albo nie mogę po prostu trafić z hasłem. Znalazłem taki fajny tutorial:
http://www.exceptionnotfound.net/entity-framework-and-wcf-loading-related-entities-with-automapper-and-reflection/
i chciałem się zapytać czy właśnie taki sposób jest OKej, czy może da się to zrobić w jakiś prostszy sposób ?

1 użytkowników online, w tym zalogowanych: 0, gości: 1