Conversation
…s) and Product calling methods in main
| this.guests = guests; | ||
| } | ||
|
|
||
| static int getAmountOfGuests(Scanner scanner) { |
There was a problem hiding this comment.
старайся использовать нестатические функции, так так статические функции постоянно висят в памяти и соответственно занимают её, чего стоит избегать, иначе на устройстве может банально закончится память и твоё приложение закроется с ошибкой
| } | ||
|
|
||
| static int getAmountOfGuests(Scanner scanner) { | ||
| int guests; |
There was a problem hiding this comment.
по возможности избегай ситуаций, когда твоя внутренняя переменная называется точно так же, как и внешняя, иначе очень легко могут возникнуть ошибки, когда нужно было присвоить значение внешней, а присвоилось внутренней и наоборот
src/main/java/Calculator.java
Outdated
| static void printResultingSum(Calculator calculator) { | ||
| double resultSum = calculator.sum / calculator.guests; | ||
| int resultSumIntDiv10 = ((int) resultSum) % 10; | ||
| if (resultSumIntDiv10 == 1) { |
There was a problem hiding this comment.
попробуй переделать на оператор switch-case, для ситуаций, когда более 2 условий он подходит гораздо лучше и код становится более понятным
src/main/java/Calculator.java
Outdated
| double resultSum = calculator.sum / calculator.guests; | ||
| int resultSumIntDiv10 = ((int) resultSum) % 10; | ||
| if (resultSumIntDiv10 == 1) { | ||
| System.out.println(String.format("%.2f рубль", resultSum)); |
There was a problem hiding this comment.
у тебя повторяется один и тот же кусок кода с выводом строки, попробуй сделать функцию, которая будет возвращать правильно склоненное слово "рубль"
No description provided.