Part 1
Ieri am promis ca revin cu un post despre ce nu ar trebuii sa contina un view.
Pornim de la o clasa PersonModel care are urmatoarea definitie:
Modelul nu contine toate datele necesare pentru a afisa tot ce este necesar. Din aceasta cauza se ajunge sa se faca un apel spre o componenta externa. Nu are importanta daca aceasta componenta este un serviciu extern sau o clasa din assemblyul nostru. Toate datele necesare trebuie sa fie continute de catre model. Orice data trebuie sa ajunga in view prin intermediul modelului.
O solutie este sa adaugam doua propietati in PersonModel care sa reprezinte coordonatele GPS( sau putem sa ne cream o clasa care sa stocheze longitudinea si latitudinea - dar tot ca si o propietate din PersonModel o sa fie).
Ieri am promis ca revin cu un post despre ce nu ar trebuii sa contina un view.
Pornim de la o clasa PersonModel care are urmatoarea definitie:
public class PersonModel
{
public string Name { get; set; }
public int Age { get; set; }
public string Address { get; set; }
}
@model PersonModel
@{
Layout = null;
}
<!DOCTYPE html>
<html>
<head>
<title>Person</title>
</head>
<body>
<fieldset>
<legend>PersonModel</legend>
<div class="display-label">Name</div>
<div class="display-field">
@Html.DisplayFor(model => model.Name)
</div>
<div class="display-label">Age</div>
<div class="display-field">
@Html.DisplayFor(model => model.Age)
</div>
<div class="display-label">Address</div>
<div class="display-field">
@Html.DisplayFor(model => model.Address)
</div>
</fieldset>
</body>
</html>
@model CIC.PersonModel
@{
Layout = null;
string coordLat;
string coordLong;
var request = WebRequest.Create(someAddress + "?location=" + Model.Address);
var webResponse = request.GetResponse();
using (var contentStream = new StreamReader(webResponse.GetResponseStream()))
{
var content = contentStream.ReadToEnd();
webResponse.Close();
string[] coords = content.Split(' ');
coordLong = coords[0];
coordLat = coords[1];
}
}
<!DOCTYPE html>
<html>
<head>
<title>Person</title>
</head>
<body>
<fieldset>
<legend>PersonModel</legend>
<div class="display-label">
Name</div>
<div class="display-field">
@Html.DisplayFor(model => model.Name)
</div>
<div class="display-label">
Age</div>
<div class="display-field">
@Html.DisplayFor(model => model.Age)
</div>
<div class="display-label">
Address</div>
<div class="display-field">
@Html.DisplayFor(model => model.Address)
</div>
<div class="display-label">
GPS Location</div>
<div class="display-field">
@coordLong
@coordLat
</div>
</fieldset>
</body>
</html>
Modelul nu contine toate datele necesare pentru a afisa tot ce este necesar. Din aceasta cauza se ajunge sa se faca un apel spre o componenta externa. Nu are importanta daca aceasta componenta este un serviciu extern sau o clasa din assemblyul nostru. Toate datele necesare trebuie sa fie continute de catre model. Orice data trebuie sa ajunga in view prin intermediul modelului.
O solutie este sa adaugam doua propietati in PersonModel care sa reprezinte coordonatele GPS( sau putem sa ne cream o clasa care sa stocheze longitudinea si latitudinea - dar tot ca si o propietate din PersonModel o sa fie).
public class PersonModel
{
public string Name { get; set; }
public int Age { get; set; }
public string Address { get; set; }
public string Longitude { get; set; }
public string Latitude { get; set; }
}
La capitolul asta - care sunt responsabilitatile controller-ului, programatorii inca au diverse pareri :)
ReplyDeleteCa model-ul e incarcat dintr-un web service, database sau fisier nu e asa important, normalvoi lucra doar cu o interfata injectata in controller, insa unii vor zice:
- modelul trebuie sa stie sa se incarce singur ("ActiveRecord", mai ales prin lumea Ruby), iar controller-ul sa zica doar model.Load()
- altii vor zice: nu, ActiveRecord e un anti-pattern, modelul trebuie sa fie cat mai dummy, si alta clasa sa fie responsabila de incarcarea lui ("service", "repository", "context" sau alte nume..) - mai ales prin lumea Java care vad modeul ca un DTO
- dupa care vor veni altii si vor zice: DTO duce la anemic domain model, care e iarasi un anti pattern, si fie se intorc la prima varianta, fie la o combinatie de domain model + viewmodel..
Lumea .NET, care a intrat mai tarziu in joc, se ia fie dupa unii fie dupa altii.. :)
Eu prefer ca action-ul din controller sa fie cit mai simplu ... asa ca , probabil, ActiveRecord, nu?
ReplyDeleteAsa cum spunea si Tudor pareriile sunt impartite. In cazul unui model pentru view eu il prefer cat mai simplu. Daca ar avea forma la ActiveRecord atunci in model pot sa ajunga obiecte sau date de care nu ai nevoie.
ReplyDeletePersonal imi place sa trag o linie f. clara si precisa intre obiectele din business layer si cele din UI( model). Chiar daca cele doua coexista impreuna, prefer sa fie diferite, iar modelul sa contina doar datele de care am nevoie in view si atata. Orice camp in plus, care nu e folosit in view nu isi are rostul.
Multi prefera ca modelul sa fie chior (doar proprietati), view-ul chior (doar "lipirea" proprietatile din model pe HTML), controller-ul subtirel (apel servicii colo-colo) si inca un service layer.
ReplyDelete