tags:

views:

659

answers:

1

I'm working with a map in Scala and doing the usual "if there's no value associated with a key, create it, put it in the map and return it":

def alphaMemory(key : AlphaMemoryKey) = {
    var am = map.getOrElse(key, null)
    if(am == null) {
        am = new AlphaMemory(key)
        map.put(key, am)
    }
    am
}

To me, this does not feel like idiomatic Scala code. It feels like Java. Is there a more succinct way of writing this? It looked like maybe I could override Map.default() to insert the new value and return it. Not sure though.

Thanks!

+7  A: 

mutable.Map has getOrElseUpdate which does exactly what you want, no idiom necessary.

sblundy
Excellent! I didn't realize that the default parameter was "by-name" so I had convinced myself it would be evaluated each time it was called. Thanks!
Dave Ray
I'd be interested in seeing how this would be done with immutable maps?
Marcus Downing