Repository navigation
Реализовано приложение калькулятор. - #1
powerrewop wants to merge 1 commit into
Conversation
| public static void calculationOfResults(int theNumberOfPersons, float allPrice) { | ||
|
|
||
| float result = allPrice / theNumberOfPersons; // итог на каждую персону | ||
| String valMessage = "", resultString = String.format("%.2f", result); //преобразуем итог в строку |
There was a problem hiding this comment.
так лучше не объединять, хоть язык и позволяет писать хоть в одну строчку всю программу, но лучше стремится к максимальной читабельности кода, поэтому лучше будет разделить на 2 строки
|
|
||
| symvolsArray = resultString.substring(0, index).toCharArray(); //получим массив символов из результата строки, возьмем только целую часть | ||
|
|
||
| if (symvolsArray.length > 1) { //если целая часть состоит более чем из одной цифры |
There was a problem hiding this comment.
если приходится для самого себя объяснять - что же здесь происходит, то тогда самым простым вариантом можем стать вынесением в функцию. Например, этот можно было бы вынести в функцию isMoreThanOneSymbol и из названия функции нам уже понятно, что эта функция проверяет, больше чем один символ или нет и нам не нужно писать комментарий
| } | ||
| } else { | ||
|
|
||
| if ((symvolOne == '0') && (symvolTwo == '0') || |
There was a problem hiding this comment.
решение рабочее, бесспорно, но я тебе рекомендую подумать над тем, как его можно улучшить
твоя проверка должна быть как воронка, сначала проверяем очень узкие кейсы, например, что число кончается на 1, а дальше всё шире и шире
К тому же лучше использовать switch-case и не разбивать на две разных проверки. Попробуй вынести в отдельный класс, который будет выдавать тебе слово с правильным окончанием
2 participants
Первое ДЗ "Калькулятор". 17-ая когорта Android разработка.