Conversation
# Conflicts: # src/main/java/Main.java
# Conflicts: # src/main/java/Main.java
# Conflicts: # src/main/java/Calculator.java
| import java.util.Scanner; | ||
|
|
||
| public class Calculator { | ||
| static int persons = 0; |
There was a problem hiding this comment.
старайся по возможности не использовать статические переменные, они живут столько же, сколько живет само приложение. В этом месте можно вполне использовать без модификатора static
| static int persons = 0; | ||
| static String menuSum = ""; | ||
| static double priceSum = 0; | ||
| public static void calculator() { |
There was a problem hiding this comment.
Имена функций должны содержать в себе действие. Функция делает что-либо. Хорошо помогает практика переводить имена функций дословно, если после перевода понятно чем занимается функция, то название хорошее
| } | ||
| } else { | ||
| System.out.println("Введите число"); | ||
| scanner.next(); |
There was a problem hiding this comment.
Рекомендация: - хорошей практикой является делать вызов scanner.close() после того, как сканнер больше не используется. Это необходимо для того, что бы этот объект не потреблял ресурсы, тогда когда это уже не требуется.
|
|
||
| public static String formatRub(){ | ||
| int priceRemainder = (int)Math.floor(rubAverage); | ||
| if (priceRemainder % 10==1) |
There was a problem hiding this comment.
В ситуациях, когда у тебя есть более 2-3 условий лучше использовать оператор switch-case - это улучшит читабельность кода и его легче будет расширить или изменить в дальнейшем
ПР ввод пользователя