Repository navigation
Спринт 2 Проверочная №1 от 21.10.2023 - #1
Nataliya2288 wants to merge 2 commits into
Conversation
| Product product = new Product(); | ||
| double sum = 0.00f; | ||
| String menu = ""; | ||
| Scanner scannerAdd = new Scanner(System.in); | ||
| int persons = Main.countOfGuests();// ссылка на число гостей из класса майн |
There was a problem hiding this comment.
Это тоже можно объявить в функции ниже, а не в классе
| if ((Math.floor(personPay) % 100 >= 11) && (Math.floor(personPay) % 100 <= 19)) { | ||
| String format = "Каждый гость должен: %.2f рублей"; | ||
| System.out.println(String.format(format, personPay)); | ||
| } else if (Math.floor(personPay) % 10 > 5) { | ||
| String format = "Каждый гость должен: %.2f рублей"; | ||
| System.out.println(String.format(format, personPay)); | ||
| } else if ((2 <= Math.floor(personPay) % 10) && (Math.floor(personPay) % 10 <= 4)) { | ||
| String format = "Каждый гость должен: %.2f рубля"; | ||
| System.out.println(String.format(format, personPay)); | ||
| } else { | ||
| String format = "Каждый гость должен: %.2f рубль"; | ||
| System.out.println(String.format(format, personPay)); | ||
| } |
There was a problem hiding this comment.
Тут код в каждой ветке повторяется, лучше повторения вынести в отдельную функцию
| @@ -0,0 +1,67 @@ | |||
| import java.util.Scanner; | |||
| public class Calculate { | |||
There was a problem hiding this comment.
Классы принято называть существительными
| String name; | ||
| double price; | ||
|
|
||
| void input() { |
There was a problem hiding this comment.
Функции принято глаголами называть, эту например можно назвать readInfo или fill
| public class Main { | ||
| public static void main(String[] args) { | ||
| System.out.println("Hello world!"); | ||
| static int Guests; |
There was a problem hiding this comment.
Названия переменных принято начинать с маленьких букв
| scanner.nextLine(); | ||
| } | ||
| } | ||
| return Guests; |
There was a problem hiding this comment.
Раз эта функция возвращает число человек, то саму переменную можно сделать private и пользоваться только этой функцией, чтобы не было путаницы при выборе чем пользоваться снаружи - переменной Guests или функцией countOfGuests
2 participants
Направляю на проверку первую работу