Как сделать проверку на ноль?

Question

[Mister_krid]

Метод берет целое число и возвращает его с расположением цифр в порядке убывания. Но если число 0 или 00 и т.д. все ломается т.к. он не кладет ноль в список и я не могу допереть как это исправить. Так же интересно что думаете на счет List в данной реализации ибо я его использовал просто потому, что захотел применить где-нибудь(может массив с циклом смотрелись бы получше). И еще можно ли избавиться от конвертации int в строчный массив, а потом обратно в int(я пытался сделать попроще, но не нашел ничего для конвертации и объединения list int)

public int DescendingOrder(int num)
        {

            var inpNum = num;
            List<int> list = new List<int>();

            while (inpNum > 0)
            {
                list.Add(inpNum % 10);
                inpNum = inpNum / 10;
            }
            list.Sort();
            string[] array = list.Select(n => n.ToString()).ToArray();

            var result = string.Join(null, array);
          
            return Convert.ToInt32(result);
            
              
        }

Comments

[Mister_krid]

если что идеальный правильный ответ выглядит так:

public int DescendingOrder(int num)
        {
            return int.Parse(string.Concat(num.ToString().OrderByDescending(x => x)));                   
        }

может кому пригодится.

Answer 1

[V Sh.]

Ну ошибка, скорее всего, в строчке Convert.ToInt32(). Насколько я помню, если передать туда null или пустую строку, то будет ошибка преобразования, а так и происходит, когда у вас inpNum = 0.
Если вы используете LINQ у себя, то используйте его по полной:

string str = "1243586701";
Console.WriteLine(String.Concat(str.OrderBy(x => x)));

Comments

[Mister_krid]

оо вот это я и искал, правда не очень понял как работает OrderBy (извлечение ключа элемента, компортатор для сравнения ключей )

Answer 2

[Rsa97]

do {
    list.Add(inpNum % 10);
    inpNum = inpNum / 10;
} while (inpNum > 0);

Comments

[Mister_krid]

я думал об этом, но если 0 попадается в числе данный способ его убирает и ответ неверный.

[Rsa97]

Mister_krid, Этот способ отличается от вашего только тем, что цикл гарантировано выполнится один раз. Соответственно, если на входе 0, то он всё равно будет занесён в массив.

[Mister_krid]

Rsa97, я знаю как работает do while, он сработет только если у нас 0, если я ввиду к примеру 2101, он выведет 211, а должен 2110.

[Mister_krid]

Rsa97, так подожди, а может ли в int быть число 0132 к примеру или с 0 на первом месте нельзя?

[Rsa97]

Mister_krid, Проверьте в отладчике. Список list после цикла будет содержать все цифры числа, включая нули. Теряете их вы позже, где-то в бессмысленных преобразованиях.

[Rsa97]

Mister_krid, 0132 === 132
Естественно, лидирующие нули игнорируются. Чтобы их обрабатывать нужно работать не с числом, а со строкой.

Answer 3

[GavriKos]

Сделать для нуля отдельную обработку.
Кстати 00 - такого числа нет.