You'll be creating this names over and over again on very invocation. I think that making them private (possibly lazy) vals might be beneficial.