永続化前後のクラスの表現について C#

質問

ものすごくくだらない質問ですみません。
リレーショナルデータベースを使わざるを得なく、O/Rマッピングツールは使えない状況です。
class Employee
があります。
StaffCodeがありますが、将来このコードが重複はありませんが変わる可能性があるので永続化を目的としてこれを主キーとして使えません。
そこで主キーにサロゲートキーを使おうと考えました。
アプリ側で対応するよりもデータベース側でオートナンバーを使います。
一応下記コードでこれから採番する社員と、永続化＆採番したあとの社員を表現する事はできます。
しかし同じクラスを使うものなのでしょうか？
クラスを分けるべきでしょうか？
そもそも考え方がおかしいですか？

Class Employee {
Public Int Id { get; private set; }
Public String Name { get; private set; }
Public Int StaffCode { get; private set; }

Public Employee(String name, Int staffCode) {
this.Name = name;
this.StaffCode = staffCode;
}

Public Employee(Int id, String name, Int staffCode) {
this.Id = id;
this.Name = name;
this.StaffCode = staffCode;
}

Public Void SetId(Int id) {
this.Id = id;
}
}

回答No.2

ques9900 · ベストアンサー

> 一つ気になることがあるのですが、一番変更してはいけないidが
>一番変更しやすい設計になっているのは割り切って考えればいいんでしょうか？
はい、割りきってしまって良いと思いますよ。
もし何か変更条件を追加したくなった場合には、SetIdに条件を追記すればよいかと。

お礼

お礼が遅くなって済みませんでした。
インフルエンザにかかって寝込んでました。
ques9900さんも気をつけてくださいね。
割り切って上記方法でやっていきたいと思います。
ありがとうございました。

回答No.1

ques9900

同じクラスで良いと思われます。

従業員の雇用形態が変わるような場合は、
差分を派生クラスにするべきと思いますが、

永続化の有無は全ての従業員に必要な属性と思うので、
基底クラスにidがあれば良いと思います。

お礼

ありがとうございます。
確かに雇用形態が変わる場合は派生クラスにすべきですよね。
考え方がずれてないようで一安心です。

補足

一つ気になることがあるのですが、一番変更してはいけないidが一番変更しやすい設計になっているのは割り切って考えればいいんでしょうか？

Public Void SetId(Int id)

既にidが設定されていれば例外を投げればいい気もしますが。
そこだけちょっと気持ち悪くて気になっています。
新しく質問すべきかもしれない内容で申し訳ないですが、よろしければアドバイスお願いします。