Repository navigation
Created BillSplitter console app - #130
nurlankkz2502 wants to merge 2 commits into
Conversation
| List<Product> productList; | ||
|
|
||
| BillSplitter(int count) { | ||
| totalAmount = 0; |
There was a problem hiding this comment.
Проще присваивать значение totalAmount сразу при объявлении, а не в конструкторе. А в данном случае можно вообще явно не присваивать ноль, потому что в java при объявлении примитивной переменной без присваивания ей явного значения у неё и так будет дефолтное значение - для инта это 0.
См. здесь в разделе Default Values:
https://docs.oracle.com/javase/tutorial/java/nutsandbolts/datatypes.html
| BillSplitter(int count) { | ||
| totalAmount = 0; | ||
| personCount = count; | ||
| productList = new Vector<>(); |
There was a problem hiding this comment.
Также лучше сразу присвоить при объявлении
| } else if (integralPart < 10) { | ||
| result = String.format("%.2f", value) + " рубля"; | ||
| } else { | ||
| result = String.format("%.2f", value) + " рублей"; |
There was a problem hiding this comment.
Есть принцип, который называется DRY - Don't repeat yourself. По этому принципу, если есть многократные одинаковые фрагменты кода, то их лучше вынести отдельно. Здесь, например, можно отдельно вынести result = String.format("%.2f", value) и к нему в зависимости от суммы добавлять "рубль", "рубля" или "рублей"
|
|
||
| productList.add(new Product(productName, productPrice)); | ||
|
|
||
| totalAmount = totalAmount + productPrice; |
There was a problem hiding this comment.
Можно ещё использовать конструкцию totalAmount += productPrice;
| totalAmount = totalAmount + productPrice; | ||
|
|
||
| System.out.println("Продукт " + productName + " был добавлен"); | ||
| System.out.println("Вы хотите добавить еще один товар?"); |
There was a problem hiding this comment.
Подряд идущие println лучше объединять в один, делая перенос строки при помощи \n
3 participants
No description provided.