Improve the messaging for "DataClassDefaultValues"

Author: ILIYANGERMANOV

src/main/kotlin/com/github/ivy/explicit/rule/DataClassDefaultValuesRule.kt

@@ -11,7 +11,7 @@ class DataClassDefaultValuesRule(config: Config) : Rule(config) {
         id = "DataClassDefaultValues",
         severity = Severity.Maintainability,
         description = "Data class properties should not have default values. " +
-                "Default values lead to implicit instance constructions and problems.",
+                "Default values lead to implicit creation of instances which leads to problems.",
         debt = Debt.TWENTY_MINS,
     )
 
@@ -33,8 +33,10 @@ class DataClassDefaultValuesRule(config: Config) : Rule(config) {
     }
 
     private fun failureMessage(klass: KtClass, parameter: KtParameter): String = buildString {
-        append("Data class '${klass.name}' should not have default values for properties. ")
+        val className = klass.name
+        append("Data class '$className' should not have default values. ")
         append("Found default value for property '${Message.parameter(parameter)}'. ")
-        append("This can lead to implicit instance constructions and problems.")
+        append("This allows for instances of '$className' to be created without explicitly specifying all properties, ")
+        append("potentially leading to unintended or inconsistent states.")
     }
 }

src/test/kotlin/com/github/ivy/explicit/rule/DataClassDefaultValuesRuleTest.kt

@@ -22,7 +22,7 @@ internal class DataClassDefaultValuesRuleTest(private val env: KotlinCoreEnviron
         findings shouldHaveSize 1
         val message = findings.first().message
         message shouldBe """
-            Data class 'A' should not have default values for properties. Found default value for property 'x: Int = 42'. This can lead to implicit instance constructions and problems.
+            Data class 'A' should not have default values. Found default value for property 'x: Int = 42'. This allows for instances of 'A' to be created without explicitly specifying all properties, potentially leading to unintended or inconsistent states.
         """.trimIndent()
     }