请问你们在辨证什么？

我也觉得没必要，求老师指点迷津。刚实习碰到一些前辈说的原则，觉得一般来说@Valid girl girl这个对象不能和domain对象混用，业务和数据库要区分开。

参数中的girl是用来获取页面传输的条件，为避免写多个条件太繁琐，所以使用girl来映射获取值。

1. @PostMapping("/girl")

2. public Girl girlAdd(@Valid girl girl, BindingResult bingdingResult) {

3.     girl.setName(girl.getName());//??为什么自己给自己赋值？

4.     girl.setAge(girl.getAge());

5.     return girlRepository.save(girl);

6. }

7. 应该楼主是指这个问题，我觉得set可以直接去掉了。直接一句return就好

话说，你的图是不是截的和问的问题不太一致？感觉你是不是想问下面这段代码？

# src/main/java/com.imooc/controller/GirlController    
    // 添加
    @PostMapping
    public Result<Girl> girlAdd(@Valid Girl girl, BindingResult bindingResult) {
        return girlRepository.save(girl);
    }

感觉没有必须先get再set，不知道是不是老师图省事没有调整，实测这样是可以的，而且get之后set确实感觉没意义