仅在一个方法中重构函数重载方法

Question

    public boolean isHorizontalOrVertical(Point firstPoint, Point secondPoint) {
        return firstPoint.getFirstComponent() == secondPoint.getFirstComponent()
                || firstPoint.getSecondComponent() == secondPoint.getSecondComponent();
    }

    public boolean isHorizontalOrVertical(List<Point> points) {
        if (points == null || points.size() < 2) {
            throw new IllegalArgumentException("invalid number of points");
        }
        Point start = points.get(0);
        return points.stream()
                .allMatch(p -> isHorizontalOrVertical(start, p));
    }

该方法需要检查两个或三个点是否相互垂直/水平。在三个点的情况下，它只需要检查最后两个点是否与起始点水平/垂直。

有人知道我如何把所有的东西都放到一个方法里吗？

Answer 1

首先，也是最重要的，我必须注意到这样一个事实：至少对我来说，计算两个实体是水平的还是垂直的，并且这些实体是点的方法是没有意义的。两个点怎么能是水平的还是垂直的？

克服上述问题，您可以创建一个计算两点是水平还是垂直的单一方法。

更改名称ishorizontalorvertice，因为它是多余的。更好的名字是isHorizontal或isvertical。该方法将返回一个布尔值，因此如果isHorizontal返回false，那么它是垂直的，反之亦然。也许一个更好的名字可以是areTwoPointsHorizontal，但我甚至很难写出来，因为它传达了错误的信息，但请随意选择你自己的名字。

    public boolean isHorizontal(Point first, Point second){
        boolean sameFirstComponents = firstPoint.getFirstComponent() == 
                secondPoint.getFirstComponent();
        boolean sameSecondComponents = firstPoint.getSecondComponent() == 
                secondPoint.getSecondComponent();          
        return sameFirstComponents || sameSecondComponents;
    }

   public boolean isHorizontal(List<Point> points){
        boolean allPointsHorizontal = true;
        for (int i=0; i<points.size(); i++) {
            
            boolean nextPointExists = i<points.size() - 1;
            if (nextPointExists) {
                Point current = points.get(i);
                Point next = points.get(i+1);
                allPointsHorizontal = allPointsHorizontal && isHorizontal(current,next);

                if (!allPointsHorizontal)
                    return false;  
            }
        }
        
        return allPointsHorizontal;
    }