Не понимаю почему не проходит валидатор ((

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

Решен

Проверял код на выполнение всех условий, однако валидатор не принимает😢. Подскажите пож. причину...

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

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

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

Изначально берется крайнее правое число из двоичного представления. С каждой итерацией цикла берется следующее число ближе к началу двоичного представления.

Подсказка: чтобы возвести число к степени можно использовать метод Math.pow(число, степень).
Метод main() не принимает участие в тестировании.

Требования:

Нужно, чтобы метод toBinary(int) был реализован согласно условию.
Нужно, чтобы метод toDecimal(String) был реализован согласно условию.
Методы Integer.toBinaryString(int) и Long.toBinaryString(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.task0906; import java.util.ArrayList; import java.util.Arrays; /* Двоичный конвертер */ public class Solution { public static void main(String[] args) { int decimalNumber = Integer.MAX_VALUE; System.out.println("Десятичное число " + decimalNumber + " равно двоичному числу " + toBinary(decimalNumber)); String binaryNumber = "1111111111111111111111111111111"; System.out.println("Двоичное число " + binaryNumber + " равно десятичному числу " + toDecimal(binaryNumber)); } public static String toBinary(int decimalNumber) { if (decimalNumber>0) { ArrayList<Integer> arrayDigit = new ArrayList<Integer>(); // архив из остатков от деления на 2 int quotient = 1; int rem = 1; while (quotient != 0) // пересчитываем в цикле 10-тичное число на 2-чное. { quotient = decimalNumber / 2; // когда частное quotient=0 - то конец вычислений rem = decimalNumber % 2; // остаток от деления на 2 arrayDigit.add(rem); // добавляем в архив остаток (0 или 1) decimalNumber = quotient; } int[] array2 = new int[arrayDigit.size()]; // создаем архив с обратным порядком элементов for (int i = 0; i < arrayDigit.size(); i++) { array2[i] = arrayDigit.get(arrayDigit.size() - 1 - i); } return Arrays.toString(array2); // переводим архив int в String для передачи в печать } else { return ""; // возврат, если входные данные не удовл условиям }} public static int toDecimal(String binaryNumber) { if (binaryNumber == null || binaryNumber.isEmpty()) // проверка входных данных на требования { return 0; }else { int a=1; int res=0; int res2=0; char[] bN = binaryNumber.toCharArray(); // переводим входн данные String в символный массив bN for (int i = bN.length-1; i>=0; i--) { // в цикле реализуем формулу перевода 2-ного числа в 10-е a=bN[i]-'0'; // перевод символа типа char в int res = (int) (a* Math.pow(2,i)); // реализуем abc(двочн) = (a×2^2 + b×2^1 + c×2^0) (десятичн) res2 = res2+res; } return res2 ; }}}

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

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

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

hidden #598481

Уровень 27

12 декабря 2022, 10:22

Проверял код на выполнение всех условий, однако валидатор не принимает😢 Кролик, расскажи пожалуйста, как ты проверял. Мне кажется, этот код ни разу не запускали. Во-первых, по алгоритму. toDecimal("10") должно возвращать 2, а не 1. toDecimal("110") должно возвращать 6, возвращает 3. Во-вторых, Arrays#toString() возвращает строку, содержащую элементы массива, перечисленные внутри квадратных скобок через запятую и пробел, а не склеенные в одну строку. Я предлагаю тебе отказаться от использования списков и массивов, потому что получается громоздко и неуклюже. Ты же каждый раз приклеиваешь по одному символу? Для этого удобно использовать StringBuilder, у него и результат можно развернуть одним вызовом reverse(), и никаких циклов для этого не понадобится, и в строку он переводится обычным toString(). Ну и наконец, много вопросов по оформлению. Выравнивай код, используй отступы, расставляй скобки. И еще маленький совет: если у тебя есть условие, по которому надо сразу что-то вернуть из метода, то лучше написать короткий if, и else тогда уже не понадобится, ведь если выполнился return, то выполнение метода прервётся, а если не прервалось, значит выполнился else:

public static String toBinary(int decimalNumber) {
    if (decimalNumber < 1) {
        return "";
    }

    // logic here without condition
}

public static String toDecimal(String binaryNumber) {
    if (binaryNumber == null || binaryNumber.isEmpty()) {
        return 0;
    }

    // logic here without condition
}

Wise Rabbit

Уровень 12

12 декабря 2022, 10:40

😱 Действительно! Оказывается код считает правильно, но выборочно 😂.

Спасибо! Эх....а так было все лихо закручено...