Skip to content

Готовая первая практическая работа - #1

Open
ILabzin wants to merge 4 commits into
mainfrom
dev
Open

ILabzin wants to merge 4 commits into
mainfrom
dev

Conversation

@ILabzin

@ILabzin ILabzin commented Oct 2, 2023

Copy link
Copy Markdown
Owner

No description provided.

@nasibullin1412

Copy link
Copy Markdown

⚠️ Если ввести строку при запросе цены блюда, то эта строка будет использоваться как название следующего блюда, а так быть не должно.

image

@nasibullin1412

Copy link
Copy Markdown

⚠️Должно быть 111 рублей, а программа выводит 111 рубль

image

Comment thread src/main/java/Calculation.java Outdated
public class Calculation {
public static String everyPeoplePrice(double price ) {

String readysum;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🍏Название переменных должно быть в camelCase, то есть readySum, а лучше resultSum

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

String readysum;

double price2 = Math.floor(price);

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/Calculation.java Outdated
readysum = "рубля.";
} else {readysum = "рублей.";
}
if(price2>=10 && price2<=20) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️Будет работать только для интервала от 10 до 20, для 111, 112, ... работать не будет

Comment thread src/main/java/Main.java Outdated
public class Main {
public static void main(String[] args) {
System.out.println("Hello world!");
Scanner sc = new Scanner(System.in);

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/PeopleNumber.java Outdated
}
} else {
System.out.println("Вы ввели не число");
count();

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🍏Не уверен, что здесь уместна рекурсия, лучше такие штуки делать в бесконечном цикле

@nasibullin1412

Copy link
Copy Markdown

⚠️Программа заканчивает свою работу при вводе строки, а должна перезапрашивать данные, пока не получит корректные

image

@nasibullin1412

Copy link
Copy Markdown

⚠️Если вводить число, то программа вообще падает с ошибкой

image

readySum = "рублей.";
}
if(priceName>=10 && priceName<=20) {
readySum = "рублей.";

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🍏priceName, который является остатком от деления на 10, никогда не будет иметь значение больше 10, следовательно - это недостижимый код

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

while (true) {
System.out.println("Введите цену блюда:");
if (scanner.hasNextInt()) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️Почему тут появилась функция hasNextInt, когда цена блюда может быть не целым числом?

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