Repository navigation
Консольное приложение - #1
KotlinPunk wants to merge 1 commit into
Conversation
|
|
||
| public class Calculator { | ||
|
|
||
| public static String calculator(int N) { |
There was a problem hiding this comment.
старайся избегать использования статических функций, так как они не очищаются и всегда занимают место в памяти
There was a problem hiding this comment.
Понял. Но вроде в курсе ещё не встречал эту информацию)
| System.out.println("Товар успешно добавлен"); | ||
|
|
||
| System.out.println("Будут ли ещё товары? Если нет, пожалуйста, введите 'Завершить'."); | ||
| String z = scanner.next(); |
There was a problem hiding this comment.
не сокращай имена переменных и не используй аббревиатуры, к коду приходится возвращаться время от времени и бывает сложно понять, что значит то или иное сокращение через месяц или два
| } | ||
| public static String rub(double midSum) { | ||
| double v = Math.floor(midSum); | ||
| if (v % 100 >= 11 && v % 100 <=14) { |
There was a problem hiding this comment.
для ситуаций когда 2 и более условия, почти всегда лучше использовать switch/case
|
|
||
| try { | ||
| numberOfPeople = scanner.nextInt(); | ||
| } catch (InputMismatchException q) { |
There was a problem hiding this comment.
с одной стороны, когда ты используешь более конкретный класc ошибки, то ты как бы делаешь свой код более точным, но с другой стороны, если произойдет любая другая ошибка, то твой код упадет и всё сломается, поэтому если на это нет острой необходимости, то проще использовать просто класс Exception
There was a problem hiding this comment.
Вот тут я просто глядел форумы. Искал ответ, как лучше будет. Поэтому решил оставить так. Но я вас понял. Спасибо.
2 participants
Проектная работа №1. Консольное приложение.