Java/Spring/REST/web service

Question

Siemanko

Próbuję napisać coś swojego. Aplikacja ma umożliwiać dodawanie produktów spożywczych z parametrami, co już zrealizowałem, dalej celem jest sumowanie wartości odżywczych, porównywanie z dziennym zapotrzebowaniem etc..)

Problem leży w wybieraniu produktów, do połączenia w posiłek. Otóż zastanawiam się, czy jest jakaś alternatywa dla funkcji o określonej liczbie opcjonalnych parametrów, co z góry narzuca pewne ograniczenie na posiłek. Tutaj mam taką prowizorkę, ale nie podoba mi się to na tyle że póki co nie rozwijam jej dalej

@RequestMapping("/add")
public Integer adding(@RequestParam("i") Long index,
                      @RequestParam("i2") Long index2,
                      @RequestParam(value="i3", required = false, defaultValue = "0") Long index3,
                      @RequestParam(value="i4", required = false, defaultValue = "0") Long index4,
                      @RequestParam(value="i5", required = false, defaultValue = "0") Long index5,
                      @RequestParam(value="i6", required = false, defaultValue = "0") Long index6
){
    Integer sum = null;


   Integer i1 = productManager.findById(index).get().getCalories();
  // Integer x = first.getCalories();
   Products second = productManager.findById(index2).get();
   Integer i2 = second.getCalories();

   Integer i3,a,b,c;
   if (index3==0){
       i3=0;
   } else {
       Products thrid = productManager.findById(index3).get();
       i3 = thrid.getCalories();
   }
    sum= i1+i2+i3;
    return sum;
}

 Ma ktoś jakąś inną, ładniejszą koncepcję..? 

Comments

Good If you really like to learn more about java and python.

I recommend you get skills from frontier technology. Link below

https://frontiertechnologyinstitute.com/python-programming-certification/

Answer 1

Skorzystaj z listy/tablicy na wejściu.

I jedna ważna rzecz, odnośnie użycia Optional, ponieważ to zdaje się jest operacja własnie na tym obiekcie:

productManager.findById(index).get().getCalories();

Tak nie należy robić. Tzn. nie wolno używać bezpośrednio get, ponieważ może to prowadzić do wyrzucenia wyjątku.

W powyższym przypadku możesz zrobić np. tak:

Integer integer = findProduct()
                     .map(Product::getCalories)
                     .orElse(ZERO);

Ponieważ sumujesz wartości, to jeśli produktu nie udało się znaleźć to powyższa konstrukcja zwróci 0 - 0 jest obojętne w przypadku sumowania (nie zmieni wyniku). A jeśli dany produkt będzie znaleziony to zwróci jego liczbę kalorii (map(Product::getCalories)).

Comments

Super, wszystko śmiga, a wygląda tak : 

@RequestMapping(value= "/addCalories")
    public String sumCalories(@RequestParam("index") List<Long> ids) {
        Float sumCal = 0F;

        for (Long id: ids) {
            sumCal += productManager.findById(id).map(Product::getCalories).orElse(0F));
        }

        return sumCal+" kcal";
}

Uświadomiłem sobie jednak, że potrzebuję raczej Mapy zamiast Listy. Każda liczba kalorii powinna być wymnożona przez gramaturę. Jeżeli mogę zażądać Mapy, to czy jest to poprawna forma ? 

    public String sumCalories(@RequestParam() Map<Long, Integer> map)

Jeżeli tak, to jak powinno wyglądać żądanie http ? 
 

Answer 2

Poszukaj informacji o RequestBody i o Listach/Setach.

Jeżeli chcesz, to możesz owrapowac tą Liste/Set w jakąś klasę, będzie lepiej wyglądać zamiast otrzymywać zwykłej Listy.