added Logging for InteractionChecks

lib/src/main/java/net/tomatentum/marinara/Marinara.java

@@ -1,11 +1,16 @@
 package net.tomatentum.marinara;
 
+import org.apache.logging.log4j.Logger;
+
 import net.tomatentum.marinara.registry.InteractionCheckRegistry;
 import net.tomatentum.marinara.registry.InteractionRegistry;
+import net.tomatentum.marinara.util.LoggerUtil;
 import net.tomatentum.marinara.wrapper.LibraryWrapper;
 
 public class Marinara {
 
+    private Logger logger = LoggerUtil.getLogger(getClass());
+    
     public static <T extends LibraryWrapper> Marinara load(LibraryWrapper wrapper) {
         return new Marinara(wrapper);
     }
@@ -18,6 +23,7 @@ public class Marinara {
         this.wrapper = wrapper;
         this.registry = new InteractionRegistry(this);
         this.checkRegistry = new InteractionCheckRegistry();
+        logger.info("Marinara loaded successfully!");
     }
 
     public InteractionRegistry getRegistry() {

lib/src/main/java/net/tomatentum/marinara/checks/AppliedCheck.java

@@ -5,10 +5,15 @@ import java.lang.reflect.InvocationTargetException;
 import java.lang.reflect.Method;
 import java.util.Arrays;
 
+import org.apache.logging.log4j.Logger;
+
+import net.tomatentum.marinara.util.LoggerUtil;
 import net.tomatentum.marinara.util.ReflectionUtil;
 
 public record AppliedCheck(InteractionCheck<?> check, Annotation annotation) {
 
+    private static Logger logger = LoggerUtil.getLogger(AppliedCheck.class);
+    
     public boolean pre(Object context) {
         Method[] methods = Arrays.stream(check.getClass().getMethods())
             .filter(x -> x.getName().equals("preExec"))
@@ -17,9 +22,10 @@ public record AppliedCheck(InteractionCheck<?> check, Annotation annotation) {
         Method method = ReflectionUtil.getMostSpecificMethod(methods, context.getClass(), annotation.annotationType());
         method.setAccessible(true);
         try {
+            logger.debug("Executing pre check {} with context {}", check.getClass().getName(), context.toString());
             return (boolean) method.invoke(check, context, annotation);
         } catch (IllegalAccessException | InvocationTargetException | SecurityException e) {
-            e.printStackTrace();
+            logger.fatal(e);
             return false;
         }
     }
@@ -32,9 +38,10 @@ public record AppliedCheck(InteractionCheck<?> check, Annotation annotation) {
         Method method = ReflectionUtil.getMostSpecificMethod(methods, context.getClass(), annotation.annotationType());
         method.setAccessible(true);
         try {
+            logger.debug("Executing pre check {} with context {}", check.getClass().getName(), context.toString());
             method.invoke(check, context, annotation);
         } catch (IllegalAccessException | InvocationTargetException | SecurityException e) {
-            e.printStackTrace();
+            logger.fatal(e);
         }
     }
 

lib/src/main/java/net/tomatentum/marinara/interaction/methods/InteractionMethod.java

@@ -7,6 +7,8 @@ import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.List;
 
+import org.apache.logging.log4j.Logger;
+
 import net.tomatentum.marinara.Marinara;
 import net.tomatentum.marinara.checks.AppliedCheck;
 import net.tomatentum.marinara.interaction.InteractionHandler;
@@ -16,6 +18,8 @@ import net.tomatentum.marinara.interaction.commands.annotation.SlashCommand;
 import net.tomatentum.marinara.interaction.commands.annotation.SubCommand;
 import net.tomatentum.marinara.parser.AnnotationParser;
 import net.tomatentum.marinara.parser.InteractionCheckParser;
+import net.tomatentum.marinara.util.LoggerUtil;
+import net.tomatentum.marinara.util.ReflectionUtil;
 
 public abstract class InteractionMethod {
 
@@ -33,6 +37,8 @@ public abstract class InteractionMethod {
     protected List<AnnotationParser> parsers;
     protected List<AppliedCheck> appliedChecks;
 
+    private Logger logger = LoggerUtil.getLogger(getClass());
+
     protected InteractionMethod(Method method, 
         InteractionHandler handler, 
         Marinara marinara
@@ -82,11 +88,14 @@ public abstract class InteractionMethod {
         List<Object> parameters = new ArrayList<>();
         
         for (int i = 0; i < parameterCount; i++) {
+            Object parameter;
             if (i == 0) {
-                parameters.add(context);
+                parameter = context;
-                continue;
+            }else
-            }
+                parameter = getParameter(context, i-1);
-            parameters.add(getParameter(context, i-1));
+
+            logger.trace("Found parameter {}={} for method {}", parameter.getClass().toString(), parameter, ReflectionUtil.getFullMethodName(method));
+            parameters.add(parameter);   
         }
         return parameters.toArray();
     }

lib/src/main/java/net/tomatentum/marinara/parser/ButtonParser.java

@@ -3,13 +3,19 @@ package net.tomatentum.marinara.parser;
 import java.lang.reflect.Method;
 import java.util.function.Consumer;
 
+import org.apache.logging.log4j.Logger;
+
 import net.tomatentum.marinara.interaction.annotation.Button;
+import net.tomatentum.marinara.util.LoggerUtil;
+import net.tomatentum.marinara.util.ReflectionUtil;
 
 public class ButtonParser implements AnnotationParser {
     
     private Method method;
     private Consumer<String> consumer;
 
+    private Logger logger = LoggerUtil.getLogger(getClass());
+
     public ButtonParser(Method method, Consumer<String> consumer) {
         this.method = method;
         this.consumer = consumer;
@@ -18,6 +24,7 @@ public class ButtonParser implements AnnotationParser {
     @Override
     public void parse() {
         Button button = getMethod().getAnnotation(Button.class);
+        logger.trace("Parsed Button annotation {} for method {}", button.toString(), ReflectionUtil.getFullMethodName(method));
         this.consumer.accept(button.value());
     }
 

lib/src/main/java/net/tomatentum/marinara/parser/InteractionCheckParser.java

@@ -6,9 +6,13 @@ import java.util.Arrays;
 import java.util.Optional;
 import java.util.function.Consumer;
 
+import org.apache.logging.log4j.Logger;
+
 import net.tomatentum.marinara.checks.AppliedCheck;
 import net.tomatentum.marinara.checks.InteractionCheck;
 import net.tomatentum.marinara.registry.InteractionCheckRegistry;
+import net.tomatentum.marinara.util.LoggerUtil;
+import net.tomatentum.marinara.util.ReflectionUtil;
 
 public class InteractionCheckParser implements AnnotationParser {
 
@@ -16,6 +20,8 @@ public class InteractionCheckParser implements AnnotationParser {
     private Method method;
     private Consumer<AppliedCheck> consumer;
 
+    private Logger logger = LoggerUtil.getLogger(getClass());
+
     public InteractionCheckParser(Method method, Consumer<AppliedCheck> consumer, InteractionCheckRegistry checkRegistry) {
         this.checkRegistry = checkRegistry;
         this.method = method;
@@ -30,8 +36,11 @@ public class InteractionCheckParser implements AnnotationParser {
 
     private void convertAnnotation(Annotation annotation) {
             Optional<InteractionCheck<?>> check = this.checkRegistry.getCheckFromAnnotation(annotation.annotationType());
-            if (check.isPresent())
+            if (check.isPresent())  {
-                consumer.accept(new AppliedCheck(check.get(), annotation));
+                AppliedCheck appliedCheck = new AppliedCheck(check.get(), annotation);
+                logger.trace("Parsed InteractionCheck {} for annotation {} for method {}", check.getClass().getName(), annotation.toString(), ReflectionUtil.getFullMethodName(method));
+                consumer.accept(appliedCheck);
+            }
     }
 
     @Override

lib/src/main/java/net/tomatentum/marinara/parser/SlashCommandParser.java

@@ -3,10 +3,13 @@ package net.tomatentum.marinara.parser;
 import java.lang.reflect.Method;
 import java.util.function.Consumer;
 
+import org.apache.logging.log4j.Logger;
+
 import net.tomatentum.marinara.interaction.commands.ExecutableSlashCommandDefinition;
 import net.tomatentum.marinara.interaction.commands.annotation.SlashCommand;
 import net.tomatentum.marinara.interaction.commands.annotation.SubCommand;
 import net.tomatentum.marinara.interaction.commands.annotation.SubCommandGroup;
+import net.tomatentum.marinara.util.LoggerUtil;
 import net.tomatentum.marinara.util.ReflectionUtil;
 
 public class SlashCommandParser implements AnnotationParser {
@@ -14,6 +17,8 @@ public class SlashCommandParser implements AnnotationParser {
     private Method method;
     private Consumer<ExecutableSlashCommandDefinition> consumer;
 
+    private Logger logger = LoggerUtil.getLogger(getClass());
+
     public SlashCommandParser(Method method, Consumer<ExecutableSlashCommandDefinition> consumer) {
         this.method = method;
         this.consumer = consumer;
@@ -37,6 +42,9 @@ public class SlashCommandParser implements AnnotationParser {
             builder.setSubCommand(subCmd);
         }
 
+        ExecutableSlashCommandDefinition def = builder.build();
+
+        logger.trace("Parsed using SlashCommandParser for method {} with the result:\n{}", ReflectionUtil.getFullMethodName(method), def.toString());
         consumer.accept(builder.build());
     }
 

lib/src/main/java/net/tomatentum/marinara/registry/InteractionCheckRegistry.java

@@ -6,19 +6,25 @@ import java.util.ArrayList;
 import java.util.List;
 import java.util.Optional;
 
+import org.apache.logging.log4j.Logger;
+
 import io.leangen.geantyref.GenericTypeReflector;
 import net.tomatentum.marinara.checks.InteractionCheck;
+import net.tomatentum.marinara.util.LoggerUtil;
 
 public class InteractionCheckRegistry {
     
     private List<InteractionCheck<?>> checks;
 
+    private Logger logger = LoggerUtil.getLogger(getClass());
+
     public InteractionCheckRegistry() {
         this.checks = new ArrayList<>();
     }
 
     public void addCheck(InteractionCheck<?> check) {
         checks.add(check);
+        logger.info("Registered Check {}", check.getClass().getName());
     }
 
     public Optional<InteractionCheck<?>> getCheckFromAnnotation(Type annotation) {

lib/src/main/java/net/tomatentum/marinara/util/ReflectionUtil.java

@@ -100,4 +100,8 @@ public final class ReflectionUtil {
         }
         return true;
     }
+
+    public static String getFullMethodName(Method method) {
+        return method.getClass().getName() + "." + method.getName();
+    }
 }