Skip to content

Conversation

@KonstantinKhan
Copy link
Contributor

Description

Kotlin Data classes features

Related Issue

Closes #41

@e5LA e5LA self-requested a review August 15, 2025 18:00
@e5LA
Copy link
Owner

e5LA commented Aug 15, 2025

Thanks for another contribution, @KonstantinKhan!

I like how this covers both the core features and the common pitfalls of data classes. Just a few small tweaks suggested, but overall, great work!

It’s our first Kotlin feature showcase, so we could also think about organizing the example to make it a bit easier to follow. For instance:

fun main() {
    demonstrateToString()
    demonstrateEquals()
    // ...

fun demonstrateToString() { /* focused example */ }
fun demonstrateEquals() { /* focused example */ }
fun demonstrateCopy() { /* focused example */ }
// etc.
}

What do you think?

@KonstantinKhan KonstantinKhan force-pushed the demonstrate-data-classes branch from 6ce5972 to 287ebdf Compare August 17, 2025 07:20
@KonstantinKhan
Copy link
Contributor Author

I think that it's a good idea.

Copy link
Owner

@e5LA e5LA left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for addressing the feedback and making those improvements - the code is now even cleaner and more educational :)
Great contribution to the collection! 🎉 thanks @KonstantinKhan

@e5LA e5LA merged commit 4fc201b into e5LA:main Aug 23, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Demonstrate data classes

2 participants