java instnaceof演算子の使い方

Question

q1とq2の二点間の距離の実行結果が異なっているため、クラスPoint2Dをinstnaceof演算子を用いて修正を行い、二点間の距離の実行結果を同じにしたいです。

ソースコード
public class Point {
protected int x =1;
Point(){}
Point(int x){this.x=x;}
double distance() {return Math.sqrt(xx);}
double distance(Point b) {
int dx;
dx = this.x-b.x;
return Math.sqrt(dxdx);

}

public String toString() {
	return String.format("(%d)", x);
}

}

public class Point2D extends Point {
protected int y = 1;
Point2D(){}
Point2D(int x,int y){super(x);this.y=y;}
double distance() {return Math.sqrt(xx+yy);}
double distance(Point2D b) {
int dx,dy;
dx=this.x-b.x;
dy=this.y-b.y;
return Math.sqrt(dxdx+dydy);
}

public String toString()
{return String.format("(%d,%d)", x,y);}

}

public class PointTester {

public static void main(String[] args) {
	Point2D q1=new Point2D(1,-1);
	Point q2 = new Point2D(0,1);
	System.out.println("二点間の距離は"+q1.distance(q2));
	System.out.println("二点間の距離は"+q1.distance((Point2D)q2));
}

}

Answer 1

実行結果が違う…というより、そもそも

System.out.println("二点間の距離は"+q1.distance(q2));

はコンパイルエラーになるのではないですか？

Answer 2

問題やクラスの定義について記述が足りないですが、
ここでは『Point2Dクラスは平面上の点、Pointクラスはx軸上の点を表す』と仮定します。

それならば、**Point2DがPointを継承しているのはおかしいです。**逆です。
前者の方がより汎化的な概念であり、特殊化されるべきは後者だからです。

---

クラスPoint2Dをinstnaceof演算子を用いて修正を行い、二点間の距離の実行結果を同じにしたいです。

なぜオーバーロードより先にinstanceof演算子の利用を検討するのでしょうか？
その考えに至った説明がないのでわかりません。

---

どこからオリジナルのコードなのかわかりませんが、このあたりを再整理してください。
上手く説明できないのならば、まだ問題への理解が足りないのです。

なお、Point2Dさえしっかり組んでしまえば、Point1Dの追加も簡単です。

class MyPoint1D extends MyPoint2D {
    MyPoint1D(int x) {
        super(x, 0);
    }
}

これで充分。

その他気になったこと

Math.sqrt(dx*dx) ⇒ Math.abs(dx)で充分。
Math.sqrt(dx*dx+dy*dy) ⇒ Math.hypot(dx, dy)で充分。

---

引数のないdistanceの意味合いが分かりづらいです。
原点Oとの距離という意味なら、次のように書いた方が分かりやすいんじゃないかなぁ。

static final MyPoint2D orizin = new MyPoint2D(0, 0);
double distance() {
    return this.distance(orizin);
}

コードの貼り方について

teratailには、上記のようにコードを見やすく表示する機能があります。
質問編集画面を開き、コードを選択した状態で<code>ボタンを押してください。

この機能が使われていない質問には回答しない方針の方もいらっしゃいます。