継承外の値を取得する方法（？）
解決済
回答 1
投稿
- 評価
- クリップ 0
- VIEW 598
前提・実現したいこと
継承外の値を取得（？）
（複数の）座標の値を表示する方法（？）
（問題点を自分自身で上手く言語化できていないため、下記を読んで下さい。）発生している問題・エラーメッセージ
・エラー
下記のソースコードから
"[円を描画] 中心点(Point@39d3030a)"と表示される
・目標
"[円を描画] 中心点(100, 100)"と表示させたい。該当のソースコード
public class Main{
public static void main(String[] args){
Figure figure = new Circle(100, 100, 20);
figure.draw();
}
}public interface Figure{
void draw();
}public class Point extends Object {
private int x; private int y;
public Point() {this.x = 0; this.y = 0;}
public Point(int x, int y){this.x = x; this.y = y;}
public int getX() {return this.x;}
public void setX(int x) {this.x = x;}
public int getY() {return this.y;}
public void setY(int y) {this.y = y;}
}public abstract class Shape implements Figure {
public abstract void draw();
}public class Circle extends Shape{
private Point center;
private int radius;
public Circle(int x, int y, int radius){
this.center = new Point(x, y);
this.radius = radius;
}
public void draw(){
System.out.println("[円を描画] 中心点(" + this.center);
}
}試したこと
Circle.draw()における、this.centerが違うということには見当がついています。
Point.getX()やPoint.getY()かな？、とも思いましたが、staticではない(=Class Fieldではない）ため、使用できませんでした。
補足質問
Figure figure = new Circle(100, 100, 20);
ではなく
Circle circle = new Circle(100, 100, 20);
の場合だと、エラーが発生したのですが、この理由もわからないため教えてください。
よろしくお願い致します。-
気になる質問をクリップする
クリップした質問は、後からいつでもマイページで確認できます。
またクリップした質問に回答があった際、通知やメールを受け取ることができます。
クリップを取り消します
-
良い質問の評価を上げる
以下のような質問は評価を上げましょう
- 質問内容が明確
- 自分も答えを知りたい
- 質問者以外のユーザにも役立つ
評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。
質問の評価を上げたことを取り消します
-
評価を下げられる数の上限に達しました
評価を下げることができません
- 1日5回まで評価を下げられます
- 1日に1ユーザに対して2回まで評価を下げられます
質問の評価を下げる
teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。
- プログラミングに関係のない質問
- やってほしいことだけを記載した丸投げの質問
- 問題・課題が含まれていない質問
- 意図的に内容が抹消された質問
- 過去に投稿した質問と同じ内容の質問
- 広告と受け取られるような投稿
評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。
質問の評価を下げたことを取り消します
この機能は開放されていません
評価を下げる条件を満たしてません
質問の評価を下げる機能の利用条件
この機能を利用するためには、以下の事項を行う必要があります。
- 質問回答など一定の行動
-
メールアドレスの認証
メールアドレスの認証
-
質問評価に関するヘルプページの閲覧
質問評価に関するヘルプページの閲覧
checkベストアンサー
0
単に出力したいだけなら、Object#toStringをオーバーライドすれば良いかと。
Pointクラスに次のメソッドを追加してみてください。
@Override
public String toString() {
return String.format("(%d, %d)", x, y);
}補足質問は再現できませんでした。
インスタンス名を変えたのにfigure.draw()のままにしていたとか、そういう凡ミスでは。
また、Shapeクラスの必要性もちょっと気になりました。
Circleクラスが直接Figureインターフェースを実装すれば良いのではないでしょうか。
その他気になるところをちょいちょい修正してみました。参考になれば幸いです。Wandbox
投稿
-
回答の評価を上げる
以下のような回答は評価を上げましょう
- 正しい回答
- わかりやすい回答
- ためになる回答
評価が高い回答ほどページの上位に表示されます。
-
回答の評価を下げる
下記のような回答は推奨されていません。
- 間違っている回答
- 質問の回答になっていない投稿
- スパムや攻撃的な表現を用いた投稿
評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。
15分調べてもわからないことは、teratailで質問しよう！
- ただいまの回答率 88.20%
- 質問をまとめることで、思考を整理して素早く解決
- テンプレート機能で、簡単に質問をまとめられる
2018/03/24 15:51
ごめんなさい、補足質問に関しては、もう一度やり直したらできました。
教材の練習問題（他にもPolygon, Triangle ,Circle, Square...classなども作成する）で、CircleクラスがShapeクラスを継承するように指定されていたため、それに従っています。
ただ、InterfaceからAbstract classにしただけなので、自分も必要性に疑問を感じていました。
2018/03/24 15:53
インターフェースFigureの定義も課題の要件の内なのでしょうか？
指定されているのならば、今後拡張するうえで必要になってくるのかもしれませんね。
2018/03/24 16:01
（本来、他のメソッドも書くように指示されていますが、質問のために省いています。）
2018/03/24 16:39
Circleクラスに似たクラス（Line, Triangle...）などを作成するにあたって、変数が複数(x1,y1, x2,y2やrなど）になり、Pointクラスのx, yだけでは対応できないことに気が付きました。
そのため、Pointクラスではなく、Circleクラス内で数値を表示する方法を教えていただけますか...？
2018/03/24 16:44
Pointクラスは『ただの点』のはずなので、x2やy2を持つことは好ましくありません。
2018/03/24 18:40
private Point p1; private Point p2;
public Line() {
this.p1 = new Point(0,0);
this.p2 = new Point(0,0);
}
public Line(int x1, int y1, int x2, int y2){
this.p1 = new Point(x1, y1);
this.p2 = new Point(x2, y2);
}
public void draw() {
System.out.println("[線を描画] 始点" + this.p1 + "から終点" + this.p2 + "まで");
質問以前から、上のソースコードを書いていましたが、利用している開発環境がきちんと機能しておらず、ソースコード自体が誤りだと勘違いしてしまいました。
他の環境で動作させたところ上記ソースコードで機能しました。。
ご迷惑をおかけしました。