Замечания по программе Shapes

MorlakDzuu / OOP_Labs

OOP labs

0 stars 0 forks source link

Open alexey-malov opened 5 years ago

alexey-malov commented 5 years ago

[x] В классах фигур не надо парсить строки для извлечения цвета - пусть этим занимается код, отвечающий за взаимодействие с пользователем

alexey-malov commented 5 years ago

alexey-malov commented 5 years ago

alexey-malov commented 5 years ago

[x] Упростить код тестов: исключения проверять через assert-ы
[x] Устранить дублирование кода в тестах. Если тесты могут обойтись одним значением тестируемого объекта, то создавать его в методе setUp

alexey-malov commented 5 years ago

[x] Для проверки того, реализует ли объект некоторый интерфейс, используйте instanceof вместо перехвата исключений

alexey-malov commented 5 years ago

[ ] Лучше выделить класс утилит со статическим методом для нахождения расстояния между между точками, чтобы не создавать отрезки прямых для вычисления периметра.
[ ] Вместо матрицы завести 4 скалярные переменные (для вычисления площади треугольника).
[ ] Воспользоваться функцией abs для нахождения модуля

alexey-malov commented 5 years ago

[ ] Сделайте так, чтобы достаточно было на холсте нарисовать фигуры один раз, а содержимое холста на Graphics-е рисовать несколько раз. Для этого примените дисплейные списки:
Класс Canvas должен запоминать рисуемые примитивы.
Класс Canvas имеет метод рисования его на Graphics