detekt report

Total: 13

complexity: 1

TooManyFunctions: 1 Too many functions inside a/an file/class/object/interface always indicate a violation of the single responsibility principle. Maybe the file/class/object/interface wants to manage too many things at once. Extract functionality which clearly belongs together.

Documentation

kotlin-stdlib/common/main/kotlin/com/javiersc/kotlin/stdlib/Strings.kt:1:1File '/home/runner/work/kotlin-stdlib/kotlin-stdlib/kotlin-stdlib/common/main/kotlin/com/javiersc/kotlin/stdlib/Strings.kt' with '11' functions detected. Defined threshold inside files is set to '11'
```
   1 package com.javiersc.kotlin.stdlib
   2 
   3 import kotlin.contracts.contract
   4 
```

naming: 6

FunctionNaming: 6 Function names should follow the naming convention set in the configuration.

Documentation

kotlin-stdlib/common/main/kotlin/com/javiersc/kotlin/stdlib/StringsTransform.kt:8:19Function names should match the pattern: [a-z][a-zA-Z0-9]*

   5 
   6 public fun String.transformstring(): String = prepare("").lowercase()
   7 
   8 public fun String.TRANSFORMSTRING(): String = prepare("").uppercase()
   9 
  10 public fun String.transformString(): String {
  11     val data = prepare(" ")

kotlin-stdlib/common/main/kotlin/com/javiersc/kotlin/stdlib/StringsTransform.kt:25:19Function names should match the pattern: [a-z][a-zA-Z0-9]*

  22         .remove(" ")
  23 }
  24 
  25 public fun String.TransformString(): String {
  26     val data = prepare(" ")
  27     return data
  28         .mapIndexed { index, char ->

kotlin-stdlib/common/main/kotlin/com/javiersc/kotlin/stdlib/StringsTransform.kt:41:19Function names should match the pattern: [a-z][a-zA-Z0-9]*

  38         .capitalize()
  39 }
  40 
  41 public fun String.transform_string(): String = prepare("_").lowercase()
  42 
  43 public fun String.TRANSFORM_STRING(): String = prepare("_").uppercase()
  44

kotlin-stdlib/common/main/kotlin/com/javiersc/kotlin/stdlib/StringsTransform.kt:43:19Function names should match the pattern: [a-z][a-zA-Z0-9]*

  40 
  41 public fun String.transform_string(): String = prepare("_").lowercase()
  42 
  43 public fun String.TRANSFORM_STRING(): String = prepare("_").uppercase()
  44 
  45 public fun String.transform_hyphen_string(): String = prepare("-").lowercase()
  46

kotlin-stdlib/common/main/kotlin/com/javiersc/kotlin/stdlib/StringsTransform.kt:45:19Function names should match the pattern: [a-z][a-zA-Z0-9]*

  42 
  43 public fun String.TRANSFORM_STRING(): String = prepare("_").uppercase()
  44 
  45 public fun String.transform_hyphen_string(): String = prepare("-").lowercase()
  46 
  47 public fun String.TRANSFORM_HYPHEN_STRING(): String = prepare("-").uppercase()
  48

kotlin-stdlib/common/main/kotlin/com/javiersc/kotlin/stdlib/StringsTransform.kt:47:19Function names should match the pattern: [a-z][a-zA-Z0-9]*

  44 
  45 public fun String.transform_hyphen_string(): String = prepare("-").lowercase()
  46 
  47 public fun String.TRANSFORM_HYPHEN_STRING(): String = prepare("-").uppercase()
  48 
  49 public fun String.tRaNsFoRmStRiNg(): String {
  50     var newStr = ""

style: 6

ReturnCount: 2 Restrict the number of return statements in methods.

Documentation

kotlin-stdlib/common/main/kotlin/com/javiersc/kotlin/stdlib/Collections.kt:89:35Function penultimateOrNull has 4 return statements which exceeds the limit of 2.

  86     }
  87 }
  88 
  89 public inline fun <T> Iterable<T>.penultimateOrNull(): T? {
  90     return when (this) {
  91         is List -> getOrNull(size - 2)
  92         else -> {

kotlin-stdlib/common/main/kotlin/com/javiersc/kotlin/stdlib/Collections.kt:132:37Function getIndexOrNull has 3 return statements which exceeds the limit of 2.

 129 }
 130 
 131 @PublishedApi
 132 internal inline fun <T> Iterable<T>.getIndexOrNull(index: Int): T? {
 133     return when (this) {
 134         is List -> getOrNull(index - 1)
 135         else -> {

ThrowsCount: 2 Restrict the number of throw statements in methods.

Documentation

kotlin-stdlib/common/main/kotlin/com/javiersc/kotlin/stdlib/Collections.kt:64:35Too many throw statements in the function penultimate. The maximum number of allowed throw statements is 2.

  61  *
  62  * Throws: `NoSuchElementException` - if the list size is 1.
  63  */
  64 public inline fun <T> Iterable<T>.penultimate(): T {
  65     return when (this) {
  66         is List ->
  67             when (size) {

kotlin-stdlib/common/main/kotlin/com/javiersc/kotlin/stdlib/Collections.kt:110:37Too many throw statements in the function getIndex. The maximum number of allowed throw statements is 2.

 107 }
 108 
 109 @PublishedApi
 110 internal inline fun <T> Iterable<T>.getIndex(index: Int): T {
 111     return when (this) {
 112         is List ->
 113             if (size >= index - 1) this[index - 1]

UnusedPrivateMember: 2 Private member is unused and should be removed.

Documentation

kotlin-stdlib/common/main/kotlin/com/javiersc/kotlin/stdlib/Collections.kt:121:18Private property `i` is unused.

 118                 throw NoSuchElementException("Collection size is lower than $index.")
 119             var value: T = iterator.next()
 120 
 121             for (i in 0 until index - 1) {
 122                 if (!iterator.hasNext())
 123                     throw NoSuchElementException("Collection size is lower than $index.")
 124                 else value = iterator.next()

kotlin-stdlib/common/main/kotlin/com/javiersc/kotlin/stdlib/Collections.kt:140:18Private property `i` is unused.

 137             if (!iterator.hasNext()) return null
 138             var value: T = iterator.next()
 139 
 140             for (i in 0 until index - 1) {
 141                 if (!iterator.hasNext()) return null else value = iterator.next()
 142             }
 143             value

Metrics

Complexity Report

Findings

complexity: 1

naming: 6

style: 6