Hi, does this make sense? I know it's generalizabl...

# getting-started

[Czar]

Hi, does this make sense? I know it's generalizable with comparables, I'm just interested if this simple version can be written better.

fun Sequence<Int>.isOrderedAscending(): Boolean =
	zipWithNext { prev, next -> prev <= next }
		.fold(true) { acc, value -> acc && value }

[Sebastian Aigner]

You could use windowed :

fun Sequence<Int>.isOrderedAscending(): Boolean =
    this.windowed(2).all { (first, second) -> first <= second }

[Sebastian Aigner]

windowed returns a Sequence of `List`s, which you’re then destructuring in the lambda to access the first and second elements and to do the comparison on all of them.

[Czar]

ok, so which is better I wonder:

zipWithNext().all { (first, second) -> first <= second }

or

windowed(2).all { (first, second) -> first <= second }

zip variant generates pairs, windowed - lists

[Tsvetan Ovedenski]

@Czar maybe also:

zipWithNext { prev, next -> prev <= next }.all { it }

That way you don't create pairs nor lists, esp. with Sequence.

[Czar]

Cool, looked at the bytecode, makes sense. Thank you!