Repository navigation
Conversation
| static ArrayList<Double> price = new ArrayList<>(); | ||
| static double score = 0; | ||
|
|
||
| public static void calculate() { |
There was a problem hiding this comment.
static можно убрать, поскольку ты вызываешь метод от создаваемого объекта класса Calculate
| import java.util.Scanner; | ||
|
|
||
| public class Calculate { | ||
| static Const aConst=new Const(); |
There was a problem hiding this comment.
static лучше убрать у всех полей и методов класса, необходимости в них нет
| //блок заполнения листа и подсчета счета | ||
| System.out.println(aConst.ADD_PRODUCT_AND_PRICE); | ||
| String nameProduct = scanner.next();//название продукта | ||
| Double priceProduct = scanner.nextDouble();//цена |
There was a problem hiding this comment.
Пользователь может ввести на вопрос про стоимость товаров вместо числа строку. Для количества человек ты обрабатываешь такую ситуацию через hasNextInt, стоит добавить обработку и для стоимости
| { | ||
| System.out.println("Вы ввели не целое число"); | ||
| scanner.nextLine(); | ||
| }} |
There was a problem hiding this comment.
Можешь применить в студии автоформатирование (в выбранном файле, сверху вкладка Code - Reformat Code, либо Ctrl+Alt+L), тогда автоматически код выправится.
| System.out.printf("Каждый должен заплатить: %.2f Рубля%n",score1); | ||
| break; | ||
| default: | ||
| System.out.printf("Каждый должен заплатить: %.2f Рублей%n",score1); |
There was a problem hiding this comment.
Код System.out.printf("Каждый должен заплатить: %.2f <...> %n",score1); повторяется, лучше его вынести за if-else, оставив только определение слова (можно записывать в переменную)
No description provided.