Conversation
Тестовое задание спринт 2
src/main/java/Main.java
Outdated
| Scanner scanner = new Scanner(System.in); | ||
|
|
||
| System.out.println("На сколько человек делить счет?"); | ||
| int Quantity = AskQuantity(scanner); |
There was a problem hiding this comment.
Переменные по код стайлу нужно называть в camelCase, как и функции, только классы можно в PascalCase
src/main/java/Main.java
Outdated
| for (Product TheProduct : Goods) { | ||
| System.out.println(String.format("Товар %d: %s цена: %.2f", i, TheProduct.ProductName, TheProduct.ProductPrice)); | ||
| Sum += TheProduct.ProductPrice; | ||
| i++; |
There was a problem hiding this comment.
Тогда лучше такой цикл for (int i = 1; i < goods.size(); i++)
src/main/java/Main.java
Outdated
| } | ||
|
|
||
| private static String TakeEnding (int SumPerPeopleInt){ | ||
| int Remainder = SumPerPeopleInt % 10; |
There was a problem hiding this comment.
src/main/java/Main.java
Outdated
| Calculate(Goods, Quantity); | ||
| } | ||
|
|
||
| private static class Product{ |
There was a problem hiding this comment.
Лучше вынести в отдельный файл, чтобы весь код не был написан в одном лишь классе
|
src/main/java/Main.java
Outdated
|
|
||
| private static Product RequestGoods(Scanner scanner){ | ||
|
|
||
| BufferedReader br = new BufferedReader(new InputStreamReader(System.in)); |
There was a problem hiding this comment.
Еще добавлю, здесь ты попробовал для общения с пользовательским вводом использовать BufferedReader, это круто, потому что он гораздо эффективнее, чем сканнер. Но поскольку во всех других местах используешь сканнер, не вижу смысла использовать это здесь. Можешь оставить в качестве эксперимента, но в дальнейшем не стоит так делать. Либо одно, либо другое
Тестовое задание спринт 2 ver 1.2
Задание выполнил достаточно объемно для получение большего кол-ва фидбэка.
Дополнительные вопросы: