поведение отношения OneToMany при использовании пружинной загрузки и пружинных данных jpa - PullRequest
Новая
       26

поведение отношения OneToMany при использовании пружинной загрузки и пружинных данных jpa

1 голос
/ 02 марта 2020

Я занимаюсь разработкой простого приложения Crud в Spring Boot с данными JPA, моя цель проста, когда у меня есть две сущности: Foo.java

@Data // lombok annotation
@Entity(name = "foos")
public class Foo{

    @Id
    @GeneratedValue(strategy = GenerationType.AUTO)
    @Column(unique = true, nullable = false)
    private Integer fooId;

    private String FooName;

    @NonNull
    @OneToMany(cascade = CascadeType.ALL)
    @JoinColumn(name = "bar_id")
    private List<Bar> barList;

}

Bar.java 

@Data
@Entity(name = "bars")
public class Bar{

    @Id
    @GeneratedValue(strategy = GenerationType.AUTO)
    @Column(name = "id", unique = true, nullable = false)
    private Integer barId;

    private String barTitle;

}

В моем контроллере я хочу сохранить Foo со списком Bars как: 

@Controller
@RequestMapping(value = "/foo")    
public class FooController {

    private final FooService fooService;
    private final BarService barService;

    public FooController(FooService fooService, BarService barService) {
        this.fooService = fooService;
        this.barService = barService;
    }

    @GetMapping(value = "/{id}/add_bar")
    public String addBar(@PathVariable("id") Integer id,  Model model){
        model.addAttribute("foo", fooService.findById(id));
        model.addAttribute("bar", new Bar());
        return "add_bar";
    }

    @PostMapping(value = "/{id}/add_bar")
    public String saveBar(
            @PathVariable("id") Integer id,
            @ModelAttribute("bar") Bar bar,
            BindingResult result
    ){
        if (result.hasErrors()) {
            return "add_bar";
        }
        // update foo by adding new bar and save
        Foo foo = getFooAndAddBar(id, bar);
        fooService.save(foo);
        // save bar
        barService.save(bar);

        return "redirect:/foo/" + foo.getFooId();
    }

    // update foo by adding new bar and save
    private Foo getFooAndAddBar(Integer id, Bar bar) {
        Foo foo = fooService.findById(id);
        ArrayList<Bar> barList = new ArrayList<>();
        barList.add(bar);
        foo.setBarList(barList);
        return foo;
    }
}

Первый бар сохраняется и выбирается по идентификатору foo, но когда я хочу добавить другой бар, он только обновляет первый бар вместо вставки новой записи бара в БД. Чего не хватает в ассоциации @OneToMany? или чего-то не хватает в другой части программы? Пожалуйста.

1 Ответ

2 голосов
/ 02 марта 2020

вы устанавливаете barlist каждый раз, когда вызываете свою функцию. Вы написали foo.setBarList(barList);. каждый раз это будет перезаписывать предыдущий список, а затем сохранять его, что приведет к перезаписи предыдущих значений. Вместо этого попробуйте это foo.getBarList().add(Bar). Он извлечет предыдущий список баров, а затем добавит новый брус в список. После этого просто сохраните сущность

...