Skip to content

Реализация приложения калькулятор. - #1

Open
SmolyakovKirill wants to merge 15 commits into
mainfrom
dev
Open

SmolyakovKirill wants to merge 15 commits into
mainfrom
dev

Conversation

@SmolyakovKirill

Copy link
Copy Markdown
Owner

Не предумал как изящно реализовать некоторые моменты, в особенности класс formatter и реализацию ввода данных.
Жду обратной связи!
Спасибо!

@SmolyakovKirill

Copy link
Copy Markdown
Owner Author

Обратная связь по замечаниям:

  • В "название товара" товара можно ввести цифры. Обычно название состоит из букв - исправлено
  • Не верное окончание для "0" и "4". Сейчас для обоих случаев это "рубль" вместо "рублей" и "рубля" соответственно. Выводить слово рубль с правильным окончанием в зависимости от суммы - для правильной обработки окончания нужно учесть округленное в меньшую сторону значение без дробной части, и на что оно оканчивается: 1 - "рубль", 2-4 - "рубля", 5-9 (и 0) - "рублей". Плюс учесть, что для чисел 11-19 - "рублей" - для этого стоит проверять, что <число> % 100 между 11 и 19 включительно находится - исправлено, а разве цена товаров может быть 0? :)
    Рекомендации, не влияющие на результат проверки работы:
  • После каждого введённого товара можно выводить что-то вроде: "Введите название товара. Или "Завершить", если вы закончили". Сейчас не понятно как закончить ввод товаров. -исправлено

Жду обратной связи, Спасибо!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant