C# のEntity frameworkで、元のデータを編集して新規のデータを追加しようとするとPrimary Keyのカラムを変更出来ない

Question

Visual Studio 2015のC#でEntity Frameworkで出来たアプリで
データの編集をしているのですが
新しいレコードを作成し、レコードに追加しようとするとプライマリーキーを変更するとエラーになってしまいます
わざわざPKがかぶらないようにしたのですが・・・
元のデータを編集して、一部のデータを変えただけで新規のデータを作りたいのですが
こういった場合に新規データを作りたい場合はどうしたらいいのでしょうか

カラム数が多いのですが一部を書き換えて増殖したいだけなのです

using (var context = TBL_DBAccess.GetEntities())
    var item = context.TABLENAME.Single(n => n.ID == id);	//編集元のデータを持ってくる
    item.ID= 100;		//ここがPK
    item.col1= 200;
    item.col2= "test";

    context.TABLENEME.Add(item);    //新しいレコードを追加★★★->プライマリーキーの変更は出来ませんというエラー
    //テーブルの書換
    context.SaveChanges();
}

Comments

[退会済みユーザー]

DB は何ですか？　対象のテーブルの構造は？　ID は IDENTITY とかですか？　「元のデータを編集して新規のデータを追加」というのがそもそも解せないです。やってることは新規データの追加ではなく既存のデータの編集にみえます。であれば普通はそういうこと（主キーの変更）はしないはず。

[退会済みユーザー]

質問者さんの回答への 2021/08/26 10:41 の私のコメントで「後で最初から最後まで Entity Framework を使った案を回答欄に書いておきます。」と書いた件、先ほど回答しておきましたので見てください。

Answer 1

質問者さんの回答への 2021/08/26 10:41 の私のコメントで「後で最初から最後まで Entity Framework を使った案を回答欄に書いておきます。」と書いた件です。

もともとの質問は、タイトルにもあるように「C# のEntity frameworkで」だったはずで、質問者さんの回答のようにするのは、ご自分でも言っているように「無理やりですが・・」だと思います。

質問者さんのコードでなぜエラーになるかと言うと、コンテキストがエンティティを追跡しているからです。

念のため、自分の環境で、以下のような SQL Server のデータベースを使って、同様なことを試してみました。（Id は IDENTITY ではありません）

質問者さんのコードと同様なことをすると、コンテキスト context がエンティティ table を追跡しているため、質問者さんの場合と同じエラーになります。

なので、追跡されないようにするというのが解決策の一つになります。それには AsNoTracking メソッドが利用できます。詳しくは以下の記事を見てください。

追跡なしのクエリ
https://docs.microsoft.com/ja-jp/ef/ef6/querying/no-tracking

AsNoTracking メソッドを使って上のコードを書き直すとエラー無く実行されます。

結果も以下の通り DB に反映されます。

お試しください。

Answer 2

掲示したコードは新規データの追加ではなく、既存データの変更をしています。

データベースの仕組み上、主キーの更新はリンクされている全てのテーブルも更新する必要があり推奨されません。ですから、Entity Framework においてもそのような操作は許可されずエラーが出ているのでしょう。

元のデータを編集して、一部のデータを変えただけで新規のデータを作りたい

まずは コンテキストとエンティティ クラスを使用してデータを追加、変更、および削除する方法 を参照してください。"データの追加" にて新たなエンティティのインスタンスを作成している部分が重要です。

つまり、データの複製をするには、既存のエンティティから新しくインスタンスを作成する必要があります。

using (var context = new BloggingContext())
{
    var original = context.Blogs.First();
    var clone = new Blog { Url = source.Url };
    context.Blogs.Add(clone);
    context.SaveChanges();
}

注意すべき点として主キーを自動採番していない場合は開発者が主キーを重複させないよう実装する必要があります。

Comments

[micchi]

Entity Frameworkで自動的作成したクラスを
var clone = new XXX{ itemorg };
としてみたところ、「IEnumerableを実装していないためXXXはコレクション初期化子で初期化することは出来ません」
というエラーになってしまい、新規にデータ作成は出来ないようでした

別の方法で無理やり解決したのでそちらをこれから回答として作成します

[BluOxy]

すみません。書き方を間違えていたので追記しました。

EFで生のSQLを書いたらEFを使用する意味が薄くなりますし、使い方次第ではセキュリティリスクにもなります。
ですから、公式のドキュメントに記載されているような方法で解決できるなら無理に解決するよりそちらを選択した方が不幸を招かないと思います。

[BluOxy]

var clone = new XXX
{
ID = 100, // ID だけは自動採番に任せるか、直接入力する
col1 = item.col1,
col2 = item.col2
};
のようなものを作り Add すればわざわざSQL文を書く必要はありません。
ちなみに、この{}を使った書き方はオブジェクト初期化子というものです。
https://docs.microsoft.com/ja-jp/dotnet/csharp/programming-guide/classes-and-structs/object-and-collection-initializers

[micchi]

重ね重ねありがとうございます。
カラムが大量にあるのと元のレコードのデータが必要なので、元のデータをなんとかしてコピーしないと行けないのと
すでに存在するテーブルなのでPKを自動採番にするとかできないです
deepcloneでも実装しようかと思ったのですが、それもVSが自動コーディングで作成したクラスなのでなんともならないので(どちらにしろ大量にメンバ変数ごとにコピーするなんてコーディングはしたくないので・・・)

Answer 3

無理やりですが自己回答です・・
こちらのページに記載されていた
SQL文で元のデータを増殖させて

string strSql ="select * into #wktbl from tbl1"+ Environment.NewLine;
strSql += "alter table #wktbl drop column id"+ Environment.NewLine;
strSql += "insert into tbl1 select 1 as id ,* from #wktbl"+ Environment.NewLine;
context.Database.ExecuteSqlCommand(strSql);//★SQL分を無理やり実行

そこから

var tmp = context.tabl.Single(n => n.ID == 999);

として引っ張り出して、キー以外の部分を変更して保存としたら上手く行きました

海外だとストアド作ってをキックするとかしてますが、そこまですることも無いかと思いこちらの方法を使いました

Comments

[退会済みユーザー]

もともとの質問はタイトルにもあるように「C# のEntity frameworkで」だったはずです。上のようにするのは話が違うと思います。

[退会済みユーザー]

後で最初から最後まで Entity Framework を使った案を回答欄に書いておきます。