線分の幅を形成する

Question

以下の問題のプログラムを書きたいのですが、どうも頭がこんがらがってわかりません。

public class LineSegment{
    private double x, y;
    private double x1, y1;

    public LineSegment(double x, double y){
	this.x = x;
	this.y = y;
    }
    public void linearTransfer(){
	this.x1 = 6 * this.x + 4 * this.y;
	this.y1 = -2 * this.x + this.y;
    }
    public final double getX1(){
	return x1;
    }
    public final double getY1(){
	return y1;
    }   
}
 class ThickLineSegment extends LineSegment{
   ThickLineSegment(double x1, double y1,double x2,double y2){
	super(x1,y1);
	super(x2,y2);
    }
   ThickLineSegment(double x, double y,int w){
	super(x,y);
	this.setWidth(w);
    }
   ThickLineSegment(double x1, double y1,double x2,double y2,int w){
	super(x1,y1);
	super(x2,y2);
	this.setWidth(w);
    }
   void  setWidth(int w){
	int awidth=w;
    }

}

ここまで書いたんですが、どう改良したらうまく動くのか。。。

Answer 1

まず、

ThickLineSegment(double x1, double y1,double x2,double y2){
    super(x1,y1);
    super(x2,y2);
}

super()のコンストラクタ呼び出しはコンストラクタの最初にしか書けないので、このように2行続けて書くことはできません。
要件の一つにあるwidthも見当たりません。makeForm,toStringの実装もめちゃくちゃです。
しかし腑に落ちないのが、これではThickLineSegmentクラスからどうやってもLineSegmentクラスの「もう一方の端点」の情報と思われるx1,y1に値を設定することができないということ。LineSegmentクラスもいじれるんですかね？

---

LineSegmentもいじれるとのことなので改良策を。
そもそも「線分」というからには始点・終点を設定するのが本来の姿のはずなので、始点・終点を設定するコンストラクタをLineSegmentに実装させる。

public class LineSegment{
    private double x, y;  //始点
    private double x1, y1;  //終点

    public LineSegment(double x, double y){
        this(x, y, 0.0, 0.0);
    }

    public LineSegment(double x, double y, double x1, double y1){
        this.x = x;
        this.y = y;
        this.x1 = x1;
        this.y1 = y1;
    }
}

で、これを継承したThickLineSegmentでは幅を設定すればいいだけにする。

class ThickLineSegment extends LineSegment{

   int width;

   ThickLineSegment(double x1, double y1, double x2, double y2){
       this(x1, y1, x2, y2, 1);
   }
   ThickLineSegment(double x, double y, int w){
       this(x, y, 0.0, 0.0, w);
   }

   ThickLineSegment(double x1, double y1, double x2, double y2,int w){
       super(x1, y1, x2, y2);
       setWidth(w);
   }

   public void setWidth(int w){
       width = w;
   }
}

makeFormの要件は、ご自身のコードの条件の部分をよく見てください。
toStringの要件がよくわからないのですが…

Comments

[indehi]

そうなんですね。確かにこのままじゃダメですよね。
もともとLineSegmentも自分で書いたものなので変えても大丈夫です。