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

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

新規登録して質問してみよう
ただいま回答率
85.50%
Java

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

Q&A

解決済

3回答

2892閲覧

親クラスの変数が参照されてしまう。

TA-KEY

総合スコア21

Java

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

0グッド

0クリップ

投稿2018/02/05 01:28

編集2018/02/05 06:58

今、javaの練習でパワプロ君のサクセスのようなものを作っています。
Playerクラス(選手全員の親クラス)を継承したHeroクラス(主人公を生み出す)の初期能力値についてです。
Heroクラスがインスタンス化されるときに、乱数で初期値が決まるようにしました。しかし、Mainのクラスで主人公の能力値を見るために、Playerクラスから継承したgetParameterメソッドを使うと、Playerクラスでとりあえず設定された初期値が戻り値として返されてしまいます。どうしたらいいでしょうか。

HeroクラスでgetParameterメソッドをオーバーライドすることも考えましたが、Playerクラスでのメソッドをそのまま継承できた方が今後楽そうです。よろしくお願いします。

[追記]
Playerを継承したクラスのインスタンスはそれぞれの能力値を持っていて、Heroクラスだけはランダムで初期値を。他の子クラスは定数で初期値を決めたいです。

java

1 2public abstract class Player implements Character{ 3 int HP = 0; 4 int condition = 0; //調子。0-4の5段階 5 int meet = 0; //ミート 6 int power = 0; //パワー 7 int speed = 0; //走力 8 int shoulder = 0; //肩力 9 int defense = 0; //守備 10 int ballSpeed = 0; //球速 11 int control = 0; //コントロール 12 int[] breakingBall = {0, 0, 0, 0, 0}; //変化球1-5の5段階でスライダー、カーブ、フォーク、シンカー、シュート 13 14 abstract void practice(int numberOfPra); 15 16/*能力値を取得する 17*/ 18 String getParameter() { 19 String conditionEva = ""; 20 21 switch(condition){ 22 case 0: conditionEva = "絶不調"; 23 break; 24 case 1: conditionEva = "悪い"; 25 break; 26 case 2: conditionEva = "普通"; 27 break; 28 case 3: conditionEva = "良い"; 29 break; 30 case 4: conditionEva = "絶好調"; 31 break; 32 } 33 34 String ability = "調子:" + conditionEva + ", ミート:" + judge(meet) + ", パワー:" + judge(power) 35 + ", 走力:" + judge(speed) + ", 肩力:" + judge(shoulder) + ", 守備力:" + judge(defense) + ", 球速:" + ballSpeed + "km, コントロール" + judge(control) 36 + ", ←:" + breakingBall[0] + ", ↙:" + breakingBall[1] + ", ↓:" + breakingBall[2] 37 + ", ↘:" + breakingBall[3] + ", ︎→:" + breakingBall[4]; 38 return ability; 39 } 40 41/*能力値をSABCDEFGで判断する 42*/ 43 String judge(int sample){ 44 if (sample < 10){ 45 return "G";//0-9 46 } 47 else if(sample < 20){ 48 return "F";//10-19 49 } 50 else if(sample < 30){ 51 return "E";//20-29 52 } 53 else if(sample < 40){ 54 return "D";//30-39 55 } 56 else if(sample < 50){ 57 return "C";//40-49 58 } 59 else if(sample < 75){ 60 return "B";//51-74 61 } 62 else if(sample < 100){ 63 return "A";//75-99 64 } 65 else if(sample == 100){ 66 return "S"; 67 } 68 return "-"; 69 } 70 71 /*以下、全てゲッターとセッターだったので省略 72 */ 73 74}

java

1public class Hero extends Player { 2 private static String name; 3 private int HP = 0;//体力 4 private int condition = 2; //調子。0-4の5段階 5 private int meet = 0; //ミート 6 private int power = 0; //パワー 7 private int speed = 0; //走力 8 private int shoulder = 0; //肩力 9 private int defense = 0; //守備 10 private int ballSpeed = 100; //球速 11 private int control = 0; //コントロール 12 private static int[] breakingBall = {0, 0, 0, 0, 0}; //変化球0-4でスライダー、カーブ、フォーク、シンカー、シュート 13 private int sliderCounter = 0; 14 private int curveCounter = 0; 15 private int forkCounter = 0; 16 private int sinkerCounter = 0; 17 private int hardSinkerCounter = 0; 18 19 public Hero(String name){ 20 this.name = name; 21 this.HP = 100; 22 23 //乱数で能力の初期値を決める。 24 Random random = new Random(); 25 condition = 2; 26 meet = random.nextInt(31); 27 power = random.nextInt(31); 28 speed = random.nextInt(21) + 90; 29 shoulder = random.nextInt(31); 30 defense = random.nextInt(31); 31 ballSpeed = random.nextInt(31); 32 control = random.nextInt(31); 33 breakingBall = new int[]{random.nextInt(2), random.nextInt(2), random.nextInt(2), random.nextInt(2), random.nextInt(2)}; //スライダー、カーブ、フォーク、シンカー、シュート 34 } 35 36//休むメソッドです。質問には関係ないので省略 37 public void rest(Hero h){ 38 39 } 40 41//練習メソッド。質問には関係ないので省略 42 public void practice(int numberOfPra){ 43 } 44 45 //HPが25以下で約50%で怪我。変化球以外の能力値が減少。質問には関係ないので省略 46 public void judgInjury() { 47 } 48 49//以下全てゲッターセッターなので省略 50 51 52} 53

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答3

0

ベストアンサー

子クラスから親クラスのメソッドを呼ぶと親クラスのフィールドを参照します。

getParameterを継承しないのであれば、Heroクラスの「name・HP・condition・meet・power・speed・shoulder・defense・ballSpeed・control・breakingBall」を削除し、コンストラクタで親クラスの該当フィールドにランダム値を与えるようにしましょう。

投稿2018/02/05 01:40

YamakawaJunichi

総合スコア630

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

TA-KEY

2018/02/05 02:09

なるほど。継承しなくてもそのような方法があるんですね。ありがとうございます。
guest

0

まず2つあります。

  • そもそもフィールドはオーバーライドされない。
  • 親クラス(Player)から子クラス(Hero)のフィールドを参照することはできない

PlayerクラスのgetParameterはHeroクラスの値を見ることはないので、初期値のままとなります。

追記
「継承されない」は、間違いでは?については、その通りです。
同名のフィールドを子クラスで定義した場合、親クラス、子クラスでそれぞれ同名のフィールドを持っています。
ですので、子クラス側で親クラスのフィールドに値を設定したつもりでも、子クラス側に設定されます。
継承自体されますが、他のクラスから参照できるかは、可視性によります。

対応としては、いくつかありますが。

前提としては、子クラス側のフィールドは削除します。

  1. YamakawaJunichiさんの言うとおり、子クラス側から親クラスの値を直接変更する
  2. 乱数設定の部分をPlayerのコンストラクタに実装する
  3. Playerに設定するパラメータを受け取るコンストラクタを用意する

が、考えられます。

子クラスで実装する場合、Playerクラスを継承した子クラスを作るごとに初期化処理を実装するので面倒です。
とはいえ、子クラスで独自の初期値を設定したいことがある場合、ランダムでは困ることがあるかもしれません。

子クラスから呼べるセッターを用意する場合、上記1の方法とほぼ同じです。
セッターで値を変更できるようにしたくないなら、2と3両方コンストラクタを用意すれば、
通常時はデフォルトコンストラクタで、ランダムに値を設定する。
特定の値を設定したい場合は、引数つきのコンストラクタを呼べばよいです。

投稿2018/02/05 02:22

編集2018/02/05 02:50
momon-ga

総合スコア4820

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

swordone

2018/02/05 02:27

「フィールドは継承されない」というのは違うのでは?
momon-ga

2018/02/05 02:34

なるほど、表現ミスりました。あとで修正します。
YamakawaJunichi

2018/02/05 02:38

2.は正しくないです。 子クラスで親クラスのフィールドと同じものを宣言するとオーバーライドされず隠蔽されるので、PlayerクラスとHeroクラスで別々の乱数値を持ち、getParameterするとPlayerクラスの乱数値を取得することになってしまいます。
momon-ga

2018/02/05 02:42 編集

追記します。削除が前提でしたね。 親クラスで設定するので子クラス側のフィールド作る意味ないっていう前提が抜けてました。
TA-KEY

2018/02/05 06:31

1,2のやり方だと、Playerクラスを継承したクラスは全て同じ能力値を共有してしまいますよね? それぞれ固有のフィールドを持たせたいのです。 申し訳ないですが、3はちょっと理解できてないです。
momon-ga

2018/02/05 06:47

能力値は共有でなくインスタンスごとに異なります(ランダム) インスタンス変数とクラス変数、クラス毎のフィールドの追加の話と、何を解決したいのか、わからないので、元の質問に追記してほしいです。
TA-KEY

2018/02/05 06:59

なるほど。追記しました。これで大丈夫でしょうか、、、
TA-KEY

2018/02/05 07:58

もしかして、Heroクラスのフィールドを消せば解決でしょうか?
swordone

2018/02/05 08:08

YamakawaJunichiさんの回答そうですよね
TA-KEY

2018/02/05 09:47

なるほど。ありがとうございました!
guest

0

この回答は間違っていました。


Player.getParameterの宣言がint HP = 0;String getParameter()となっているために、これはpackage privateとなっていて、Heroには継承されません。

protected、あるいはpublicにしてきちんと継承させましょう。

投稿2018/02/05 01:34

編集2018/02/05 02:16
maisumakun

総合スコア145121

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

swordone

2018/02/05 01:49

問題の原因と関係ないような
maisumakun

2018/02/05 01:57 編集

いえ、HeroクラスにgetParameterがない(親クラスからは継承されてこない)以上、Mainから呼んでいるのはpackage privateな親クラスのメソッドで、参照先も親クラスにしかないフィールド、ということになると判断したのですが、何か違いますでしょうか。
TA-KEY

2018/02/05 02:08

継承できてないんですか??オーバーライドしない限り全て継承されると思ってました。ちょっと勉強してきます。
swordone

2018/02/05 02:14

パッケージプライベートであることは全く関係ないかと。
maisumakun

2018/02/05 02:16

すみません、いま確認したら「同じパッケージ内で継承を行った場合、package privateなものも継承される」とのことでした。ありがとうございます。 ただ、パッケージ外に出さないクラスは別として、継承されるものはprotected/publicをきちんと管理して置いたほうがいいのは間違いないです。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問