С алгоритмической точки зрения и некоторых других моментов (например, постоянное использование var) скорее говнокод, чем нет, приемлемого среднего уровня 5 \ 10. Но по этому куску трудно судить о всём коде в целом - он слишком прост.
это характеризует его с положительной или отрицательной стороны?
Если в макете \ ТЗ было четко написано "рублей", то это характеризует его как человека, внимательного к мелочам. Это положительное качество.
P.S. Библиотеки для склонения
числительных существительных в данном случае избыточны.