Feat: AOP args

Author: J-minkuk

src/main/java/com/springboot/aop/AopApplication.java

@@ -1,5 +1,6 @@
 package com.springboot.aop;
 
+import com.springboot.aop.aspect.Performance;
 import com.springboot.aop.domain.Board;
 import com.springboot.aop.domain.User;
 import com.springboot.aop.repository.BoardRepository;

src/main/java/com/springboot/aop/Performance.java src/main/java/com/springboot/aop/aspect/Performance.java

@@ -1,4 +1,4 @@
-package com.springboot.aop;
+package com.springboot.aop.aspect;
 
 import lombok.extern.slf4j.Slf4j;
 import org.aspectj.lang.ProceedingJoinPoint;

src/main/java/com/springboot/aop/aspect/UserHistory.java

@@ -0,0 +1,37 @@
+package com.springboot.aop.aspect;
+
+import com.springboot.aop.domain.History;
+import com.springboot.aop.domain.User;
+import com.springboot.aop.repository.HistoryRepository;
+import org.aspectj.lang.annotation.AfterReturning;
+import org.aspectj.lang.annotation.Aspect;
+import org.aspectj.lang.annotation.Pointcut;
+import org.springframework.beans.factory.annotation.Autowired;
+import org.springframework.stereotype.Component;
+
+@Aspect
+@Component
+public class UserHistory {
+
+  @Autowired
+  private HistoryRepository historyRepository;
+
+  /**
+   * args(user) 표현식을 통해 타겟 메소드의 인자와 어드바이스의 인자가 매칭된다.
+   * updateUser 라는 포인트컷이 user 라는 인자를 사용하도록 args(user) 표현식으로 지정한 것이다.
+   *
+   * @param user
+   */
+  @Pointcut("execution(* com.springboot.aop.service.UserService.update(*)) && args(user)")
+  public void updateUser(User user) {}
+
+  /**
+   * '@AfterReturning' 을 통해 정상적으로 타겟 메소드가 실행 된 후, DB 에 저장되도록 함
+   * @param user
+   */
+  @AfterReturning("updateUser(user)")
+  public void saveHistory(User user) {
+    historyRepository.save(new History(user.getIdx()));
+  }
+
+}

src/main/java/com/springboot/aop/domain/History.java

@@ -0,0 +1,38 @@
+package com.springboot.aop.domain;
+
+import lombok.Getter;
+import lombok.Setter;
+
+import javax.persistence.Column;
+import javax.persistence.Entity;
+import javax.persistence.GeneratedValue;
+import javax.persistence.Id;
+import java.util.Date;
+
+@Getter
+@Setter
+@Entity
+public class History {
+
+  @Id
+  @GeneratedValue
+  private long idx;
+
+  @Column
+  private long userIdx;
+
+  @Column
+  private Date updateDate;
+
+  public History() {}
+
+  public History(long userIdx) {
+    this.userIdx = userIdx;
+  }
+
+  public History(long userIdx, Date updateDate) {
+    this.userIdx =  userIdx;
+    this.updateDate = updateDate;
+  }
+
+}

src/main/java/com/springboot/aop/repository/HistoryRepository.java

@@ -0,0 +1,7 @@
+package com.springboot.aop.repository;
+
+import com.springboot.aop.domain.History;
+import org.springframework.data.jpa.repository.JpaRepository;
+
+public interface HistoryRepository extends JpaRepository<History, Long> {
+}

src/main/java/com/springboot/aop/service/UserService.java

@@ -8,4 +8,6 @@ public interface UserService {
 
   List<User> getUsers();
 
+  void update(User user) throws Exception;
+
 }

src/main/java/com/springboot/aop/service/impl/UserServiceImpl.java

@@ -19,4 +19,9 @@ public List<User> getUsers() {
     return users;
   }
 
+  @Override
+  public void update(User user) throws Exception {
+    userRepository.save(user);
+  }
+
 }

src/test/java/com/springboot/aop/aop/BoardServiceTest.java src/test/java/com/springboot/aop/aop/ServiceTest.java

@@ -1,5 +1,8 @@
 package com.springboot.aop.aop;
 
+import com.springboot.aop.domain.History;
+import com.springboot.aop.domain.User;
+import com.springboot.aop.repository.HistoryRepository;
 import com.springboot.aop.service.BoardService;
 import com.springboot.aop.service.UserService;
 import org.junit.Assert;
@@ -9,14 +12,18 @@
 import org.springframework.boot.test.context.SpringBootTest;
 import org.springframework.test.context.junit4.SpringRunner;
 
+import java.util.List;
+
 @RunWith(SpringRunner.class)
 @SpringBootTest
-public class BoardServiceTest {
+public class ServiceTest {
 
   @Autowired
   private BoardService boardService;
   @Autowired
   private UserService userService;
+  @Autowired
+  private HistoryRepository historyRepository;
 
   @Test
   public void findBoards() throws Exception {
@@ -28,4 +35,17 @@ public void findUsers() throws Exception {
     Assert.assertEquals(userService.getUsers().size(), 100);
   }
 
+  @Test
+  public void updateUsers() throws Exception {
+    List<User> users = userService.getUsers();
+    for (int i = 0; i < 5; ++i) {
+      User user = users.get(i);
+      user.setEmail("[email protected]");
+      userService.update(user);
+    }
+
+    List<History> histories = historyRepository.findAll();
+    Assert.assertEquals(histories.size(), 5);
+  }
+
 }