spring boot列表中的Getters和setters

Question

我正在spring boot创建我的API。我认为一个产品是由以下部件组成的：

这里是我的代码：（实体层）

import javax.persistence.*;
import java.util.List;
import java.util.ArrayList;

@Table
@Entity
public class Product{
   @Id @GeneratedValue
   private Long id;
   private String name;
   @OneToMany
   @JoinColumn
   private List<Component> productComponents = new ArrayList<>();

   //default constructor
   public Product(){

   }
   
   public Product(Long id, String name, List<Component> productComponents){
      this.id = id;
      this.name = name;
      this.productComponents = productComponents;
   }

  //getters
  public Long getId(){
     return this.id;
  }

  public String getName(){
     return this.name;
  }

  public List<Component> getProductComponents(){
     return this.productComponents;
  }


  //setters
  public void setId(Long id){
     this.id = id;
  }

 public void setName(String name){
    this.name = name;
 }

 public void setProductComponents(List<Component> productComponents){
    this.productComponents = productComponents;
 }

}

下面是我的Controller类：

package com.accounting.accounting.Controller;
import java.util.List;

@CrossOrigin
@RestController
@RequestMapping("/api/products/product")
public class ProductController {
    private static ProductRepository myProductRepository;

    @GetMapping("/get/all")
    public List<Product> listProducts(){
        return myProductRepository.findAll();
    }


    @PostMapping("/new")
    public Product createNewProduct(@RequestBody Product myProduct){
       return myProductRepository.save(myProduct);
    }

    @PutMapping("/update/{id}")
    public Product updateProduct(@RequestBody Product myProduct, @PathVariable Long id){
        return myProductRepository.findById(id).map((product) ->{
           product.setName(myProduct.getName());
           product.setProductComponents(myProduct.getProductComponents());
        return myProductRepository.save(product);
    }).orElseGet(() ->{
       myProduct.setId(id);
       return myProductRepository.save(myProduct);
    });
}

}

我的问题：

在实体类“Product”中，对于类型为List的productComponents变量，getter和setter应该如上面所述，还是应该如下所示：

 public List<Component> getProductComponents(){
     return this.productComponents;
  }

 public void setProductComponents(Component productComponent){
    this.productComponents.add(productComponent);
 }

并将controller类中的update方法更改为：

    @PutMapping("/update/{id}")
    public Product updateProduct(@RequestBody Product myProduct, @PathVariable Long id){
        return myProductRepository.findById(id).map((product) ->{
           product.setName(myProduct.getName());
           product.setProductComponents((Component) myProduct.getProductComponents());
        return myProductRepository.save(product);
    }).orElseGet(() ->{
       myProduct.setId(id);
       return myProductRepository.save(myProduct);
    });
}

还是有更好的解决方案？

Answer 1

我更喜欢第一个。理性存在

1. 我可以通过传递组件列表轻松创建初始对象

product.setProductComponents((Component) myProduct.getProductComponents())

GetProductComponents我假设很好地返回列表。这样做是行不通的

更好的主意是欢快的getter和setter dumbest。优点是您可以在将来使用lombok这样的库时省略它们

并且始终可以添加实用程序方法

Answer 2

我建议

public List<Component> getProductComponents(){
    return this.productComponents;
}

public void setProductComponents(List<Component> productComponents){
    this.productComponents = productComponents;
}

另外为了方便起见：

public void addProductComponent(Component component) {
   this.productComponents.add(component);
}

public void removeProductComponent(Component component) {
   this.productComponents.remove(component);
}