Conversation
src/main/java/Calculator.java
Outdated
| public class Calculator { | ||
|
|
||
| int persons; | ||
| Double summ = 0.0; |
There was a problem hiding this comment.
Double можно просто написать double, как и другие численные значения
There was a problem hiding this comment.
Спасибо, не понятно почему я вообще с большой буквы написал.
| switch ((int) lastDigit) { | ||
|
|
||
| case 1: | ||
| return "рубль"; |
There was a problem hiding this comment.
Неверно преобразовывает склонение слова рубль
Не думаю, что стоит перечислять сколько заплатит каждый. Это же будет одно и то же число
There was a problem hiding this comment.
А как же быть с ситуацией когда будет, например 3 человека, а сумма 133 рубля? т.е. при делении будет 44.33x3 = 132.99. В моём коде последнему покажет система не 44.33, а 44.34. Т.е. будет 44.33, 44.33, 44.34. Все копейки учтены.
There was a problem hiding this comment.
Хм, кейс редкий, но ты молодец что подумал об этом
There was a problem hiding this comment.
There was a problem hiding this comment.
Мне кажется баг не пофикшен.
Очень стыдно, честно. Отправляю третий, последний раз.
There was a problem hiding this comment.
Проверил, теперь корректно. Надо было банально потратить время и хорошо протестировать, зациклился на значениях свыше 100 и остатке от деления % 100. Буду впредь внимательнее.
src/main/java/Formatter.java
Outdated
| @@ -0,0 +1,32 @@ | |||
| public class Formatter { | |||
|
|
|||
| //public String RubDesc = ""; | |||
There was a problem hiding this comment.
Комментарии, не поясняющие код, лучше не пушить
There was a problem hiding this comment.
Спасибо, забыл почистить мусор.
src/main/java/Calculator.java
Outdated
| System.out.println(arrayOfProductsPrices.get(i).productName); | ||
| } | ||
|
|
||
| void cutSummPpl() { |
There was a problem hiding this comment.
cutSummPpl - если имелось ввиду "Разделить счет на всех", то я бы назвал splitSumToPerson
There was a problem hiding this comment.
Спасибо, слово не пришло нормальное для функции.
|
Не понял как отправить пулл реквест повторно. Мои коммиты видны? |
Проектная работа №1.
Ткач Евгений.