Spring MVC @PathVariable

Question

Witam, mam problem ze zrozumieniem pewnej rzeczy. Załóżmy, że chcę utworzyć metodę mapującą w kontrolerze formularz do akcji "dodawania" nowego obiektu oraz użyć tego samego formularza do edycji istniejących obiektów, oczywiście mapując obie akcje w tej samej metodzie. Do tego oczywiście chcę używać zmiennych URL.

Przykładowy kod:

    @GetMapping(value ={"/osoba/add" , "/osoba/edit/{id}"})
    public String showForm(Model model, @PathVariable(required = false, value = "id") int id){
            if(id == 0){
                o = new Osoba();
            } else {
                o = OsobaController.lista.stream().filter(g -> g.getId()== id).findFirst().get();
            }
            model.addAttribute("osoba" , o);
        

Kod błędu :

Request processing failed; nested exception is java.lang.IllegalStateException: Optional int parameter 'id' is present but cannot be translated into a null value due to being declared as a primitive type. Consider declaring it as object wrapper for the corresponding primitive type.

 

Czyli po prostu czegoś nie rozumiem bo pomimo required = false, kontroler próbuje odczytać tą zmienna. Mogę to oczywiście rozbić na 2 mapowania, ale nie o to tu chodzi :)

Answer 1

Tak jak w podpowiedzi masz wspomniane. Zmień na Integer i obsłuż w przypadku, gdy ktoś nie poda parametru (będzie nullem). Nie zmapujesz nulla na typ prymitywny.

Answer 2

Według mnie nie jest to dobry pomysł, żeby dwie funkcjonalności pchać do jednej metody. Łamie to m.in. pierwszą zasadę SOLID, bo w tym momencie, jeśli będziesz musiał zmienić coś w edycji czy dodawaniu to za to odpowiada jedna metoda, czyli jedna metoda ma dwa powody do zmiany.

Dodatkowo to może spowodować, że w o będzie null, czyli gdzieś później może wystąpić NullPointerException:

 o = OsobaController.lista.stream().filter(g -> g.getId()== id).findFirst().get();

Jeśli rozbijesz to na drugą metodę to można zrobić tak:

OsobaController.lista.stream()
    .filter(g -> g.getId()== id)
    .findFirst()
    .ifPresent(user -> model.addAttribute("osoba", user);

wtedy jest jak to się mówi null-safety. 

Dodatkowo jeśli o jest zmienną klasową to narażasz się na to, że jeśli pójdą dwa requesty na raz to mogą sobie one wzajemnie nadpisywać tą zmienną. Najlepiej, jeśli o będzie zmienną lokalną w metodzie. Miej też na uwadze, że to samo będzie z OsobaController.lista jednak pewnie używasz jej jako "bazy danych" więc się tym na razie nie przejmuj - miej tylko tego świadomość, być może rozwiązaniem tego jest użycie jakiejś kolekcji z pakietu java.util.concurrent.