Hacker News Reader

Comment by himata4113 | Hacker News Reader

  items.filter(|(i)| cond(i)).map(Pointer::idx).collect()

you can probably get away with this if you implement a Trait, not sure how but I know for a fact this is possible, idk why there's an enumerate there when you aren't even using it.

items is already an iteratible so you can do direct .filter on it as well

tl;dr if the code looks ugly you're probably not taking advantage of a language feature that allows it to look pretty.

View on HN

This is in fact more readable, thank you.

I still don’t know that I find it more intuitive or readable than the simple loop, but it’s much less awful than before.

I am sure there's a way to filter directly on cond variable and collect isn't needed most of the time since iterators are way more useful in general (unless you want to print that data)

   items.filter(cond).map(Pointer::idx)

I know this is possible, but you would have to consult some rust wizard for this.

Reply on news.ycombinator.com