Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Домашнее задание #106

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
23 changes: 23 additions & 0 deletions src/main/java/Calculator.java
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
public class Calculator {
float priceForPerson;
String outputCalculate;
String rub;

Choose a reason for hiding this comment

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

Можно сделать 3 переменные приватными


void calculate(int persons, float totalPrice) {
priceForPerson = totalPrice/persons;
}

void correctOutput() {
if (priceForPerson%10 == 1) {

Choose a reason for hiding this comment

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

priceForPerson%10 считается несколько раз, лучше посчитать 1 раз и записать в переменную

rub = "рубль";
}
else if (priceForPerson%10 == 2 && priceForPerson%10 == 3 && priceForPerson%10 == 4){

Choose a reason for hiding this comment

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

Можно упростить условие: priceForPerson%10 >= 2 && priceForPerson%10 <= 4

rub = "рубля";
}
else {
rub = "рублей";

Choose a reason for hiding this comment

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

Стоит учесть, что для чисел 11-19 окончание слова - "рублей". Для обработки можно до этого if-else написать ещё один, с проверкой (например, priceForPerson%100 >= 11 и <= 19, или другим вариантом)

}
outputCalculate = String.format("%.2f", priceForPerson);
System.out.println("Каждый человек должен заплатить по: " + outputCalculate + " " + rub);

Choose a reason for hiding this comment

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

Хорошо, что повторяющийся код вынесен за if-else, молодец

}
}
33 changes: 29 additions & 4 deletions src/main/java/Main.java
Original file line number Diff line number Diff line change
@@ -1,8 +1,33 @@
import java.util.Scanner;

public class Main {

public static void main(String[] args) {
// ваш код начнется здесь
// вы не должны ограничиваться только классом Main и можете создавать свои классы по необходимости
System.out.println("Привет Мир");
Scanner scanner = new Scanner(System.in);
System.out.println("На скольких человек необходимо разделить счёт?");
while (!scanner.hasNextInt()){ // Проверяем, ввели ли значение типа int
scanner.next();
System.out.println("Вы ввели неверное значение, попробуйте еще раз: ");
}
int amountOfPersons = scanner.nextInt();
while (amountOfPersons < 2) { // если значенние 0,1 или отрицательное, запрашиваем ввод снова

Choose a reason for hiding this comment

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

Можно вместо 3 while сделать так: один бесконечный цикл while(true), до него объявить переменную amountOfPersons, после него идет строчка 20 с выводом количества персон. В самом бесконечном цикле if-else, в условии проверяем, ввели ли значение типа int, если нет, то как у тебя строчки 9-10, если да - считываем инт значение в amountOfPersons и затем проверяем в if-else, если <2, то как ты выводишь, что считать нечего, иначе - break; выходим из цикла. Это немного уберет дублирование ввода при неверном значении

System.out.println("В таком случае нет смысла считать :\nВведите значение еще раз: ");
while (!scanner.hasNextInt()) {
scanner.next();
}

Choose a reason for hiding this comment

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

Можно добавить System.out.println("Вы ввели неверное значение, попробуйте еще раз: ");, иначе будет запрашиваться число, но пользователю ничего не выведется, он может не понять, что от него ждут

amountOfPersons = scanner.nextInt();
}
System.out.println("Кол-во персон: " + amountOfPersons);

Product products = new Product();
products.saveProducts();
Calculator calculator = new Calculator();
calculator.calculate(amountOfPersons, products.totalSumOfProducts);
calculator.correctOutput();

}

}
}



43 changes: 43 additions & 0 deletions src/main/java/Product.java
Original file line number Diff line number Diff line change
@@ -0,0 +1,43 @@
import java.util.Scanner;

public class Product {
Scanner scanner = new Scanner(System.in);
float priceOfProduct; // переменная отвечающая за стоимость каждого товара
float totalSumOfProducts; // переменная отвечающая за суммирование
String nameOfProduct; // строка содержащая название продукта
String totalProducts = ""; // строка сохраняющая все продукты
String endOfCalculate; // строка для завершения бесконечного цикла

Choose a reason for hiding this comment

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

Тут тоже можно сделать переменные приватными, если вне класса они не используются


void saveProducts() {
while(true) {
System.out.println("Введите название товара: ");
nameOfProduct = scanner.next();
totalProducts = totalProducts.concat(nameOfProduct + "\n");
System.out.println("Введите стоимость товара в формате 'рубли.копейки' [10.45, 11.40]");
testFloatPrice();
totalSumOfProducts += priceOfProduct;
System.out.println("Товар успешно добавлен");
System.out.println("Желаете ли Вы добавить еще 1 товар? (Введите 'Завершить' для завершения программы, любой другой ввод продолжит выполнение программы)");
endOfCalculate = scanner.next();
if (endOfCalculate.equalsIgnoreCase("Завершить"))
break;
}
System.out.println("Итоговая сумма всех товаров: " + totalSumOfProducts);
System.out.println("Добавленные товары:\n" + totalProducts);
}

void testFloatPrice() { // проверка значения float
while(!scanner.hasNextFloat()) {

Choose a reason for hiding this comment

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

Можно преобразовать алгоритм метода по аналогии с вводом количества человек

System.out.println("Неверный формат, попробуйте еще раз:");
scanner.next();
}
priceOfProduct = scanner.nextFloat();
while(priceOfProduct < 0) {
System.out.println("Значение меньше нуля, попробуйте еще раз:");
while (!scanner.hasNextFloat()) {
scanner.next();
}

Choose a reason for hiding this comment

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

Тут тоже можно добавить System.out.println("Вы ввели неверное значение, попробуйте еще раз: ");

priceOfProduct = scanner.nextFloat();
}
}
}