Код-ревью Senior AS-IS, TO-BE и BPMN обновлено
Ревью: текстовое описание BPMN-схемы
Вопрос
Аналитик описал схему процесса согласования отпуска текстом (так её выгрузил инструмент). Найдите ошибки моделирования.
Пул «Компания», дорожки: Сотрудник, Руководитель, Отдел кадров
1. Начальное событие: «Начало»
2. Задача (Сотрудник): «Заявление»
3. Задача (Руководитель): «Рассмотреть заявление»
4. Шлюз XOR «Одобрено?» -> «да»: задача 5; без подписи: задача 6
5. Задача (Отдел кадров): «Оформить приказ»
6. Задача (Сотрудник): «Изменить даты» -> задача 2
7. Шлюз AND (слияние) входы: задача 5, задача 6 -> Конечное событие «Конец»
8. Поток управления из задачи 5 в пул «Бухгалтерия» к задаче «Начислить отпускные»
Коротко
Схема нарушает правила BPMN: смешение XOR и AND, поток управления между пулами. Плюс названия без глаголов, событие без триггера и безымянные ветки делают схему нечитаемой.
Что должен найти ревьюер
- Стильстрока 2Начальное событие без триггера: лучше «Сотрудник решил взять отпуск» или по графику отпусков.
- Стильстрока 3Задача названа существительным: должно быть действие, например «Подать заявление на отпуск».
- Стильстрока 5Вторая ветка XOR без условия: должно быть «нет».
- Ошибкастрока 8Слияние AND после разветвления XOR: процесс зависнет, ожидая обе ветки. К тому же ветка «нет» уходит в цикл, а не к концу.
- Стильстрока 8Конечное событие не названо по результату: «Отпуск оформлен».
- Ошибкастрока 9Между пулами нельзя использовать поток управления: только поток сообщений.
Разбор
Схема нарушает правила BPMN: смешение XOR и AND, поток управления между пулами. Плюс названия без глаголов, событие без триггера и безымянные ветки делают схему нечитаемой.
Ответьте сами, а не прочитайте
В тренажёре вопрос попадёт в повторения и вернётся, когда начнёт забываться. Плюс билеты с зачётом: 20 вопросов, не больше 2 ошибок.