Замечания по заданию Shape (lw.4)

iandreyshev commented 4 years ago

[ ] Выделять интерфейс SolidShape не самый хороший вариант, хотя бы потому что он нигде не используется. Стоит придумать, как можно получать оба цвета используя только интерфейс Shape, при этом не у всех фигур есть цвет заливки. Например, у линии нет цвета заливки

iandreyshev commented 4 years ago

[x] В названии теста не нужно писать Correct, потому что в задании подразумевается что фигуры могут вычислять только правильные периметры. Достаточно shouldCalculateArea

iandreyshev commented 4 years ago

[ ] Код фигур не должен иметь возможность очистить холст

iandreyshev commented 4 years ago

[x] Не стоит создавать линии внутри теугольников только чтобы посчитать расстояние между точками. В таком случае код треугольников начинает зависеть от кода линий, хотя нужно только вычислить расстояние между точками.

override fun getPerimeter() = LineSegment(outlineColor, vertex1, vertex2).getLength() +
        LineSegment(outlineColor, vertex1, vertex3).getLength() +
        LineSegment(outlineColor, vertex2, vertex3).getLength()

Можно сделать расширение для Point, или функцию верхнего уровня и т.д.

iandreyshev commented 4 years ago

[x] Нет возможности вводить фигуры через консоль, получить фигуру с наибольшей площадью, периметром и т.д.

Из задания:

iandreyshev commented 4 years ago

Коэффициент по основной части - 0.5

iandreyshev commented 4 years ago

[ ] Непонятно зачем осталвена возможность вводить параметры руками, если можно все делать мышкой. Стоит убрать эти контролы
[ ] Добавь возможность при "рисовании фигур мышкой" видеть начертания фигур пока пользователь удерживает клавишу, как это происходит в Paint

Коэффициент по визуализации - 0.9

alexeyminay commented 4 years ago

[ ] Выделять интерфейс SolidShape не самый хороший вариант, хотя бы потому что он нигде не используется. Стоит придумать, как можно получать оба цвета используя только интерфейс Shape, при этом не у всех фигур есть цвет заливки. Например, у линии нет цвета заливки

согласен, но это было частью задания

alexeyminay commented 4 years ago

[ ] Код фигур не должен иметь возможность очистить холст

возможно я неправильно понял комментарий.. посмотрел реализацию, canvas чистит сам себя.

alexeyminay commented 4 years ago

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

Оставлена так как во многих графических редакторах она есть, тем более без этого не нарисовать треугольник любой формы, только прямоугольной. Возможность рисовать треугольник по трем точкам выходила далеко за рамки лабораторной, по этому не стал тратить время, переключился на другую.

iandreyshev commented 4 years ago

[ ] Код фигур не должен иметь возможность очистить холст

возможно я неправильно понял комментарий.. посмотрел реализацию, canvas чистит сам себя.

Интерфейс ICanvas содержит метод clearAll(), который можно вызвать из фигуры и очистить его.

iandreyshev commented 4 years ago

[ ] Код вычисления фигур с наибольшей площадью и наименьшим периметром стоит выделить в функции и написать на них тесты. Функции не должны выводить сообщения в Output, они должны получать список фигур и возвращать нужную. Эти функции стоит покрыть тестами.

Сейчас код получения фигуры с нименьшим периметром не выполняет свою задачу.

iandreyshev commented 4 years ago

Коэффициент по основной части - 0.7

alexeyminay / labs

Замечания по заданию Shape (lw.4) #7