Repository navigation
Калькулятор счёта - #1
Kosmargaret wants to merge 1 commit into
Conversation
| public HashMap<String, Integer> Chek(String productPrice){ | ||
| double price; | ||
| //убедимся, что строка не пустая и не содержит пробелов | ||
| if(!productPrice.trim().isEmpty() && productPrice.indexOf(" ") == -1) { |
There was a problem hiding this comment.
интересная проверка на то, что там нет пробела, если об этом беспокоится, то лучше пройтись по строке и убрать все пробелы
|
|
||
| public class ChekFormat { | ||
|
|
||
| public HashMap<String, Integer> Chek(String productPrice){ |
There was a problem hiding this comment.
Имена функций должны содержать в себе действие. Функция делает что-либо. Хорошо помогает практика переводить имена функций дословно, если после перевода понятно чем занимается функция, то название хорошее
Текущее имя функции не дает понятия о том, чем эта функция занимается
| return null; | ||
| } | ||
|
|
||
| public String GetRubleAddition(int num) |
There was a problem hiding this comment.
имена функций должны начинаться с маленькой буквы. Можно для примера ознакомиться с этой статьей https://habr.com/ru/articles/112042/
| Calculate(); | ||
| } | ||
|
|
||
| public static void SetPeople() { |
There was a problem hiding this comment.
старайся избегать использования статических функций, в будущем из-за частого использования оных, можно получать ошибку недостатка памяти на слабых телефонах
| People = Value; | ||
| chek = true; | ||
| } | ||
| } catch (NumberFormatException Error) { |
There was a problem hiding this comment.
у оператора try/catch есть еще одна часть "finally", с помощью этой части можно удобно закрывать ресурсы, которые нам уже не нужны, таким например является сканер в данной программе, его нужно закрывать после использования с помощью метода close() ->
scanner.close()
| } | ||
| public void splitAccount(int People){ | ||
| int rubEach = RUB / People; | ||
| int val2 = RUB - rubEach * People; |
There was a problem hiding this comment.
старайся всегда именовать переменные осмысленно, избегай непонятных названий, иначе вернувшись к этому коду через некоторое время не сможешь понять его
2 participants
Консольное приложение №1.