Блог

В текущем проекте мы решили использовать dto для передачи данных между сервером и клиентом.

Dto плоские, проблема не в том, чтобы льстить, это можно сделать без особых проблем. Но нелестный dto может стать намного сложнее реализовать, потому что пользователь может удалять, создавать и обновлять некоторые части сглаженного графа объектов.

Итак, это пример кода для одного из методов веб-сервиса:

     [Update, EmpresaHasPermissions("PERMIT_INS_Employee")]
    public void UpdateBackground(EmployeeBackgroundDTO dto)
    {
        using (var context = GetObjectContext())
        {
            var user = EmpresaAuthentication.Current.User;

            Employee employee = context.Employees
                .Include(it => it.Nationality)
                .Include(it => it.EthnicOrigin)
                .Include(it => it.MaritalStatus)
                .Include(it => it.Religion)
                .Include(it => it.CRB)
                .Include(it => it.Passport)
                .Single(it => it.OwnerOrganizationId == user.OrganizationId amp;amp;
                              !it.Deleted amp;amp; it.Id == dto.Id);

            var updater = new EmployeeBackgroundUpdater(context);

            updater.UpdateEntity(employee, dto);

            context.SaveChanges();

            dto.MaritalStatusId = employee.MaritalStatusId;
            dto.EthnicOriginId = employee.EthnicOriginId;
            dto.ReligionId = employee.ReligionId;
        }
    } 
  

Как вы можете видеть, здесь смешано много всего:
создание контекста
выбор данных
вызов программы обновления данных
отправка идентификатора вновь созданного dto обратно клиенту

Все начинает приобретать смысл, когда вы видите, как реализован EmployeeBackgroundUpdater:

 public override void UpdateEntity(Employee employee, EmployeeBackgroundDTO dto)
{
    employee.InjectFrom(dto);

    if (!IsPassportNull(dto))
    {
        if (employee.Passport == null)
        {
            employee.Passport = new Passport();
        }

        employee.Passport.IssueDate = dto.PassportIssueDate.Value;
        employee.Passport.ExpiryDate = dto.PassportExpiryDate.Value;
        employee.Passport.PassportNo = dto.PassportPassportNo;
        employee.Passport.IssuingCountryId = dto.PassportIssuingCountryId.Value;
        employee.Passport.OwnerUserId = UserId;
    }
    else
    {
        if (employee.Passport != null)
        {
            DeleteObject(employee.Passport);
            employee.Passport = null;
        }
    }

    if (!IsCRBNull(dto))
    {
        if (employee.CRB == null)
        {
            employee.CRB = new CRB();
        }

        employee.CRB.IssueDate = dto.CRBIssueDate.Value;
        employee.CRB.ExpiryDate = dto.CRBExpiryDate.Value;
        employee.CRB.Registration = dto.CRBRegistration;
        employee.CRB.Notes = dto.CRBNotes;
    }
    else
    {
        if (employee.CRB != null)
        {
            DeleteObject(employee.CRB);
            employee.CRB = null;
        }
    }

    var epmpresaContext = (EmpresaEntities)ObjectContext;

    AddMaritalStatus(employee, dto, epmpresaContext);

    AddReligion(employee, dto, epmpresaContext);

    AddEthnicOrigin(employee, dto, epmpresaContext);

    employee.NationalityId = dto.NationalityId;
}

private void AddMaritalStatus(Employee employee, EmployeeBackgroundDTO dto, EmpresaEntities epmpresaContext)
{
    if (!dto.MaritalStatusId.HasValue amp;amp; !String.IsNullOrWhiteSpace(dto.MaritalStatusDescription))
    {
        var item = epmpresaContext.MaritalStatuses.FirstOrDefault(
            it => it.Description.ToUpper() == dto.MaritalStatusDescription.ToUpper());

        if (item == null)
        {
            employee.MaritalStatus = new MaritalStatus
            {
                Description = dto.MaritalStatusDescription
            };
        }
        else
        {
            employee.MaritalStatus = item;
        }
    }
    else
    {
        employee.MaritalStatusId = dto.MaritalStatusId;
    }
}
  

Код одинаков по структуре, единственное отличие заключается в типах наборов сущностей и типах сущностей. И это пугает, потому что мне придется переписывать один и тот же код в куче разных мест в проектах, если мы решим обновить логику проверки или что-то подобное.

Спасибо, что дочитали до этого места. И у меня есть множество вопросов:

1) Как преобразовать плоские dto, которые могут иметь дочерние объекты, в действительный граф объектов?

2) Должен ли dto (или, возможно, модель представления) содержать иерархию объектов?

3) Как избавиться от повторяющегося кода?

Для преобразования объектов в DTO и наоборот вы можете проверить AutoMapper. DTO может быть иерархией объектов (это не обязательно должно быть сглажено). Я боюсь, что вам никогда не избежать некоторого повторяющегося кода, потому что каждый тип сущности каким-то образом особенный, и вам приходится иметь с ним дело вручную — в этом сложность, связанная с использованием DTO.