Je suis d'accord avec Thomas, mais je vais montrer d'autres points, donc pouvez-vous ajuster votre diagramme. S'il vous plaît ne pas voir comme quelque chose de destructeur. Je crois simplement que ces conseils peuvent vous aider.
- multiplicités show de chaque connexion
- ne pas utiliser getter et setter *
- ne pas utiliser les attributs ID *
- Ordre rend la composition avec l'utilisateur (n'a pas de sens ont un ordre avec un utilisateur lié avec elle et OrderLine fait la composition avec l'ordre;
- Produit font l'agrégation avec OrderLine et revue font la composition avec le produit;
- Le fabricant fait l'agrégation avec le produit. (dépend de votre système, ce peut être une composition, mais plus susceptible d'être une agrégation).
Rappelez-vous (la partie) fait quelque chose avec (l'ensemble)
- il n'a pas besoin de créer un attribut dans une classe si vous avez un lien avec cette classe, sauf si vous avez une liste de cela, (par exemple: attribut CreditPayment in Payment et CreditPayment de classe);
- Vous pourriez faire une liste de produits dans l'ordre. Après cela, vous pouvez supprimer la classe Orderline.
* si vous ne générez pas de code à partir du modèle.