Блог

Использование EF Core

Мы пытаемся получить всю информацию об оценке, которая включает ее группы и всех назначенных пользователей. Смотрите схему базы данных

Что работает в следующем порядке;

1. HttpPost (api / Assessment / aID / groups) пустой группы для оценки
2. HttpPost (api / Group / gID / users) пользователей в существующую группу

Чего мы пытаемся достичь (ссылочный код — это другой пример, но тот же принцип);

• HttpPost (api / Assessment / aID / groups), где группа уже содержит список пользователей. При попытке выполнить это, a possible object cycle was detected which is not supported.

 This piece of code is currently throwing a NullReference on Address
-------------------------------------------------------------------

   Group groupToCreate = new Group { Name = dto.Name, Description = dto.Description };

   foreach (var u in dto.Users)
   {
       groupToCreate.AddUser(new User
        {
             Name = u.Name,
             Email = u.Email,
             Address = new Address
             {
                 Country = u.Address.Country,
                 City = u.Address.City,
                 PostalCode = u.Address.PostalCode,
                 Street = u.Address.Street,
                 HouseNr = u.Address.HouseNr,
                 BusNr = u.Address.BusNr
             }
        });
    }

   _groupRepository.Add(groupToCreate);
   _groupRepository.SaveChanges();

   return groupToCreate;
  

• HttpGet (api / Assessment), который отображает назначенные ему группы и связанных пользователей.

 This seems to be working
------------------------
   groupList = _groups.Select(g => new GroupDTO
   {
       Name = g.Name,
       Description = g.Description,
       Users = g.GroupUsers.Select(u => new UserDTO
       {
           Name = u.User.Name,
           Email = u.User.Email,
           Address = new AddressDTO
           {
              Country = u.User.Address.Country,
              City = u.User.Address.City,
              PostalCode = u.User.Address.PostalCode,
              Street = u.User.Address.Street,
              HouseNr = u.User.Address.HouseNr,
              BusNr = u.User.Address.BusNr
           }
       }).ToList()
   }).ToList();
  

Ссылки:
Оценка группы пользователей


AssessmentRepo

Трудно сказать, учитывая детали, которые вы предоставляете, но я предполагаю, что это связано с наличием свойств двусторонней навигации? Вы используете EF здесь?

Например, если у вашего пользователя есть свойство навигации, позволяющее получить доступ к свойству пользователя Group , но Group у a есть коллекция User объектов, тогда каждый из этих пользователей сам будет иметь Group выраженное внутри них … тогда при попытке выразить это он может легко застрять в цикле, например, пользователь будет выглядеть как:

 {
  "Name":"user name",
  "Group":{
     "Name":"group1",
     "Users":[
       {
          "Name":"user name",
          "Group":{
            "Name":"group1",
            "Users":{
              ....
            }
          }
       }
     ]
  }
}
  

.. потому что a User имеет a Group , и Group у него есть список User объектов, и у каждого из них есть Group … и т.д.

Это своего рода проблема, возникающая из-за смешивания вашего уровня данных и объектов DTO. Измените свою систему так, чтобы объекты, возвращаемые вашими методами REST, были новыми объектами, разработанными в соответствии с требованиями API / интерфейса. Эти объекты могут выглядеть очень похожими на ваши модели БД (по крайней мере, изначально), но они не должны быть одинаковыми объектами.

Создавайте совершенно новые объекты, которые не имеют никаких логических или навигационных свойств и существуют только для передачи информации обратно потребителям API. Например, простой класс для предоставления списка групп пользователей и пользователей в этих группах может быть определен как:

 public class UserDto
{
   public string UserName { get; set; }
   public IEnumerable<string> Groups { get; set; }
}

public class UserListDto
{
   public IEnumerable<UserDto> Users { get; set; }
}
  

И тогда ваше действие контроллера может выполнять что-то вроде:

 var users = userService.GetAllUsers();

var result = new UserListDto { 
    Users = users.Select(u => new UserDto{
      UserName = u.Name,
      Groups = u.Groups.Select(g => g.Name)
      }
   };

return Ok(result);
  

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

Заманчиво думать, что «Данные, которые мне нужно вернуть, в основном такие же, как и то, как я храню их внутри, поэтому просто повторно используйте эти классы», но это не очень хорошая идея и только в долгосрочной перспективе создаст проблемы.

Чтобы избежать необходимости (повторного) написания большого количества кода для «преобразования» одного объекта в другой, я бы рекомендовал изучить что-то вроде AutoMapper, поскольку это может сделать это довольно легко повторно используемым и позволить вам поместить все эти «переводные» вещи в одно место.