roman numerals working needs re-factoring