Skip to content

Пулл реквест ПР #1 - #94

Open
Gammus87 wants to merge 2 commits into
Yandex-Practicum:masterfrom
Gammus87:master
Open

Gammus87 wants to merge 2 commits into
Yandex-Practicum:masterfrom
Gammus87:master

Conversation

@Gammus87

Copy link
Copy Markdown

No description provided.

Comment thread src/main/java/Main.java
// вы не должны ограничиваться только классом Main и можете создавать свои классы по необходимости
System.out.println("Привет Мир");
System.out.println("Делитель счёта v.100500. Привет!");
Calculate cal = new Calculate();

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Не получилось. Не совсем понял.

Comment thread src/main/java/Main.java Outdated

}
}
public static class Calculate{

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Лучше вынести класс отдельно, а не делать вложенным классом, так как сказывается на читаемости кода.

Comment thread src/main/java/Main.java Outdated
persons= sc.nextInt();
if (persons<=1){
System.out.println("Людей не может быть меньше двух! Попробуйте ещё раз!");
continue;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

continue здесь не нужен, так как он необходим для завершения итерации цикла и начале новой итерации. Здесь continue последняя команда цикла, то есть итерация и без него закончится

Comment thread src/main/java/Main.java Outdated
if (getMore.equalsIgnoreCase("нет")) {
break;
}else{
continue;}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

continue также не нужен

Comment thread src/main/java/Main.java Outdated
public void payPeople(){

System.out.println("Добавленные товары: \n"+items);
double payPerPerson=total/persons;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

неиспользуемая переменная

Comment thread src/main/java/Main.java Outdated
}
System.out.println(endRuble); // это шок
//Я не понимаю как иначе вывести переменную endRuble. Её нельзя добавить в верхний метод.
} //Если поменять параметры метода, то потом он просто не встанет в Main.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Можно и не вызывать в main, а вызывать в payPeople(). Это же служебный метод, обслуживающий payPeople - так что строго говоря к Main он не имеет особого отдельного отношения

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants