Refactor: Fix checkstyle lint warnings

Author: sreeraj-keyvalue

src/main/java/com/ecommerce/order/OrderApplication.java

@@ -6,8 +6,8 @@
 @SpringBootApplication
 public class OrderApplication {
 
-	public static void main(String[] args) {
-		SpringApplication.run(OrderApplication.class, args);
-	}
+  public static void main(String[] args) {
+    SpringApplication.run(OrderApplication.class, args);
+  }
 
 }

src/main/java/com/ecommerce/order/controller/CustomerController.java

@@ -1,7 +1,12 @@
 package com.ecommerce.order.controller;
 
+import com.ecommerce.order.dto.CreateCustomerDto;
+import com.ecommerce.order.dto.UpdateCustomerDto;
+import com.ecommerce.order.exception.EntityNotFoundException;
+import com.ecommerce.order.model.Customer;
+import com.ecommerce.order.service.CustomerService;
+import jakarta.validation.Valid;
 import java.util.List;
-
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.web.bind.annotation.DeleteMapping;
 import org.springframework.web.bind.annotation.GetMapping;
@@ -12,43 +17,37 @@
 import org.springframework.web.bind.annotation.RequestMapping;
 import org.springframework.web.bind.annotation.RestController;
 
-import com.ecommerce.order.dto.CreateCustomerDto;
-import com.ecommerce.order.dto.UpdateCustomerDto;
-import com.ecommerce.order.exception.EntityNotFoundException;
-import com.ecommerce.order.model.Customer;
-import com.ecommerce.order.service.CustomerService;
-
-import jakarta.validation.Valid;
-
 @RestController
 @RequestMapping(path = "/customers")
 public class CustomerController {
-    @Autowired
-    private CustomerService customerService;
-
-    @GetMapping("/")
-    public List<Customer> findAll() {
-        return customerService.findAll();
-    }
-
-    @GetMapping("/{id}")
-    public Customer findOne(@PathVariable Integer id) throws EntityNotFoundException {
-        return customerService.findOne(id);
-    }
-
-    @PostMapping("/")
-    public Customer create(@Valid @RequestBody CreateCustomerDto customerDto) {
-        return customerService.create(customerDto);
-    }
-
-    @PutMapping("/{id}")
-    public Customer update(@PathVariable Integer id, @Valid @RequestBody UpdateCustomerDto customerDto)
-            throws EntityNotFoundException {
-        return customerService.update(id, customerDto);
-    }
-
-    @DeleteMapping("/{id}")
-    public void delete(@PathVariable Integer id) throws EntityNotFoundException {
-        customerService.delete(id);
-    }
+
+  @Autowired
+  private CustomerService customerService;
+
+  @GetMapping("/")
+  public List<Customer> findAll() {
+    return customerService.findAll();
+  }
+
+  @GetMapping("/{id}")
+  public Customer findOne(@PathVariable Integer id) throws EntityNotFoundException {
+    return customerService.findOne(id);
+  }
+
+  @PostMapping("/")
+  public Customer create(@Valid @RequestBody CreateCustomerDto customerDto) {
+    return customerService.create(customerDto);
+  }
+
+  @PutMapping("/{id}")
+  public Customer update(@PathVariable Integer id,
+      @Valid @RequestBody UpdateCustomerDto customerDto) throws EntityNotFoundException {
+    return customerService.update(id, customerDto);
+  }
+
+  @DeleteMapping("/{id}")
+  public void delete(@PathVariable Integer id) throws EntityNotFoundException {
+    customerService.delete(id);
+  }
+
 }

src/main/java/com/ecommerce/order/controller/HealthController.java

@@ -10,8 +10,8 @@
 @RequestMapping("/health")
 public class HealthController {
 
-    @GetMapping("/")
-    public ResponseEntity<String> healthCheck() {
-        return new ResponseEntity<String>("Order management service is up", HttpStatus.OK);
-    }
+  @GetMapping("/")
+  public ResponseEntity<String> healthCheck() {
+    return new ResponseEntity<String>("Order management service is up", HttpStatus.OK);
+  }
 }

src/main/java/com/ecommerce/order/controller/OrderController.java

@@ -1,5 +1,10 @@
 package com.ecommerce.order.controller;
 
+import com.ecommerce.order.dto.CreateOrderDto;
+import com.ecommerce.order.dto.UpdateOrderDto;
+import com.ecommerce.order.model.Order;
+import com.ecommerce.order.service.OrderService;
+import jakarta.validation.Valid;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.data.domain.Page;
 import org.springframework.web.bind.annotation.DeleteMapping;
@@ -11,42 +16,35 @@
 import org.springframework.web.bind.annotation.RequestMapping;
 import org.springframework.web.bind.annotation.RestController;
 
-import com.ecommerce.order.dto.CreateOrderDto;
-import com.ecommerce.order.dto.UpdateOrderDto;
-import com.ecommerce.order.model.Order;
-import com.ecommerce.order.service.OrderService;
-
-import jakarta.validation.Valid;
-
 @RestController
 @RequestMapping(path = "/orders")
 public class OrderController {
 
-    @Autowired
-    private OrderService orderService;
-
-    @GetMapping("/")
-    public Page<Order> findAll() {
-        return orderService.findAll(0, 5);
-    }
-
-    @GetMapping("/{id}")
-    public Order findOne(@PathVariable Integer id) {
-        return orderService.findOne(id);
-    }
-
-    @PostMapping("/")
-    public Order create(@Valid @RequestBody CreateOrderDto orderDto) {
-        return orderService.create(orderDto);
-    }
-
-    @PutMapping("/{id}")
-    public Order update(@PathVariable Integer id, @Valid @RequestBody UpdateOrderDto orderDto) {
-        return orderService.update(id, orderDto);
-    }
-
-    @DeleteMapping("/{id}")
-    public void delete(@PathVariable Integer id) {
-        orderService.delete(id);
-    }
+  @Autowired
+  private OrderService orderService;
+
+  @GetMapping("/")
+  public Page<Order> findAll() {
+    return orderService.findAll(0, 5);
+  }
+
+  @GetMapping("/{id}")
+  public Order findOne(@PathVariable Integer id) {
+    return orderService.findOne(id);
+  }
+
+  @PostMapping("/")
+  public Order create(@Valid @RequestBody CreateOrderDto orderDto) {
+    return orderService.create(orderDto);
+  }
+
+  @PutMapping("/{id}")
+  public Order update(@PathVariable Integer id, @Valid @RequestBody UpdateOrderDto orderDto) {
+    return orderService.update(id, orderDto);
+  }
+
+  @DeleteMapping("/{id}")
+  public void delete(@PathVariable Integer id) {
+    orderService.delete(id);
+  }
 }

src/main/java/com/ecommerce/order/dto/CreateCustomerDto.java

@@ -11,34 +11,35 @@
 @Setter
 public class CreateCustomerDto {
 
-    @NotEmpty
-    @NotNull
-    @Size(min = 2, message = "Name must have at least 2 characters")
-    private String name;
-
-    @NotEmpty
-    @NotNull
-    private String phone;
-
-    @NotEmpty
-    @NotNull
-    @Email
-    private String email;
-
-    @NotNull
-    @NotEmpty
-    @Size(min = 2, message = "Street must have at least 2 characters")
-    private String street;
-
-    @NotNull
-    @NotEmpty
-    @Size(min = 2, message = "City must have at least 2 characters")
-    private String city;
-
-    @NotNull
-    @NotEmpty
-    @Size(min = 2, message = "State must have at least 2 characters")
-    private String state;
-
-    private String profilePic;
+  @NotEmpty
+  @NotNull
+  @Size(min = 2, message = "Name must have at least 2 characters")
+  private String name;
+
+  @NotEmpty
+  @NotNull
+  private String phone;
+
+  @NotEmpty
+  @NotNull
+  @Email
+  private String email;
+
+  @NotNull
+  @NotEmpty
+  @Size(min = 2, message = "Street must have at least 2 characters")
+  private String street;
+
+  @NotNull
+  @NotEmpty
+  @Size(min = 2, message = "City must have at least 2 characters")
+  private String city;
+
+  @NotNull
+  @NotEmpty
+  @Size(min = 2, message = "State must have at least 2 characters")
+  private String state;
+
+  private String profilePic;
+
 }

src/main/java/com/ecommerce/order/dto/CreateOrderDto.java

@@ -1,10 +1,9 @@
 package com.ecommerce.order.dto;
 
-import java.util.List;
-
 import jakarta.validation.Valid;
 import jakarta.validation.constraints.NotEmpty;
 import jakarta.validation.constraints.NotNull;
+import java.util.List;
 import lombok.Builder;
 import lombok.Getter;
 import lombok.Setter;
@@ -13,10 +12,11 @@
 @Setter
 @Builder
 public class CreateOrderDto {
-    @NotNull
-    private Integer customerId;
+  @NotNull
+  private Integer customerId;
+
+  @NotEmpty
+  @Valid
+  private List<CreateOrderItemDto> orderItems;
 
-    @NotEmpty
-    @Valid
-    private List<CreateOrderItemDto> orderItems;
 }

src/main/java/com/ecommerce/order/dto/CreateOrderItemDto.java

@@ -11,10 +11,11 @@
 @Builder
 public class CreateOrderItemDto {
 
-    @NotNull
-    private Integer productId;
+  @NotNull
+  private Integer productId;
+
+  @NotNull
+  @Min(1)
+  private Integer quantity;
 
-    @NotNull
-    @Min(1)
-    private Integer quantity;
 }

src/main/java/com/ecommerce/order/dto/UpdateOrderDto.java

@@ -1,14 +1,15 @@
 package com.ecommerce.order.dto;
 
 import com.ecommerce.order.constants.OrderStatus;
-
 import jakarta.validation.constraints.NotNull;
 import lombok.Getter;
 import lombok.Setter;
 
 @Getter
 @Setter
 public class UpdateOrderDto {
-    @NotNull
-    private OrderStatus status;
+
+  @NotNull
+  private OrderStatus status;
+
 }

src/main/java/com/ecommerce/order/exception/CommonException.java

@@ -1,14 +1,15 @@
 package com.ecommerce.order.exception;
 
-import org.springframework.http.HttpStatusCode;
-
 import lombok.AllArgsConstructor;
 import lombok.Getter;
+import org.springframework.http.HttpStatusCode;
 
 @Getter
 @AllArgsConstructor
 public class CommonException extends RuntimeException {
-    private final HttpStatusCode statusCode;
-    private final String errorCode;
-    private final String message;
-}
+
+  private final HttpStatusCode statusCode;
+  private final String errorCode;
+  private final String message;
+
+}

src/main/java/com/ecommerce/order/exception/EntityNotFoundException.java

@@ -4,10 +4,11 @@
 
 public class EntityNotFoundException extends CommonException {
 
-    private static final long serialVersionUID = 1L;
+  private static final long serialVersionUID = 1L;
 
-    public <T> EntityNotFoundException(Class<T> entity) {
-        super(HttpStatus.NOT_FOUND, ExceptionCodes.ENTITY_NOT_FOUND, entity.getSimpleName() + " not found");
-    }
+  public <T> EntityNotFoundException(Class<T> entity) {
+    super(HttpStatus.NOT_FOUND, ExceptionCodes.ENTITY_NOT_FOUND,
+        entity.getSimpleName() + " not found");
+  }
 
 }