Я запутался и совершенно не понимаю как исправить ошибку.

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

Решен

На данный момент, я пытаюсь справиться хотя бы с первой частью задачи — переводом из двоичной системы в шестнадцатиричную. Столкнулся с тем, что компиляция кода банально не доходит до цикла for, в котором и происходит преобразование. ПОгуглил, поспрашивал на StackOverflow - оказалось дело в огромном количестве "return", предшествующих циклу. Однако, не понимаю как исправить код таким образом, чтобы это самое количество возвращений уменьшить. Объясните плиз, я на грани отчаяния (хоть и знаю, что дальше будет только хуже).

Публичный статический метод 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 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) { String hexNumber = ""; if (binaryNumber == "" || binaryNumber == null) { return ""; } else if (binaryNumber.contains("[a-zA-Z]+") || binaryNumber.contains("[2-9]+")) { return ""; } else { if (binaryNumber.length() % 4 != 0) { String bN2 = '0' + binaryNumber; bN2 = binaryNumber; return binaryNumber; } else { return binaryNumber; } } //code is here for (int i = 0; i < (binaryNumber.length()); i += 4) { switch (binaryNumber.substring(i, i + 4)) { case ("0000"): hexNumber = hexNumber + "0"; break; case ("0001"): hexNumber = hexNumber + "1"; break; case ("0010"): hexNumber = hexNumber + "2"; break; case ("0011"): hexNumber = hexNumber + "3"; break; case ("0100"): hexNumber = hexNumber + "4"; break; case ("0101"): hexNumber = hexNumber + "5"; break; case ("0110"): hexNumber = hexNumber + "6"; break; case ("0111"): hexNumber = hexNumber + "7"; break; case ("1000"): hexNumber = hexNumber + "8"; break; case ("1001"): hexNumber = hexNumber + "9"; break; case ("1010"): hexNumber = hexNumber + "a"; break; case ("1011"): hexNumber = hexNumber + "b"; break; case ("1100"): hexNumber = hexNumber + "c"; break; case ("1101"): hexNumber = hexNumber + "d"; break; case ("1110"): hexNumber = hexNumber + "e"; break; case ("1111"): hexNumber = hexNumber + "f"; break; } } return hexNumber; } public static String toBinary(String hexNumber) { if (hexNumber == "" || hexNumber == null) { return ""; } else if (hexNumber.contains("[g-zA-Z]+")) { return ""; } return null; } }

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

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

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

Dmitriy QA Automation Engineer

18 июля 2022, 20:28

решение

Привет. Основная проблема кроется в условии, начиная со строки 30 и до 41. Выход из этого блока кода всегда один: 'return binaryNumber;'. Нужно изменить условие, чтобы после 'else' на 40 строке, выполнение шло в цикл 'for'(другими словами, нужно удалить все с 38 по 41 строку включительно). И несколько дополнений касаемо оптимизации: Условия в этом блоке кода возвращают одно и то же - пустую строку. 'else if' на 26 строке выглядит избыточным.

if (binaryNumber == "" || binaryNumber == null)
        {
            return "";
        } else if (binaryNumber.contains("[a-zA-Z]+") || binaryNumber.contains("[2-9]+"))
        {
            return "";
        }

Можно сделать так:

if (binaryNumber == "" ||
                binaryNumber == null ||
                binaryNumber.contains("[a-zA-Z]+") ||
                binaryNumber.contains("[2-9]+")) {
            return "";
        }

Еще можно обратить внимание на 3ю строку, там не зря импортирован Pattern. С его помощью можно сделать так(обрати внимание на 'НЕ' в виде знака !).

if (binaryNumber == null ||
                binaryNumber.equals("") ||
                !Pattern.matches("[0-1]+", binaryNumber)) {
            return "";
        }

Другими словами условие звучит так: Если binaryNumber не совпадает с шаблоном [0-1], то возвращаем пустую строку. Надеюсь помог. Удачи!

The_Red_Baron43

Уровень 31

26 июля 2022, 16:27

Огромное спасибище! И проблема решилась и код оптимизировался немного, прям конфетка! Спасибо!