Repository navigation
Rychkov PW1 - #84
Rychkov PW1#84Saharok1209 wants to merge 4 commits into
Conversation
# Conflicts: # src/main/java/Main.java
| public String calculationNameProduct(){ | ||
| for (int i = 0; i < product.getNameProduct().size(); i++) { | ||
| allProducts = allProducts + " " + product.getNameProduct().get(i); | ||
| }return allProducts; |
There was a problem hiding this comment.
Можешь применить в студии автоформатирование (в выбранном файле, сверху вкладка Code - Reformat Code, либо Ctrl+Alt+L), тогда автоматически код выправится
| System.out.println("Your price per guest: " + calculation.calculationPriceProduct() / people + " 'rubles.penni'"); | ||
|
|
||
| } | ||
| public static int people(){ |
There was a problem hiding this comment.
Студия подсказывает, что возвращаемый тип функции никак не используется, тогда можно его убрать, или всё-таки использовать. Методы лучше называть с глаголом, например, readPeople или getPeople (это если возвращаемый результат будет использоваться)
| } | ||
| return people; | ||
| } | ||
| public static Product addProduct(){ |
There was a problem hiding this comment.
Студия подсказывает, что возвращаемый тип функции никак не используется, тогда можно его убрать, или всё-таки использовать
| @@ -0,0 +1,22 @@ | |||
| import java.util.ArrayList; | |||
|
|
|||
| public class Product { | |||
There was a problem hiding this comment.
Класс содержит информацию не о продукте, а обо всех продуктах, так что можно назвать его Menu или хотя бы Products
| import java.util.ArrayList; | ||
|
|
||
| public class Product { | ||
| static ArrayList<String> nameProduct = new ArrayList<>(); |
There was a problem hiding this comment.
Поскольку в этой переменной хранится не имя одного продукта, а все продукты, лучше ее назвать productNames
|
|
||
| public class Product { | ||
| static ArrayList<String> nameProduct = new ArrayList<>(); | ||
| static ArrayList<Double> priceProduct = new ArrayList<>(); |
| Product.priceProduct.add(priceProduct); | ||
| } | ||
|
|
||
| public ArrayList<String> getNameProduct() { |
There was a problem hiding this comment.
Методы возвращают список имен продуктов, поэтому тоже лучше переименовать понятнее
| } | ||
| }else{ | ||
| System.out.println("invalid input"); | ||
| scan.nextInt(); |
There was a problem hiding this comment.
Здесь нужен просто scan.next();, иначе приложение упадет, поскольку мы проверили, что !scan.hasNextInt()
|
|
||
| if(Math.round(productPrice)%10 == 1 && Math.round(productPrice)%100 != 11){ | ||
| return "ruble"; | ||
| }else if((Math.round(productPrice)%10) <= 4 && (Math.round(productPrice)%100 >= 2) && (Math.round(productPrice)%100)/10 != 2){ |
There was a problem hiding this comment.
Повторяющийся код Math.round(productPrice) и Math.round(productPrice)%10, Math.round(productPrice)%100 можно вынести в переменные
2 participants
Привет, это моя проектная работа№1