Субъективный Си / Говнокод #46

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
Console.WriteLine("\nС каким номером карту, вы хотите поменять?");
            string numberOfCards = Console.ReadLine();
            string[] temp = numberOfCards.Split(new char[] { ' ' }, StringSplitOptions.RemoveEmptyEntries);
            int[] template = new int[temp.Length];
            for (int i = 0; i < temp.Length; i++)
            { template[i] = Convert.ToInt32(temp[i]); }
            return template;

Запостил: lizerka lizerka,

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

  • ava guest # 0

    Давайте я обосру код аргументировано.

    Работать с IO нужно в терминах TextWriter/TextReader, без завязки на консоль.
    Локализацию нужно делать через `Microsoft.Extensions.Localization` а не хардкодить строки.
    Явно указывать типы не нужно: есть var.
    Попытка считать номер карты ужасна. Вот ей богу: тут подошли бы регуляррные выражения.
    Обработка ошибок тоже совершенно кошмарная: нужно в случае неудачного чтения возвращать нормалную ошибку, а не то исключеие, что киент ToIn32
    Ответить

Добавить комментарий

Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

    А не использовать ли нам bbcode?

    ↑
    8
    ↓