1

Estou tendo problema na hora de fazer a atualização com relacionamento many to many.

Eu quero que no método atualizar ele atualize todas os campos do itemEntity também como os fornecedores.

Para exemplificar coloquei apenas o campo descrição e fornecedores conforme a imagem da tela.

No meu código abaixo se eu removo a linha que da o erro (mostrado abaixo) ele grava apenas os fornecedores.

Eu tentei setar a itemEntity como modificado mas eu recebo o erro.

Alguém pode por favor me ajudar?

Minha Tela

Minha Tela

Erro

Attaching an entity of type 'Domain.Entities.ItemEntity' failed because another entity of the same type already has the same primary key value. This can happen when using the 'Attach' method or setting the state of an entity to 'Unchanged' or 'Modified' if any entities in the graph have conflicting key values. This may be because some entities are new and have not yet received database-generated key values. In this case use the 'Add' method or the 'Added' entity state to track the graph and then set the state of non-new entities to 'Unchanged' or 'Modified' as appropriate.

inserir a descrição da imagem aqui

Tabela

inserir a descrição da imagem aqui

Relacionamento

inserir a descrição da imagem aqui

Codigo

    public override ItemEntity Atualizar(ItemEntity entity)
    {
        var novos = new List<FornecedorEntity>(entity.Fornecedores);

        entity.Fornecedores = _dbContext.TbItem
           .Where(p => p.Id == entity.Id).FirstOrDefault().Fornecedores;

        _dbContext.Entry(entity).State = EntityState.Modified;
        //Sem a linha acima ele funciona mais não atualiza as demais propriedades do ItemEntity, 
        //atualiza somente os fornecedores.

        var deletetadoFornecedores = entity.Fornecedores
            .Except(novos).ToList();

        deletetadoFornecedores.ForEach(c => entity.Fornecedores.Remove(c));

        var AdicionadoFornecedores = novos
            .Except(entity.Fornecedores).ToList();

        foreach (FornecedorEntity c in AdicionadoFornecedores)
        {
            if (_dbContext.Entry(c).State == EntityState.Detached)
            {
                _dbContext.TbFornecedor.Attach(c);
            }

            entity.Fornecedores.Add(c);
        }

        _dbContext.SaveChanges();      

        return entity;
    }

Editado

Esse código faz exatamente o que preciso, porem tenho que ficar mapeando as propriedades.

    public override ItemEntity Atualizar(ItemEntity entity)
    {
        var itemExistente = _dbContext.TbItem
                .Find(entity.Id);

        itemExistente.Descricao = entity.Descricao;
        itemExistente.Descricao1 = entity.Descricao1;
        itemExistente.Descricao2 = entity.Descricao2;
        itemExistente.Descricao3 = entity.Descricao3;
        itemExistente.Descricao4 = entity.Descricao4;
        //Esse mapeamento acima não tem como fazer de forma automatica?
        //Toda vez que eu criar uma nova propriedade vou ter que lembrar de vir aqui e alterar
        //isso gera maior dificuldade na manutenão e aumenta a chance de esquecer e gerar um bug.


        var fornecedoresDeletados = itemExistente.Fornecedores
            .Except(entity.Fornecedores).ToList();

        var fornecedoresAdicionados = entity.Fornecedores
            .Except(itemExistente.Fornecedores).ToList();

        fornecedoresDeletados.ForEach(c => itemExistente.Fornecedores.Remove(c));

        foreach (FornecedorEntity c in fornecedoresAdicionados)
        {
            if (_dbContext.Entry(c).State == EntityState.Detached)
                _dbContext.TbFornecedor.Attach(c);

            itemExistente.Fornecedores.Add(c);
        }

        _dbContext.SaveChanges();

        return entity;
    }
8
  • Mauricio, no lugar da imagem do erro poderia por a mensagem? Commented 26/05/2017 às 14:31
  • Posso mais qual mensagem? Commented 26/05/2017 às 14:35
  • Não sei kkkk, é que não consigo ver as imagens daqui :(. Não é uma mensagem que tem na imagem do Erro? Commented 26/05/2017 às 14:37
  • rsrs, Da um clique nela que ela fica grande.Coloquei em cima de qualquer forma. Commented 26/05/2017 às 14:38
  • 1
    O problema é bem maior. Já estou elaborando alguma coisa. Commented 26/05/2017 às 14:43

1 Resposta 1

2

Você está colocando o Item duas vezes em contexto, e seus Fornecedores também:

    entity.Fornecedores = _dbContext.TbItem
       .Where(p => p.Id == entity.Id).FirstOrDefault().Fornecedores;

O que eu faria no seu lugar são dois ViewModels:

public class ItemViewModel
{
    public int ItemId { get; set; }
    public String Nome { get; set; }

    public ICollection<FornecedorViewModel> Fornecedores { get; set; }
}

public class FornecedorViewModel
{
    public int FornecedorId { get; set; }
    public String Nome { get; set; }
    public bool Selecionado { get; set; }
}

Aqui eu faria:

public override ItemEntity Atualizar(ItemViewModel entity)
{
    var item = _dbContext.TbItem
                         .Include(i => i.Fornecedores)
                         .FirstOrDefault(i => i.Id == entity.Id);

    // Excluídos
    var idsFornecedoresExcluidos = item.Fornecedores
                                       .Where(f => !f.Selecionado)
                                       .Select(f => f.Id)
                                       .ToList();

    foreach (var fornecedor in item.Fornecedores.Where(f => idsFornecedoresExcluidos.Contains(f.Id)))
    {
        item.Fornecedores.Remove(fornecedor);
    }

    __dbContext.SaveChanges();

    // Incluídos
    var idsFornecedoresIncluidos = item.Fornecedores
                                       .Where(f => f.Selecionado)
                                       .Select(f => f.Id)
                                       .ToList();

    foreach (var novoFornecedor in __dbContext.Fornecedores.Where(f => idsFornecedoresIncluidos.Contains(f.Id)))
    {
        item.Fornecedores.Add(novoFornecedor);
    }

    __dbContext.SaveChanges();

    // Alterando item.
    item.Nome = entity.Nome;
    _dbContext.Entry(item).State = EntityState.Modified;
    __dbContext.SaveChanges();

    return item;
}

Como são várias operações, um escopo transacional aqui seria interessante:

public override ItemEntity Atualizar(ItemViewModel entity)
{
    using (var scope = new TransactionScope()) // Adicione a referência para System.Transactions para ter isso funcionando
    {
        var item = _dbContext.TbItem
                             .Include(i => i.Fornecedores)
                             .FirstOrDefault(i => i.Id == entity.Id);

        // Excluídos
        var idsFornecedoresExcluidos = item.Fornecedores
                                           .Where(f => !f.Selecionado)
                                           .Select(f => f.Id)
                                           .ToList();

        foreach (var fornecedor in item.Fornecedores.Where(f => idsFornecedoresExcluidos.Contains(f.Id)))
        {
            item.Fornecedores.Remove(fornecedor);
        }

        __dbContext.SaveChanges();

        // Incluídos
        var idsFornecedoresIncluidos = item.Fornecedores
                                           .Where(f => f.Selecionado)
                                           .Select(f => f.Id)
                                           .ToList();

        foreach (var novoFornecedor in __dbContext.Fornecedores.Where(f => idsFornecedoresIncluidos.Contains(f.Id)))
        {
            item.Fornecedores.Add(novoFornecedor);
        }

        __dbContext.SaveChanges();

        // Alterando item.
        item.Nome = entity.Nome;
        _dbContext.Entry(item).State = EntityState.Modified;
        __dbContext.SaveChanges();

        scope.Complete(); // Não esqueça do `scope.Complete()` para completar a transação.
        return item;
    }
}

Se precisar de ajuda sobre como carregar o ViewModel, só se manifestar por comentário.

8
  • não tem como fazer todas operações com um único _dbContext.SaveChanges(); ? estou usando UnitOfWork que no final de tudo eu chamo o _dbContext.SaveChanges();, depois que eu resolvesse o problema eu ia remover o _dbContext.SaveChanges(); dessa classe. Commented 26/05/2017 às 15:59
  • 1
    Por que está usando UnitOfWork? Leia isto. Commented 26/05/2017 às 16:03
  • Estou utilizando uma arquitetura DDD que estou aprendendo e nem sempre todas as alterações estarão dentro do mesmo repositório. Commented 26/05/2017 às 16:06
  • 2
    @MauricioFerraz Te aconselho a ler o artigo que o Cigano referênciou no comentario dele, resolveu muitos problemas que eu tinha e pode te ajudar a ter um entendimento melhor de todo o projeto. Commented 26/05/2017 às 16:28
  • 1
    Estou lendo, Obrigado Cigano por sua ajuda e aos demais. Commented 26/05/2017 às 16:30

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .