-
Notifications
You must be signed in to change notification settings - Fork 0
Comments
Conversation
...прашивает количество персон для разделения счета, но пока еще не может отработать ввод числа буквами. Также создал класс Product, с помощью которого буду создавать объекты данного класса, для создания списка заказанных позиций из общего счета.
...тформатированный вывод стоимости позиций из счета и суммы распила, а также добавить обозначение валюты.
... счету, данное значение выводся в консоль с двумя знаками после запятой. + чуть переписал README.md и добавил новый счетчик в класс Main.
..., но он пока не до конца правильно работает, буду исправлять.
...а Рубль работает коректно.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
значение name для этой переменной никогда не будет достигнуто, т к при создание объекта (вызове конструктора) оно перезатрется. об этом подсказывает idea ;)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
кажется, что этот цикл здесь совсем не обязателен - его можно удалить
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
переменным нужно стараться присваивать говорящие, а не абстрактные имена :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
слово рублей повторяется трех местах данного класса - его лучше вынести в константы (как и другие слова)
private static final String RUB = "рублей"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
довольно громоздкая конструкция для восприятия. для удобочитаемости можно вынести подставляемые значения в отдельные переменные, либо воспользоваться StringBuilderом
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
не нужно создавать много сканнеров - достаточно одного экземпляра этого класса
public class Main {
private static final Scanner scanner = new Scanner(System.in)
public static void main(String[] args) {...
Выполнил практическое задание 2-ого спринта, реализовал вроде все, что просили в ТЗ к заданию.