Skip to content

Sprint02consoleAppPaffn#1

Merged
Paffnuty merged 6 commits intomainfrom
dev
Mar 21, 2023
Merged

Sprint02consoleAppPaffn#1
Paffnuty merged 6 commits intomainfrom
dev

Conversation

@Paffnuty
Copy link
Copy Markdown
Owner

Training console app sprint #2




// public static void checkInt() {
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⏫ Желательно не оставлять в итоговой работе мертвый код в комментариях

int iMunit = (int)Munit;
int mCh = iMunit%10;

if (iMunit%100>=11 && iMunit<=14) {
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Отличная идея проверять остаток от деления на 10 и выводить значение через форматирование

// }


Calculator (String itemNameList, float price, int numItems ) {
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Хорошая идея использовать отдельный класс, для хранения стейта работы программы.

⏫ Я бы посоветовал разделить логику хранения стейта и вывода верного значения "рублей" на разные классы, т.к. это две разные не связанные друг с другом логики

System.out.println("Привет Мир");

Scanner scanner = new Scanner(System.in);
Calculator calcPersonBill = new Calculator("", 0, 0);
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⏫ Можно для полей класса Calculator проинициализировать дефолтными значениями внутри самого класса без необходимости передавать их в конструкторе

calcPersonBill.itemNameList = calcPersonBill.itemNameList + "\n" + scanner.nextLine(); // добавляем имя в строку

System.out.println("Введите цену товара");
calcPersonBill.price = calcPersonBill.price + scanner.nextFloat(); // Складываем полученые цены по мере ввода
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Если ввести в качестве цены товара не число, то произойдет ошибка программы. Кроме того, у пользователя есть возможность добавить отрицательное значение цены товара

@Paffnuty Paffnuty merged commit 726c458 into main Mar 21, 2023
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.

2 participants