thing?.takeIf { anotherCondition }?.run(::doSomething)

I believe would work

thing?.takeIf { anotherCondition }?.doSomething()

should work, too

ah of course

Actually I over simplified my problem. Here is what it looks like

if (mCachedTasks != null && !mCacheIsDirty) {
    callback.onTasksLoaded(ArrayList(mCachedTasks!!.values))
    return
}

For null checking I do: thing?.let { ... it.something()...}. Is that ok? Or is takeIf better solution?

.takeIf

returns the object if the condition is true, or

null

if it was false. So it's a good way to filter out on some condition other than null. For example, you might not want to do

employee?.foo()

if the employee is non-null but invalid, so you can do

employee?.takeIf { it.isValid }?.foo()

Who not simply assign

var

to local

val

and leave

if

?

DRY - Don't repeat youself. When you operate on one thing it is eaiser to follow logic with

it

as opposed to repeating the name of this thing over and over.

DRY is about business rules, not low level code, but that's not for Slack 😉