From 0cdf53896dbc6190c6b2a205f33bc539d852d1d9 Mon Sep 17 00:00:00 2001
From: Mark Alsip <mark.alsip@comcast.net>
Date: Thu, 12 Nov 2020 09:48:03 -0700
Subject: [PATCH] pushing up logger aspect

---
 .../java/com/revature/aop/AopController.java  | 54 +++++++++++++++++++
 .../com/revature/services/UserService.java    |  2 +-
 2 files changed, 55 insertions(+), 1 deletion(-)
 create mode 100644 src/main/java/com/revature/aop/AopController.java

diff --git a/src/main/java/com/revature/aop/AopController.java b/src/main/java/com/revature/aop/AopController.java
new file mode 100644
index 0000000..22132eb
--- /dev/null
+++ b/src/main/java/com/revature/aop/AopController.java
@@ -0,0 +1,54 @@
+package com.revature.aop;
+
+import org.aspectj.lang.JoinPoint;
+import org.aspectj.lang.annotation.AfterReturning;
+import org.aspectj.lang.annotation.AfterThrowing;
+import org.aspectj.lang.annotation.Aspect;
+import org.aspectj.lang.annotation.Before;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+import org.springframework.stereotype.Component;
+
+@Aspect
+@Component
+public class AopController {
+
+    Logger logger = LoggerFactory.getLogger(AopController.class);
+
+
+    @Before(value = "within(com.revature..*)")
+    public void logBefore(JoinPoint jp) {
+
+    	// logger.info(" Method "+jp.toShortString()+" called on "+jp.getTarget()+" args:
+    	// "+Arrays.toString(jp.getArgs()));
+    	
+    	
+    	System.out.println("Before: method" + jp.toShortString());
+    }
+
+    @AfterReturning(value = "within(com.revature..*)", returning = "result")
+    public void afterReturning(JoinPoint jp, Object result) {
+
+	  
+    	logger.info(" method " + jp.toShortString());
+    	// if(result!=null) {
+    	// logger.info(" method "+jp.toShortString()+"returned "+result.toString());
+    	// }else {
+    	// logger.info(" method "+jp.toShortString()+" returned "+" it returns null");
+    	// }
+
+    	
+    	System.out.println("AfterReturning: method" + jp.toShortString());
+    }
+
+    @AfterThrowing(value = "within(com.revature..*)", throwing = "error")
+    public void afterThrowing(JoinPoint jp, Throwable error) {
+
+    	logger.error(" method " + jp.toShortString() + " error " + error.getMessage());
+
+
+    	System.out.println("AfterThrowing: method" + jp.toShortString());
+    }
+
+
+}
diff --git a/src/main/java/com/revature/services/UserService.java b/src/main/java/com/revature/services/UserService.java
index f3d7170..3479f28 100644
--- a/src/main/java/com/revature/services/UserService.java
+++ b/src/main/java/com/revature/services/UserService.java
@@ -23,7 +23,7 @@ public List<User> getAllUser(){
 		List<User> users = userRepository.findAll();
 
 		if(users.size() != 0) {
-			System.out.print(users.size());
+			System.out.println(users.size());
 			return users;		
 		}