java - 从 Line2D 值添加多边形点

Question

我有一小段代码从名为 lineList 的数组(在不同的类中)读取 Line2D 值并将它们存储在名为 list 的新数组中。从这里开始，我一直在尝试将所有线值转换为多边形点(线端的每个 x、y 坐标对应一个点)。

到目前为止，我已经可以正常工作，但它不适用于添加的数组中第一行的第一个点(我怀疑是这样)，我在尝试找到解决方案时遇到了麻烦将其包含在第一个 if 语句中。

我将非常感谢任何人能够为此提供的任何帮助。

下面是我用来从 Line2D 值添加点的代码:

Polygon p = new Polygon();
    ArrayList<Line2D> list = new ArrayList<Line2D>();
    Color pixel;
    boolean firstTime = true;

    list = segmentation.getLineList();

    //loop through lineList and add all x and y coordinates to relative x and y arrays
    for(int i = 0; i < list.size(); i++) {
        if(firstTime == true){
            Line2D line = list.get(i);
            Point2D startPoint = line.getP1();
            Point2D endPoint = line.getP2();
            int startX = (int) startPoint.getX();
            int startY = (int) startPoint.getY();
            int endX = (int) endPoint.getX();
            int endY = (int) endPoint.getY();
            p.addPoint(p.xpoints[i] = startX, p.ypoints[i] = startY);
            p.addPoint(p.xpoints[i] = endX, p.ypoints[i] = endY);
            startPoint = null;
            endPoint = null;
            line = null;
            firstTime = false;
        }
        else {
            Line2D line = list.get(i);
            Point2D endPoint = line.getP2();
            int endX = (int) endPoint.getX();
            int endY = (int) endPoint.getY();
            p.addPoint(p.xpoints[i] =  endX, p.ypoints[i] = endY);
            endPoint = null;
            line = null;    
        }
    }

下面是第一个点(最低点)未包含在多边形点中的示例。

Answer 1

对我来说似乎有很多重复的代码。在我们尝试更多调试之前，让我们重构代码并使其更易于理解和调试。

重构

我们可以提取的第一段代码是向多边形添加一个点的代码。这是新方法。

protected void addPoint(Polygon p, Point2D point) {
    int x = (int) point.getX();
    int y = (int) point.getY();
    p.addPoint(x, y);
}

现在，我没有在一次重构中做到这一点。我首先提取了端点代码，因为它是相同的。在对代码进行了更多思考之后，我对其进行了概括，以便将其用于起点代码。

当我第一次看到这行代码时

 p.addPoint(p.xpoints[i] = startX, p.ypoints[i] = startY);

我想，WT​​F？我从未见过有人在方法调用中设置值。当然，在 where 子句中。

经过大约 5 分钟的思考，我意识到 Polygon 类的内部值是在执行 addPoint 方法之后设置的。虽然这对于其他一些方法调用可能很有用，但在这里没有必要。方法调用可以简化为

p.addPoint(x, y);

Java 开发人员，如果您还需要另一个理由让您的类变量成为非公共(public)的，那么这是一个非常好的理由。在您在 setter 方法中设置类变量后，防止人们设置它们。

入门阅读

如果我们使用一种鲜为人知的称为启动读取的算法，我们可以摆脱第一次开关和大量代码。

大多数 for 循环都将输入语句作为循环中的第一条语句。循环的 for (String s : stringList) 构造隐藏了输入语句是循环中的第一条语句这一事实。

但有时，您有一种方法需要启动阅读。这种方法就是其中之一。

在伪代码中，启动读取是这样工作的。

Read input
for loop
    process input
    read input
end loop
process last input

通过使用启动读取，我能够极大地简化 createPolygon 方法。

任何读到这个想法的 Cobol 程序员，“是的，读一读。”

Java 程序员，请牢记这个入门阅读的想法。您不会经常使用它，但如您所见，它在某些情况下大大减少了您需要的代码量。

重构代码

public Polygon createPolygon(Segmentation segmentation) {
    Polygon p = new Polygon();
    List<Line2D> list = segmentation.getLineList();
    if (list.size() < 2) return p;

    Line2D line = list.get(0);
    addPoint(p, line.getP1());

    // loop through lineList and add all x and y coordinates to relative x
    // and y arrays
    for (int i = 1; i < list.size(); i++) {
        addPoint(p, line.getP2());
        line = list.get(i);
    }

    addPoint(p, line.getP2());
    return p;
}

protected void addPoint(Polygon p, Point2D point) {
    int x = (int) point.getX();
    int y = (int) point.getY();
    p.addPoint(x, y);
}

我对代码做了两件事。

1. 我添加了少于 2 行的测试。基本上，创建一个三角形(多边形)至少需要 2 条线。对 1 行或零行执行该方法没有意义。

2. 我将 ArrayList 引用更改为 List。在 Java 中，最好在具体类上使用接口(interface)。由于我们在代码中使用的唯一 List 方法是 get 方法，因此我们可以使用该接口(interface)。使用该接口(interface)的优点是 createPolygon 方法不关心 getLineList 方法是否返回 ArrayList、LinkedList 或实现 List 的自定义类。这使得将来的修改更容易。