LinkChecker: Use logging instead of direct printing into console

Author: valery1707

build.gradle.kts

@@ -27,6 +27,9 @@ repositories {
 dependencies {
   implementation(libs.jackson.databind)
   implementation(libs.mockneat)
+  implementation(libs.bundles.logger.api)
+
+  runtimeOnly(libs.logger.impl)
 
   testImplementation(kotlin("test"))
   testImplementation(platform(libs.junit))

gradle/libs.versions.toml

@@ -11,8 +11,12 @@ junit = { module = "org.junit:junit-bom", version.ref = "junit" }
 assertj = { module = "org.assertj:assertj-core", version = "3.23.1" }
 mockneat = { module = "net.andreinc:mockneat", version = "0.4.8" }
 jmh-ann = { module = "org.openjdk.jmh:jmh-generator-annprocess", version.ref = "jmh.tools" }
+logger-api4j = { module = "org.slf4j:slf4j-api", version = "2.0.6" }
+logger-api4k = { module = "io.github.microutils:kotlin-logging-jvm", version = "3.0.4" }
+logger-impl = { module = "ch.qos.logback:logback-classic", version = "1.4.5" }
 
 [bundles]
+logger-api = ["logger.api4j", "logger.api4k"]
 
 [plugins]
 jmh = { id = "me.champeau.jmh", version = "0.6.8" }

src/main/kotlin/name/valery1707/problem/LinkChecker.kt

@@ -40,7 +40,7 @@ class LinkChecker(private val root: Path) {
             .filter { it.second.second.first != 200 }
             .toList()
         // todo remove
-        println("filePos2uriCheck = $filePos2uriCheck")
+        logger.debug { "filePos2uriCheck = $filePos2uriCheck" }
         return filePos2uriCheck
             .associateBy(
                 { "${it.first.first}:${it.first.second}" },
@@ -59,6 +59,8 @@ class LinkChecker(private val root: Path) {
     }
 
     companion object {
+        private val logger = mu.KotlinLogging.logger {}
+
         /**
          * https://stackoverflow.com/a/45690571
          */
@@ -104,8 +106,7 @@ class LinkChecker(private val root: Path) {
             val request = HttpRequest.newBuilder(this).GET().build()
             // todo Cache
             return try {
-                // todo Logging
-                println("Check: $this")
+                logger.info("Check: $this")
                 val response = client.send(request, HttpResponse.BodyHandlers.discarding())
                 when (response.statusCode()) {
                     //Redirects: extract new location
@@ -128,16 +129,15 @@ class LinkChecker(private val root: Path) {
 
                             ?: 500
 
-                        // todo Logging
-                        println("Await: $await ms")
+                        logger.debug("Await: $await ms")
                         Thread.sleep(await)
                         check(client)
                     }
 
                     else -> response.statusCode() to response.uri()
                 }
             } catch (e: Exception) {
-                // todo Logging
+                logger.error(e) { "Handle error on checking $this" }
                 -1 to URI.create("http://host?message=${e.message?.replace(" ", "%20")}")
             }
         }

versions.lock

@@ -1,21 +1,25 @@
 # Run ./gradlew --write-locks to regenerate this file
+ch.qos.logback:logback-classic:1.4.5 (1 constraints: 0c050136)
+ch.qos.logback:logback-core:1.4.5 (1 constraints: 0b0d071d)
 com.fasterxml.jackson:jackson-bom:2.14.1 (3 constraints: 7f37c0d3)
 com.fasterxml.jackson.core:jackson-annotations:2.14.1 (2 constraints: 3f21f093)
 com.fasterxml.jackson.core:jackson-core:2.14.1 (2 constraints: 3f21f093)
 com.fasterxml.jackson.core:jackson-databind:2.14.1 (2 constraints: f1138582)
 com.github.mifmif:generex:1.0.2 (1 constraints: e00921ac)
 commons-codec:commons-codec:1.15 (1 constraints: b6094aa2)
 dk.brics.automaton:automaton:1.11-8 (1 constraints: 900be8e7)
+io.github.microutils:kotlin-logging-jvm:3.0.4 (1 constraints: 0905fe35)
 net.andreinc:aleph:0.1.1 (1 constraints: df091eac)
 net.andreinc:markovneat:1.8 (1 constraints: 88099b98)
 net.andreinc:mockneat:0.4.8 (1 constraints: 0e05ff35)
 org.apache.commons:commons-lang3:3.12.0 (2 constraints: ab17f565)
 org.apache.commons:commons-text:1.9 (1 constraints: 89099c98)
 org.jetbrains:annotations:13.0 (1 constraints: df0e795c)
 org.jetbrains.kotlin:kotlin-stdlib:1.7.22 (3 constraints: 463049c4)
-org.jetbrains.kotlin:kotlin-stdlib-common:1.7.22 (1 constraints: 450fd27a)
+org.jetbrains.kotlin:kotlin-stdlib-common:1.7.22 (2 constraints: 71207b44)
 org.jetbrains.kotlin:kotlin-stdlib-jdk7:1.7.22 (1 constraints: e310fbd2)
-org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.7.22 (1 constraints: 3e05453b)
+org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.7.22 (2 constraints: 6a16ed11)
+org.slf4j:slf4j-api:2.0.6 (3 constraints: 0723153f)
 
 [Test dependencies]
 net.bytebuddy:byte-buddy:1.12.10 (1 constraints: 7b0bbcea)