Conversation
| private String args; | ||
|
|
||
| public Task1(String args) { | ||
|
|
||
| this.args = args; | ||
| } |
There was a problem hiding this comment.
Зачем args передавать в конструктор, если все равно он потом приходит в функцию time(String args)?
Можно удалить это поле из класса, а конструктор сделать приватным
| String minets = ""; | ||
| String seconds = ""; | ||
| int ch = 0; | ||
| String lol = ""; |
| if (args.charAt(i) == ':') { | ||
| ch = 1; | ||
| } else if (ch == 0) { | ||
| minets += args.charAt(i); | ||
| } else { | ||
| seconds += args.charAt(i); | ||
| } |
There was a problem hiding this comment.
Можно попроще сделать args.split(":")
Но если хочешь так, то seconds и mitutes нужно собирать через StringBuilder
| private int args; | ||
|
|
||
| //public countDigits(int args) { | ||
| public Task2(int args) { | ||
| this.args = args; | ||
| } |
There was a problem hiding this comment.
То же самое. Не очень понимаю логику передачи в конструктор переменной, которая потом также передается в функцию.
| answer += swap.charAt(i + 1); | ||
| answer += swap.charAt(i); |
|
|
||
| public static final int INDEX_CHAR = 48; | ||
|
|
||
| public boolean proverochka(Integer x) { |
There was a problem hiding this comment.
proverochka можно сделать pivate, т.к. функция для внутреннего использования
|
|
||
| public boolean isPalindromeDescendant(Integer palindrom) { | ||
| this.palindrom = palindrom; | ||
| String y = String.valueOf(palindrom); |
There was a problem hiding this comment.
Поработай над названием переменных. Тяжело читать код, приходится постоянно возвращаться наверх и вспоминать, что такое y, что lenn это не длина, а половина длины и т.д.
| x1 = massiv[MASSIV_SIZE - 1] * SIZE_10 * SIZE_10 * SIZE_10 + massiv[2] * SIZE_10 * SIZE_10 + massiv[1] * SIZE_10 | ||
| + massiv[0] - massiv[MASSIV_SIZE - 1] - massiv[2] * SIZE_10 | ||
| - massiv[1] * SIZE_10 * SIZE_10 - massiv[0] * SIZE_10 * SIZE_10 * SIZE_10; |
There was a problem hiding this comment.
Это сильно. Но лучше разбить на 3 операции:
- Получаем первое число
- Получаем второе число
- Находим разность
Потому что мы понимаем, что тут происходит, пока помним условие задачи. Если коллега будет смотреть на этот код, ему придется долго разбираться зачем это все нужно.
| if (x1.equals(ANSWER)) { | ||
| return 0; | ||
| } else { | ||
| return (1 + swap(x1)); | ||
| } |
There was a problem hiding this comment.
Эта проверка есть в функции countK(). Почему бы ее здесь и не вызвать?
|
Привет, Иван! Нужно поправить замечания. Помимо замечаний, есть пара пунктов:
|
There was a problem hiding this comment.
А почему не все файлы перенесены из project-template? Это тоже должно быть в корне проекта.
|
Привет! При переносе из project-template потерялись некоторые важные файлы. Сейчас зачту эту ДЗ, в следующем нужно будет добавить. 👍 |
2 participants
No description provided.