Repository navigation
My first HW - #1
realreired wants to merge 2 commits into
Conversation
SeveNChaK
left a comment
SeveNChaK
left a comment
There was a problem hiding this comment.
В некоторых файлах код прыгает, отступы не одинаковые - рекомендую использовать встроенные форматтер. Если ты программируешь в IDE от JetBrains, то можешь пройтись по всем файлам и нажать комбинацию (Windows: Shift + Ctrl + Alt + L, MacOS: Shift + Option + Command + L), затем выбрать, какую часть кода хочешь изменить, и выполнить команду. Среда разработки автоматически сделает нужные отступы и переносы (как она это делает можно изменить в настройках, но дефолтные значения вполне нормальные).
| // вы не должны ограничиваться только классом Main и можете создавать свои классы по необходимости | ||
|
|
||
| Basket.fillProductInfo(); | ||
|
|
There was a problem hiding this comment.
⏫ Кучу пустых строк, конечно, лучше не плодить)
|
|
||
| public class Rubles { | ||
|
|
||
| public static String writeRouble() { |
There was a problem hiding this comment.
0
| public static String writeRouble() { | ||
| String rouble; | ||
| double payPerFriend = 0; | ||
| int intPayPerFriend = (int) payPerFriend; |
There was a problem hiding this comment.
Учитывая, что payPerFriend == 0, то тут бы можно было тоже просто 0 написать. Скорее всего, payPerFriend должно приходить снаружи, а не создаваться здесь.
| rouble = " рубля"; | ||
| break; | ||
| default: | ||
| rouble = " рублей"; |
| System.out.println("Введите число большее одного"); | ||
| } else if (friends < 1) { | ||
| System.out.println("Продавцы съели сами. Попробуйте ещё раз"); | ||
| } else if (friends > 1) { |
There was a problem hiding this comment.
⏫ Тут можно просто else { break; }, так как другие случаи ты уже проверил
| } | ||
| } | ||
|
|
||
| public String writeRouble() { |
There was a problem hiding this comment.
Rubles
|
|
||
| Scanner productScanner = new Scanner(System.in); | ||
| int friends = Friends.comeFriends(); | ||
| String rouble = Rubles.writeRouble(); |
There was a problem hiding this comment.
| System.out.println("Что взяли? "); | ||
| while ((name = productScanner.next()) != null) { | ||
|
|
||
| nameList += " " + name + "\n"; |
There was a problem hiding this comment.
⏫ Так как String является неизменяемым объектом, то при каждом сложении строк создаются новые объекты, что расходует память. Для таких случаев лучше использовать StringBuilder
| sum += price; | ||
|
|
||
| payPerFriend = sum / friends; | ||
| double result = Double.parseDouble(String.format("%.2f", payPerFriend)); |
There was a problem hiding this comment.
2 participants
I spent a lot of time. Cant write anything else