質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.51%

  • Java

    13762questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

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

解決済

回答 3

投稿

  • 評価
  • クリップ 0
  • VIEW 318

stynjoe

score 7

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

(例)

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("ヤマダ");

    }

}


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

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 3

+2

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

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;
    }
}

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

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/11/20 17:05

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

    キャンセル

checkベストアンサー

+1

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

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つめは、連鎖的にコンストラクタを呼び出しています。

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

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


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

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

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/11/20 16:53

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

    キャンセル

+1

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

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さんのご指摘のとおりです。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/11/20 17:01

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.51%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • Java

    13762questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。