```if (data in selectedItems) { selectedItems....

# announcements

[Jérôme Gully]

if (data in selectedItems) {
    selectedItems.remove(data)
} else {
    selectedItems.add(data)
}

It is a better way to write this type of condition in Kotlin ?
selectedItems is an arraylist and data is an object of arraylist type

[Stephan Schroeder]

if(!selectedItems.remove(data)) {
    selectedItems.add(data)
}

[Jonathan Mew]

If that works, I guess you have the option of

selectedItems.remove(data) || selectedItems.add(data)

but personally I find your original structure the most readable, which I would consider more important

[Stephan Schroeder]

and you could put that into an extension function

fun <T> MutableCollection<T>.toogle(value: T) {
    remove(value) || add(value)
}

[tddmonkey]

Personally I wouldn’t let that last form through PR. It might work but the intent is just not clear

[Jérôme Gully]

Thank you all for your messages. My colleague started with something like with(selectedItems) { if (data in this) ... else ... } but I said "you do kotlin like code just for doing kotlin code 🧌 . And I proposed to wrote it with a simple if else, i guess I was right, for me too it's the most readable, but I'm not a Kotlin expert for now, so I asked to see if a most simple/idiomatic/readable code was possible.