Skip to content

Calc2 - #51

Open
Volkark wants to merge 3 commits into
Yandex-Practicum:masterfrom
Volkark:master
Open

Volkark wants to merge 3 commits into
Yandex-Practicum:masterfrom
Volkark:master

Conversation

@Volkark

@Volkark Volkark commented Sep 27, 2022

Copy link
Copy Markdown

Внесенные изменения:

  1. Слова "рубль","копейка","человек" склоняются в зависимости от количества и падежа
  2. При вводе чисел ввод проверяется, если введено не число, выдается сообщение об этом и просьба ввести еще раз.

…с латинскими буквами. С русскими не работает, поэтому добавил в проект возможность выхода из цикла по слову Complete, при вводе котрого регистр не важен.
1) Слова "рубль","копейка","человек" склоняются в зависимости от количества и падежа
2) При вводе чисел ввод проверяется, если введено не число, выдается сообщение об этом и просьба ввести еще раз.
1) Слова "рубль","копейка","человек" склоняются в зависимости от количества и падежа
2) При вводе чисел ввод проверяется, если введено не число, выдается сообщение об этом и просьба ввести еще раз.
Comment thread src/main/java/Main.java
System.out.println("Обратите внимание! Бесплатный товар, но мы его добавим");
} while(price < 0);
Calc.calculate(name, price);
System.out.println(jl;lString.format("Добавлен товар \"%s\" по цене %.2f, каждому теперь нужно внести уже по %s\n",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Тут случайно попало jl;l

Comment thread src/main/java/Main.java
String test1 = "Завершить";
String test2 = "Complete";
double price;
double summ;

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
@@ -1,8 +1,145 @@
public class Main {
import java.io.IOError;
import java.io.IOException;

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
// вы не должны ограничиваться только классом Main и можете создавать свои классы по необходимости
System.out.println("Привет Мир");
Scanner sc = new Scanner(System.in);
int n;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Можно использовать более понятное имя, например, person

Comment thread src/main/java/Main.java
int n;
String name;
String test1 = "Завершить";
String test2 = "Complete";

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

Comment thread src/main/java/Main.java

// Вывод денежной суммы в формате
// для var == 1: хх рубль/рубля/рублей xx копейка/копейки/копеек
// для var == 2: хх рублю/рубля/рублей xx копейке/копейки/копеек

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Можно завести две константы типа Int, например, NOMINATIVE (именительный падеж) и DATIVE (дательный), и передавать их. Таким образом по коду будет сразу понятно, для чего это

Comment thread src/main/java/Main.java
String Out = "рубль"; // именительный падеж
if (var == 2)
Out = "рублю"; // дательный падеж
int p1 = rub % 100;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Можно дать имя, например, latestDigits

Comment thread src/main/java/Main.java
if ((p1 >= 5) && (p1 <= 20)) // Обработка исключений от 11 до 14: рублей
Out = "рублей";
else {
int p2 = p1 % 10;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Можно не заводить вторую переменную, а изменять предыдущую, просто записав p1 = p1 % 10; или p1 %= 10;

Comment thread src/main/java/Main.java
if (var == 2)
Out = "копейке"; // дательный падеж

int p1 = kop % 100;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

По идее, копеек не может быть больше 100, тогда это преобразование не нужно

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