Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Docs: Первые диаграммы #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Docs: Первые диаграммы #9

wants to merge 1 commit into from

Conversation

KruchDmitriy
Copy link

No description provided.

@valentina-kustikova
Copy link
Contributor

@KruchDmitriy, замечания по use case-диаграмме:

  1. Система не просто должна позволять изменять количество классов, но и загружать имеющийся перечень классов объектов, добавлять новый класс (со вводом названия класса), удалять класс объектов (с перемаркировкой всех сегментов, отмеченных этим классом).
  2. Polygon, Brush, Brush for segmentation -- этих компонент не должно быть в use case, т.к. относится к реализации.

Замечания по sequece диаграмме:

  1. В цикле загрузки данных не хватает шагов отображения загруженных изображений/разметки на вьюшке.
  2. После установки количества классов пользователь также ничего не видит в результате.
  3. При изменении размера кисти (операция 8) та же ситуации, а пользователь узнает, что размер изменился?
  4. При отрисовке многоугольника (операция 9) аналогично ничего не возвращается, хотя пользователь должен видеть контур, который рисует.
  5. Операция добавления точки не совсем понятна (операция 10).
  6. При переходе к следующему изображению надо сохранять результат работы с предыдущим (операция 19).
  7. При закрытии (операция 23) надо сохранить результат.
  8. Операции 24 и 25 подвешенные и не понятно, для чего они.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants