diagram klas uml ocena

Witam,

Mam do zrobienia na zajęcia diagram klas, zacząłem coś robić, ale nie jestem przekonany czy idę w dobrym kierunku.
Proszę o jakąkolwiek pomoc, co źle, co dobrze, jak powinno być, może wyrobie sobie jakiś obraz dzięki waszym uwagą.
Uprzedzam że to mój pierwszy kontakt z UML dlatego proszę o wyrozumiałość.

Pozdrawiam i z góry dziękuje za wszelkie pomocne uwagi.

Wszystko tu jest źle bo diagram klas TO NIE TO SAMO co diagram ERD / diagram Bazy Danych (a to co wstawiłeś to nawet jako diagram bazy danych jest bardzo słabe)...

1. Po co ci ID w tych klasach?
2. "Zamówienie" a nie "Zamówienia"!
3. O ile jestem w stanie zrozumieć Kompozycje pomiędzy PozycjąZamówienia a Zamówieniem, o tyle kompozycja Klienta w Zamówieniu to jest jakiś horror! Kompozycja oznacza, ze na przykład koła są częścią samochodu...
4. Nie rozumiem co robi metoda get_klienta w klasie Klient. Jak to niby potem wygląda wg ciebie w kodzie? Masz obiekt klasy Klient i wołasz na nim get_klienta i to ci zwraca jakąś listę stringów? WTF? O co w ogóle chodzi? Analogiczne pytanie do wyswietl_zamowienia.

Poza tym gdybyś faktycznie to implementował to pewnie zrobiłbyś to zupełnie inaczej. Na przykład Adres byłby osobną klasą a nie polami w klasie Klient. Poza tym na pewno nie miałbyś żadnej "nazwy produktu" w pozycji zamówienia, bo po prostu miałyś referencje do Produktu. Tak samo nie miałbyś żadnych pól "wartość" bo i czemu miałbyś mieć? Skoro masz cenę Produktu w klasie Produkt... Analogicznie dla całego Zamówienia. Cenę wyliczyłbyś sumując ceny z listy pozycji, a nie przechowywał w osobnym polu.

Dzięki za uwagi, postaram coś się z tym zrobić i zamieszczę poprawioną wersję.
Pozdrawiam, jeszcze raz dzięki!

Witam,

Poczytałem trochę o UML i stworzyłem stosując się do waszych rad coś takiego (patrz. załącznik)
Bardzo proszę o informację co jest źle i dlaczego ułatwi mi to dalszy rozwój i zrozumienie tematu.
Z góry dziękuje i pozdrawiam