Refactor dependency injection and add SLF4J for logging

Revised the DI lifecycle by introducing an initializer and renaming `startDI` to `startDependencyInjection`. Added SLF4J JDK14 for improved logging capabilities and updated related configurations in build scripts. This enhances code readability and maintainability while enabling better logging practices.

Author: Newspicel

build.gradle.kts

@@ -25,6 +25,11 @@ dependencies {
     implementation(libs.guice)
     implementation(libs.reflections)
 
+    // Logging
+    implementation(libs.slf4j.jdk14)
+
+
+    // Tests
     testImplementation(libs.kotlin.test)
     testImplementation(libs.kotlin.test.junit5)
     testImplementation(libs.kotlinx.coroutines.test)

gradle/libs.versions.toml

@@ -15,6 +15,7 @@ mockbukkit = { group = "org.mockbukkit.mockbukkit", name = "mockbukkit-v1.21", v
 reflections = { group = "org.reflections", name = "reflections", version = "0.10.2" }
 guice = { group = "com.google.inject", name = "guice", version = "7.0.0" }
 kotest = { group = "io.kotest", name = "kotest-assertions-core-jvm", version = "5.9.1" }
+slf4j-jdk14 = { group = "org.slf4j", name = "slf4j-jdk14" , version = "2.0.16" }
 
 [plugins]
 kotlin-jvm = { id = "org.jetbrains.kotlin.jvm", version.ref = "kotlin" }

src/main/kotlin/dev/newspicel/di/DIJavaPlugin.kt

@@ -14,41 +14,43 @@ import org.bukkit.command.CommandSender
 import org.bukkit.plugin.java.JavaPlugin
 import org.reflections.Reflections
 import org.reflections.util.ConfigurationBuilder
+import java.util.logging.Level
 import kotlin.reflect.KClass
 import kotlin.reflect.KParameter
 import kotlin.reflect.full.findAnnotations
 import kotlin.reflect.full.functions
 
 abstract class DIJavaPlugin : JavaPlugin() {
 
-    private val stereotypes = arrayOf(MinecraftController::class, Service::class, Repository::class)
+    private val stereotypes = listOf(MinecraftController::class, Service::class, Repository::class)
 
-    protected lateinit var stereotypesClasses: List<KClass<*>>
-    private lateinit var controllerClasses: List<KClass<*>>
+    protected val stereotypesClasses: List<KClass<*>>
+    private val controllerClasses: List<KClass<*>>
 
-    open val projectPackagePath: String
+    private val projectPackagePath: String
         get() = this.javaClass.`package`.name
 
-    abstract fun startDI()
+    init {
+        stereotypesClasses = scanForMinecraftStereotypes()
+        controllerClasses = stereotypesClasses.filter { it.findAnnotations(MinecraftController::class).isNotEmpty() }
+        logger.log(Level.FINE, "Found ${stereotypesClasses.size} Minecraft Stereotypes in $projectPackagePath")
+        startDependencyInjection()
+    }
+
+    abstract fun startDependencyInjection()
     abstract fun <T : Any> getExistingBinding(type: KClass<T>, qualifier: String? = null): T?
     abstract fun <T : Any> getInstance(type: KClass<T>, qualifier: String? = null): T?
 
     abstract fun getQualifier(annotation: List<Annotation>): String?
 
     override fun onLoad() {
-        logger.info("Scanning for Minecraft controllers in $projectPackagePath")
-        stereotypesClasses = scanForMinecraftStereotypes()
-        controllerClasses = stereotypesClasses.filter { it.findAnnotations(MinecraftController::class).isNotEmpty() }
-        logger.info("Found ${stereotypesClasses.size} Minecraft Stereotypes in $projectPackagePath")
-        startDI()
         scanForMinecraftAnnotationsInClassesOnLoad()
-
         ServerEventsHelper.triggerOnLoad(this)
     }
 
     final override fun onEnable() {
-        ServerEventsHelper.triggerOnEnable(this)
         scanForMinecraftAnnotationsInClassesOnEnable()
+        ServerEventsHelper.triggerOnEnable(this)
     }
 
     final override fun onDisable() {
@@ -78,43 +80,43 @@ abstract class DIJavaPlugin : JavaPlugin() {
             ?: error("Could not find a value for parameter ${parameter.name} of type ${parameter.type}")
     }
 
-    private fun scanForMinecraftAnnotationsInClassesOnEnable() {
+    private fun scanForMinecraftAnnotationsInClassesOnLoad() {
         controllerClasses.forEach { clazz ->
             clazz.functions.forEach { func ->
                 func.annotations.forEach { annotation ->
                     when (annotation) {
-                        is Command -> {
-                            CommandsHelper.registerCommand(this, annotation, func)
+                        is OnEnable -> {
+                            ServerEventsHelper.registerOnEnable(func)
                         }
 
-                        is TabComplete -> {
-                            CommandsHelper.registerTabComplete(this, annotation, func)
+                        is OnDisable -> {
+                            ServerEventsHelper.registerOnDisable(func)
                         }
 
-                        is Event -> {
-                            EventHelper.registerEvent(this, annotation, func)
+                        is OnLoad -> {
+                            ServerEventsHelper.registerOnLoad(func)
                         }
                     }
                 }
             }
         }
     }
 
-    private fun scanForMinecraftAnnotationsInClassesOnLoad() {
+    private fun scanForMinecraftAnnotationsInClassesOnEnable() {
         controllerClasses.forEach { clazz ->
             clazz.functions.forEach { func ->
                 func.annotations.forEach { annotation ->
                     when (annotation) {
-                        is OnEnable -> {
-                            ServerEventsHelper.registerOnEnable(func)
+                        is Command -> {
+                            CommandsHelper.registerCommand(this, annotation, func)
                         }
 
-                        is OnDisable -> {
-                            ServerEventsHelper.registerOnDisable(func)
+                        is TabComplete -> {
+                            CommandsHelper.registerTabComplete(this, annotation, func)
                         }
 
-                        is OnLoad -> {
-                            ServerEventsHelper.registerOnLoad(func)
+                        is Event -> {
+                            EventHelper.registerEvent(this, annotation, func)
                         }
                     }
                 }

src/main/kotlin/dev/newspicel/di/GuiceJavaPlugin.kt

@@ -15,8 +15,7 @@ abstract class GuiceJavaPlugin : DIJavaPlugin() {
 
     private lateinit var injector: Injector
 
-    override fun startDI() {
-        logger.info("Starting Guice injector")
+    override fun startDependencyInjection() {
         val module = MinecraftGuiceModule(this, stereotypesClasses, classLoader)
 
         try {