Entity Framework 6を​​使用してレコードを更新する方法は?


245

EF6を使用してレコードを更新しようとしています。最初にレコードが見つかったら、それを更新します。これが私のコードです:-

var book = new Model.Book
{
    BookNumber =  _book.BookNumber,
    BookName = _book.BookName,
    BookTitle = _book.BookTitle,
};
using (var db = new MyContextDB())
{
    var result = db.Books.SingleOrDefault(b => b.BookNumber == bookNumber);
    if (result != null)
    {
        try
        {
            db.Books.Attach(book);
            db.Entry(book).State = EntityState.Modified;
            db.SaveChanges();
        }
        catch (Exception ex)
        {
            throw;
        }
    }
}

上記のコードを使用してレコードを更新しようとするたびに、次のエラーが発生します。

{System.Data.Entity.Infrastructure.DbUpdateConcurrencyException:ストアの更新、挿入、または削除ステートメントが予期しない行数(0)に影響を与えました。エンティティが読み込まれた後に、エンティティが変更または削除された可能性があります。ObjectStateManagerエントリを更新


7
補足:catch (Exception ex){throw;}冗長であり、完全に削除できます。
Sriram Sakthivel 2014

try catch blockは、失敗した理由を理解するためのものです。しかし、それでもこのコードが失敗する理由はわかりませんか?
user1327064 14

2
このトピックの専門家ではないので、この質問には答えられません。ただし、try catchを使用せずに、例外がスローされたときにブレーク機能を使用して、例外が発生したときにデバッガをブレークすることもできます。
Sriram Sakthivel 2014

1
何も変更していません。エンティティの状態を操作しても、オブジェクトが実際に変更されていないという事実は変わりません。
ジョナサンアレン

1
まあ、私はあなたと同じことをしました、そして、エラーを得ませんでした。例外はDbUpdateConcurrencyExceptionを示します。どのように並行性を処理しましたか?タイムスタンプを使用しましたか、オブジェクトを複製してから再度マージしましたか、または自己追跡エンティティを使用しましたか?(3つの最もよく使われるアプローチ)。同時実行性を処理しなかった場合は、それが問題だと思います。
El Mac

回答:


344

レコードを更新しようとしている(これは、「既存のレコードの値を変更して保存する」という意味です)。したがって、オブジェクトを取得して変更を加え、保存する必要があります。

using (var db = new MyContextDB())
{
    var result = db.Books.SingleOrDefault(b => b.BookNumber == bookNumber);
    if (result != null)
    {
        result.SomeValue = "Some new value";
        db.SaveChanges();
    }
}

16
値を割り当ててもデータベースは更新されませんdb.SaveChanges()。コンテキスト内の変更されたオブジェクトで呼び出すと、データベースが更新されます。
Craig W.

6
それでも私は魅力的です... var resultは実際にはdbcontextに接続されます...つまり、dbcontextメンバーによってインスタンス化された変数は実際にはデータベースに関連付けられているため、その変数に変更が適用されます、それも適用または永続化されますか?
WantIt

6
コンテキストはオブジェクトを生成したため、コンテキストはオブジェクトへの変更を含め、オブジェクトを追跡できます。SaveChangesコンテキストを呼び出すと、コンテキストは追跡しているすべてのオブジェクトを評価して、それらが追加、変更、または削除されているかどうかを判断し、接続されているデータベースに適切なSQLを発行します。
Craig W.

3
同じ問題に直面しているiam-EF6を使用して、エンティティを更新しようとしています。Attach + EntityState.Modifiedが機能しない。機能しているのは、オブジェクトを取得し、必要な変更を加えて、db.SaveChanges();で保存することだけです。
Gurpreet Singh 2015

7
オブジェクトを更新するために、最初にオブジェクトを取得する必要はありません。主キー値の1つ(複合キー)を変更しようとしていることに気づくまで、同じ問題が発生しました。テーブルに定義されている他の整合性制約を壊さない限り、正しい主キーを提供する限り、EntityStateをModifiedに設定してSaveChanges()が機能します。
adrianz 2017年

165

私はEntity Frameworkのソースコードを確認していて、Keyプロパティがわかっている場合にエンティティを実際に更新する方法を見つけました。

public void Update<T>(T item) where T: Entity
{
    // assume Entity base class have an Id property for all items
    var entity = _collection.Find(item.Id);
    if (entity == null)
    {
        return;
    }

    _context.Entry(entity).CurrentValues.SetValues(item);
}

それ以外の場合は、AddOrUpdate実装でアイデアを確認してください。

この助けを願っています!


12
いいね!すべてのプロパティを列挙する必要はありません。SaveChanges()値を設定した後、呼び出しが必要だと思います。
JanZahradník16年

3
はい、変更はSaveChanges()で永続化されます
Miguel

1
すばらしい答えです。IntelliSenseでは、次のようなことは機能しないことは明確ではありません。_context.MyObj = newObj; 次にSaveChanges()または.... _context.MyObj.Update(newObj)次にSaveChanges(); ソリューションは、すべてのプロパティをループする必要なく、オブジェクト全体を更新します。
アダム

7
これは私がIDフィールドを編集しようとしていると私に不平を言います
Vasily Hall

3
@VasilyHall-これは、IDフィールド(または主キーとして定義したもの)がモデル間で異なる場合に発生します(モデルの1つでnull / 0を含む)。2つのモデル間でIDが一致していることを確認してください。正しく更新されます。
Gavin Coates

51

次のAddOrUpdate方法を使用できます。

db.Books.AddOrUpdate(book); //requires using System.Data.Entity.Migrations;
db.SaveChanges();

1
IMOのベストソリューション
Norgul

112
.AddOrUpdate()はデータベースの移行中に使用されます。移行以外でこのメソッドを使用しないことを強くお勧めしEntity.Migrationsます。そのため、名前空間にあるのです。
アダムヴィンセント

1
@AdamVincentが言ったように、このAddOrUpdate()メソッドは移行を目的としており、既存の行を更新するだけでよい状況には適していません。検索参照(ID)のある本がない場合は、新しい行が作成され、場合によっては問題になることがあります(たとえば、APIがあり、404-NotFound応答を返す必要がある場合存在しない行に対してPUTメソッドを呼び出そうとする)。
マルコ

4
何をしているかわからない場合は、これを使用しないでください!!!!!!!!!!!!!!!! 読み:michaelgmccarthy.com/2016/08/24/...
勇者

4
今日もこれに戻ってきました。これは望ましいユースケースの良い解決策ではないことを警告しますか
Yusha

23

更新されたエンティティがあり、最小限のコードでデータベースを更新したいとします...

並行性は常にトリッキーですが、私はあなたがあなたのアップデートが勝つことを望んでいると仮定しています。これは、同じケースでこれを実行し、クラスを模倣するように名前を変更した方法です。言い換えれば、に変更attachするだけでadd、私にとってはうまくいきます。

public static void SaveBook(Model.Book myBook)
{
    using (var ctx = new BookDBContext())
    {
        ctx.Books.Add(myBook);
        ctx.Entry(myBook).State = System.Data.Entity.EntityState.Modified;
        ctx.SaveChanges();
    }
}

10

オブジェクトのすべてのフィールドを更新する場合は、Entry()メソッドを使用する必要があります。また、フィールドID(キー)を変更することはできないので、最初に編集時と同じIDを設定してください。

using(var context = new ...())
{
    var EditedObj = context
        .Obj
        .Where(x => x. ....)
        .First();

    NewObj.Id = EditedObj.Id; //This is important when we first create an object (NewObj), in which the default Id = 0. We can not change an existing key.

    context.Entry(EditedObj).CurrentValues.SetValues(NewObj);

    context.SaveChanges();
}

2
コードを投稿するだけでなく、少なくとも質問に回答するようにしてください
StaceyGirl 2017年

質問者をよりよく支援するために、コードスニペットを残すだけでなく、質問について説明してください。
feanor07 2017年

9

このコードは、最初にレコードを返すクエリを実行せずに列のセットのみを更新するテストの結果です。最初にEntity Framework 7コードを使用します。

// This function receives an object type that can be a view model or an anonymous 
// object with the properties you want to change. 
// This is part of a repository for a Contacts object.

public int Update(object entity)
{
    var entityProperties =  entity.GetType().GetProperties();   
    Contacts con = ToType(entity, typeof(Contacts)) as Contacts;

    if (con != null)
    {
        _context.Entry(con).State = EntityState.Modified;
        _context.Contacts.Attach(con);

        foreach (var ep in entityProperties)
        {
            // If the property is named Id, don't add it in the update. 
            // It can be refactored to look in the annotations for a key 
            // or any part named Id.

            if(ep.Name != "Id")
                _context.Entry(con).Property(ep.Name).IsModified = true;
        }
    }

    return _context.SaveChanges();
}

public static object ToType<T>(object obj, T type)
{
    // Create an instance of T type object
    object tmp = Activator.CreateInstance(Type.GetType(type.ToString()));

    // Loop through the properties of the object you want to convert
    foreach (PropertyInfo pi in obj.GetType().GetProperties())
    {
        try
        {
            // Get the value of the property and try to assign it to the property of T type object
            tmp.GetType().GetProperty(pi.Name).SetValue(tmp, pi.GetValue(obj, null), null);
        }
        catch (Exception ex)
        {
            // Logging.Log.Error(ex);
        }
    }
    // Return the T type object:         
    return tmp;
}

ここに完全なコードがあります:

public interface IContactRepository
{
    IEnumerable<Contacts> GetAllContats();
    IEnumerable<Contacts> GetAllContactsWithAddress();
    int Update(object c);
}

public class ContactRepository : IContactRepository
{
    private ContactContext _context;

    public ContactRepository(ContactContext context)
    {
        _context = context;
    }

    public IEnumerable<Contacts> GetAllContats()
    {
        return _context.Contacts.OrderBy(c => c.FirstName).ToList();
    }

    public IEnumerable<Contacts> GetAllContactsWithAddress()
    {
        return _context.Contacts
            .Include(c => c.Address)
            .OrderBy(c => c.FirstName).ToList();
    }   

    //TODO Change properties to lambda expression
    public int Update(object entity)
    {
        var entityProperties = entity.GetType().GetProperties();

        Contacts con = ToType(entity, typeof(Contacts)) as Contacts;

        if (con != null)
        {
            _context.Entry(con).State = EntityState.Modified;
            _context.Contacts.Attach(con);

            foreach (var ep in entityProperties)
            {
                if(ep.Name != "Id")
                    _context.Entry(con).Property(ep.Name).IsModified = true;
            }
        }

        return _context.SaveChanges();
    }

    public static object ToType<T>(object obj, T type)
    {
        // Create an instance of T type object
        object tmp = Activator.CreateInstance(Type.GetType(type.ToString()));

        // Loop through the properties of the object you want to convert
        foreach (PropertyInfo pi in obj.GetType().GetProperties())
        {
            try
            {
                // Get the value of the property and try to assign it to the property of T type object
                tmp.GetType().GetProperty(pi.Name).SetValue(tmp, pi.GetValue(obj, null), null);
            }
            catch (Exception ex)
            {
                // Logging.Log.Error(ex);
            }
        }
        // Return the T type object
        return tmp;
    }
}    

public class Contacts
{
    public int Id { get; set; }
    public string FirstName { get; set; }
    public string LastName { get; set; }
    public string Email { get; set; }
    public string Company { get; set; }
    public string Title { get; set; }
    public Addresses Address { get; set; }    
}

public class Addresses
{
    [Key]
    public int Id { get; set; }
    public string AddressType { get; set; }
    public string StreetAddress { get; set; }
    public string City { get; set; }
    public State State { get; set; }
    public string PostalCode { get; set; }  
}

public class ContactContext : DbContext
{
    public DbSet<Addresses> Address { get; set; } 
    public DbSet<Contacts> Contacts { get; set; } 
    public DbSet<State> States { get; set; }

    protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
    {
        var connString = "Server=YourServer;Database=ContactsDb;Trusted_Connection=True;MultipleActiveResultSets=true;";
        optionsBuilder.UseSqlServer(connString);
        base.OnConfiguring(optionsBuilder);
    }
}

7

.netコアの場合

context.Customer.Add(customer);
context.Entry(customer).State = Microsoft.EntityFrameworkCore.EntityState.Modified;
context.SaveChanges();

5

この問題の最善の解決策は次のとおりです。ビューですべてのID(キー)を追加します。(First、Second、Third)という名前の複数のテーブルを持つことを検討してください

@Html.HiddenFor(model=>model.FirstID)
@Html.HiddenFor(model=>model.SecondID)
@Html.HiddenFor(model=>model.Second.SecondID)
@Html.HiddenFor(model=>model.Second.ThirdID)
@Html.HiddenFor(model=>model.Second.Third.ThirdID)

C#コードでは、

[HttpPost]
public ActionResult Edit(First first)
{
  if (ModelState.Isvalid)
  {
    if (first.FirstID > 0)
    {
      datacontext.Entry(first).State = EntityState.Modified;
      datacontext.Entry(first.Second).State = EntityState.Modified;
      datacontext.Entry(first.Second.Third).State = EntityState.Modified;
    }
    else
    {
      datacontext.First.Add(first);
    }
    datacontext.SaveChanges();
    Return RedirectToAction("Index");
  }

 return View(first);
}

5

Attachエンティティを実行すると、追跡状態がに設定されUnchangedます。既存のエンティティを更新するには、追跡状態をに設定するだけですModifiedEF6ドキュメントによると:

データベースにすでに存在しているが、変更が加えられている可能性があるエンティティがある場合は、エンティティにアタッチしてその状態を変更済みに設定するようコンテキストに指示できます。例えば:

var existingBlog = new Blog { BlogId = 1, Name = "ADO.NET Blog" };

using (var context = new BloggingContext())
{
    context.Entry(existingBlog).State = EntityState.Modified;

    // Do some more work...  

    context.SaveChanges();
}

4
using(var myDb = new MyDbEntities())
{

    user user = new user();
    user.username = "me";
    user.email = "me@me.com";

    myDb.Users.Add(user);
    myDb.users.Attach(user);
    myDb.Entry(user).State = EntityState.Modified;//this is for modiying/update existing entry
    myDb.SaveChanges();
}

4

私はうまく機能する方法を見つけました。

 var Update = context.UpdateTables.Find(id);
        Update.Title = title;

        // Mark as Changed
        context.Entry(Update).State = System.Data.Entity.EntityState.Modified;
        context.SaveChanges();


1

これが私のRIA後のエンティティ更新メソッドです(Ef6タイムフレーム用):

public static void UpdateSegment(ISegment data)
{
    if (data == null) throw new ArgumentNullException("The expected Segment data is not here.");

    var context = GetContext();

    var originalData = context.Segments.SingleOrDefault(i => i.SegmentId == data.SegmentId);
    if (originalData == null) throw new NullReferenceException("The expected original Segment data is not here.");

    FrameworkTypeUtility.SetProperties(data, originalData);

    context.SaveChanges();
}

これFrameworkTypeUtility.SetProperties()は、NuGetのAutoMapperのかなり前に私が書いた小さなユーティリティ関数です。

public static void SetProperties<TIn, TOut>(TIn input, TOut output, ICollection<string> includedProperties)
    where TIn : class
    where TOut : class
{
    if ((input == null) || (output == null)) return;
    Type inType = input.GetType();
    Type outType = output.GetType();
    foreach (PropertyInfo info in inType.GetProperties())
    {
        PropertyInfo outfo = ((info != null) && info.CanRead)
            ? outType.GetProperty(info.Name, info.PropertyType)
            : null;
        if (outfo != null && outfo.CanWrite
            && (outfo.PropertyType.Equals(info.PropertyType)))
        {
            if ((includedProperties != null) && includedProperties.Contains(info.Name))
                outfo.SetValue(output, info.GetValue(input, null), null);
            else if (includedProperties == null)
                outfo.SetValue(output, info.GetValue(input, null), null);
        }
    }
}

注:プロパティがモデル内で、保存されているViewModelオブジェクトとまったく同じ場合にのみ機能します。
vapcguy

1

Renatが言ったように、削除: db.Books.Attach(book);

また、「AsNoTracking」を使用するように結果クエリを変更します。これは、このクエリがエンティティフレームワークのモデル状態をスローするためです。それは「結果」が今追跡するべき本であり、あなたがそれを望んでいないと考えています。

var result = db.Books.AsNoTracking().SingleOrDefault(b => b.BookNumber == bookNumber);

1

それを試してみてください....

UpdateModel(book);

var book = new Model.Book
{
    BookNumber =  _book.BookNumber,
    BookName = _book.BookName,
    BookTitle = _book.BookTitle,
};
using (var db = new MyContextDB())
{
    var result = db.Books.SingleOrDefault(b => b.BookNumber == bookNumber);
    if (result != null)
    {
        try
        {
            UpdateModel(book);
            db.Books.Attach(book);
            db.Entry(book).State = EntityState.Modified;
            db.SaveChanges();
        }
        catch (Exception ex)
        {
            throw;
        }
    }
}

1

私はそれがすでに何度か良い答えをされていることを知っていますが、私はこれを行う以下の方法が好きです。誰かのお役に立てば幸いです。

//attach object (search for row)
TableName tn = _context.TableNames.Attach(new TableName { PK_COLUMN = YOUR_VALUE});
// set new value
tn.COLUMN_NAME_TO_UPDATE = NEW_COLUMN_VALUE;
// set column as modified
_context.Entry<TableName>(tn).Property(tnp => tnp.COLUMN_NAME_TO_UPDATE).IsModified = true;
// save change
_context.SaveChanges();

1

これはEntity Framework 6.2.0の場合です。

DbSet更新または作成する必要がある特定のアイテムがある場合:

var name = getNameFromService();

var current = _dbContext.Names.Find(name.BusinessSystemId, name.NameNo);
if (current == null)
{
    _dbContext.Names.Add(name);
}
else
{
    _dbContext.Entry(current).CurrentValues.SetValues(name);
}
_dbContext.SaveChanges();

ただし、これはDbSet単一の主キーまたは複合主キーを持つジェネリックにも使用できます。

var allNames = NameApiService.GetAllNames();
GenericAddOrUpdate(allNames, "BusinessSystemId", "NameNo");

public virtual void GenericAddOrUpdate<T>(IEnumerable<T> values, params string[] keyValues) where T : class
{
    foreach (var value in values)
    {
        try
        {
            var keyList = new List<object>();

            //Get key values from T entity based on keyValues property
            foreach (var keyValue in keyValues)
            {
                var propertyInfo = value.GetType().GetProperty(keyValue);
                var propertyValue = propertyInfo.GetValue(value);
                keyList.Add(propertyValue);
            }

            GenericAddOrUpdateDbSet(keyList, value);
            //Only use this when debugging to catch save exceptions
            //_dbContext.SaveChanges();
        }
        catch
        {
            throw;
        }
    }
    _dbContext.SaveChanges();
}

public virtual void GenericAddOrUpdateDbSet<T>(List<object> keyList, T value) where T : class
{
    //Get a DbSet of T type
    var someDbSet = Set(typeof(T));

    //Check if any value exists with the key values
    var current = someDbSet.Find(keyList.ToArray());
    if (current == null)
    {
        someDbSet.Add(value);
    }
    else
    {
        Entry(current).CurrentValues.SetValues(value);
    }
}
弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.