Как лучше сократить код return на Java?

Question

[Jake Taylor]

Такой вопрос.
Имеется метод, который в качестве аргумента принимает четырёхугольник (внутри содержит четыре объекта типа Point. Point содержит в себе координаты Х и Y). Метод определяет, является ли четырёхугольник трапецией.

Справочно:
Трапеция - это четырёхугольник, у которого две стороны параллельны, а другие две - не параллельны.

Метод boolean isSidesParallel(side1, side2) определяет, параллельны ли стороны.

Собственно, сам код, который выполняет свою задачу. Но мне не нравится как выглядит return. Как привести его к более удобочитаемому виду и сократить его?

Код (жми)

@Override
    public boolean isTrapezoid(Quadrangle quadrangle) {
        // создаем точки
        Point p1 = quadrangle.getP1();
        Point p2 = quadrangle.getP2();
        Point p3 = quadrangle.getP3();
        Point p4 = quadrangle.getP4();

        // создаем стороны АB, BC, CD, DA
        Line sideAb = new Line(p1, p2);
        Line sideBc = new Line(p2, p3);
        Line sideCd = new Line(p3, p4);
        Line sideDa = new Line(p1, p4);

        // проверка, чтобы две противоположыне стороны были параллельны, а другие - не параллельны
        return (isSidesParallel(sideAb, sideCd) && !isSidesParallel(sideBc, sideDa)) || (!isSidesParallel(sideAb, sideCd) && isSidesParallel(sideBc, sideDa));
    }

Answer 1

[koperagen]

Условие "две стороны параллельны, другие две - нет" похоже на оператор xor
Для него таблица истинности
1 1 -> 0
1 0 -> 1
0 1 -> 1
0 0 -> 0
Т.е. можно записать так
isSidesParallel(sideAb, sideCd) ^ isSidesParallel(sideBc, sideDa)
(^ - это xor в джаве)

Comments

[Alex Wells]

и.е. это стало более понятно чем вариант автора? Ни в коем случае.

[Jake Taylor]

Alex Wells, Так меньше кода занимает, понятно)

Answer 2

[Alex Wells]

return (isSidesParallel(sideAb, sideCd) && !isSidesParallel(sideBc, sideDa)) || 
    (!isSidesParallel(sideAb, sideCd) && isSidesParallel(sideBc, sideDa));

и are sides parallel, а не is

Comments

[Jake Taylor]

Так в итоге не сократилось ничего, просто перенос строки :)
Спасибо за are

[Alex Wells]

Jake Taylor, не сократилось конечно же, но стало вполне читабельно.

Answer 3

[Therapyx]

Написать его по человечески, с комментариями и чтобы его мог прочитать другой человек. А потом вернуть true или false.
мой лучший тебе совет. Не вые... Тренируйся писать хороший код. А хороший код - это не красивый на твой взгляд, а понятный как тебе, так и другим людям.
Ибо поверь, если в таком ключе будет что-то посложнее и ты вернешься к этому коду через год, то ты потом и глаза и голову сломаешь))

Comments

[Alex Wells]

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

[Therapyx]

Alex Wells, Я не говорил о "ясно" или "не ясно". Тут надо менять весь стиль. Он спрашивал тут не об ошибках. А именно, что ему не нравится стиль.
И я описал что тут не так. А именно - все. Такие привычки = это вредители. И я его прекрасно понимаю, потому-что когда только связывался с айти миром, я точно так же болел перфекционизмом и красотой "1-й строчки". А сейчас я передаю ему свой опыт, что так делать нельзя.