Search code, repositories, users, issues, pull requests...

Copy link

@Saharok1209 Saharok1209 commented Oct 24, 2022

Привет, это моя проектная работаNo1

Saharok1209 added 4 commits

October 24, 2022 18:33


 Rychkov PW1

49775af


 Rychkov PW1

b3d0747


 Rychkov PW1 v2

1c8387e


 Merge remote-tracking branch 'origin/master'

fa838ba

# Conflicts:
#	src/main/java/Main.java

MagicUnderHood

MagicUnderHood reviewed

Oct 25, 2022

View reviewed changes

src/main/java/Calculation.java

public String calculationNameProduct(){

for (int i = 0; i < product.getNameProduct().size(); i++) {

allProducts = allProducts + " " + product.getNameProduct().get(i);

}return allProducts;

Copy link

@MagicUnderHood MagicUnderHood Oct 25, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Можешь применить в студии автоформатирование (в выбранном файле, сверху вкладка Code - Reformat Code, либо Ctrl+Alt+L), тогда автоматически код выправится

System.out.println("Your price per guest: " + calculation.calculationPriceProduct() / people + " 'rubles.penni'");

}

public static int people(){

Copy link

@MagicUnderHood MagicUnderHood Oct 25, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Студия подсказывает, что возвращаемый тип функции никак не используется, тогда можно его убрать, или всё-таки использовать. Методы лучше называть с глаголом, например, readPeople или getPeople (это если возвращаемый результат будет использоваться)

}

return people;

}

public static Product addProduct(){

Copy link

@MagicUnderHood MagicUnderHood Oct 25, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Студия подсказывает, что возвращаемый тип функции никак не используется, тогда можно его убрать, или всё-таки использовать

@@ -0,0 +1,22 @@

import java.util.ArrayList;

public class Product {

Copy link

@MagicUnderHood MagicUnderHood Oct 25, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Класс содержит информацию не о продукте, а обо всех продуктах, так что можно назвать его Menu или хотя бы Products

import java.util.ArrayList;

public class Product {

static ArrayList<String> nameProduct = new ArrayList<>();

Copy link

@MagicUnderHood MagicUnderHood Oct 25, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Поскольку в этой переменной хранится не имя одного продукта, а все продукты, лучше ее назвать productNames

public class Product {

static ArrayList<String> nameProduct = new ArrayList<>();

static ArrayList<Double> priceProduct = new ArrayList<>();

Copy link

@MagicUnderHood MagicUnderHood Oct 25, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Аналогично с nameProduct

Product.priceProduct.add(priceProduct);

}

public ArrayList<String> getNameProduct() {

Copy link

@MagicUnderHood MagicUnderHood Oct 25, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Методы возвращают список имен продуктов, поэтому тоже лучше переименовать понятнее

}

}else{

System.out.println("invalid input");

scan.nextInt();

Copy link

@MagicUnderHood MagicUnderHood Oct 25, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Здесь нужен просто scan.next();, иначе приложение упадет, поскольку мы проверили, что !scan.hasNextInt()