Не пропускает методы из-за null

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

Решен

Вот что пишет валидатор: Убедись, что метод toHex(String) возвращает пустую строку, если на вход получает null. Убедись, что метод toBinary(String) возвращает пустую строку, если на вход получает null. Но у меня есть проверки на null в обоих методах на 18 и 51 строках соответственно, что за бред?

Публичный статический метод 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) { Pattern pattern = Pattern.compile("^[01]+$"); if (binaryNumber.isEmpty() || binaryNumber == null || !pattern.matcher(binaryNumber).matches()) return ""; String solution = ""; int number = binaryNumber.length() % 4; if (number > 0) { for (int i = 0; i < number; i++) { binaryNumber = 0 + binaryNumber; } } for (int i = 0; i < binaryNumber.length(); i += 4) { String part = binaryNumber.substring(i, i + 4); if (part.equals("0000")) part = "0"; else if (part.equals("0001")) part = "1"; else if (part.equals("0010")) part = "2"; else if (part.equals("0011")) part = "3"; else if (part.equals("0100")) part = "4"; else if (part.equals("0101")) part = "5"; else if (part.equals("0110")) part = "6"; else if (part.equals("0111")) part = "7"; else if (part.equals("1000")) part = "8"; else if (part.equals("1001")) part = "9"; else if (part.equals("1010")) part = "a"; else if (part.equals("1011")) part = "b"; else if (part.equals("1100")) part = "c"; else if (part.equals("1101")) part = "d"; else if (part.equals("1110")) part = "e"; else if (part.equals("1111")) part = "f"; solution += part; } return solution; } public static String toBinary(String hexnumber) { Pattern pattern = Pattern.compile("^[09][af]+$"); if (hexnumber.isEmpty() || hexnumber == null || pattern.matcher(hexnumber).matches()) return ""; String solution = ""; for (int i = 0; i < hexnumber.length(); i++) { String part = "" + hexnumber.charAt(i); if (part.equals("0")) part = "0000"; else if (part.equals("1")) part = "0001"; else if (part.equals("2")) part = "0010"; else if (part.equals("3")) part = "0011"; else if (part.equals("4")) part = "0100"; else if (part.equals("5")) part = "0101"; else if (part.equals("6")) part = "0110"; else if (part.equals("7")) part = "0111"; else if (part.equals("8")) part = "1000"; else if (part.equals("9")) part = "1001"; else if (part.equals("a")) part = "1010"; else if (part.equals("b")) part = "1011"; else if (part.equals("c")) part = "1100"; else if (part.equals("d")) part = "1101"; else if (part.equals("e")) part = "1110"; else if (part.equals("f")) part = "1111"; solution += part; } return solution; } }

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

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

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

Стас ПасинковSoftware Developer в Zipy

Master

7 сентября 2022, 20:40

сначала делаете проверку на нулл, а только потом уже остальные проверки (пустая строка/не подходит под регулярку) сейчас у вас если передать нулл - то сначала у вас срабатывает проверка binaryNumber.isEmpty() в случае, если передали нулл - то у вас получается null.isEmpty(), а так делать нельзя и поэтому ваша программа падает с NullPointerException сразу же, и до вашей второй проверки на нулл уже дело даже не доходит аналогично и с hexnumber

Сергей

Уровень 17

8 сентября 2022, 06:01

Спасибо огромное, этот момент вообще не учел.

Сергей

Уровень 17

8 сентября 2022, 06:31

Только теперь не пропускает паттерны.

Стас ПасинковSoftware Developer в Zipy

Master

8 сентября 2022, 19:46

попробуйте убрать ^ и $ из паттернов. паттерн [09][af]+ реализован неправильно :)

Сергей

Уровень 17

9 сентября 2022, 09:43

В принципе паттерны не работают. Не понимаю, в чем проблема. В онлайн проверке паттерны работают

Сергей

Уровень 17

9 сентября 2022, 10:05

Паттерны не работали из-за метода matches. Он ищет не символы, а всю строку. Решил сам так:

Pattern pattern = Pattern.compile("[^01]+");
        if (binaryNumber == null || binaryNumber.isEmpty() || pattern.matcher(binaryNumber).find()) return "";

И в функционале добавлении нулей поменял условие на такое:

if (number > 0) {
            for (int i = 0; i < 4 - number; i++) {
                binaryNumber = "0" + binaryNumber;
            }
        }

Стас ПасинковSoftware Developer в Zipy

Master

9 сентября 2022, 11:15

так вам же это и надо. проверить, что вся строка соответствует определенному шаблону. случай, когда где-то посреди строки будут нужные вам символы, а вокруг куча левых символов - вас же наоборот не устраивает. регулярка [09][af]+ у вас не работает. в тест-стрингах указаны корректные значения для чисел в 16тиричном формате. ваша регулярка на каждое такое значение говорит, что оно не подходит. справа вверху почитайте "расшифровку" вашей регулярки. она "примет" только такую строку, где первым символом обязательно стоит 0 или 9, и после него один или более символов a или f. только эти символы, а не все, что между ними вы же хотели сделать что-то типа такого