Артем GskyANAСервис - это
бизнес-логика .
Это как если бы Вы что-то делали, делали, проверяли, сверяли, получили результат (вот это сервис)... и на листочек записали (вот это репозиторий).
Просто многие пихают первое в контроллеры, а потом задаются вопросом: а чем у нас сервисы от репозиториев-то отличаются? :)
Вот тут созрел вопрос. Интересно как бы Вы реализовали.
Есть Stage (один) и Step (ко многим) в базе.
При создании Step на странице сайта нужно выбрать Stage из DropDownList.
Вот модель
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
public class StepViewModel
{
[Key]
[DatabaseGenerated(DatabaseGeneratedOption.Identity)]
public int Id { get; set; }
[Required]
[StringLength(100)]
public string Name { get; set; }
[Column(TypeName = "ntext")]
public string Goal { get; set; }
[Column(TypeName = "ntext")]
public string Description { get; set; }
public int StageId { get; set; }
public StageViewModel Stage { get; set; }
public IEnumerable<StageViewModel> StageDropDownListItems { get; set; } // здесь отличие от db и dto классов
}
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
[HttpGet]
public ActionResult Create()
{
var stages = stageService.GetStages();
var stagesVM = mapper.Map<IEnumerable<StageViewModel>>(stages);
var model = new StepViewModel()
{
StageDropDownListItems = stagesVM
};
return View(model);
}
Видно что какая-то часть логики в контроллере. Вы бы оставили все как есть в контроллере или перенесли бы это в логику?
PS. Сейчас у меня все это в контроллере, есть желание перенести в логику, но тогда придется менять DTO Step в бизнес логике, добавив IEnumerable<Stage>. Напомню что Stage (один) и Step (ко многим) в базе и не логично как-то в DTO Step иметь IEnumerable<Stage>. Заранее спасибо.
"выбрать Stage из DropDownList" к логике не имеет никакого отношения. Это некое UI решение.
Отображение модели (Model) на модель представления (ViewModel) я бы вынес в отдельный класс, назовём его Converter, или Formatter, или Mapper, и было бы как-то так:
1.
2.
3.
4.
5.
6.
7.
8.
[HttpGet]
public ActionResult Create()
{
var stages = stageService.GetStages();
var model = mapper.ToViewModel(stages);
return View(model);
}
