Repository navigation
Консольное приложение №1 - #1
exMarsianin wants to merge 3 commits into
Conversation
| public class Main { | ||
| public static void main(String[] args) { | ||
| System.out.println("Hello world!"); | ||
| Scanner scan = new Scanner(System.in); |
There was a problem hiding this comment.
Рекомендация: - хорошей практикой является делать вызов scanner.close() после того, как сканнер больше не используется. Это необходимо для того, что бы этот объект не потреблял ресурсы, тогда когда это уже не требуется.
| @@ -0,0 +1,38 @@ | |||
|
|
|||
|
|
|||
| public class Calculator { | |||
There was a problem hiding this comment.
В java принят определенный стиль форматирования кода. Если не вдаваться в подробности, то отформатировать код можно быстрой комбинаций клавиш Ctrl+Alt+L(Windows) или (⌘+⌥+L)(Mac)
| this.people = people; | ||
| } | ||
|
|
||
| void addmember(Product product){ |
There was a problem hiding this comment.
Рекомендация: В яве и котлине принят нейминг в стиле CamelCase, т.е. лучше переименовть в addMember ( но имя метода должно быть с маленькой буквы)
| } | ||
|
|
||
| public static String getSuffix(double rubles) { | ||
| if (rubles % 100 >= 11 && rubles%100 <= 14){ |
There was a problem hiding this comment.
rubles % 100 и rubles % 10 считаются несколько раз, чтоит вынести эти значения в локальные переменные метода
2 participants
Попытка номер 2 :)