C# と VB.NET の質問掲示板

■No100800 (しーや さん) に返信
> 勘違いだといけないので、下記のコードと自分の考えを書いたものが正しいか評価お願いします。

間違ってはいないと思います。
ただコンストラクタについては、処理を共通化するために次のように書くことが多いです。

public class Character1
{
    public string name;
    public int exp;

    public Character1() : this("no name") { }

    public Character1(string name)  {
        this.name = name;
    }
}

引数の無いコンストラクタが呼ばれたら、引数のあるコンストラクタを呼び出します。

パターンはもう一つあって

public class Character2
{
    public string name;
    public int exp;

    public Character2() {
        Initialize("no name");
    }

    public Character2(string name)  {
        Initialize(name);
    }

    private void Initialize(string name) {
        this.name = name;
    }
}

これは共通処理の前にいろいろ処理を行う場合に使います。

System.Drawing.Font クラスがこのパターンです。
https://referencesource.microsoft.com/#System.Drawing/commonui/System/Drawing/Advanced/Font.cs,7c692e253eb24b01

■No100801 (KOZ) に返信

あまり見かけませんが、最初のパターンの逆で引数のあるコンストラクタから
引数の無いコンストラクタを呼び出すケース。
実装したあとで機能追加していったのかなぁ

public class Character3
{
    public string name;
    public int exp;

    public Character3() {
        // 初期処理
    }

    public Character3(string name) : this() {
        this.name = name;
    }
}

直接の回答は、みなさんのお話通りですが、なぜ、
みなさん、コンストラクタから別のコンストラクタを呼ぼうとしてるか分かりますか？

コンストラクタの役目は初期化です。
たとえ、複数のコンストラクタがあっても、共通して絶対に行いたい初期化というのが
あることもあるわけです。

たとえば、
  class Catalog
みたいなクラスがあって、カタログのデータを取得できるものとします。
カタログの実体は、実はXMLファイルでCatalogクラスに隠蔽したいとします。

そんな感じの設計なら、さくっとコンストラクタでXMLファイル読み込んじゃうのが
いいですよね。

これらのコンストラクタなら、
  Catalog(Kind kind)         // 製品種別を引数に持つコンストラクタ
  Catalog(string kind_name)  // 製品種別名称を引数に持つコンストラクタ
  Catalog()                  // 全製品のXMLを一気に読み込んじゃうコンストラクタ

XMLファイルを読み込む処理自体は共通なので、どれかのコンストラクタに、
MLファイルを読み込む処理を実装して、他のコンストラクタからは、そのコンストラクタ
を呼ぶようにします。

これをやらないと何が不都合かというと...
将来、XMLファイルのフォーマットが変わった時に、３つのコンストラクタ全てを同じ
ように修正しなきゃいけないわけです。

同様に、
異常系の処理、「そんなタグ名知らん」「必要なタグが見つからん」...などを
いちいち３つのコンストラクタにコピーコードで入れられたらたまらんというわけです。

なんとなく伝わったかな？