在Java中合并凸壳

4
我已经写了一个Java程序,使用分治算法在笛卡尔坐标系中查找多边形的凸包。我有一个名为Coord的类,它有两个“double”字段X和Y,而我正在使用该方法的“this”是坐标(集合)的集合。我的方法应该返回多边形的壳(集合)。
我的代码如下:
    public Collection<Coord> territoire()
    {
        Collection<Coord> sommets = new ArrayList<Coord>();
        ArrayList<Coord> thisToArrList = new ArrayList<Coord>();
        for(Coord c : this)
            thisToArrList.add(c);

        ArrayList<Coord> sortedPointsByX = new ArrayList<Coord>(); 


        int n = this.size();
        if (n <= 2)
            return this;

        //sorting the points by their X coordinates 
        sortedPointsByX = sortedArrayByX(thisToArrList);

        //>>>>>>>>>>>>>>>>>> works good till here <<<<<<<<<<<<<<<<<<<<<<<<

        // now sortedPointsByX array contains the points with increasing X 
        // splitting the sortedPointsByX into two arrays  
        ArrayList<Coord> firstPart = new ArrayList<Coord>();
        ArrayList<Coord> secondPart = new ArrayList<Coord>();

        // if the number of the points is prime, the leftmost and the rightmost half
        // both have same number of points
        if(sortedPointsByX.size() % 2 == 0)
        {   

            for(int i = 0; i < sortedPointsByX.size()/2; i++)
            {
                firstPart.add(sortedPointsByX.get(i));

            }

            for(int i = sortedPointsByX.size()/2; i < sortedPointsByX.size(); i++)
            {
                secondPart.add(sortedPointsByX.get(i));
            }

        }
        //>>>>>>>>>>>>>>>>>>>>>works good till here<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<


        // if the number of points is odd, the leftmost half have the extra points 
        else 
        {
            for(int i = 0; i < sortedPointsByX.size()/2+1; i++)
            {
                firstPart.add(sortedPointsByX.get(i));
            }

            for(int i = sortedPointsByX.size()/2+1; i < sortedPointsByX.size(); i++)
            {
                secondPart.add(sortedPointsByX.get(i));
            }
        }

        //>>>>>>>>>>>>>>>>>>>>>>>works good till here<<<<<<<<<<<<<<<<<<<<<<<<<

        CoordSet firstSet = new CoordSet(firstPart);
        CoordSet secondSet = new CoordSet(secondPart);


        // converting the arrays to list of coordinates in order to use recursion over them 


        //recursion for sub coordsets
        Collection<Coord> firstSetSommet = firstSet.territoire();
        Collection<Coord> secondSetSommet = secondSet.territoire();

        ArrayList<Coord> firstHull = new ArrayList<Coord>(firstSetSommet);
        ArrayList<Coord> secondHull = new ArrayList<Coord>(secondSetSommet);

        sommets = mergeHulls(firstHull, secondHull);


        return sommets;
    }


    public Collection<Coord> mergeHulls(ArrayList<Coord> firstHull, ArrayList<Coord> secondHull) 
    {

        Collection<Coord> pointsInside = new ArrayList<Coord>();
        Collection<Coord> sommets = new ArrayList<Coord>();

      //********************upper tangent***************       

        //find the highest point of the leftmost part
        Coord firstPartHighestPoint = getMaxY(firstHull);
        //find the highest point of the rightmost part 
        Coord secondPartHighestPoint = getMaxY(secondHull);

        for(int i = 0; i< firstHull.size(); i++)
        {
            // check if the points lie on the line between highest point in leftmost and in rightmost
            // if true, the current point is above the line
            if(isCollinear(firstPartHighestPoint, secondPartHighestPoint, firstHull.get(i))>0)
            {
                // the current point is above the line
                firstPartHighestPoint = firstHull.get(i);
            }
            pointsInside.add(firstPartHighestPoint);
        }

        for(int i = 0; i < secondHull.size(); i++)
        {
            if(isCollinear(firstPartHighestPoint, secondPartHighestPoint, secondHull.get(i))>0)
            {
                // the current point is above the line
                secondPartHighestPoint = secondHull.get(i);
            }
            pointsInside.add(secondPartHighestPoint);

        }

        //******************lower tangent***************     

        //find the lowest point of the leftmost part 
        Coord firstPartLowestPoint = getMinY(firstHull);
        // find the lowest point of the rightmost part
        Coord secondPartLowestPoint = getMinY(secondHull);

        for(int i = 0; i< firstHull.size(); i++)
        {
            // check if the points lie on the line between highest point in leftmost and in rightmost
            // if true, the current point is above the line
            if(isCollinear(firstPartLowestPoint, secondPartLowestPoint, firstHull.get(i)) < 0)
            {
                // the current point is above the line
                firstPartLowestPoint = firstHull.get(i);
            }
            pointsInside.add(firstPartLowestPoint);
        }

        for(int i = 0; i < secondHull.size(); i++)
        {
            if(isCollinear(firstPartLowestPoint, secondPartLowestPoint, secondHull.get(i)) < 0)
            {
                // the current point is above the line
                secondPartLowestPoint = secondHull.get(i);
            }
            pointsInside.add(firstPartLowestPoint);
        }

        sommets.addAll(firstHull);
        sommets.addAll(secondHull);
        sommets.removeAll(pointsInside);

        return sommets;
    }     

//**********************************Auxiliary méthods****************************************************

    // if the equation is equal to 0, the points are collinear
    // the method returns the determinant of the point matrix
    // This determinant tells how far point 'c' is from vector ab and on which side
    // it is 
    // < 0 if the point 'c' is below the line (assumption : horizontal line) 
    // > 0 if the point 'c' is above the line 
    public double isCollinear(Coord a, Coord b, Coord c)
    {
         return ((b.x - a.x)*(c.y - a.y) - (b.y - a.y)*(c.x - a.x));
    }



//************************************** line equation ************************************************ 

    // find the slope of the line between two points
    public static double findSlope(Coord point1, Coord point2)
    {
        return (point2.y - point1.y)/(point2.x-point1.x);
    }

    // finding the constant 'b' of the line equation y = xm + b 
    public static double constantB(Double slope, Coord point)
    {
        return point.y - slope* point.x;

    }

//*************************************** Minimum and Maximum "Y" *****************************************

    // the point with maximum Y
    public static Coord getMaxY(ArrayList<Coord> points) 
    {
        double maxY = points.get(0).y;   // start with the first value
        Coord maxPoint = points.get(0);
        for (int i=1; i<points.size(); i++) {
            if (points.get(i).y > maxY) 
            {
                maxY = points.get(i).y; // new maximum
                maxPoint = points.get(i);
            }
        }
        return maxPoint;
    }

    // a method to find the Point with the minimum y 
    public static Coord getMinY(ArrayList<Coord> points)
    {  
          double minValue = points.get(0).y;
          Coord minPoint = points.get(0);
          for(int i=1;i<points.size();i++){  
            if(points.get(i).y < minValue)
            {
                minPoint = points.get(i);
                minValue = points.get(i).y;  
            }  
          }  
          return minPoint;  
    } 

//************************************** sorting the points ******************************************** 

    //sorting the points by their x in ascending order
    public static ArrayList<Coord> sortedArrayByX(ArrayList<Coord> arrayOfPoints)
    {
        //double minval = arrayOfPoints[0].x;
        Coord temp = null;
        for(int i = 0; i< arrayOfPoints.size(); i++)
        { 
            for(int j = 0; j< arrayOfPoints.size()-1; j++)
            {
                if(arrayOfPoints.get(j+1).x < arrayOfPoints.get(j).x)
                {
                    temp = arrayOfPoints.get(j+1);
                    arrayOfPoints.set(j+1, arrayOfPoints.get(j));
                    arrayOfPoints.set(j, temp);  
                }
            }
        }
        return arrayOfPoints;
    }

我不明白为什么当我运行程序时,会出现以下消息:
Exception in thread "main" java.lang.IndexOutOfBoundsException: Index: 0, Size: 0
    at java.util.ArrayList.rangeCheck(ArrayList.java:604)
    at java.util.ArrayList.get(ArrayList.java:382)
    at miniprojet2.CoordSet.getMaxY(CoordSet.java:270)
    at miniprojet2.CoordSet.mergeHulls(CoordSet.java:154)
    at miniprojet2.CoordSet.territoire(CoordSet.java:139)
    at miniprojet2.CalculeTerritoire.main(CalculeTerritoire.java:36)

如果您能告诉我犯了哪些错误,我将非常高兴。


那是运行时错误,而非编译时错误。 - wchargin
是的,你说得对。我的错 :) 但我甚至不知道为什么会出现这个错误。 - Navid Koochooloo
1个回答

0

注意: 我假设你的代码在firstPart.add(sortedPointsByX.get(i))处失败。如果您发布SSCCE,我可以提供更好的帮助。

sortedPointsByX大小为零。

您的代码:

for(int i = 0; i < sortedPointsByX.size()/2+1; i++) {
    firstPart.add(sortedPointsByX.get(i));
}

因此评估为:

for (int i=0; i<(0 / 2) + 1; i++) {
    firstPart.add(sortedPointsByX.get(i));
}

这是:

for (int i=0; i<1; i++) {
    firstPart.add(sortedPointsByX.get(i));
}

这句话可以转化为:

firstPart.add(sortedPointsByX.get(0));

然而,你无法获取sortedPointsByX的第零个元素,因为它是空的。


你可以从错误信息中了解到这一切:
Exception in thread "main" java.lang.IndexOutOfBoundsException: Index: 0, Size: 0
    at java.util.ArrayList.rangeCheck(ArrayList.java:604)

IndexOutOfBoundsException 意味着您试图访问一些索引 i,以至于 i < 0i >= size()。错误报告表示大小为零,您尝试访问索引为 0 的位置; 因此,0 ≥ 0,您的代码失败了。


但是我已经测试过(使用System.out.println())我在“直到这里都工作正常”的部分编写的代码,并且我打印了sortedPointsByX中的元素,它们是正确的元素。 - Navid Koochooloo
我正在测试这组数字: 0 0, 0 3.75, 1 2.5, 1 0.5, 1.5 1.5, -4 1.5, -3 1.3, -2 1.1, -1 0.9, 0 0.7, -2.9 1.5, -1.8 1.5, -0.7 1.5, 0.6 1.5, -3 1.7, -2 1.9, -1 2.1, 0 2.3, -3.2 1.2, -2.4 0.9, -1.6 0.6, -0.8 0.3, -3.2 1.95, -2.4 2.4, -1.6 2.85, -0.8 3.3, - Navid Koochooloo

网页内容由stack overflow 提供, 点击上面的
可以查看英文原文,
原文链接