Hi guys .... hope you doing will what is the diff and what should i choose ```private val empID ...

# codereview

[hisham bin awiad]

Hi guys .... hope you doing will

what is the diff and what should i choose

private val empID 
             get() = (userModelId ?: session.getUserModelId().toSafetyString()).toSafetyString()

	private val empID2 by lazy { (userModelId ?: session.getUserModelId().toSafetyString()).toSafetyString()  }

[sbyrne]

With get(), it will be evaluated every time empID is referenced. With by lazy it will only be evaluated once at most.

[sbyrne]

If userModelId is immutable and session.getUserModelId() will always return the same thing, use by lazy.

[hisham bin awiad]

Thx bro🌹

[Robert Williams]

Remember by lazy also has to synchronise threads on every access to guarantee the single execution

[Robert Williams]

You can remove some of this overhead by using LazyThreadSafetyMode.NONE (this will break the single execution guarantee but in this case it probably doesn't matter)

[Robert Williams]

But if the value really is immutable and not particularly expensive to compute but you still only want to do it once you should consider the simple

private val empID = (userModelId ?: session.getUserModelId().toSafetyString()).toSafetyString()

[Matteo Mirk]

minor: session.getUserModelId() can be changed to property accessor session.userModelId