spring-projects#31 Use @slf4j annotation from Lomnok

JonasKunz · Jan 8, 2017 · 8b366e7 · 8b366e7
1 parent 184a7b8
commit 8b366e7
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 29 deletions.
diff --git a/...vice/src/main/java/org/springframework/samples/petclinic/customers/web/OwnerResource.java b/...vice/src/main/java/org/springframework/samples/petclinic/customers/web/OwnerResource.java
@@ -16,26 +16,16 @@
 package org.springframework.samples.petclinic.customers.web;
 
 import lombok.RequiredArgsConstructor;
-
-import java.util.List;
-
-import javax.validation.Valid;
-
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
+import lombok.extern.slf4j.Slf4j;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.http.HttpStatus;
 import org.springframework.samples.petclinic.customers.model.Owner;
 import org.springframework.samples.petclinic.customers.model.OwnerRepository;
 import org.springframework.samples.petclinic.monitoring.Monitored;
-import org.springframework.web.bind.annotation.GetMapping;
-import org.springframework.web.bind.annotation.PathVariable;
-import org.springframework.web.bind.annotation.PostMapping;
-import org.springframework.web.bind.annotation.PutMapping;
-import org.springframework.web.bind.annotation.RequestBody;
-import org.springframework.web.bind.annotation.RequestMapping;
-import org.springframework.web.bind.annotation.ResponseStatus;
-import org.springframework.web.bind.annotation.RestController;
+import org.springframework.web.bind.annotation.*;
+
+import javax.validation.Valid;
+import java.util.List;
 
 /**
  * @author Juergen Hoeller
@@ -47,10 +37,9 @@
 @RequestMapping("/owners")
 @RestController
 @RequiredArgsConstructor(onConstructor = @__(@Autowired))
+@Slf4j
 class OwnerResource {
 
-    private static final Logger LOG = LoggerFactory.getLogger(OwnerResource.class);
-
     private final OwnerRepository ownerRepository;
 
     /**
@@ -92,7 +81,7 @@ public Owner updateOwner(@PathVariable("ownerId") int ownerId, @Valid @RequestBo
         ownerModel.setCity(ownerRequest.getCity());
         ownerModel.setAddress(ownerRequest.getAddress());
         ownerModel.setTelephone(ownerRequest.getTelephone());
-        LOG.info("Saving owner {}", ownerModel);
+        log.info("Saving owner {}", ownerModel);
         return ownerRepository.save(ownerModel);
     }
 }
diff --git a/...ervice/src/main/java/org/springframework/samples/petclinic/customers/web/PetResource.java b/...ervice/src/main/java/org/springframework/samples/petclinic/customers/web/PetResource.java
@@ -16,8 +16,7 @@
 package org.springframework.samples.petclinic.customers.web;
 
 import lombok.RequiredArgsConstructor;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
+import lombok.extern.slf4j.Slf4j;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.http.HttpStatus;
 import org.springframework.samples.petclinic.customers.model.*;
@@ -34,10 +33,9 @@
  */
 @RestController
 @RequiredArgsConstructor(onConstructor = @__(@Autowired))
+@Slf4j
 class PetResource {
 
-    private static final Logger LOG = LoggerFactory.getLogger(PetResource.class);
-
     private final PetRepository petRepository;
 
     private final OwnerRepository ownerRepository;
@@ -76,7 +74,7 @@ private void save(final Pet pet, final PetRequest petRequest) {
         petRepository.findPetTypeById(petRequest.getTypeId())
             .ifPresent(pet::setType);
 
-        LOG.info("Saving pet {}", pet);
+        log.info("Saving pet {}", pet);
         petRepository.save(pet);
     }
 

diff --git a/...service/src/main/java/org/springframework/samples/petclinic/visits/web/VisitResource.java b/...service/src/main/java/org/springframework/samples/petclinic/visits/web/VisitResource.java
@@ -16,8 +16,7 @@
 package org.springframework.samples.petclinic.visits.web;
 
 import lombok.RequiredArgsConstructor;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
+import lombok.extern.slf4j.Slf4j;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.http.HttpStatus;
 import org.springframework.samples.petclinic.visits.model.Visit;
@@ -36,11 +35,9 @@
  */
 @RestController
 @RequiredArgsConstructor(onConstructor = @__(@Autowired))
+@Slf4j
 public class VisitResource {
 
-    private static final Logger LOG = LoggerFactory.getLogger(VisitResource.class);
-
-
     private final VisitRepository visitRepository;
 
     @PostMapping("owners/*/pets/{petId}/visits")
@@ -50,7 +47,7 @@ public void create(
         @PathVariable("petId") int petId) {
 
         visit.setPetId(petId);
-        LOG.info("Saving visit {}", visit);
+        log.info("Saving visit {}", visit);
         visitRepository.save(visit);
     }