The error I am having is that the console is printing out all zeros in the point values when it should be showing something similar to (1,2) (3,4) (5,6) (7,8)
I tracked down the error to the setters, where it does not load in the value. Example
public void setPoint1(double newX, double newY) {
this.N1x = newX;
this.N1y = newY;
}
This does not update the private double N1.x to the corresponding constructor value of 1. I believe this to be the problem but I am new to coding and still have a lot to learn.
As I understand it, the call to the setters in the constructor should take the 1 through 8 values from the constructor call and then send then to the setter where they are then distributed to the private doubles then the Point Class operation executes and the values are saved into the point1 through point4 objects which is then called from the main method returning their stored values.
Below is the code I have regarding my problem. Thank you in advance!
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());
}
}
This is the point class
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()+")";
}
}
If I understand your question right you should do this:
//creates new point objects
Point point1;
Point point2;
Point point3;
Point point4;
and this:
public void setPoint1(double newX, double newY) {
this.N1x = newX;
this.N1y = newY;
point1 = new Point(newX, newY);
}
(Same thing for the other point setters too)
In your code you only initialized the points while the coordinates are null, before you even first set the coordinates in your setPoint methods