Ошибка при вставке / обновлении записей в ядре asp.net

Эта ошибка возникает, когда вы хотите отслеживать изменения, внесенные в model, а не оставлять не отслеживаемый model в memory. У меня есть небольшое предложение или альтернативный подход, который решит вашу проблему.

EntityFramework автоматически отслеживает изменения. Но вы можете OvverrideSaveChanges() в вашем DbContext.

public override int SaveChanges()
{
    foreach (var ent in ChangeTracker.Entries<Client>())
    {
        if (ent.State == EntityState.Modified)
        {
            // Get the changed values
            var modifiedProps = ObjectStateManager.GetObjectStateEntry(ent.EntityKey).GetModifiedProperties();
            var currentValues = ObjectStateManager.GetObjectStateEntry(ent.EntityKey).CurrentValues;
            foreach (var propName in modifiedProps)
            {
                var newValue = currentValues[propName];
                //log your changes
            }
        }
    }

    return base.SaveChanges();
}

Прежде всего, многие люди пропускают эти проверки, что приводит к исключениям во время выполнения. Поэтому вы всегда должны проверять состояние своей сущности:

context.YourEntities.Local.Any(e => e.Id == id);

Или

context.ChangeTracker.Entries<YourEntity>().Any(e => e.Entity.Id == id);

Чтобы убедиться, что ваша сущность 'безопасно использовать', если вы хотите использовать удобный метод, вы можете использовать это:

/// <summary>
    /// Determines whether the specified entity key is attached is attached.
    /// </summary>
    /// <param name = "context">The context.</param>
    /// <param name = "key">The key.</param>
    /// <returns>
    ///   <c>true</c> if the specified context is attached; otherwise, <c>false</c>.
    /// </returns>
    internal static bool IsAttached(this ObjectContext context, EntityKey key)
    {
        if (key == null)
        {
            throw new ArgumentNullException("key");
        }

        ObjectStateEntry entry;
        if (context.ObjectStateManager.TryGetObjectStateEntry(key, out entry))
        {
            return (entry.State != EntityState.Detached);
        }
        return false;
    }

а потом:

if (!_objectContext.IsAttached(entity.EntityKey))
{
   _objectContext.Attach(entity);
}

Это избавит вас от проблемы переопределения метода SaveChanges(), что не рекомендуется ни в коем случае, если вам действительно не нужно.

Я вижу, что вы используете один и тот же метод для объектов «Добавить» и «Обновить», мое первое предложение - это отдельные проблемы и сделать разные методы: один для добавления, а другой - для обновления, если это возможно.

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

Повторите ошибку, которую вы можете преодолеть, проверив, отслеживается ли объект, и отсоедините его, затем прикрепите измененный объект и обновите его.

здесь вы можете увидеть измененную версию вашего метода:

public bool SaveDataCapDetails(List<TDataCapDetails> lstDataCapDetails)
    {
        bool IsSuccess = false;

        using (var dbContextTransaction = _objContext.Database.BeginTransaction())
        {
            try
            {
                int? id = _objContext.TDataCapDetails.Max(x => (int?)x.Id);
                id = id == null ? 0 : id;

                // entities with Id == 0 --> new entities
                // you may need to check if Id == null as well (depends on your data model)
                var entitiesToAdd = lstDataCapDetails.Where(x => x.Id == 0);
                foreach(var entity in entitiesToAdd)
                {
                    entity.Id = id++;

                    // new entities is not tracked, its state can be changed to Added
                    _objContext.Entry(entity).State = EntityState.Added;
                }

                // entities with Id > 0 is already exists in db and needs to be updated
                var entitiesToUpdate = lstDataCapDetails.Where(x => x.Id > 0);
                foreach (var entity in entitiesToUpdate)
                {
                    // check if entity is being tracked
                    var local = _objContext.Set<TDataCapDetails>().Local.FirstOrDefault(x => x.Id.Equals(entity.Id));

                    // if entity is tracked detach it from context
                    if (local != null)
                        _objContext.Entry<TDataCapDetails>(local).State = EntityState.Detached;

                    // attach modified entity and change its state to modified
                    _objContext.Attach(entity).State = EntityState.Modified;
                }

                // optional: assign value for IsSuccess
                IsSuccess = _objContext.SaveChanges() > 0;                    

                dbContextTransaction.Commit();
            }
            catch (Exception ex)
            {
                dbContextTransaction.Rollback();
                throw ex;
            }
        }

        return IsSuccess;
    }

Если вы хотите иметь таблицу с Primary-Key, а также с Identity-Incremental, вы должны создать свой Table после установки ForeignKey, вы должны установить Identity-Incremental для этого ForeignKey. нравиться:

С этой проблемой ваш код изменится на этот:

    public bool SaveDataCapDetails(List<TDataCapDetails> lstDataCapDetails)
    {
        bool IsSuccess = false;

        using (var dbContextTransaction = _objContext.Database.BeginTransaction())
        {
            try
            {

                List<TDataCapDetails> lstDataCapDetailsRecords = null;

                if (lstDataCapDetails.Where(x => x.Id == 0).Count() > 0)
                {
                    lstDataCapDetailsRecords = new List<TDataCapDetails>();
                    _objContext.AddRange(lstDataCapDetailsRecords);
                    _objContext.SaveChanges();
                }

                if (lstDataCapDetails.Where(x => x.Id > 0).Count() > 0)
                {
                    lstDataCapDetailsRecords = new List<TDataCapDetails>();
                    lstDataCapDetailsRecords = lstDataCapDetails.Where(x => x.Id > 0).ToList();
                    _objContext.UpdateRange(lstDataCapDetailsRecords);
                    _objContext.SaveChanges();
                }

                dbContextTransaction.Commit();
            }
            catch (Exception ex)
            {
                dbContextTransaction.Rollback();
                throw ex;
            }
        }

        return IsSuccess;
    }