<@U68N4SR27> Don't call `.stream()` it gives you a...

# announcements

[Hawk]

@ianandrewclark Don't call .stream() it gives you a java stream not a kotlin list, you should be able to call .forEach { println(it) } directly on the collection.

[Shawn]

Am I missing context? Why would you suggest mapping the stream contents with println? Might he not need the list for something else?

[Shawn]

Also it might be more concise to just .forEach(::println)

[Hawk]

My apologies, I made an assumption he was try to convert something to a list in order to perform some sort of list functions on it. map/filter/foreach. Furthermore if he can call stream on the object then isn't it most likely already a collection?

[Hawk]

Any yes that is more concise 😉

[Ian]

Sorry, I’m getting the Stream from a BufferedReader.lines() method

[Ian]

Is there a convenient way I can get a Sequence instead?

[Shawn]

hmm, there’s supposed to be an extension function, BufferedReader#lineSequence(), provided by Kotlin, but I’m not sure that’ll be terribly helpful if you can’t use any of the extension methods

[Ian]

Hmm, let me check that out…

[Ian]

Nice, that worked, thank you @Shawn;)

[Shawn]

np @Ian. Might be worth mentioning that the source for that method includes this comment:

We suggest you try the method [useLines] instead which closes the stream when the processing is complete.

I’m not sure if this is directly relevant to you, but it may be worth investigating

[Shawn]

useLines takes in a lambda, specifically (Sequence<String>) -> T, and handles cleanup for you

[Ian]

Yes, in this case I actually needed a List, so toList() worked out fine 🙂