Программа перевода из десятичной системы счисления в двоичную, правильно?

Question

[Moskaa]

Составил программу на перевод из десятичной системы счисления в двоичную, я проверил несколько чисел и перевод итоги перевода были удовлетворительными. Но я не уверен, есть ли такие числа, которые будут переведены неправильно(Учитывая, что метод не принимает отрицательные значения).

И ещё, если есть какие-то замечания по написанной программе, то напишите, пожалуйста. Буду очень благодарен.

using System;

namespace ConsoleApp
{
    class Class
    {
        static void Main(string[] args)
        {
            BinaryTranslation(10);
        }

        static void BinaryTranslation(int number)
        {
            List<int> binar = new List<int>();

            int quotient = 1;

            while(quotient != 0)
            {
                if(number % 2 == 1)
                {
                    number -= 1;

                    binar.Add(1);
                }
                else
                {
                    binar.Add(0);
                }

                quotient = number / 2;
                number = number / 2;
            }

            binar.Reverse();
            Console.WriteLine(string.Join("", binar));
        }
    }
}

Answer 1

[Rsa97]

На отрицательных числах будет ошибка.
Ну и проще можно сделать.

static void BinaryTranslation(int number)
{
    List<int> binar = new List<int>();
    do
    {
        binar.Add(number % 2);
        number /= 2;
    }
    while (number > 0);

    binar.Reverse();
    Console.WriteLine(string.Join("", binar));
}

Comments

[Moskaa]

Забыл написать, что метод не рассчитан на отрицательные числа(( но всё равно спасибо
Изменено: интересно, я опробую этот вариант кода завтра, потому что сейчас уже не могу, но отмечу как решения вопроса, спасибо!!

Answer 2

[rPman]

Отрицательные числа

Comments

[Moskaa]

Забыл написать, что метод не рассчитан на отрицательные числа(( но всё равно спасибо

Answer 3

[GavriKos]

Учитывая, что метод не принимает отрицательные значения

Нет, согласно объявлению метода - расчитан. Но это можно все предусмотреть. Как и в объявлении метода, так и в обрабоке метода. И это бы как раз стоило переделать - т.е. или ограничить входные данные типом, или обработать ошибку.

Дальше. По красоте - метод должен возвращать бинарное представление, а дальше уже выводить в консоль или еще куда - это задача не метода.

Дальше. Честно я не понял зачем нужна переменная quotient. По моему можно без нее.

Ну и еще метод должен называться глаголом в идеале )