クラス内でプロパティを通してフィールドにアクセスすることの是非

前提・実現したいこと

はじめまして。
c#で、あるクラスを作成しているのですが、
プロパティの扱いに関して質問があります。

プロパティは一般には、クラス内のメンバ変数に外からアクセスするときの
インターフェース的なもので、クラス内部のコードでは対応するメンバ変数を
使うという理解なのですが、クラス内部でそのままプロパティをメンバ変数のように
使用するのは良くないのでしょうか。

該当のソースコード

例えば、あるクラスの中身が以下のようだとして、
クラス内で、プロパティ"Name"をそのまま使っていますが、
何か問題あるのでしょうか。

csharp
1private string _name;
2public string Name
3{
4  get { return this._name; }
5  set { this._name = value; }
6}
7
8private string Hogehoge()
9{
10    Name = "yamada"; // this._name = "yamada";のほうが適切？
11    var xxx = Name; // var xxx = this._name;のほうが適切？
12
13    return xxx;
14}
15

またそもそも、特にアクセスレベルの制限などする必要のないような場合、
以下のように、メンバ変数を省略するというのはありでしょうか。
(この場合、単にメンバ変数pulic string Name;を使うのとほぼ同じ・・・？)

csharp
1public string Name { get; set; }
2
3private string Hogehoge()
4{
5    Name = "yamada"; 
6    var xxx = Name; 
7
8    return xxx;
9}
10

よろしくお願いいたします。

退会済みユーザー

2020/11/16 12:28 編集

> プロパティは一般には、クラス内のメンバ変数に外からアクセスするときのインターフェース的なもので、クラス内部のコードでは対応するメンバ変数を使うという理解なのですが、オブジェクト指向プログラミングの「カプセル化」という考えに沿って、フィールドは private それに外部からアクセスする必要がある場合はパブリックプロパティを使うという基本で良いと思いますよ。

行動規範の内容に同意します

回答2件

ベストアンサー

質問のコメント欄に書いたことを含め、プロパティを使う理由を以下に書いておきます。

(1) オブジェクト指向の概念の一つ「カプセル化」を実現するため、通常クラス内の各フィールドへの直接アクセスは禁止するようにしておき、外部からはパブリックプロパティで各フィールドの値を取得したり設定したりするということがもともとのプロパティの目的です。

(2) プロパティを使う目的には、開発者が意図した規則に基づいてフィールドを正しく使用できるよう保証するということもあります。

(3) プロパティでなければダメというケースもあります。例えば、Entity Framework Code First でのモデルを定義を行う場合はフィールドではダメで、プロパティの定義が必要です。他には、ASP.NET Web Forms アプリのデータバインド式でもプロパティでないとダメです。

【追記】

下のコメントで書いた「自動実装プロパティでフィールドが定義してないという場合はプロパティを使わざるを得ないですね」の件です。Configuration プロパティは Visual Studio のテンプレートで生成されるコードです。

投稿2020/11/16 12:34

編集2020/11/16 13:28

退会済みユーザー

総合スコア0

xxx.ty

2020/11/16 12:43

コメント欄含め、ご回答ありがとうございます。クラス内ではやはり、プロパティでget,setするのではなく、フィールド変数を直接使うのが普通なのでしょうか。「クラス内からであっても、private のメンバー変数には直接アクセスせず、プロパティを通してアクセスする方が後々の保守がしやすかったりします」という記事を見つけまして、どっちがいいのかなと思いまして。。

退会済みユーザー

2020/11/16 13:13

> クラス内からであっても、private のメンバー変数には直接アクセスせず、プロパティを通してアクセスする方が後々の保守がしやすかったりしますクラス内からしかアクセスしないフィールドにわざわざパブリックプロパティを定義するということはないのでは？　クラス外からアクセスするのでパブリックプロパティも定義してあるような場合は、そういう話（クラス内でもプロパティを使う）もあるかもしれませんけど。それと、自動実装プロパティでフィールドが定義してないという場合はプロパティを使わざるを得ないですね。

退会済みユーザー

2020/11/16 13:24

上のコメントで書いた、 > 自動実装プロパティでフィールドが定義してないという場合はプロパティを使わざるを得ないですね。という例を回答欄に追記しておきます。

xxx.ty

2020/11/16 13:25

なるほど、ありがとうございます。＞クラス外からアクセスするのでパブリックプロパティも定義してあるような場合主にこちらの意味で質問しておりました。こういう場合で、フィールド変数も特に定義していなかった場合（自動実装プロパティ）、プロパティをそのままクラス内で使っても別に変ではないんですね。すっきりしました。ありがとうございます。