3つ以上の同じ処理内容を含むコンストラクタの利用方法

Question

複数のコンストラクタの内容に重複部分がある場合、コンストラクタからコンストラクタを呼び出せば手間が省けるというの学んだのですが

（例）

public class Main {
 public static void main(String[] args) {
	 Hero h1 = new Hero("タナカ");
	 Hero h2 = new Hero();
	 
	 System.out.println(h1.hp);
	 System.out.println(h1.name);
	 
	 System.out.println(h2.hp);
	 System.out.println(h2.name);
 }
}

public class Hero {
	String name;
	int hp;
	
	Hero(String name) {
		this.hp = 100;
		this.name = name;
	}
	Hero() {
		this("スズキ");
		
	}
	
}

実行結果

100
タナカ
100
スズキ

もう一つ(3つ以上)インスタンスを生成して、同じ内容のコンストラクタ(hpは同じで名前はまた個別に与える)を利用したい場合どうしたらいいですか？

同じように

public class Main {
 public static void main(String[] args) {
	 Hero h1 = new Hero("ミナト");
	 Hero h2 = new Hero();
	 Hero h3 = new Hero();
	 System.out.println(h1.hp);
	 System.out.println(h1.name);
	 
	 System.out.println(h2.hp);
	 System.out.println(h2.name);
         System.out.println(h3.name);
 }
}

public class Hero {
	String name;
	int hp;
	
	Hero(String name) {
		this.hp = 100;
		this.name = name;
	}
	Hero() {
		this("スガワラ");
		
	}
        Hero() {
		this("ヤマダ");
		
	}
	
}

と書こうと思ったのですが、”メソッド名が重複しています”　とエラーメッセージが出ます。
でもメソッド名とクラス名が同じでないとコンストラクタと見なされないし、どうすれば良いのでしょうか？

Answer 1

結論から言うと、次のようなコンストラクタは書くべきではありません。

public class Hero {
    ...

Hero() {
    this("スズキ");
}

}

引数のないコンストラクタはこの場合デフォルトの動作をすべきで、具体性を伴ってはいけません。
『知らないヒーローを見たらスズキだと思え』みたいなイメージになってしまっています。

次のようなコンストラクタなら、まだ許容範囲ではあります。

public class Hero {
    ...

    Hero() {
        this("unknown");
    }
}

サンプル

ついでに、アクセッサという重要な要素も取り入れています。

public class Main {
    public static void main(String[] args) {
        Hero h1 = new Hero("ミナト");
        Hero h2 = new Hero("スガワラ");
        Hero h3 = new Hero("ヤマダ");

        System.out.println(h1.getHp());
        System.out.println(h1.getName());

        System.out.println(h2.getHp());
        System.out.println(h2.getName());

        System.out.println(h3.getHp());
        System.out.println(h3.getName());
    }
}

class Hero {
    private final String name;
    private int hp;

    Hero(String name) {
       this.hp = 100;
       this.name = name;
    }

    String getName() {
        return name;
    }
    int getHp() {
        return hp;
    }
}

このコードもまだ改良可能です。
配列やリストなどの学習を終えたら、ちょっと考えてみてください。

Comments

[退会済みユーザー]

そうなんですね。コンストラクタがどういう動作をするものなのかというのを学んだばかりなので、あまり質の事はよくわかっていないので参考になります。ありがとうございました。

Answer 2

ご質問を見て違和感を感じたのは、まず、
変数の値が異なるだけなら、引数を変えれば良いということです。

Javaのコンストラクタはオーバーロードできますが、
オーバーロードするには引数の型か個数が異なる必要があります。
が、これで十分なのです。具体的な値は引数で変えれば良いからです。

上手くできた仕組みです。ご質問のサンプルコードのように、
具体的な値が異なるたびに、コンストラクタを作っていては、
キリがありませんので、まずしません。そもそも、やる必要がないのです。

ただ、やろうとしている延長線上の書き方は思いつくので、一応紹介します。
言葉だけで説明すると分かりにくいので、具体的なコードを示します。
以下、すごく長くなってしまいましたが、コード自体はシンプルです。

---

public class Main {
 public static void main(String[] args) {

     Hero h1 = new Hero();
     Hero h2 = new Hero("スズキ");
     Hero h3 = new Hero("タナカ", 200);
 
     System.out.println(h1.name);
     System.out.println(h1.hp);

     System.out.println(h2.name);
     System.out.println(h2.hp);

     System.out.println(h3.name);
     System.out.println(h3.hp);
 }
}

public class Hero {
    String name;
    int hp;

    Hero() {
        this.hp = 0;
        this.name = "Anonymous";
    }
    Hero(String name) {
        this.hp = 0;
        this.name = name;
    }
    Hero(String name, int hp) {
        this.hp = hp;
        this.name = name;
    }
}

public class Hero {
    String name;
    int hp;

    Hero() {
        this("Anonymous", 0);
    }
    Hero(String name) {
        this(name, 0);
    }
    Hero(String name, int hp) {
        this.hp = hp;
        this.name = name;
    }
}

public class Hero {
    String name;
    int hp;

    Hero() {
        this("Anonymous");
    }
    Hero(String name) {
        this(name, 0);
    }
    Hero(String name, int hp) {
        this.hp = hp;
        this.name = name;
    }
}

// 動作結果
Anonymous
0
スズキ
0
タナカ
200

まず、Mainクラスは、ご質問のサンプルコードを
説明の便宜上、少しだけ改変しました。

それから、Heroクラスの書き方を3つ示しましたが、
これらを交換しても、同じ最後の動作結果になります。

上のコードは何かというと、デフォルト値を設定する書き方です。
たとえば、RPGなら（クラス名はそのイメージでしょう）、
NPCがたくさんいて、街を歩いてるけど、名前も戦闘も関係ない、
そういう場合に省略できる書き方です。

3つのHeroクラスは、コンストラクタのオーバーロードを利用していて、
最初のものは単純に別々に初期化しています。
2つめは、別のコンストラクタを呼び出しており、
3つめは、連鎖的にコンストラクタを呼び出しています。

後の方ほど短くなります。サンプルでは大差ないように感じるでしょうが、
設定する変数や引数が多くなると、もっと違いが出てきます。
ただし、その一方で記述が複雑になりやすいデメリットもあります。

実際に使うかどうかは別にして、こういう書き方があるということです。

---

さてここまで、なるべくご質問のやりたいことに沿って、
私の価値観を押しつけないようにしましたが、
私自身はこういうコンストラクタを多用しません。

もし、インスタンスの生成が複雑になれば、
たとえば、ファクトリやビルダー、
デコレータなどの利用を検討します。

そもそも、引数なしのコンストラクタより、
完全コンストラクタの方が良いと思っています。
また、スモールオブジェクトにするので、
コンストラクタがやたら複雑化することもあまりないです。

これらを逐一説明すると長くなるので省略しますが、
デザインパターンなどを学習すると分かってきます。

Comments

[退会済みユーザー]

すごくわかりやすかったです。ありがとうございます。まだまだ入門教本をノソノソやっているので、これから色んなやり方を学んでいこうと思います。ありがとうございました。

Answer 3

コンストラクタをそのままに、引数を変えればいいだけです。

public class Main {
 public static void main(String[] args) {
     Hero h1 = new Hero("ミナト");
     Hero h2 = new Hero("スガワラ");
     Hero h3 = new Hero();
     System.out.println(h1.hp);
     System.out.println(h1.name);

     System.out.println(h2.hp);
     System.out.println(h2.name);
         System.out.println(h3.name);
 }
}

もっとも、引数なしのコンストラクタで具体名を持っているのがあまり褒められたものではないのはLouiS0616さんのご指摘のとおりです。

Comments

[退会済みユーザー]

単純に考えればその通りでしたね・・・実際に書き込んでみるとより一層感じました。
お恥ずかしい。ありがとうございました。