Originally posted by jason adam:
Don't know if you would get nitpicked for doing it that way, though. Do whatever is the most readable
Yup, this gets nitpicked.
Using a loop to load the HashMap seemed so clever (and one way of not repeating that method 28 times), but after looking at how clear the code is when you just "put" in the keys and values directly, there's no question about which option is more readable.
Live and learn.
Pauline