I have programmed a mini-class for practicing Kotlin. I'm getting really confused with how I could properly initialize these class properties without repeating the operations. I'm trying that whenever you initialize the

Time

class with more than 59 minutes, or more than 59 seconds, it automatically gets adjusted

That's a weird ctor IMO. I think you should just assert that the params are within range, instead of handling the overflow.

Honestly it's just for the sake of practice, although I see it very convenient that some kind of "Value Object" like this could handle the overflow by itself

Actually, I think you're missing an edge case

Umm according to the tests the implementation of the pic is correct 🤔

Ohhh I see.

In case you needed that kind of overflow handling what would you do instead of making the constructor handle it?

Are those static methods? Sorry, I don't understand very well what do you mean

They should exists in the companion object of the

Time

class

Thank you! More people think the same, that constructor is a bit ugly

If you don’t want to repeat the calculations as you originally asked, just create private methods and use them in the constructor