Skip to content

vvverman tochno pull request... da? - #1

Open
vvverman wants to merge 7 commits into
mainfrom
dev
Open

vvverman wants to merge 7 commits into
mainfrom
dev

Conversation

@vvverman

Copy link
Copy Markdown
Owner

и снова в бой

@gusar-off gusar-off left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

Comment thread src/main/java/Main.java Outdated
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, будут иметь разные окончания

Comment thread src/main/java/Main.java Outdated

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

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

Comment thread src/main/java/Main.java Outdated




Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

Comment thread src/main/java/Main.java Outdated

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

Comment thread src/main/java/Main.java Outdated
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