Search code examples
kotlinintellij-ideakodein

Kodein factory with interface. IDE suggestion produces incorect code


I have the following code in Kodein module

    bind<Manager>() with factory { strategy: OrderStrategyType ->
        val manager: Manager = when (strategy) {
            OrderStrategyType.VOLATILITY -> VolatilityManager()
            else -> SimpleManager()
        }

        return@factory manager
    }

where Manager is interface and VolatilityManager() and SimpleManager() are implementing it.

IntelliJ suggests to inline variable manager, if I apply suggestion I receive the code:

    bind<Manager>() with factory { strategy: OrderStrategyType ->
        return@factory when (strategy) {
            OrderStrategyType.VOLATILITY -> VolatilityManager()
            else -> SimpleManager()
        }
    }

However, while IDE detects no problems with this code, it doesn't compile with

Type inference failed: infix fun <C, A> with(binding: KodeinBinding<in C, in A, out Manager>): Unit
cannot be applied to
(Factory<Any?, OrderStrategyType, Any>)

Honestly, I don't understand compiler error. Inference looks obvious for me. Should I rewrite my code, if yes, how?


Solution

  • Regarding this code, with Kotlin 1.3.72 their is no issue.

    interface A
    class B: A
    class C: A
    
    bind<A>() with factory { b: Boolean ->
        when(b) {
            true -> B()
            false -> C()
       }
    }
    

    If your classes implements/extends more than one interfaces/classes type inference may not now what type you want to return.

    Force cast seems to do the trick

    interface A
    interface X
    class B: A, X
    class C: A, X
    
    bind<A>() with factory { b: Boolean ->
        when(b) {
            true -> B()
            false -> C()
        } as A
    }
    

    PS: you don't need the return@factory as when is an expression.