this looks pretty nice (though

polocode

is apparently something very different from

polacode

lol)

oh, oops

hmm, can I not edit comments on a file upload?

good question, file comments have always been super weird

@nish commented on @perryprog’s file : Did you do this in VScode?

@perryprog commented on @perryprog’s file : Yes, that’s the main limitation

fun getYear(): String? {
    for (node in childNodes) {
        val matcher = yearPattern.matcher(node.attr("href").trim())
        if (matcher.find()) {
            return matcher.group("year")
        }
    }
    return null
}

Is there a more concise way to do this via collection functions?

I think in my head I’m wanting something like “mapFirst”

Which I guess I could just create that as an extension function if I really wanted to.

I wanna say there’s something like that already in the collections

Right?

That would return the node

Indeed, I should read more closely

something like

childNodes.toSequence()
	.map {
        yearPattern.matcher(it.attr("href").trim())
    }.firstOrNull {
        it.find()
    }?.group("year")

Not too bad, though I’ve been wondering about Kotlin’s efficiency on this kind of thing

it's a sequence so it's lazily evaluated

Ahh

I was going to ask about the sequence bit next.

the sequence approach is what I’d recommend as well

So that’s how I can be sure that it doesn’t first map every item in the collection?

By calling toSequence()?

yes, it'll stop on the first one matching the

first

condition

(btw It should be

asSequence

)

my bad, didn't write it in IDEA

TIL! That asSequence bit was something I’ve been wanting

Thanks for the help :]

honestly I'd just keep the original "for (node in childNodes)", shorter != clearer

That’s a fair point, but I have a total lack of empathy for our junior dev and I usually run with concise 😛

Is there a way to use

find

here while still catching the exception?