pytanie i ocena kodu - REST (CRUD)

0

Utworzyć warstwę serwisów. Co prawda miej ciągle w pamięci, że rypanie miliona warstwa nie zawsze jest dobre :).

0

@tomek1221 zrób obie opcje, zobacz czym się różnią, jakie są zalety i wady takiego rozwiązania ;) Jak już wybierzesz to będziemy radzić dalej. W ten sposób więcej się nauczysz, niż jak damy Ci gotowe rozwiązanie. Ponadto widzę Twoj kod na githubie. Zamiast korzystac z findOne korzystaj z findById customowo napisanego. Niech zwraca optionala i będziesz mógł sobie ładnie to obsłużyć.

edit:
ajjj @Schadoow zepsułeś. Dałeś rybkę zamiast wędki :(

0

@artur52: @Shadoow Spoko, też chce z tego jak najwięcej wynieść, po prostu zacząłem się gubić w tych warstwach i w momencie jak przychodzi, żeby jeszcze to połączyć to w ogóle głupieje.

A co z DAO? Bo np. patrzę sobie przykład, że gość tworzy controllers, entity, service, dao. Tworząc serwisy powinienem też mieć DAO?

0

Używając Spring DATA w Twoim przypadku nie ma takiej potrzeby.

0

@artur52: pytanie techniczne, tworzę sobie ten cały serwis, czy ja dobrze to rozumiem, logikę działania np. tworzenia Customer'a z controllera, który sobie używa repository powinienem PRZENIEŚĆ do serwisu a w kontrolerze tylko użyć metody tego serwisu? Dobrze to rozumiem?

0

Tak, dokładnie

0

@Shadoow: @artur52 Panowie, dodałem warstwę serwisów, tutaj kod https://github.com/tomek1221/orchestrationService

Możecie mi teraz poradzić jak zrobić to ostatnie zadanie? Tak łopatologicznie, bo tak jak powiedziałem z tego schematu niestety nie mogę tego ogarnąć.

0

@artur52:
@Schadoow

Panowie mam problem, zrobiłem te mapowania w taki sposób:

W Customer.java dodałem:

@OneToMany(cascade = CascadeType.ALL, mappedBy = "customer")
private List<Product> products;

W Product.java dodałem:

@ManyToOne
private Customer customer;

I obecnie wywala mi ciągle błąd, próbowałem też na kilka innych sposobów ale jest to samo.

[PersistenceUnit: default] Unable to build Hibernate SessionFactory

0
  1. Jednego produktu nie może miec wielu klientów tzn jabłka może kupic tylko jeden klient ?
  2. Czy produkt musi wiedziec kto go kupił ? Po co jabłku informacja, że ma go Krzysiek ?
  3. Od połączenia co kupił klient powinien byc koszyk bo jest to relacja wiele do wielu ;p A jeszcze lepiej koszyk aktualny i zamówienia wykonane wcześniej ^^. Pomyśl o modelu jak ma to wyglądac i poczytaj o soft delete :)
0

@Schadoow: Chciałem to zrobić tak jak wcześniej próbowałeś opisać, tylko ja do końca nie potrafię tego przełożyć, mówiłeś wcześniej, że spróbujesz to tak lepiej rozpisać, mógłbym Cię o to prosić?

1 użytkowników online, w tym zalogowanych: 0, gości: 1