So I want to calculate the number of points within any given triangle. I know I have to use Pick's Theorem, but my code ends up being ridiculously long with the amount of if-else if statements to test for every case. I've been using this as a guide How many integer points within the three points forming a triangle?, but then end up with this (vertices is an array of 3 arrays. Each array is the (x, y) coordinates of a vertex of the triangle):
int maxX = Math.max(Math.max(vertices[0][0], vertices[1][0]), vertices[2][0]),
minX = Math.min(Math.min(vertices[0][0], vertices[1][0]), vertices[2][0]),
maxY = Math.max(Math.max(vertices[0][1], vertices[1][1]), vertices[2][1]),
minY = Math.min(Math.min(vertices[0][1], vertices[1][1]), vertices[2][1]);
int height = Math.abs(maxY - minY),
width = Math.abs(maxX - minX);
double area = Math.abs(((vertices[0][0] * (vertices[1][1] - vertices[2][1]
)) + (vertices[1][0] * (vertices[2][1] - vertices[0][1]))
+ vertices[2][0] * (vertices[0][1] - vertices[1][1])) / 2);
if ((vertices[0][0] == vertices[1][0]) && (vertices[0][1] == vertices[2][1]))
{
area = ((Math.abs(vertices[0][1] - vertices[1][1]) - 1) *
(Math.abs(vertices[0][0] - vertices[2][0]) - 1)) / 2;
}
else if ((vertices[0][0] == vertices[2][0]) && (vertices[0][1] == vertices[1][1]))
{
area = ((Math.abs(vertices[0][1] - vertices[2][1]) - 1) *
(Math.abs(vertices[0][0] - vertices[1][0]) - 1)) / 2;
}
else if ((vertices[1][0] == vertices[2][0]) && (vertices[1][1] == vertices[0][1]))
{
area = ((Math.abs(vertices[1][1] - vertices[2][1]) - 1) *
(Math.abs(vertices[1][0] - vertices[0][0]) - 1)) / 2;
}
else if ((vertices[1][0] == vertices[2][0]) && (vertices[1][1] == vertices[0][1]))
{
area = ((Math.abs(vertices[1][1] - vertices[2][1]) - 1) *
(Math.abs(vertices[1][0] - vertices[0][0]) - 1)) / 2;
}
else if(vertices[0][0] == vertices[1][0])
{
int b = Math.abs(vertices[0][1] - vertices[1][1]);
/*double dist1 = Math.sqrt(Math.pow(vertices[0][0] - vertices[2][0], 2) +
Math.pow(vertices[0][1] - vertices[2][1], 2)),
dist2 = Math.sqrt(Math.pow(vertices[1][0] - vertices[2][0], 2) +
Math.pow(vertices[1][1] - vertices[2][1], 2));*/
if (vertices[0][1] > vertices[1][1])
{
area = (width - 1) * (height - 1) /*- dist1 - dist2*/ - (((width - 1) *
(height - 1) /*- dist1*/) / 2) - (((width - 1) * (height - b - 1)
/*- dist2*/) / 2);
}
else if (vertices[0][1] < vertices[1][1])
{
area = (width - 1) * (height - 1) /*- dist1 - dist2*/ - (((width - 1) *
(height - 1) /*- dist2*/) / 2) - (((width - 1) * (height - b - 1)
/*- dist1*/) / 2);
}
}
else if(vertices[0][0] == vertices[2][0])
{
int b = Math.abs(vertices[0][1] - vertices[2][1]);
/*double dist1 = Math.sqrt(Math.pow(vertices[0][0] - vertices[1][0], 2) +
Math.pow(vertices[0][1] - vertices[1][1], 2)),
dist2 = Math.sqrt(Math.pow(vertices[2][0] - vertices[1][0], 2) +
Math.pow(vertices[2][1] - vertices[1][1], 2));*/
if (vertices[0][1] > vertices[2][1])
{
area = (width - 1) * (height - 1) /*- dist1 - dist2*/ - (((width - 1) *
(height - 1) /*- dist1*/) / 2) - (((width - 1) * (height - b - 1)
/*- dist2*/) / 2);
}
else if (vertices[0][1] < vertices[2][1])
{
area = (width - 1) * (height - 1) /*- dist1 - dist2*/ - (((width - 1) *
(height - 1) /*- dist2*/) / 2) - (((width - 1) * (height - b - 1)
/*- dist1*/) / 2);
}
}
else if(vertices[1][0] == vertices[2][0])
{
int b = Math.abs(vertices[1][1] - vertices[2][1]);
/*double dist1 = Math.sqrt(Math.pow(vertices[1][0] - vertices[0][0], 2) +
Math.pow(vertices[1][1] - vertices[0][1], 2)),
dist2 = Math.sqrt(Math.pow(vertices[2][0] - vertices[0][0], 2) +
Math.pow(vertices[2][1] - vertices[0][1], 2));*/
if (vertices[1][1] > vertices[2][1])
{
area = (width - 1) * (height - 1) /*- dist1 - dist2*/ - (((width - 1) *
(height - 1) /*- dist1*/) / 2) - (((width - 1) * (height - b - 1)
/*- dist2*/) / 2);
}
else if (vertices[1][1] < vertices[2][1])
{
area = (width - 1) * (height - 1) /*- dist1 - dist2*/ - (((width - 1) *
(height - 1) /*- dist2*/) / 2) - (((width - 1) * (height - b - 1)
/*- dist1*/) / 2);
}
}
else if(vertices[0][1] == vertices[1][1])
{
int b = Math.abs(vertices[0][0] - vertices[1][0]);
/*double dist1 = Math.sqrt(Math.pow(vertices[0][1] - vertices[2][0], 2) +
Math.pow(vertices[0][1] - vertices[2][1], 2)),
dist2 = Math.sqrt(Math.pow(vertices[1][0] - vertices[2][0], 2) +
Math.pow(vertices[1][1] - vertices[2][1], 2));*/
if (vertices[0][0] > vertices[1][0])
{
area = (width - 1) * (height - 1) /*- dist1 - dist2*/ - (((width - 1) *
(height - 1) /*- dist1*/) / 2) - (((width - 1) * (height - b - 1)
/*- dist2*/) / 2);
}
else if (vertices[0][0] < vertices[1][0])
{
area = (width - 1) * (height - 1) /*- dist1 - dist2*/ - (((width - 1) *
(height - 1) /*- dist2*/) / 2) - (((width - 1) * (height - b - 1)
/*- dist1*/) / 2);
}
}
else if(vertices[0][1] == vertices[2][1])
{
int b = Math.abs(vertices[0][0] - vertices[2][0]);
/*double dist1 = Math.sqrt(Math.pow(vertices[0][1] - vertices[1][0], 2) +
Math.pow(vertices[0][1] - vertices[1][1], 2)),
dist2 = Math.sqrt(Math.pow(vertices[2][0] - vertices[1][0], 2) +
Math.pow(vertices[2][1] - vertices[1][1], 2));*/
if (vertices[0][0] > vertices[2][0])
{
area = (width - 1) * (height - 1) /*- dist1 - dist2*/ - (((width - 1) *
(height - 1) /*- dist1*/) / 2) - (((width - 1) * (height - b - 1)
/*- dist2*/) / 2);
}
else if (vertices[0][0] < vertices[2][0])
{
area = (width - 1) * (height - 1) /*- dist1 - dist2*/ - (((width - 1) *
(height - 1) /*- dist2*/) / 2) - (((width - 1) * (height - b - 1)
/*- dist1*/) / 2);
}
}
else if(vertices[1][1] == vertices[2][1])
{
int b = Math.abs(vertices[1][0] - vertices[2][0]);
/*double dist1 = Math.sqrt(Math.pow(vertices[1][1] - vertices[0][0], 2) +
Math.pow(vertices[1][1] - vertices[0][1], 2)),
dist2 = Math.sqrt(Math.pow(vertices[2][0] - vertices[0][0], 2) +
Math.pow(vertices[2][1] - vertices[0][1], 2));*/
if (vertices[1][0] > vertices[2][0])
{
area = (width - 1) * (height - 1) /*- dist1 - dist2*/ - (((width - 1) *
(height - 1) /*- dist1*/) / 2) - (((width - 1) * (height - b - 1)
/*- dist2*/) / 2);
}
else if (vertices[1][0] < vertices[2][0])
{
area = (width - 1) * (height - 1) /*- dist1 - dist2*/ - (((width - 1) *
(height - 1) /*- dist2*/) / 2) - (((width - 1) * (height - b - 1)
/*- dist1*/) / 2);
}
}
else if (minX == vertices[0][0])
{
int a = 0,
b = 0;
/*double dist1 = 0,
dist2 = 0,
dist3 = 0;*/
if(Math.min(vertices[1][0], vertices[2][0]) == vertices[1][0])
{
a = width - (vertices[1][0] - vertices[0][0]);
b = height - (vertices [1][1] - vertices[0][1]);
/*dist1 = Math.sqrt(Math.pow(vertices[0][0] - vertices[1][0], 2) +
Math.pow(vertices[0][1] - vertices[1][1], 2));
dist2 = Math.sqrt(Math.pow(vertices[1][0] - vertices[2][0], 2) +
Math.pow(vertices[1][1] - vertices[2][1], 2));
dist3 = Math.sqrt(Math.pow(vertices[0][0] - vertices[2][0], 2) +
Math.pow(vertices[0][1] - vertices[2][1], 2));*/
}
else if(Math.min(vertices[1][0], vertices[2][0]) == vertices[2][0])
{
a = width - (vertices[2][0] - vertices[0][0]);
b = height - (vertices [2][1] - vertices[0][1]);
/*dist1 = Math.sqrt(Math.pow(vertices[0][0] - vertices[2][0], 2) +
Math.pow(vertices[0][1] - vertices[2][1], 2));
dist2 = Math.sqrt(Math.pow(vertices[1][0] - vertices[2][0], 2) +
Math.pow(vertices[1][1] - vertices[2][1], 2));
dist3 = Math.sqrt(Math.pow(vertices[0][0] - vertices[1][0], 2) +
Math.pow(vertices[0][1] - vertices[1][1], 2));*/
}
area = (width - 1) * (height - 1) /*- dist1 - dist2 - dist3*/ - (((a - 1)
* (b - 1) /*- dist1*/) / 2) - (((width - a - 1) * (height - 1)
/*- dist2*/) / 2) - (((width - 1) * (height - b - 1) /*- dist3*/) / 2);
}
else if (minX == vertices[1][0])
{
int a = 0,
b = 0;
/*double dist1 = 0,
dist2 = 0,
dist3 = 0;*/
if(Math.min(vertices[0][0], vertices[2][0]) == vertices[0][0])
{
a = width - (vertices[0][0] - vertices[1][0]);
b = height - (vertices [0][1] - vertices[1][1]);
/*dist1 = Math.sqrt(Math.pow(vertices[1][0] - vertices[0][0], 2) +
Math.pow(vertices[1][1] - vertices[0][1], 2));
dist2 = Math.sqrt(Math.pow(vertices[0][0] - vertices[2][0], 2) +
Math.pow(vertices[0][1] - vertices[2][1], 2));
dist3 = Math.sqrt(Math.pow(vertices[1][0] - vertices[2][0], 2) +
Math.pow(vertices[1][1] - vertices[2][1], 2));*/
}
else if(Math.min(vertices[0][0], vertices[2][0]) == vertices[2][0])
{
a = width - (vertices[2][0] - vertices[1][0]);
b = height - (vertices [2][1] - vertices[1][1]);
/*dist1 = Math.sqrt(Math.pow(vertices[1][0] - vertices[2][0], 2) +
Math.pow(vertices[1][1] - vertices[2][1], 2));
dist2 = Math.sqrt(Math.pow(vertices[0][0] - vertices[2][0], 2) +
Math.pow(vertices[0][1] - vertices[2][1], 2));
dist3 = Math.sqrt(Math.pow(vertices[0][0] - vertices[1][0], 2) +
Math.pow(vertices[0][1] - vertices[1][1], 2));*/
}
area = (width - 1) * (height - 1) /*- dist1 - dist2 - dist3*/ - (((a - 1)
* (b - 1) /*- dist1*/) / 2) - (((width - a - 1) * (height - 1)
/*- dist2*/) / 2) - (((width - 1) * (height - b - 1) /*- dist3*/) / 2);
}
else if (minX == vertices[2][0])
{
int a = 0,
b = 0;
/*double dist1 = 0,
dist2 = 0,
dist3 = 0;*/
if(Math.min(vertices[0][0], vertices[1][0]) == vertices[0][0])
{
a = width - (vertices[0][0] - vertices[2][0]);
b = height - (vertices [0][1] - vertices[2][1]);
/*dist1 = Math.sqrt(Math.pow(vertices[2][0] - vertices[0][0], 2) +
Math.pow(vertices[2][1] - vertices[0][1], 2));
dist2 = Math.sqrt(Math.pow(vertices[0][0] - vertices[1][0], 2) +
Math.pow(vertices[0][1] - vertices[1][1], 2));
dist3 = Math.sqrt(Math.pow(vertices[1][0] - vertices[2][0], 2) +
Math.pow(vertices[1][1] - vertices[2][1], 2));*/
}
else if(Math.min(vertices[0][0], vertices[1][0]) == vertices[1][0])
{
a = width - (vertices[1][0] - vertices[2][0]);
b = height - (vertices [1][1] - vertices[2][1]);
/*dist1 = Math.sqrt(Math.pow(vertices[2][0] - vertices[1][0], 2) +
Math.pow(vertices[2][1] - vertices[1][1], 2));
dist2 = Math.sqrt(Math.pow(vertices[0][0] - vertices[2][0], 2) +
Math.pow(vertices[0][1] - vertices[2][1], 2));
dist3 = Math.sqrt(Math.pow(vertices[0][0] - vertices[2][0], 2) +
Math.pow(vertices[0][1] - vertices[2][1], 2));*/
}
area = (width - 1) * (height - 1) /*- dist1 - dist2 - dist3*/ - (((a - 1)
* (b - 1) /*- dist1*/) / 2) - (((width - a - 1) * (height - 1)
/*- dist2*/) / 2) - (((width - 1) * (height - b - 1) /*- dist3*/) / 2);
}
Could someone help me either fix the algorithm or give me an easier/better way to do this? This code pretty much never works.
Sorry about the long code, I didn't know what parts I should add so I put the whole algorithm.
Edit: So I changed the algorithm to this (Thanks to MBo):
public static int answer(int[][] vertices)
{
int a = (Math.abs((vertices[1][0] - vertices[0][0]) * (vertices[2][1]
- vertices[0][1]) - (vertices[2][0] - vertices[0][0]) *
(vertices[1][1] - vertices[0][1]))) / 2,
b = pointsOnLine(vertices[0][0], vertices[0][1], vertices[1][0],
vertices[1][1]) + pointsOnLine(vertices[1][0],
vertices[1][1], vertices[2][0], vertices[2][1]) +
pointsOnLine(vertices[0][0], vertices[0][1],
vertices[2][0], vertices[2][1]),
area = (2 * a - 2 - b) / 2; // Also tried a + (b / 2) - 1;
return (int)area;
}
public static int pointsOnLine(int x0, int y0, int x1, int y1)
{
BigInteger b1 = BigInteger.valueOf(Math.abs(x1 - x0)),
b2 = BigInteger.valueOf(Math.abs(y1 - y0));
return b1.gcd(b2).intValue();
}
But, I don't always get the right answer. Did I do something wrong?