We can avoid the loop and if with this solution:
public class RomanNumber {
private final static Supplier<TreeMap<Integer, String>> romanNumerals = () -> {
final TreeMap<Integer, String> map = new TreeMap<>();
map.put(1000, "M");
map.put(900, "CM");
map.put(500, "D");
map.put(400, "CD");
map.put(100, "C");
map.put(90, "XC");
map.put(50, "L");
map.put(40, "XL");
map.put(10, "X");
map.put(9, "IX");
map.put(5, "V");
map.put(4, "IV");
map.put(1, "I");
return map;
};
private static Function<TreeMap<Integer, String>, Function<Integer, String>> numeralConverter = map -> number -> Optional
.ofNullable(map.floorKey(number))
.filter(number::equals)
.map(map::get)
.or(() -> Optional
.ofNullable(map.floorKey(number))
.map(num -> map.get(num) + RomanNumber.numeralConverter.apply(map).apply(number - num))
).orElse("NaN");
public static Function<Integer, String> toRoman = numeralConverter.apply(romanNumerals.get());
}