Refactor item model to remove duplicate code

src/main/java/bg/softuni/invoice/model/bind/ItemAddBindingModel.java

@@ -3,6 +3,8 @@
 import jakarta.validation.constraints.NotBlank;
 import jakarta.validation.constraints.NotNull;
 import jakarta.validation.constraints.Positive;
+import lombok.Getter;
+import lombok.Setter;
 import org.hibernate.validator.constraints.Length;
 import org.springframework.web.multipart.MultipartFile;
 
@@ -12,6 +14,8 @@
 import static bg.softuni.invoice.constant.ErrorMsg.PRICE_POSITIVE;
 import static bg.softuni.invoice.constant.ErrorMsg.STRING_MIN_LENGTH;
 
+@Setter
+@Getter
 public class ItemAddBindingModel {
 
     @NotBlank(message = NAME_MIN_LENGTH)
@@ -27,35 +31,4 @@ public class ItemAddBindingModel {
 
     private MultipartFile imageUrl;
 
-    public String getName() {
-        return name;
-    }
-
-    public void setName(String name) {
-        this.name = name;
-    }
-
-    public BigDecimal getPrice() {
-        return price;
-    }
-
-    public void setPrice(BigDecimal price) {
-        this.price = price;
-    }
-
-    public MultipartFile getImageUrl() {
-        return imageUrl;
-    }
-
-    public void setImageUrl(MultipartFile imageUrl) {
-        this.imageUrl = imageUrl;
-    }
-
-    public String getVatValue() {
-        return vatValue;
-    }
-
-    public void setVatValue(String vatValue) {
-        this.vatValue = vatValue;
-    }
 }

src/main/java/bg/softuni/invoice/model/bind/ItemEditBindingModel.java

@@ -3,6 +3,8 @@
 import jakarta.validation.constraints.NotBlank;
 import jakarta.validation.constraints.NotNull;
 import jakarta.validation.constraints.Positive;
+import lombok.Getter;
+import lombok.Setter;
 import org.hibernate.validator.constraints.Length;
 import org.springframework.web.multipart.MultipartFile;
 
@@ -13,6 +15,8 @@
 import static bg.softuni.invoice.constant.ErrorMsg.PRICE_POSITIVE;
 import static bg.softuni.invoice.constant.ErrorMsg.STRING_MIN_LENGTH;
 
+@Setter
+@Getter
 public class ItemEditBindingModel {
 
     private String id;
@@ -33,51 +37,4 @@ public class ItemEditBindingModel {
 
     private MultipartFile newImageUrl;
 
-    public String getId() {
-        return id;
-    }
-
-    public void setId(String id) {
-        this.id = id;
-    }
-
-    public String getName() {
-        return name;
-    }
-
-    public void setName(String name) {
-        this.name = name;
-    }
-
-    public BigDecimal getPrice() {
-        return price;
-    }
-
-    public void setPrice(BigDecimal price) {
-        this.price = price;
-    }
-
-    public String getVatValue() {
-        return vatValue;
-    }
-
-    public void setVatValue(String vatValue) {
-        this.vatValue = vatValue;
-    }
-
-    public String getImageUrl() {
-        return imageUrl;
-    }
-
-    public void setImageUrl(String imageUrl) {
-        this.imageUrl = imageUrl;
-    }
-
-    public MultipartFile getNewImageUrl() {
-        return newImageUrl;
-    }
-
-    public void setNewImageUrl(MultipartFile newImageUrl) {
-        this.newImageUrl = newImageUrl;
-    }
 }

src/main/java/bg/softuni/invoice/model/service/ItemServiceModel.java

@@ -1,9 +1,13 @@
 package bg.softuni.invoice.model.service;
 
 import bg.softuni.invoice.model.enumerated.VatValue;
+import lombok.Getter;
+import lombok.Setter;
 
 import java.math.BigDecimal;
 
+@Setter
+@Getter
 public class ItemServiceModel {
 
     private String id;
@@ -13,51 +17,4 @@ public class ItemServiceModel {
     private String imageUrl;
     private int quantity;
 
-    public String getId() {
-        return id;
-    }
-
-    public void setId(String id) {
-        this.id = id;
-    }
-
-    public String getName() {
-        return name;
-    }
-
-    public void setName(String name) {
-        this.name = name;
-    }
-
-    public BigDecimal getPrice() {
-        return price;
-    }
-
-    public void setPrice(BigDecimal price) {
-        this.price = price;
-    }
-
-    public VatValue getVatValue() {
-        return vatValue;
-    }
-
-    public void setVatValue(VatValue vatValue) {
-        this.vatValue = vatValue;
-    }
-
-    public String getImageUrl() {
-        return imageUrl;
-    }
-
-    public void setImageUrl(String imageUrl) {
-        this.imageUrl = imageUrl;
-    }
-
-    public int getQuantity() {
-        return quantity;
-    }
-
-    public void setQuantity(int quantity) {
-        this.quantity = quantity;
-    }
 }

src/main/java/bg/softuni/invoice/model/view/ItemViewModel.java

@@ -1,7 +1,12 @@
 package bg.softuni.invoice.model.view;
 
+import lombok.Getter;
+import lombok.Setter;
+
 import java.math.BigDecimal;
 
+@Setter
+@Getter
 public class ItemViewModel {
 
     private String id;
@@ -11,51 +16,4 @@ public class ItemViewModel {
     private String imageUrl;
     private int quantity;
 
-    public String getId() {
-        return id;
-    }
-
-    public void setId(String id) {
-        this.id = id;
-    }
-
-    public String getName() {
-        return name;
-    }
-
-    public void setName(String name) {
-        this.name = name;
-    }
-
-    public BigDecimal getPrice() {
-        return price;
-    }
-
-    public void setPrice(BigDecimal price) {
-        this.price = price;
-    }
-
-    public String getVatValue() {
-        return vatValue;
-    }
-
-    public void setVatValue(String vatValue) {
-        this.vatValue = vatValue;
-    }
-
-    public String getImageUrl() {
-        return imageUrl;
-    }
-
-    public void setImageUrl(String imageUrl) {
-        this.imageUrl = imageUrl;
-    }
-
-    public int getQuantity() {
-        return quantity;
-    }
-
-    public void setQuantity(int quantity) {
-        this.quantity = quantity;
-    }
 }

src/main/java/bg/softuni/invoice/service/impl/ItemServiceImpl.java

@@ -5,26 +5,21 @@
 import bg.softuni.invoice.model.service.ItemServiceModel;
 import bg.softuni.invoice.repository.ItemRepository;
 import bg.softuni.invoice.service.ItemService;
+import lombok.RequiredArgsConstructor;
 import org.modelmapper.ModelMapper;
-import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.stereotype.Service;
 
 import java.util.List;
 
 import static bg.softuni.invoice.constant.ErrorMsg.ITEM_NOT_FOUND;
 
 @Service
+@RequiredArgsConstructor
 public class ItemServiceImpl implements ItemService {
 
     private final ItemRepository itemRepository;
     private final ModelMapper modelMapper;
 
-    @Autowired
-    public ItemServiceImpl(ItemRepository itemRepository, ModelMapper modelMapper) {
-        this.itemRepository = itemRepository;
-        this.modelMapper = modelMapper;
-    }
-
     @Override
     public List<ItemServiceModel> getAllItems() {
         return this.itemRepository

src/main/java/bg/softuni/invoice/web/controller/ItemController.java

@@ -8,13 +8,18 @@
 import bg.softuni.invoice.service.ItemService;
 import bg.softuni.invoice.web.annotation.PageTitle;
 import jakarta.validation.Valid;
+import lombok.RequiredArgsConstructor;
 import org.modelmapper.ModelMapper;
-import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.security.access.prepost.PreAuthorize;
 import org.springframework.stereotype.Controller;
 import org.springframework.ui.Model;
 import org.springframework.validation.BindingResult;
-import org.springframework.web.bind.annotation.*;
+import org.springframework.web.bind.annotation.GetMapping;
+import org.springframework.web.bind.annotation.ModelAttribute;
+import org.springframework.web.bind.annotation.PathVariable;
+import org.springframework.web.bind.annotation.PostMapping;
+import org.springframework.web.bind.annotation.RequestMapping;
+import org.springframework.web.bind.annotation.RequestParam;
 import org.springframework.web.multipart.MultipartFile;
 import org.springframework.web.servlet.mvc.support.RedirectAttributes;
 
@@ -26,6 +31,7 @@
 
 @Controller
 @RequestMapping("/item")
+@RequiredArgsConstructor
 public class ItemController {
 
     private static final String ITEM_ADD_BINDING_MODEL = "itemAddBindingModel";
@@ -34,13 +40,6 @@ public class ItemController {
     private final ModelMapper modelMapper;
     private final CloudinaryService cloudinaryService;
 
-    @Autowired
-    public ItemController(ItemService itemService, ModelMapper modelMapper, CloudinaryService cloudinaryService) {
-        this.itemService = itemService;
-        this.modelMapper = modelMapper;
-        this.cloudinaryService = cloudinaryService;
-    }
-
     @GetMapping("/add")
     @PageTitle("Item add")
     @PreAuthorize("hasRole('ADMIN')")