引数コンストラクタを引数に入れる

Question

Main javaについての質問です。
下記の「引数ありのコンストラクタ」を作成したのですが、それを引数に入れるのをしてくださいと言われたのですが、よく分かりません。
どのようにすればいいのかご教授いただきたいです。

First p = new First();
p.name = "ピカチュウ";
p.type = "でんき";
p.number = 25;
p.pikachu = new Pikachu();
p.pikachu.name = "10まんボルト";
p.pikachu.damage = 20;

public class First {
	String name;
	String type;
	int number;
	
	Pikachu pikachu;
	
	public First() {
		this.name = "NoName";
		this.type = "NoType";
		this.number = 25;
		this.pikachu = new Pikachu();
	}
	
	public First(String name, String type, int number) {
		this.name = name;
		this.type = type;
		this.number = number;
		this.pikachu = new Pikachu();
	}
	
	public First(String name, String type, int number, String Name, int Damage) {
		this.name = name;
		this.type = type;
		this.number = number;
		this.pikachu = new Pikachu(Name, Damage);
	}
	
	public void attack() {
		System.out.println(this.name + "は攻撃した！");
		System.out.println("コイキングに" + this.pikachu.damage + "ポイントのダメージを与えた！");
	}
}

Main java
public class Main {
	public static void main(String[] args) {
		First p = new First();
		
		p.name = "ピカチュウ";
		p.type = "でんき";
		p.number = 25;
		p.pikachu = new Pikachu();
		p.pikachu.name = "10まんボルト";
		p.pikachu.damage = 20;
		
		p.attack();
		Waza w = new Waza("10まんボルト", 30);
		
		p.pikachu = w.kougeki();
		p.attack();
	}
}

Pikachu java

public class Pikachu {
	String name;
	int damage;
	
	public Pikachu() {
		this.name = "";
		this.damage = 0;
	}
	
	public Pikachu(String name, int damage) {
		this.name = name;
		this.damage = damage;
	}
}

Waza java

public class Waza {
	String name;
	int damage;
	
	public Waza() {
		this.name ="";
		this.damage =0;
	}
	
	public Waza(String name, int damage) {
		this.name = name;
		this.damage = damage;
	}
	
	public Pikachu kougeki() {
		System.out.println("ピカチュウの必殺技" +this.name + "を使った");
		Pikachu s = new Pikachu();
		s.name = "10まんボルト";
		s.damage =30;
		return s;
	}
}

Comments

[maisumakun]

> それを引数に入れるのをしてくださいと言われた

人間にですか？何かしらのツールが表示したエラーメッセージですか？

（後者の場合、エラーメッセージを「正確に」ご提示ください）

[m.ts10806]

前者であれば指示者に聞いてください。

[jimbe]

コードは出来るだけファイル毎にマークダウンで囲ってください。

[xebme]

this( ... )を使って、コンストラクターのコードを節約しろということですか。引数の数が最も多いコンストラクタのコードを残して、他のコンストラクタの内部でthis()を呼べ。という指示でしょうか。

Answer 1

First p = new First();
p.name = "ピカチュウ";
p.type = "でんき";
p.number = 25;
p.pikachu = new Pikachu();
p.pikachu.name = "10まんボルト";
p.pikachu.damage = 20;

この部分のコードを、 public First(String name, String type, int number) か public First(String name, String type, int number, String Name, int Damage) を使って書き直せということでしょう。
どちらも第一引数が name ですから、おそらく First p = new First("ピカチュウ", ...以下省略) のようになるでしょうね。