Re: Should I leave behind beautiful code or readable code?

[jwkrahn]

I have two questions that would concern me as a maintainer:

1.   When you use split(/ /) are you saying that you know that the data will always be separated by exactly one space character or did you really mean to use split(/ +/) or split(/\s+/) or preferably split(' ') instead?

2.   Why are you using map twice when you only need to use it once?   For example:

return map split( / /, uc, 2 ), @somearray;