ASP.NET MVC-同じタイプの別のエンティティに同じ主キー値がすでにあるため、タイプ「MODELNAME」のエンティティのアタッチに失敗しました

一言で言えば、ラッパーモデルのPOST中に、1つのエントリの状態を「変更済み」に変更するときに例外がスローされます。状態を変更する前に、状態は「Detached」に設定されていますが、Attach（）を呼び出しても同じエラーがスローされます。EF6を使用しています。

以下のコードを見つけてください（読みやすくするためにモデル名が変更されています）

型番

// Wrapper classes
        public class AViewModel
        {
            public A a { get; set; }
            public List<B> b { get; set; }
            public C c { get; set; }
        }   

コントローラ

        public ActionResult Edit(int? id)
        {
            if (id == null)
            {
                return new HttpStatusCodeResult(HttpStatusCode.BadRequest);
            }

            if (!canUserAccessA(id.Value))
                return new HttpStatusCodeResult(HttpStatusCode.Forbidden);

            var aViewModel = new AViewModel();
            aViewModel.A = db.As.Find(id);

            if (aViewModel.Receipt == null)
            {
                return HttpNotFound();
            }

            aViewModel.b = db.Bs.Where(x => x.aID == id.Value).ToList();
            aViewModel.Vendor = db.Cs.Where(x => x.cID == aViewModel.a.cID).FirstOrDefault();

            return View(aViewModel);
        }

[HttpPost]
        [ValidateAntiForgeryToken]
        public ActionResult Edit(AViewModel aViewModel)
        {
            if (!canUserAccessA(aViewModel.a.aID) || aViewModel.a.UserID != WebSecurity.GetUserId(User.Identity.Name))
                return new HttpStatusCodeResult(HttpStatusCode.Forbidden);

            if (ModelState.IsValid)
            {
                db.Entry(aViewModel.a).State = EntityState.Modified; //THIS IS WHERE THE ERROR IS BEING THROWN
                db.SaveChanges();
                return RedirectToAction("Index");
            }
            return View(aViewModel);
        }

上記のように

db.Entry(aViewModel.a).State = EntityState.Modified;

例外をスローします：

同じタイプの別のエンティティがすでに同じ主キー値を持っているため、タイプ「A」のエンティティのアタッチに失敗しました。これは、「アタッチ」メソッドを使用するか、グラフ内のエンティティにキー値の競合がある場合にエンティティの状態を「未変更」または「変更」に設定すると発生する可能性があります。これは、一部のエンティティが新しく、データベースで生成されたキー値をまだ受け取っていないことが原因である可能性があります。この場合、「Add」メソッドまたは「Added」エンティティの状態を使用してグラフを追跡し、新しいエンティティ以外の状態を「Unchanged」または「Modified」に適宜設定します。

誰かが私のコードで何か悪いことを見たり、モデルの編集中にそのようなエラーをスローする状況を理解したりしますか？

問題が解決しました！

Attachメソッドは潜在的に誰かを助けることができますが、編集GETコントローラー関数でロードされている間にドキュメントが既に追跡されているため、この状況では役に立ちません。アタッチはまったく同じエラーをスローします。

ここで遭遇した問題はcanUserAccessA()、オブジェクトaの状態を更新する前にAエンティティをロードする関数が原因でした。これは追跡対象のエンティティを台無しにし、オブジェクトの状態をに変更していましたDetached。

解決策は、canUserAccessA()ロードしているオブジェクトが追跡されないように修正することでした。AsNoTracking()コンテキストのクエリ中に関数を呼び出す必要があります。

// User -> Receipt validation
private bool canUserAccessA(int aID)
{
    int userID = WebSecurity.GetUserId(User.Identity.Name);
    int aFound = db.Model.AsNoTracking().Where(x => x.aID == aID && x.UserID==userID).Count();

    return (aFound > 0); //if aFound > 0, then return true, else return false.
}

何らかの理由で私は使用couldntの.Find(aID)とAsNoTracking()、それは本当に問題で、私は、クエリを変更することで、同じことを達成できるようしません。

これが同様の問題を抱えている人を助けることを願っています！

興味深いことに：

_dbContext.Set<T>().AddOrUpdate(entityToBeUpdatedWithId);

または、まだ一般的ではない場合：

_dbContext.Set<UserEntity>().AddOrUpdate(entityToBeUpdatedWithId);

問題はスムーズに解決したようです。

変更しようとしているエンティティは正しく追跡されていないため、編集されたものとして認識されず、代わりに追加されたようです。

状態を直接設定する代わりに、次のことを試してください。

//db.Entry(aViewModel.a).State = EntityState.Modified;
db.As.Attach(aViewModel.a); 
db.SaveChanges();

また、コードに潜在的なセキュリティの脆弱性が含まれていることを警告します。エンティティをビューモデルで直接使用している場合、送信されたフォームに正しく名前が付けられたフィールドを追加することにより、誰かがエンティティのコンテンツを変更する危険性があります。たとえば、ユーザーが「A.FirstName」という名前の入力ボックスを追加し、エンティティにそのようなフィールドが含まれている場合、ユーザーがアプリケーションの通常の操作で値を変更できない場合でも、値はviewmodelにバインドされ、データベースに保存されます。 。

更新：

前述のセキュリティの脆弱性を克服するには、ドメインモデルをビューモデルとして公開せず、代わりに別のビューモデルを使用する必要があります。次に、アクションはビューモデルを受け取り、AutoMapperなどのマッピングツールを使用してドメインモデルにマッピングし直すことができます。これにより、ユーザーが機密データを変更するのを防ぐことができます。

ここに拡張説明があります：

http://www.stevefenton.co.uk/Content/Blog/Date/201303/Blog/Why-You-Never-Expose-Your-Domain-Model-As-Your-MVC-Model/

これを試して：

var local = yourDbContext.Set<YourModel>()
                         .Local
                         .FirstOrDefault(f => f.Id == yourModel.Id);
if (local != null)
{
  yourDbContext.Entry(local).State = EntityState.Detached;
}
yourDbContext.Entry(applicationModel).State = EntityState.Modified;

私にとって、ローカルコピーが問題の原因でした。これで解決しました

var local = context.Set<Contact>().Local.FirstOrDefault(c => c.ContactId == contact.ContactId);
                if (local != null)
                {
                    context.Entry(local).State = EntityState.Detached;
                }

私のケースでは、MVCアプリからEFコンテキストに直接アクセスできませんでした。

したがって、エンティティの永続化に何らかのリポジトリを使用している場合、明示的にロードされたエンティティをデタッチし、バインドされたEntityStateをModifiedに設定するのが適切です。

サンプル（抽象）コード：

MVC

public ActionResult(A a)
{
  A aa = repo.Find(...);
  // some logic
  repo.Detach(aa);
  repo.Update(a);
}

リポジトリ

void Update(A a)
{
   context.Entry(a).EntityState = EntityState.Modified;
   context.SaveChanges();
}

void Detach(A a)
{
   context.Entry(a).EntityState = EntityState.Detached;
}

すぐに気づかなかったので少しばかげたと感じましたが、私はこれについて私の経験を共有したいと思いました。

私のコントローラーに注入されたリポジトリインスタンスでリポジトリパターンを使用しています。具体的なリポジトリーはIDisposable、コントローラーによって破棄されるリポジトリーの存続期間を持続する私のModelContext（DbContext）をインスタンス化します。

私の問題は、エンティティに変更されたスタンプと行のバージョンがあるため、インバウンドヘッダーと比較するためにそれらを最初に取得することでした。もちろん、これにより、後で更新されるエンティティが読み込まれ、追跡されました。

修正は、単にリポジトリをコンストラクタで一度更新するコンテキストから、次のメソッドを持つように変更することでした：

    private DbContext GetDbContext()
    {
        return this.GetDbContext(false);
    }


    protected virtual DbContext GetDbContext(bool canUseCachedContext)
    {
        if (_dbContext != null)
        {
            if (canUseCachedContext)
            {
                return _dbContext;
            }
            else
            {
                _dbContext.Dispose();
            }
        }

        _dbContext = new ModelContext();

        return _dbContext;
    }

    #region IDisposable Members

    public void Dispose()
    {
        this.Dispose(true);
    }

    protected virtual void Dispose(bool isDisposing)
    {
        if (!_isDisposed)
        {
            if (isDisposing)
            {
                // Clear down managed resources.

                if (_dbContext != null)
                    _dbContext.Dispose();
            }

            _isDisposed = true;
        }
    }

    #endregion

これにより、リポジトリメソッドは、使用するたびにを呼び出してコンテキストインスタンスを更新しGetDbContextたり、trueを指定して必要に応じて以前のインスタンスを使用したりできます。

この回答を追加したのは、問題がより複雑なデータパターンに基づいて説明されていて、ここでは理解しにくいためです。

かなりシンプルなアプリケーションを作成しました。このエラーは、POSTの編集アクション内で発生しました。アクションは入力モデルとしてViewModelを受け入れました。ViewModelを使用する理由は、レコードが保存される前に何らかの計算を行うためでした。

アクションがのような検証を通過したif(ModelState.IsValid)後、私の問題はViewModelの値をEntityの完全に新しいインスタンスに投影することでした。更新されたデータを格納するために新しいインスタンスを作成し、そのようなインスタンスを保存する必要があると思いました。

後で気付いたのは、データベースからレコードを読み取らなければならないことでした。

Student student = db.Students.Find(s => s.StudentID == ViewModel.StudentID);

このオブジェクトを更新しました。今はすべてうまくいきます。

私はローカル変数でこの問題があり、次のように切り離しました：

if (ModelState.IsValid)
{
    var old = db.Channel.Find(channel.Id);
    if (Request.Files.Count > 0)
    {
        HttpPostedFileBase objFiles = Request.Files[0];
        using (var binaryReader = new BinaryReader(objFiles.InputStream))
        {
            channel.GateImage = binaryReader.ReadBytes(objFiles.ContentLength);
        }

    }
    else
        channel.GateImage = old.GateImage;
    var cat = db.Category.Find(CatID);
    if (cat != null)
        channel.Category = cat;
    db.Entry(old).State = EntityState.Detached; // just added this line
    db.Entry(channel).State = EntityState.Modified;
    await db.SaveChangesAsync();
    return RedirectToAction("Index");
}
return View(channel);

同じキーを持つロードされたオブジェクトの問題の原因なので、最初にそのオブジェクトを切り離して更新を行い、同じキーを持つ2つのオブジェクト間の競合を回避します

EFは変更を追跡せず、オブジェクトがアタッチされていない限り変更がないと見なすため、2〜3日間プローブした後、「。AsNoTracking」を削除する必要があります。また、.AsNoTrackingを使用しない場合、EFは保存/更新するオブジェクトを自動的に認識するため、Attach / Addedを使用する必要はありません。

AsNoTracking()クエリを取得する場所を使用します。

  var result = dbcontext.YourModel.AsNoTracking().Where(x => x.aID == aID && x.UserID==userID).Count();

私はこのエラーに遭遇しました

• ApplicationDbContextの同じインスタンスを使用した、両方の単一のコントローラ内の2つの方法、A＆B、及び
• メソッドAがメソッドBを呼び出した

    private ApplicationDbContext db;
    // api methods
    public JsonResult methodA(string id){
        Resource resource = db.Resources.Find(id);
        db.Entry(resource).State = EntityState.Modified;
        db.SaveChanges();
        return methodB()
    }

    public JsonResult methodB(string id){
        Resource resource = db.Resources.Find(id);
        db.Entry(resource).State = EntityState.Modified;
        db.SaveChanges();
        return new JsonResult();
    }

メソッドBを使用してusingステートメントを作成し、ローカルdb2のみに依存するようにしました。後：

    private ApplicationDbContext db;    
    // api methods    
    public JsonResult methodA(string id){
        Resource resource = db.Resources.Find(id);
        db.Entry(resource).State = EntityState.Modified;
        db.SaveChanges();
        return methodB()
    }

    public JsonResult methodB(string id){
        using (var db2 = new ApplicationDbContext())
        {
            Resource resource = db2.Resources.Find(id);
            db2.Entry(resource).State = EntityState.Modified;
            db2.SaveChanges();
        }
        return new JsonResult();
    }

Luke Puplettが言っているのと同様に、この問題はコンテキストを適切に破棄または作成していないことが原因である可能性があります。

私の場合、次のコンテキストを受け入れるクラスがありましたContextService。

public class ContextService : IDisposable
{
    private Context _context;

    public void Dispose()
    {
        _context.Dispose();
    }
    public ContextService(Context context)
    {
        _context = context;
    }
//... do stuff with the context

私のコンテキストサービスには、インスタンス化されたエンティティオブジェクトを使用してエンティティを更新する関数がありました。

        public void UpdateEntity(MyEntity myEntity, ICollection<int> ids)
        {
            var item = _context.Entry(myEntity);
            item.State = EntityState.Modified;
            item.Collection(x => x.RelatedEntities).Load();
            myEntity.RelatedEntities.Clear();
            foreach (var id in ids)
            {
                myEntity.RelatedEntities.Add(_context.RelatedEntities.Find(id));
            }
            _context.SaveChanges();
        }

これはすべて問題なく、サービスを初期化したコントローラーが問題でした。私のコントローラーはもともとこのように見えました：

    private static NotificationService _service = 
        new NotificationService(new NotificationContext());
    public void Dispose()
    {
    }

私はそれをこれに変更し、エラーは消えました：

    private static NotificationService _service;
    public TemplateController()
    {
        _service = new NotificationService(new NotificationContext());
    }
    public void Dispose()
    {
        _service.Dispose();
    }

この問題は、中に見ることができるViewModelまでEntityModel（使用してマッピングAutoMapperなど）とが挙げられるしようcontext.Entry().Stateとcontext.SaveChanges()、以下に示すような使用ブロックする問題を解決するだろう。ブロックを使用する場合にも必要なので、context.SaveChanges()メソッドを直後に使用するのではなく2回使用することに注意してくださいif-block。

public void Save(YourEntity entity)
{
    if (entity.Id == 0)
    {
        context.YourEntity.Add(entity);
        context.SaveChanges();
    }
    else
    {
        using (var context = new YourDbContext())
        {
            context.Entry(entity).State = EntityState.Modified;
            context.SaveChanges(); //Must be in using block
        }
    }            
}

お役に立てれば...

ここで私が同様のケースで何をしたか。

そのシチュエーションは、同じエンティティがコンテキスト内にすでに存在していることを意味します。

最初に、エンティティがコンテキスト内にあるかどうかをChangeTrackerから確認します

var trackedEntries=GetContext().ChangeTracker.Entries<YourEntityType>().ToList();

var isAlreadyTracked =
                    trackedEntries.Any(trackedItem => trackedItem.Entity.Id ==myEntityToSave.Id);

存在する場合

  if (isAlreadyTracked)
            {
                myEntityToSave= trackedEntries.First(trackedItem => trackedItem.Entity.Id == myEntityToSave.Id).Entity;
            } 

else
{
//Attach or Modify depending on your needs
}

状態を更新して問題を修正するように管理します。同じレコードで検索またはその他のクエリ操作をトリガーすると、状態が変更されて更新されたので、ステータスをDetachedに設定する必要があります。更新の変更を起動できます。

     ActivityEntity activity = new ActivityEntity();
      activity.name="vv";
    activity.ID = 22 ; //sample id
   var savedActivity = context.Activities.Find(22);

            if (savedActivity!=null)
            {
                context.Entry(savedActivity).State = EntityState.Detached;
                context.SaveChanges();

                activity.age= savedActivity.age;
                activity.marks= savedActivity.marks; 

                context.Entry(activity).State = EntityState.Modified;
                context.SaveChanges();
                return activity.ID;
            }

「using」ブロックでこの問題を解決します

using (SqlConnection conn = new SqlConnection(connectionString))

    {

       // stuff to do with data base
    }

    // or if you are using entity framework 
    using (DataBaseEntity data = new DataBaseEntity)
{

    }

ここで私はアイデアを取得しますhttps://social.msdn.microsoft.com/Forums/sqlserver/es-ES/b4b350ba-b0d5-464d-8656-8c117d55b2af/problema-al-modificar-en-entity-framework?forum = vcses はスペイン語です（2番目の答えを探してください）

次のような追加メソッドを使用できます。

_dbContext.Entry(modelclassname).State = EntityState.Added;

しかし、多くの場合、その時点で複数のモデルを使用したい場合、エンティティは既に別のエンティティにアタッチされているため、これは機能しません。そのため、そのときは、オブジェクトを単に別のオブジェクトに移行するADDOrUpdateエンティティ移行メソッドを使用でき、その結果、エラーは発生しません。

_dbContext.Set<modelclassname>().AddOrUpdate(yourmodel);

すべての州をクリア

dbContextGlobalERP.ChangeTracker.Entries（）。Where（e => e.Entity！= null）.ToList（）。ForEach（e => e.State = EntityState.Detached）;