Добавление точек многоугольника из значений Line2D

Мне кажется, что много дублированного кода. Прежде чем мы попытаемся продолжить отладку, давайте реорганизуем код и упростим его понимание и отладку.

Рефакторинг

Первый фрагмент кода, который мы можем извлечь, — это код для добавления точки к многоугольнику. Вот новый метод.

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);

Я подумал, ВТФ? Я никогда не видел, чтобы кто-то устанавливал значения в вызове метода. Конечно, в предложении where.

Примерно через 5 минут размышлений я понял, что внутренние значения класса Polygon устанавливаются после выполнения метода addPoint. Хотя это может быть полезно при вызове какого-либо другого метода, здесь в этом нет необходимости. Вызов метода можно упростить до

p.addPoint(x, y);

Разработчики Java, если вам нужна еще одна причина для того, чтобы сделать ваши переменные класса закрытыми, это действительно хорошая причина. Не позволяет людям устанавливать переменные вашего класса после того, как вы установили их в методе установки.

Предварительное чтение

Мы можем избавиться от первого переключения времени и большого количества кода, если воспользуемся малоизвестным алгоритмом, называемым первичным чтением.

Большинство циклов 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 лучше использовать интерфейс над конкретным классом. Поскольку единственным методом List, который мы используем в коде, является метод get, мы можем использовать интерфейс. Преимущество использования интерфейса заключается в том, что методу createPolygon не важно, возвращает ли метод getLineList ArrayList, LinkedList или пользовательский класс, реализующий List. Это упрощает будущие модификации.