Почему метод toHex не реализован согласно условию?

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

Архивный

При проверке задания пишет, что метод toHex не реализован согласно условию, все остальные условия выполнены. Не понимаю, в чём проблема, вроде toBinary выполнен аналогично, но программа проверки на него не ругается

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

Методы работают только с не пустыми строками.
Если входящий параметр — пустая строка или null, то оба метода возвращают пустую строку.
Если входящий параметр метода toHex(String) содержит любой символ, кроме 0 или 1, то метод возвращает пустую строку.
Если входящий параметр метода toBinary(String) содержит любой символ, кроме цифр от 0 до 9 или латинскую букву от a до f (в нижнем регистре), то метод возвращает пустую строку.

Твоя задача — реализовать эти методы.

Один из алгоритмов перевода строкового представления двоичного числа в строковое представление шестнадцатеричного числа следующий:

Проверяем длину строки, полученной входящим параметром. Она должна быть кратна 4.
Если это не так, то добавляем нужное количество 0 в начало строки.
Берем каждые четыре символа (бита) и проверяем, какому символу шестнадцатеричной кодировки он соответствует.

Например:

двоичное представление — "100111010000", где "1001" — "9", "1101" — "d", "0000" — "0",
шестнадцатеричное представление — "9d0".

Один из алгоритмов перевода строкового представления шестнадцатеричного числа в строковое представление двоичного числа следующий:
Берем каждый символ и проверяем какому двоичному числу (4 бита) он соответствует.

Например:

шестнадцатеричное представление — "9d0", где "9" — "1001", "d" — "1101", "0" — "0000",
двоичное представление — "100111010000".

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

Требования:

Нужно, чтобы метод toHex(String) был реализован согласно условию.
Нужно, чтобы метод toBinary(String) был реализован согласно условию.
Методы Integer.toBinaryString(int) и Long.toBinaryString(int) использовать нельзя.
Методы 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.task0908; import java.util.regex.Pattern; /* Двоично-шестнадцатеричный конвертер */ public class Solution { public static String code = "00102001130100401015011060111710008100191010a1011b1100c1101d1110e1111f"; public static void main(String[] args) { String binaryNumber = "100111010000"; System.out.println("Двоичное число " + binaryNumber + " равно шестнадцатеричному числу " + toHex(binaryNumber)); String hexNumber = "9d0"; System.out.println("Шестнадцатеричное число " + hexNumber + " равно двоичному числу " + toBinary(hexNumber)); } public static String toHex(String binaryNumber) { if (binaryNumber == "" || binaryNumber == null || !Pattern.matches("[0-1]+", binaryNumber)) return ""; String hex = ""; while ((binaryNumber.length() % 4) != 0) { binaryNumber = binaryNumber + 0; } for (int i = 0; i < binaryNumber.length(); i += 4) { String sub = binaryNumber.substring(i, i + 4); if (sub.equals("0000")) sub = "0"; else if (sub.equals("0001")) sub = "1"; else { int index = code.indexOf(sub); sub = code.substring(index + 4, index + 5); } hex = hex + sub; } return hex; } public static String toBinary(String hexNumber) { if (hexNumber == null || hexNumber == "" || !Pattern.matches("[0-9a-f]+", hexNumber)) return ""; String dub = ""; for (int i = 0; i < hexNumber.length(); i++) { String sub = hexNumber.substring(i, i + 1); if (sub.equals("0")) sub = "0000"; else if (sub.equals("1")) sub = "0001"; else { int index = code.indexOf(sub); sub = code.substring(index - 4, index); } dub = dub + sub; } return dub; } }

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

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

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

rushchishin_petr

Уровень 35

19 декабря 2020, 11:21

while ((binaryNumber.length() % 4) != 0) { binaryNumber = binaryNumber + 0; } не с той стороны нули прибавляете

Duha

Уровень 42

20 ноября 2020, 16:07

if (binaryNumber == "" || binaryNumber == null || !Pattern.matches("[0-1]+", binaryNumber))

Нет проверки на пустую строку, при сравнении через "==" сравниваются ссылки а не значения, для сравнения строк по значению есть метод equals(). А так же существует метод isEmpty() на проверку пустой строки.

Stepan Zakharov

Уровень 13

21 ноября 2020, 12:49

public static String toHex(String binaryNumber) {
       String hex = "";
       if (binaryNumber == null || binaryNumber.isEmpty()|| !Pattern.matches("[0-1]+", binaryNumber))
           return hex;
       while ((binaryNumber.length() % 4) != 0) {
           binaryNumber = binaryNumber + 0;
       }
       for (int i = 0; i < binaryNumber.length(); i += 4) {
           String sub = binaryNumber.substring(i, i + 4);
           if (sub.equals("0000"))
               sub = "0";
           else if (sub.equals("0001"))
               sub = "1";
           else {
               int index = code.indexOf(sub);
               sub = code.substring(index + 4, index + 5);
           }
           hex = hex + sub;
       }
       return hex;
   }

Изменил код следующим образом, по-прежнему пишет, что toHex неправильно сделан