JAVA, setter не получает значения, отправленные в мою конструкторскую программу, возвращающую нули

Question

У меня ошибка в том, что консоль печатает все нули в значениях точек, когда она должна показывать что-то похожее на (1,2) (3,4) (5,6) (7,8)

Я отследил ошибку до установщиков, где она не загружается в значение.Пример

 public void setPoint1(double newX, double newY) {
  this.N1x = newX;
  this.N1y = newY;

}

Это не обновляет закрытый двойной N1.x до соответствующего значения конструктора 1. Я считаю, что это проблема, но я новичок в кодировании и до сих пормне нужно многому научиться.

Насколько я понимаю, вызов сеттеров в конструкторе должен принимать значения от 1 до 8 из вызова конструктора, а затем отправлять их затем сеттеру, где они затем передаются вЗначение private удваивается, затем выполняется операция Point Class, и значения сохраняются в объектах point1 - point4, которые затем вызываются из основного метода, возвращая их сохраненные значения.

Ниже приведен код, касающийся моей проблемы.Заранее спасибо!

public class Quadrilateral {

   //Receives data from constructor 
   //initializes the variables for point objects  
   private double N1x; private double N1y;
   private double N2x; private double N2y;
   private double N3x; private double N3y;
   private double N4x; private double N4y;

   //creates new point objects
   Point point1 = new Point(N1x,N1y);
   Point point2 = new Point(N2x,N2y);
   Point point3 = new Point(N3x,N3y);
   Point point4 = new Point(N4x,N4y);

   // Encapsulation for point 1
   public Point getPoint1() {
      return point1;
   }
   public void setPoint1(double newX, double newY) {
      this.N1x = newX;
      this.N1y = newY;
   }

   // Encapsulation for point 2
   public Point getPoint2() {
      return point2;
   }
   public void setPoint2(double newX, double newY) {
      this.N2x = newX;
      this.N2y = newY;
   }

   // Encapsulation for point 3
   public Point getPoint3() {
      return point3;
   }
   public void setPoint3(double newX, double newY) {
      this.N3x = newX;
      this.N3y = newY;
   }

   // Encapsulation for point 4
   public Point getPoint4() {
      return point4;
   }
   public void setPoint4(double newX, double newY) {
      this.N4x = newX;
      this.N4y = newY;
   }

   //constructor for Quadrilateral
   //takes in 4 sets of point values
   Quadrilateral(
         double N1x, double N1y,
         double N2x, double N2y,
         double N3x, double N3y,
         double N4x, double N4y){

      setPoint1(N1x, N1y);
      setPoint2(N2x, N2y);
      setPoint3(N3x, N3y);
      setPoint4(N4x, N4y);

   }

   // gets the (x,y) values 
   //remember to override to add additional values and change shape

   public String toString() {
      return "Quadrilateral "+"\n"+ 
            "Node points are "+"\n"+
            getPoint1()+"\n"+
            getPoint2()+"\n"+
            getPoint3()+"\n"+
            getPoint4();
   }

public static void main(String[] args) {

   Quadrilateral quadrilateral = new Quadrilateral(1,2,3,4,5,6,7,8);
   System.out.println(quadrilateral.toString());
   }

}

Это точка класса

public class Point {
   private double x;
   private double y;

   public double getX() {
      return x;
   }
   public double getY() {
      return y;
   }
   public void setX(double newX) {
      this.x = newX;
   }
   public void setY(double newY) {
      this.y= newY;
   }
   Point(double x, double y){//when this constructor is called it performs encapsulation 
      setX(x);
      setY(y);
   }
   public String toString() {
      return "("+getX()+","+getY()+")";
   }
}

Answer 1

Если я правильно понимаю ваш вопрос, вы должны сделать это:

//creates new point objects
Point point1;
Point point2;
Point point3;
Point point4;

и это:

public void setPoint1(double newX, double newY) {
    this.N1x = newX;
    this.N1y = newY;
    point1 = new Point(newX, newY);
}

(То же самое и для других сеттеров)

В вашем коде вы только инициализировали точки, в то время как координаты равны нулю, прежде чем вы даже сначала установили координаты в ваших методах setPoint

Answer 2

Поскольку вы инициализируете свои объекты Point, переменные-члены четырехугольника еще до вызова конструктора с N1x, N2x и т. Д., Которые не инициализированы, вы устанавливаете их значения по умолчанию, равные 0 в объектах Point (переменных-членах).Измените четырехугольное определение класса следующим образом

public class Quadrilateral {

    // creates new point objects
    Point point1 ;
    Point point2;
    Point point3 ;
    Point point4 ;

    // Encapsulation for point 1
    public Point getPoint1() {
        return point1;
    }

    public void setPoint1(double newX, double newY) {
        this.point1 = new Point(newX, newY);        
    }

    // Encapsulation for point 2
    public Point getPoint2() {
        return point2;
    }

    public void setPoint2(double newX, double newY) {
        this.point2 = new Point(newX, newY);
    }

    // Encapsulation for point 3
    public Point getPoint3() {
        return point3;
    }

    public void setPoint3(double newX, double newY) {
        this.point3 = new Point(newX, newY);
    }

    // Encapsulation for point 4
    public Point getPoint4() {
        return point4;
    }

    public void setPoint4(double newX, double newY) {
        this.point4 = new Point(newX, newY);
    }

    // constructor for Quadrilateral
    // takes in 4 sets of point values
    Quadrilateral(double N1x, double N1y, double N2x, double N2y, double N3x, double N3y, double N4x, double N4y) {

        setPoint1(N1x, N1y);
        setPoint2(N2x, N2y);
        setPoint3(N3x, N3y);
        setPoint4(N4x, N4y);

    }

    // gets the (x,y) values
    // remember to override to add additional values and change shape

    public String toString() {
        return "Quadrilateral " + "\n" + "Node points are " + "\n" + getPoint1() + "\n" + getPoint2() + "\n"
                + getPoint3() + "\n" + getPoint4();
    }

    public static void main(String[] args) {

        Quadrilateral quadrilateral = new Quadrilateral(1, 2, 3, 4, 5, 6, 7, 8);
        System.out.println(quadrilateral.toString());
    }

}