Repository navigation
new project "calculator for group of people" - #2
Selin-Eglence wants to merge 2 commits into
Conversation
| } No newline at end of file | ||
|
|
||
|
|
||
| public static int people() { |
There was a problem hiding this comment.
- private
- По хорошему надо выносить всю эту логику из класса Main. этот класс обычно призван только стартануть программу, а все остальное должно быть сделано в специальных классах.
| double productPrice = scanner.nextDouble(); | ||
| System.out.println("Товар успешно добавлен в калькулятор."); | ||
| scanner.nextLine(); | ||
| Items items = new Items(productPrice, productName); |
There was a problem hiding this comment.
Сейчас тут создается экземпляр класса Items, в котором все поля static, то есть для обращения к ним не нужен создавать экземпляр класса. Тут вообще нет необходимости передавать в методы productsSum и addNewProducts экземпляры класса Items, можно просто в этим методах вызывать Items.costOfProduct() и другие методы.
Так что рекомендую почитать про static и немного подправить логику. А вообще - в разработке под андроид мы обычно работаем с объектами, то есть создаем их и т.д., так что я рекомендую попрбовать переписать программу вообще без использования static - будет полезно
|
|
||
| public static void paymentPerPerson() { | ||
| Scanner scanner = new Scanner(System.in); | ||
| Calculator calculator = new Calculator(); |
There was a problem hiding this comment.
оба объекта создаются и не используются
| String item = ""; | ||
| static double sum = 0; | ||
|
|
||
| public double productsSum(Items items) { |
There was a problem hiding this comment.
метод возвращает double, но при этом возвращаемое значение нигде не используется
2 participants
No description provided.