Heyo! I've got a KMP project an am using value cla...

# multiplatform

[Matt Nelson]

Heyo! I've got a KMP project an am using value classes, but cannot for the life of me figure out how to require their use in Java from an interface method.

sealed interface Key {
    fun descriptor(address: Address): String
}

@JvmInline
value class Address(@JvmField val value: String)

Calling From Java:

public class SomeClass {
    public void someFunction(Key key) {
        // Key.descriptor accepts a String, not an Address
        String descriptor = key.descriptor("");

        Address address = new Address("someaddress");

        // key.descriptor does not accept an Address and IDE shows error
        String descriptor2 = key.descriptor(address);
    }
}

Any Ideas?

[George]

Hello, probably this is what you are looking for https://kotlinlang.org/docs/inline-classes.html#calling-from-java-code

[mike.holler]

In Java, you would call SomeClass().someFunction("someaddress") -- value types are not enforced by the JVM

[Matt Nelson]

With some ingenuity, I was able to remove value classes from my public APIs using sealed interfaces and operator function

sealed interface Port {
    val value: Int

    companion object {
        @JvmStatic
        @Throws(IllegalArgumentException::class)
        operator fun invoke(port: Int): Port {
            return RealPort(port)
        }
    }
}

@JvmInline
private value class RealPort(override val value: Int): Port {
    init {
        require(value in 0..65535) {
            "Invalid port range"
        }
    }
}

And calling from Java:

Port port = Port.invoke(65536); // Will throw IllegalArgumentException

This way, all my interfaces that take a Port now compile to the sealed interface, instead of the primitive type.

[Matt Nelson]

Ty George and Michael for your responses!

[mike.holler]

@Matt Nelson interesting approach. Worth noting that this has two hits against it, in addition to the obvious positives: 1) far less efficient 2) difficult to read

[mike.holler]

At this point, why not do it like this: data class Port(val value: Int)?

[mike.holler]

There will still be a boxing/unboxing on the JVM side for all of this, so doing data class Port(...) would have the same affect I'd think.

[Matt Nelson]

Yeah, I'm still playing with things to get it right.

Performance wise, data classes are very bad. See my test results HERE

[mike.holler]

What application are you writing that is using so many port numbers you have to worry about inefficiency?

[Matt Nelson]

It's a wrapper for Tor

[mike.holler]

You know, that's just about the one valid answer you could have given to that question right?

[Matt Nelson]

lol, yes.

[mike.holler]

I still think it'd be fully wrapped on the java side, and maybe on the kotlin side too. It might lose the perceived benefit. You should check out the bytecode.

[mike.holler]

Perhaps throw an example together that calls it from the java side and measure the time?

[Matt Nelson]

Decompiled byte code shows it's boxed.

[mike.holler]

What are your memory constraints? Caching every instance is only 0.5 MB

[Matt Nelson]

As minimal as possible, as running Tor on Android is a hog.

[mike.holler]

It does bear asking the question -- why not just use ints straight up? Maybe create a typealias if you're feeling curious.

[mike.holler]

I'm all for kotlinic code but here it feels like a good place to optimize

[Matt Nelson]

For sure, but Tor is very picky about the strings that it receives via it's control port. Using value classes is a very low cost way to offload knowledge library consumers have to have when interacting with the APIs (how many people know that a v3 onion address is a 56 character base32 encoded lowercase string?)

by handling all of that and returning a wrapper, it can be passed around everywhere straight from the controller response all the way down to the DB. Very low friction on library consumer side as the APIs "guide" things before even making it to the request being sent to the controller, so.

[mike.holler]

@Matt Nelson my friend, I am forever grateful to you. I have used this to create a WrappedLong implementation that is compatible with JS but performs very quickly on JVM. You are a genius.

[Matt Nelson]

Created a library for this specific issue of Kotlin value class not compiling to platform code.

https://github.com/05nelsonm/component-value-clazz