Предлагаю передавать количество quantity в качестве дополнительного параметра в функцию samib. Такой подход делает класс Calculator более независимым от класса Main, что повышает его переиспользуемость.

Также лучше перенести логику округления в функцию класса Formatter, поскольку он отвечает за форматирование данных и их представление.

Вместо использования переменной padezh лучше получать отформатированную строку через метод formatRub (поскольку метод уже возвращает строку). Это позволит избавиться от лишней переменной и уменьшит сложность класса, так как логика форматирования будет инкапсулирована в соответствующем методе.

Условие (x >= 5 && x <= 9) можно сократить до x >= 5, т.к. переменная x никогда не будет больше 10.
Также если у тебя получится избавится от свойства класса padezh, можно будет переписать метод следующим образом:

        if(...){
            return "рублей";
        }else if (x == 1 ) {
            return "рубль";
        }else {
            return  "рубля";
        }

Но еще лучше будет попробовать переписать эти условия на switch ... case

Было бы лучше создать отдельный класс Product, содержащий следующие поля:

• name - имя продукта;
• price - его цена.

Таким образом, вместо списка строк мы будем использовать список объектов класса Product. Это также позволит избавиться от необходимости хранить отдельный список prices в классе Calculator.