2 условие не выполняется, хотя ответ правильный

Вопрос по задаче

Решен

Проверял с другими числами, все работало правильно. Подскажите, что не так, пожалуйста

Публичный статический метод toHex(int) должен переводить целое число, полученное в качестве входящего параметра, из десятичной системы счисления в шестнадцатеричную и возвращать его строковое представление. А публичный статический метод toDecimal(String) наоборот — из строкового представления шестнадцатеричного числа в десятичное число.
Методы работают только с положительными числами и не пустыми строками. Если входящий параметр меньше или равен 0, метод toHex(int) возвращает пустую строку. Если входящий параметр — пустая строка или null, то метод toDecimal(String) возвращает 0.
Твоя задача реализовать эти методы.

Также в классе Solution есть константа HEX, которая содержит в себе все символы, используемые в шестнадцатеричной системе.

Один из алгоритмов перевода десятичного числа в шестнадцатеричное следующий:
while(десятичное число не равно 0) { представление шестнадцатеричного числа = символ из строки HEX с индексом, равным остатку от деления десятичного числа на 16 + представление шестнадцатеричного числа десятичное число = десятичное число / 16 }

Один из алгоритмов перевода шестнадцатеричного числа в десятичное следующий:
for (int i = 0; i < длина входящей строки; i++) { десятичное число = 16 * десятичное число + индекс символа из строки HEX, равного символу из входящей строки с индексом i }

Метод main() не принимает участие в тестировании.

Требования:

Нужно, чтобы метод toHex(int) был реализован согласно условию.
Нужно, чтобы метод toDecimal(String) был реализован согласно условию.
Методы Integer.toHexString(int) и Long.toHexString(int) использовать нельзя.
Методы Integer.parseInt(String, int) и Long.parseLong(String, int) использовать нельзя.
Методы Integer.toString(int, int) и Long.toString(long, int) использовать нельзя.
Объект типа BigInteger использовать нельзя.
Объект типа BigDecimal использовать нельзя.

package com.javarush.task.pro.task09.task0907; import java.util.regex.Pattern; /* Шестнадцатеричный конвертер */ public class Solution { private static final String HEX = "0123456789abcdef"; public static void main(String[] args) { int decimalNumber = 1256; System.out.println("Десятичное число " + decimalNumber + " равно шестнадцатеричному числу " + toHex(decimalNumber)); String hexNumber = "4e8"; System.out.println("Шестнадцатеричное число " + hexNumber + " равно десятичному числу " + toDecimal(hexNumber)); } public static String toHex(int decimalNumber) { String hexNum = ""; while (decimalNumber > 0) { char index = HEX.charAt(decimalNumber%16); hexNum = index + hexNum; decimalNumber = decimalNumber/16; } return hexNum; } public static int toDecimal(String hexNumber) { int decimal = 0; for(int i = 0; i < hexNumber.length() && !hexNumber.isEmpty() && hexNumber != null; i++) { int index = Character.getNumericValue(HEX.charAt(Character.getNumericValue(hexNumber.charAt(i)))); decimal = 16 * decimal + index; } return decimal; } }

Комментарии (2)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

hidden #3303140

Уровень 2

25 октября 2024, 06:42

решение

1. В методе toDecimal() твоя проверка на null не сработает, и программа выбросит NullPointerException:

System.out.println(toDecimal(null))
// Exception in thread "main" java.lang.NullPointerException: Cannot invoke "String.length()" because "hexNumber" is null

На null надо проверять до того, как вызываешь какой угодно метод у объекта, в данном случае length() и isEmpty(). 2. Хорошая практика - это не интегрировать терминальную логику в логику работы метода, а провести все проверки в начале, и если они не сработали, то тогда переходить к логике метода. Это во-первых позволит визуально разделить случаи, когда метод не нужно выполнять от случаев, когда нужно, а во-вторых облегчит понимание логической части:

if (hexNumber == null || hexNumber.isEmpty()) {
    return 0;
}
// остальная логика метода после, else не нужен

3. Зачем два раза перегонять туда-сюда символ в число, потом обратно в символ, потом снова в число? Достаточно одного раза. Кроме того, если у тебя есть строка HEX с символами 0-f, то проще вообще один раз воспользоваться методом indexOf и всё:

int num = HEX.indexOf(hexNumber.charAt(i));

indexOf - это действие обратное методу charAt. charAt возвращает тебе символ в строке по его индексу, indexOf - возвращает индекс символа в строке. Кстати именно поэтому в методе toHex у тебя плохое имя у переменной:

char index = HEX.charAt(decimalNumber % 16);

Оно вводит в заблуждение, т.к. в переменной index хранится не индекс символа, а сам символ записи числа, или другими словами цифра. Я бы назвал эту переменную digit или hexDigit.

CrSimple

Уровень 25

25 октября 2024, 14:58

Спасибо, учту