CODEWITHSUNDEEP
kotlinenums
Braden Steffaniak
Braden Steffaniak

Reputation: 2262

Kotlin enum constructor argument 'must be initialized' with companion object constant without qualifier name

This code compiles successfully with the qualified name used to access the constants from the companion object:

enum class CampsiteCategoryCode(val code: String) {
  TENT(CampsiteCategoryCode.TENT_CODE), // intellij says 'Redundant qualifier name'
  OTHER(CampsiteCategoryCode.OTHER_CODE), // intellij says 'Redundant qualifier name'
  LODGING(CampsiteCategoryCode.LODGING_CODE), // intellij says 'Redundant qualifier name'
  RV(CampsiteCategoryCode.RV_CODE); // intellij says 'Redundant qualifier name'

  override fun toString() = code

  companion object {
    const val TENT_CODE = "tent"
    const val OTHER_CODE = "other"
    const val LODGING_CODE = "lodging"
    const val RV_CODE = "rv"
  }
}

However, the same code without the qualifier name fails compilation:

enum class CampsiteCategoryCode(val code: String) {
  TENT(TENT_CODE), // Variable 'TENT_CODE' must be initialized
  OTHER(OTHER_CODE), // Variable 'OTHER_CODE' must be initialized
  LODGING(LODGING_CODE), // Variable 'LODGING_CODE' must be initialized
  RV(RV_CODE); // Variable 'RV_CODE' must be initialized

  override fun toString() = code

  companion object {
    const val TENT_CODE = "tent"
    const val OTHER_CODE = "other"
    const val LODGING_CODE = "lodging"
    const val RV_CODE = "rv"
  }
}

Why does specifying the qualifier name allow this code to compile? Or put another way, why does not having the qualifier name make the code fail to compile?

Upvotes: 16

Views: 5245

Answers (2)

nepa
nepa

Reputation: 1462

There are two solutions for that:

  1. make the variable a top-level declaration (as @baguIO mentioned)
  2. mention the scope explicitly (use CampsiteCategoryCode.TENT_CODE instead of just TENT_CODE)

Upvotes: 4

baguIO
baguIO

Reputation: 439

That companion object will be instantiated the moment your class becomes available (compile time). Then you are telling your class that, for it to become available, it needs a class that is not yet available (because you're compiling it at the moment and, apparently, Kotlin makes enum cases available before companion objects)

I can't tell for sure if this is intended from Kotlin, but to avoid this kind of cases, declare your constants outside this class

Upvotes: 16

Related Questions