Task 3 - #3
Open
ValchukDmitry wants to merge 2 commits into
Open
Conversation
yurii-litvinov
suggested changes
Jun 1, 2019
yurii-litvinov
left a comment
There was a problem hiding this comment.
- На IDEF0-диаграммах стрелки какие-то неканоничные
- На контекстной IDEF0-диаграмме "План комплектации" и "Сборщики" дважды
- "Текущий план" --- внутреннее дело завода (если у нас не плановая экономика), так что я бы не стал рисовать его на контекстной диаграмме вообще
- На диаграмме первого уровня "Производственный план" --- вряд ли "Материалы", конвейер же не перерабатывает планы в автомобили. Это скорее управление.
- Следовало бы отразить процесс ремонта на диаграмме первого уровня, ведь конечная наша цель --- приложение по учёту дефектов, и мы хотим видеть место нашей части в общем бизнес-процессе завода.
- На BPMN-диаграмме у Вас вроде как один пул, разделённый на дорожки. Раз так, то внутри одного пула сообщения не посылаются (внутри пула --- единый бизнес-процесс), активности просто связываются потоком управления. Кроме того, посылка и приём сообщения всё равно должны обозначаться специальным символом внутри активности (конвертиком), так что это в любом случае не совсем соответствует синтаксису BPMN. Вот, кстати, набор "каноничных" примеров диаграмм бизнес-процессов: https://www.omg.org/cgi-bin/doc?dtc/10-06-02
- Стендовые испытания и блок ветвления после сборки комплектующих, наверное, должны быть соединены.
- Use Case-диаграмму хотелось конкретно для приложения учёта дефектов, дилер на ней не нужен.
- Зато не помешала бы граница системы --- прямоугольник, внутри которого случаи использования, снаружи --- акторы. Чтобы визуально отделить то, что нам надо делать, от того, что нам делать не надо.
|
Ветвления в BPMN более каноничны с X внутри (исключающее "или"). И "Отправка на..." --- это прямо настоящая активность? Это возможная интерпретация, потому что действительно автомобиль надо снять, погрузить на тележку, отвезти и т.д., но возможно было и просто соединить стрелкой выявление и устранение дефекта сразу, без промежуточного шага (если действия по отправке нам не важны, а нам как разработчикам приложения по учёту дефектов они не важны на самом деле). В остальном ок, так что зачтена. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.