Repository navigation
Conversation
| @@ -0,0 +1,91 @@ | |||
| import java.util.Scanner; | |||
|
|
|||
| public class newGoodGenerator { // Добавить с большой буквы; Классы в JAVA с большой буквы. | |||
There was a problem hiding this comment.
Да, классы в Java пишутся в PascalCase:)
| @@ -1,6 +1,16 @@ | |||
|
|
|||
| public class Main { | |||
| int result; | |||
There was a problem hiding this comment.
Переменная объявляется, но нигде не используется
| double totalPrice = 00.00; | ||
| public String allGoods = "Список всех товаров: " + "\n"; | ||
|
|
||
| public String generateNewGood() { |
There was a problem hiding this comment.
Этот метод возвращает строку, но при этом возвращаемое значение нигде не используется - метод просто вызывается без присваивания его переменной: generator.generateNewGood();. Лучше тогда сделать его void.
| return goodCreated; | ||
| } | ||
|
|
||
| public double priceSet(double price) { |
There was a problem hiding this comment.
Этот метод возвращает строку, но при этом возвращаемое значение нигде не используется - метод вызывается, в нем происходит присваивание this.price = price;, после чего return this.price; в пустоту. Лучше сделать void и ничего не возвращать
| return this.price; | ||
| } | ||
|
|
||
| public String nameSet(String name) { |
| System.out.println(allGoods); | ||
| } | ||
|
|
||
| public String showTotalPrice() { // Метод показывает общую форму заказа; |
|
@faritowich правки для вывода рубля внесены (196a8b3) остальные комментарии проработаю когда закрою ЖД и направлю вторую ПР (24.05) |
2 participants
Вторая попытка