is my UML use case and class diagram correct? [closed]

Question

So I am very new to creating design diagrams. I have a scenario where a user who is an admin of pizza hut selects registration option and the system prompts them to enter customer details.

Upon entering the customer details, the system creates a username and password for the customer and notifies the admin that the customer has been created and also emails the customer to let them know they are fully registered.

Now based on this, here is the UML use case diagram (on the left) and UML class diagram (on the right) I created.

Is it correct at all? If not please give some feedbacks!

Answer 1

No, neither diagram is correct.

In the use case diagram, there are several problems

• Actors (the stick people in the diagram) are by definition things usually persons) that are outside the system you are designing. Therefor, having an actor with the stereotype <<system>> is a contradiction.
• Each use-case bubble in a use-case diagram represents an entire use-case. A use-case consists of a success scenario and one or more alternative scenarios.

As you are basing your diagrams on a single scenario, this means that your use-case diagram would consists of a single use-case bubble and a single actor. Depending on how important it is that the actions are performed by an admin user, I would either use the customer or the admin as actor in the diagram.

---

In the class diagram, there are also a few problems.

• A class diagram should represent how the system you are designing sees the world.
• The CustName, CustAge, CustEmail and CustNumber should not be properties of the Admin object, but they should be properties of a Customer object.
• It is not wrong, but it is unconventional to have a class called system in your design.

Answer 2

Quick Short Answer

Feedback:

Your first "Use Case" is missing a Customer actor. Usually, each actor maybe represented by a class in a class diagram, that's why have fields like CustNo, your class / actor is missing.

Warning, there was another answer that tells not to put actors into a class.

The system is the whole use case diagram, that is why is not represented as an actor.

Usually, you should don't put the name of the company, Awsome Quick Service Pizzas, instead, an ID or very short description, of the system, maybe PizzaOrderSystem .

Good Luck.