Is there a function in List that can achieve the u...

# getting-started

[Chachako]

Is there a function in List that can achieve the uniform behavior of String removeSuffix or removePrefix?

[Chachako]

It seems that there is no, eventually solved by creating an extension function:

fun <T> List<T>.endsWith(vararg slice: T) = this.toList().takeLast(slice.size) == slice.toList()

fun <T> List<T>.removeSuffix(vararg suffix: T) =
  if (endsWith(*suffix)) this.dropLast(suffix.size) else this

I hope this can help others.

[ephemient]

one possibly interesting alternative: it can be implemented on Sequence, lazily:

fun <T> Sequence<T>.removeSuffix(vararg slice: T): Sequence<T> {
    require(slice.isNotEmpty())
    return sequence {
        val iter = [e-mail](slice.size).iterator()
        if (!iter.hasNext()) return@sequence
        while (true) {
            val window = iter.next()
            if (iter.hasNext()) {
                yield(window.first())
            } else {
                if (window != slice.asList()) yieldAll(window)
                break
            }
        }
    }
}

[Chachako]

windowed operator I have not touched yet, I will go see! Very interesting

[ephemient]

actually what I wrote has a bug in that if the slice is longer than the sequence it returns nothing. could be fixed at the cost of some complexity. anyway, your original extensions do work on eager Lists

[Joffrey]

Why this.toList() before calling takeLast() though? It's already a list, and creating a new instance doesn't seem to serve any purpose here

[Klitos Kyriacou]

Also, @ephemient’s require(slice.isNotEmpty()) shouldn't be needed, as "xyz".endsWith("") is allowed (returns true).

[Chachako]

@Joffrey You are right, it is a redundant code

[ephemient]

it's there because .windowed(0) is nonsense. that case should probably be handled separately.