Repository navigation
Pull Request к практическому заданию - #8
LeonidVonarab wants to merge 7 commits into
Conversation
| this.sum = sum; | ||
| } | ||
|
|
||
| public static void calculate(String names, float sum, int number) { |
There was a problem hiding this comment.
тут нет необходимости использовать статику (ключевое слово static)
There was a problem hiding this comment.
Так же тебе не надо передавать переменные класса в функцию, у тебя к ним и так есть доступ внутри функции.
| if (name.equalsIgnoreCase("Завершить")) { | ||
| System.out.println("Спасибо, что заглянули! Хорошего дня!"); | ||
| } else { | ||
| while (!(name.equalsIgnoreCase("Завершить"))) { |
There was a problem hiding this comment.
если пользователь напишет "завершить на после ввода названия первого продукта, то ему не выведется прощальное сообщение "Спасибо, что заглянули..."
| } | ||
|
|
||
| public static void conclude(int n, float s){ | ||
| float l = n; |
There was a problem hiding this comment.
Здесь я полагаю происходит преобразование из типа int в float, это можно сделать проще, изменив следующую страну таким образом: float split = s/(float) n;
| String str = Integer.toString(a); | ||
| char lastLetter = str.charAt(str.length() - 1); |
There was a problem hiding this comment.
не обязательно преобразовывать число в строку, что бы получить последний символ, для это можно использовать различные виды деления, например целочисленное деление или деление с остатком
| System.out.println(String.format("Вы набрали продуктов на сумму: %.2f%s", sum, rightRubles(sum))); | ||
| } | ||
|
|
||
| public static String rightRubles(float sum) { |
There was a problem hiding this comment.
Так же здесь обработаны не все кейсы, например при сумме 111, будет не верное окончание
| } | ||
| } | ||
|
|
||
| public static void conclude(int n, float s){ |
There was a problem hiding this comment.
тут нет необходимости использовать статику (ключевое слово static)
| System.out.println(String.format("Каждому надо заплатить по %.2f%s",split, rightRubles(split))); | ||
| } | ||
|
|
||
| public static void getResult(String names,float sum){ |
There was a problem hiding this comment.
тут нет необходимости использовать статику (ключевое слово static)
| this.number = number; | ||
| } | ||
|
|
||
| public static int countBuyers(int number) { |
There was a problem hiding this comment.
тут нет необходимости использовать статику (ключевое слово static)
There was a problem hiding this comment.
Так же тебе не надо передавать переменные класса в функцию, у тебя к ним и так есть доступ внутри функции.
2 participants
No description provided.