c# 変数の共通化方法

Question

こんにちは。

掲題の件、良い方法があればアドバイスください。

以下のような複数のクラスで共通の方の変数を利用したいと考えています。

• 商品コード（int型）
• 商品名（string型）
• 金額（float型）

例えば、売上や在庫クラスなどでこれらの変数を使いたい場合、
共通クラスを定義してそれを継承するという方法があると思います。
※独自のクラスで商品コードなどは定義したくない。

class Item
{
  string item_code;
  string item_text;
  string item_value;
}

class Sell : Item
{
}

この場合、SellはItemの機能を引き継ぐクラスではないので、継承と言うのも違和感があります。

何かほかに良い方法があれば教えていただければ幸いです。

宜しくお願い致します。

Comments

[Zuishin]

プロパティでいいのでは？

class Sell { public Item Item { get; }; }

Answer 1

話がよく見えませんが、
is-aかhas-aで悩んでいるのでしたら、

is-a関係とhas-a関係: 継承と包含
https://think-on-object.blogspot.com/2011/11/is-ahas-is-ahas-top-is-a-is-b.html

あたりが参考になりますかね？

Answer 2

複数クラスで共通した性質を利用するが、継承に違和感がある場合はだいたい包含を使えば解決するので、単純に共通する部分をオブジェクトとして変数やプロパティで持ってしまえばいいのでは。（質問のコメント欄にも書いてあるけど）

適当にググって出てくるこういう記事とか読んでみるといいのかも。

Answer 3

例えば、売上や在庫クラスなどでこれらの変数を使いたい場合、共通クラスを定義してそれを継承するという方法があると思います。

※独自のクラスで商品コードなどは定義したくない。

そもそも、「売上」や「在庫」をクラスにしません。それらは「商品」の状態、あるいは状態変化に起因する計算結果ではないでしょうか。

Comments

[Zuishin]

すると思いますよ。DB のテーブルはだいたいクラスになる可能性があります。日付・顧客・商品・金額などが売上クラスのプロパティになります。

[Q71]

質問を読んで持った違和感がなんだか分かったような。
在庫や売上というのは、商品の状態だと思います。状態は、属性にはしてもクラスにはしないよなぁ、と思っていたのですが、状態によって振る舞いを変えるなら、継承しますね、確かに。
ただ、それだとこういう質問にはならないはずで、「共通クラス」という言葉に強く違和感を覚えました。
最近はデータベースを触ることはないのですが、20年くらい前には、テーブルにあわせてクラスを作っていましたね。今作るなら、管理の主体を考えて、それとの関係からクラスを作ると思います。

[Zuishin]

売上や在庫が状態だとして、それに付随して、売れた商品なら顧客、個数、金額、納品日や入金日、在庫なら倉庫など特定の状態特有のプロパティが業務上必要になってくるので、それぞれの状態にクラスを作るのがいいんじゃないかと思います。