Ревью кода. Что можно улучшить?

Question

[GmDvlpr]

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

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

namespace ConsoleApp1
{
    class Program
    {
        static void Main(string[] args)
        {
            int[] a = new int[99];
            Random ran = new Random() ;
            while(true){

                for (int i = 0; i < a.Length; i++)
                {
                    a[i] = ran.Next(0,99);
                    Console.WriteLine(a[i]);
                    System.Threading.Thread.Sleep(100);
                    Console.Clear();
                }
            }

            Console.ReadLine();
        }
    }
}

Comments

[Илья]

А какая цель у этого кода?

[freeExec]

От массива a можно избавиться.

[Модератор]

GmDvlpr дорогой пользователь, настоятельно рекомендуем еще раз обратить самое пристальное внимание на п. 3.1 регламента работы сервиса (и, в особенности, на его последний абзац). В противном случае, ваши вопросы будут удаляться по причине тег-спама, а систематические нарушения приведут к блокировке учетной записи.

Answer 1

[GavriKos]

1) Зачем вам вообще массив чисел?
2) Паузить основной поток - так себе идея, ИМХО
3) Есть бесконечный цикл без всяких возможностей его прервать.
4) Куча лишних переносов строк, левые пробелы, нет пробела между параметрами... В общем кодстайл так себе.

Comments

[Илья]

А как сделать задержку если она нужна, не используя Thread.Sleep?

[Foggy Finder]

Илья, Task.Delay

[Илья]

Foggy Finder, ну то есть в этом случае надо написать Task.Delay(100).Wait() ?

[Foggy Finder]

Илья, не совсем, зависит от того что вы хотите получить. Можно, например, вынести получение числа в отдельный асинхронный метод.

[Илья]

Foggy Finder, я просто не очень понимаю использование TPL в случае если нет асинхронности в коде. И поток один. Microsoft конечно рекомендует использовать таски, мотивируя тем что потоки это «сложна-сложна», но в случае когда поток один, все синхронно, почему бы и не написать Thread.Sleep

[Василий Банников]

Илья, вообще действительно не понятно, что это за код и почему он именно такой. Придирка к Thread.Sleep действительно лишняя - в данном случае это не страшно.
Я бы написал так:

var rand = new Random();
while (true)
{
    var number = rand.Next(0, 99);
    Console.Clear();
    Console.Write(number);
    Thread.Sleep(100);
}

[ddd329]

Foggy Finder, привет! Скажи пожалуйста, а есть ли большая разница между Task.Delay и Thread.Sleep?

[Foggy Finder]

ddd329, Привет. Разница действительно большая. Этот вопрос очень часто задают и информации по нему довольно много и, к тому-же, ее довольно легко найти:

When to use Task.Delay, when to use Thread.Sleep?

[Foggy Finder]

просто не очень понимаю использование TPL в случае если нет асинхронности в коде. И поток один. Microsoft конечно рекомендует использовать таски, мотивируя тем что потоки это «сложна-сложна», но в случае когда поток один, все синхронно, почему бы и не написать Thread.Sleep

Илья, Но ведь они правы. Очень много вопросов от новичков (и не только) связаны с проблемами от неправильного использования Thread.Sleep. Скопируют код и ожидают что он будет работать. Особенно это часто встречается когда от консольных приложений переходят к приложениям с графическим интерфейсам.

[ddd329]

Foggy Finder, спасибо за ответ)

[Илья]

Foggy Finder, мой комментарий относился именно к этому примеру. Я говорю что новичку предлагают для задержки вывода на консоль взять нетривиальную библиотеку, которая предназначена для многопоточности и асинхронности. А эти концепции ему не нужны в этом случае.
Что касается графических интерфейсов, то новичку не поможет простое использование правило "используй TPL". Ему нужно будет понять идеи многопоточности и асинхронности, понять что есть UI Thread и он делает определенные задачи. А без этого понимания он так же эффективно заблочит себе UI вызовом Task.Delay(1000).Wait() как и вызовом Thread.Sleep(1000). Абстракция Task "протекает" как и все остальные, приходится понимать внутреннее устройство.

[Foggy Finder]

мой комментарий относился именно к этому примеру. Я говорю что новичку предлагают для задержки вывода на консоль взять нетривиальную библиотеку, которая предназначена для многопоточности и асинхронности. А эти концепции ему не нужны в этом случае.

Илья, Тут согласен, но если ответ и обсуждение под ним подтолкнет автора к ознакомлению с асинхронностью, то разве оно того не стоит?

Что касается графических интерфейсов, то новичку не поможет простое использование правило "используй TPL". Ему нужно будет понять идеи многопоточности и асинхронности, понять что есть UI Thread и он делает определенные задачи.

Огромное число принятых ответов по типу: добавь вот тут async а вот тут delay говорят об обратном. Он просто скопирует любезно предоставленное и пойдет дальше.

К тому же если все что требуется это не вешать интерфейс и периодически сообщать о прогрессе, то даже представления об асинхронности не нужно, не говоря уже о понимании концепций. Особенно, когда всю реализацию обернули в удобную для использования AsyncCommand.