Repository navigation
Conversation
|
|
||
|
|
||
|
|
||
| class Product |
There was a problem hiding this comment.
- Форматирование кода - в Java принят немного другой стиль форматирования кода. Если не вдаваться в детали, то легко и быстро отформатировать код в Android Studio можно следующей комбинацией клавиш: в Windows Ctrl + Alt + L , в MacOs ⌘ + ⌥ + L.
|
|
||
|
|
||
| import java.util.Scanner; | ||
|
|
There was a problem hiding this comment.
В оформлении кода на Java принято делать так: один класс-один файл
| return "ей"; | ||
| } else { | ||
| switch (money) { | ||
| case 1: |
There was a problem hiding this comment.
рекомендую повторить синтаксис switch - case. Кажется, здесь необходимо использование break
| } | ||
|
|
||
| } | ||
| if (members <=0) |
There was a problem hiding this comment.
здесь взаимоисключающие условия, оптимальнее использовать конструкцию if-else
# Conflicts: # src/main/java/Main.java
|
|
||
| Product productList = new Product(); | ||
|
|
||
|
|
| } | ||
|
|
||
|
|
||
| if (members > 1) { |
There was a problem hiding this comment.
тк members < 0 и members > 1 - взаимоисключающие условия, то их стоит реализовать через блок if - else if
| productList.price = 0; //обязательная инициализация для компилятора | ||
|
|
||
|
|
||
| String tmp = scanner.nextLine(); // scanner.nextInt() нельзя выдаст ошибку, т.к. введён не int |
There was a problem hiding this comment.
При проверке значенийй удобно использовать методы scanner.hasNextInt()/hasNextFloat() или try-catch
|
|
||
| productList.listName += input; | ||
|
|
||
| productList.listName = productList.listName + "=" + String.format("%.2f", productList.price) + "р.\n"; |
There was a problem hiding this comment.
Рекомендация: строки лучше соединять через stringBuilder - это оптимальнее по использованию памяти
| int money = (int) summ; | ||
|
|
||
|
|
||
| if (money % 100 >= 5 && money % 100 <= 20) { |
There was a problem hiding this comment.
money % 100 используется больше одного раза - стоит вынести в переменную
# Conflicts: # src/main/java/Main.java
# Conflicts: # src/main/java/Main.java
3 participants
No description provided.