Repository navigation
Практическая работа 1 - #134
EkimovaDaria2004 wants to merge 2 commits into
Conversation
| System.out.println("Привет Мир"); | ||
| Scanner scanner = new Scanner(System.in); | ||
| int people = addPerson(scanner); | ||
| Calculator calculator = new Calculator(); |
There was a problem hiding this comment.
Так как на протяжении жизни программы у нас используется один экземпляр класса Calculator, можно сделать его метод productsLists() static, чтобы не приходилось создавать единственный экземпляр калькулятора и напрямую вызывать метод Calculator.productsLists(scanner);
| @@ -0,0 +1,26 @@ | |||
| import java.util.Scanner; | |||
| public class Calculator { | |||
| public void productsLists( Scanner scanner) { | |||
There was a problem hiding this comment.
В Java принята конвенция, по которой методы нучше называть глаголами. Например, можно было было назвать createProductsList или как-то так.
| productsLists(scanner); | ||
| case "Завершить": | ||
| System.out.println(answer); | ||
| System.out.println("Общая сумма"+sum); |
There was a problem hiding this comment.
Такие подряд идущие println лучше объединять в один, делая перенос строки при помощи \n. Некритично, но для небольшой экономии производительности один вызов лучше, чем два
|
Также два момента, которые я забыл упомянуть в комментарии к заданию.
|
4 participants
No description provided.