Skip to content

vvverman tochno pull request... da? #1

Open
vvverman wants to merge 7 commits intomainfrom
dev
Open

vvverman tochno pull request... da? #1
vvverman wants to merge 7 commits intomainfrom
dev

Conversation

@vvverman
Copy link
Copy Markdown
Owner

и снова в бой

Copy link
Copy Markdown

@gusar-off gusar-off left a comment

Choose a reason for hiding this comment

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

⚠️ Нужно разбить логику программы на разные классы

public static String GetRubleAddition(double formattedResult) {

}
int lastCharacter = (int) (Math.floor(formattedResult) % 10);
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️Немного некорректно рассчитывается окончание. Получается если цена будет 14 рублей, метод вернет рубль. Правильный алгоритм звучит так - Проверить лежит ли остаток от деления на 100 в интервале от 11 до 14 включительно, если да, то возвращаем рублей, если нет, то берем остаток от деления на 10 и проверяем - если 1, то рубль, 2-4 то рубля, в остальных случаях - рублей. То есть нужно во-первых проверять именно остаток от деления, во-вторых учесть, что числа заканчивающиеся на 11-12-13-14 и заканчивающиеся на 1-2-3-4, будут иметь разные окончания


// МЕТОД НИЖЕ РАЗБИРАЕТСЯ С ПРОБЛЕМОЙ "-БЛЯ"

public static String GetRubleAddition(double formattedResult) {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Метожды по код стайлу нужно называть в camelCase





Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Слишком огромные отступы, достаточно одной пустой строки


Scanner scanner = new Scanner(System.in);

int number = scanner.nextInt();
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Если передать в этот момент произвольную строку программа завершит работу с исключением. Перед тем, как прочитать инт из сканнера, нужно сначала проверить возможно ли это при помощи соответствующего метода hasNextInt

String name = scanner.nextLine();

System.out.println("Введите цену товара");
double price = scanner.nextDouble();
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Тоже самое только hasNextDouble

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