Technically, a Locu item price can contain non-numeric content such as "Market", "Seasonal", as well as "10 and up" etc.  

Looking at the old code, I understand that this is an improvement over parsing the string as a float, but I'm wondering if a better solution would be to just make the MenuItem class take a string instead (or alternatively we could subclass the Money class and make it also supports strings). What do you think?