how can I fix this? :smile: <https://pastebin.com/9iLG5E6B>

# codereview

[oday]

how can I fix this? 😄 https://pastebin.com/9iLG5E6B

[Andreas Sinz]

@oday what do you mean by fix? make it more idiomatic?

[oday]

One minute

[oday]

Ok @Andreas Sinz well yes I know there's no glaring issue in the code and it isn't wrong to do this, but it seems so exhaustive, and imagine like this method there are many others, each handling tracking

[oday]

And it just looks ugly

[oday]

For example is there a way that I can look for an object's properties and find a property by a string?

[oday]

If this property's name is "country_code"

[oday]

Im not sure really how to improve on this but it's definitely too exhaustive looking and ugly

[Andreas Sinz]

you could look for properties based by their name with reflection, but it looks like you aren't even using any properties of lets say mpProperties

[oday]

it’s the issue that the name of the property matches the name of the field i will send to mixpanel

[oday]

yea…not much value added if there’s no loop, and you can’t loop if you’re only selecting a limited number of properties from the object

[oday]

i’ll come back with a clearer problem

[Andreas Sinz]

you can make the code cleaner with with(mpProperties) { ... }

[gildor]

For example is there a way that I can look for an object’s properties and find a property by a string?

Only using reflection or code generation.
Also kotlin.serialization can do this for you

[gildor]

So this is only for fully automatic solution

[gildor]

but you can improve your code just using apply on mpProperties or even writing some dsl like:

mpProperties.putAll {
   "car_id" to listing.id
}

[oday]

I see

[oday]

ok, yea either full automatic or not, i noticed

[oday]

i will ask more about this DSL if you dont mind, later 🙂

[oday]

I actually asked this in #getting-started but I don’t think it falls into what they can help with

how can I print the name of any variable? var1.javaClass.getFieldByName(String) is what im using right now but im not sure what to feed getFieldByName

[oday]

im in the process of fixing what you suggested the other day

[gildor]

Do you want to use reflections for that?

[gildor]

Not sure what exactly do.ypu want to achive, because there are many different ways to implement such mapping with reflections