2

Суть вопроса вот в чем. Есть класс, в нем функция отправки сообщения с WebRequest в отдельном потоке с бесконечном цикле, с задержкой в 1 секунду. Создаем массив этого класса, например 10 элементов. Циклом пробегаемся по массиву, в запускаем потом на бесконечную отправку сообщений. И чем больше элементов в массиве, тем дольше будет отправка. Хотя, если не делать массив, а запустить разные копии программы, то отправка сообщений будет мгновенная. Приложу абстрактный код ниже

    class Request {
       public void Start() {
          new Thread(()=>{
             while(true) {
                SendRequest();
                Thread.Sleep(1000);
             }
          }).Start();
       }
    }

public Main() {
   var requests = new List<Request>();
   requests.Add(new Request());

   foreach(var request...) {
      request.Start();
   }
}
6
  • 2
    1) попробуйте отправлять запросы асинхронно 2) используйте для отправки HttpClient
    – tym32167
    18 мая 2018 в 20:01
  • 2
    Кстати, получается, что вы генерите кучу потоков, и чем больше потоков, тем медленней работает программа. Я поясню - в многопоточном приложении переключение между потоками - дорогая операция по ресурсам и времени. Таким образом, чем больше потоков - тем больше переключений между потоками, тем больше времени тратится на переключение и меньше на полезную работу. Другими словами, если вы создадите много много потоков, ваша программа не будет работать быстрее, а скорее всего просто зависнет, так как все время будет тратиться на переключение между потоками.
    – tym32167
    18 мая 2018 в 20:16
  • 2
    Штука в том, что сейчас, когда вы создаете 100500 потоков, то бОльшую часть времени каждый поток либо ждет ответа от сервера, либо ждет Thread.Sleep() - то есть, по сути, простаивает. Если вы переделаете на асинхронные запросы, то вам не поняобится куча потоков, так как асинхронная операция не занимает поток вовсе. А раз будет меньше потоков - то меньше переключений между ними, и, как следствие, все должно работать побыстрее
    – tym32167
    18 мая 2018 в 20:22
  • 1
    @tym32167 а почему не ответом?
    – rdorn
    18 мая 2018 в 20:48
  • 1
    @rdorn добавил ответом
    – tym32167
    18 мая 2018 в 21:05

2 ответа 2

3

Получается, что вы генерите кучу потоков, и чем больше потоков, тем медленней работает программа. Я поясню - в многопоточном приложении переключение между потоками - дорогая операция по ресурсам и времени. Таким образом, чем больше потоков - тем больше переключений между потоками, тем больше времени тратится на переключение и меньше на полезную работу. Другими словами, если вы создадите много много потоков, ваша программа не будет работать быстрее, а скорее всего просто зависнет, так как все время будет тратиться на переключение между потоками.

Штука в том, что сейчас, когда вы создаете 100500 потоков, то бОльшую часть времени каждый поток либо ждет ответа от сервера, либо ждет Thread.Sleep() - то есть, по сути, простаивает. Если вы переделаете на асинхронные запросы, то вам не поняобится куча потоков, так как асинхронная операция не занимает поток вовсе. А раз будет меньше потоков - то меньше переключений между ними, и, как следствие, все должно работать побыстрее.

То есть вам надо:

  • Попробуйте отправлять запросы асинхронно
  • используйте для отправки HttpClient

Как пример (за корректность не ручаюсь - не запускал, но идея должна быть понятна)

public class Request
{
    private bool _runned = false;

    public async void Start()
    {
        _runned = true;
        while(_runned)
        {
            await SendRequest("http://google.com");
            await Task.Delay(1000);
        }
    }

    public void Stop()
    {
        _runned = false;
    }

    private static async Task<string> SendRequest(string url)
    {
        using(var client = new HttpClient())
        {
            var result = await client.GetStringAsync(url);
            // do smthg
            return result;
        }
    }
}
1

По моему с версии c# 4.0 рекомендуется использовать класс Task, а не Thread. Task будет использовать потоки из пула потоков. Если вкратце, то пул оптимизирован для выделения потоков.

14
  • 1
    Task будет использовать потоки из пула потоков - это не совсем верно. Это, во первых, зависит от того, как таск создан. Во вторых, таски могут вообще не использовать потоки. Таски - это такая абстракция кусочка какой то работы, а как она выполняется - это скрыто внутри таска.
    – tym32167
    18 мая 2018 в 21:04
  • 1
    Если использовать стандартный Task.Run() или Task.Factory.StartNew() то будет использоваться пул. Остальные случаи бывают очень редко (ни разу не видел самописный task scheduler). Ну и результат из Taska можно вернуть сразу, тогда поток использоваться не будет, но это тоже редкий случай. 18 мая 2018 в 21:22
  • 1
    Метод Task.Factory.StartNew() содержит много параметров, ему можно явно указать, что таск нуждается в отдельном потоке или даже можно передать свой шедулер. Но это не важно, так как тут сам клиент поддерживает асинхронные операции и вручную таски создавать не нужно. Поглядите на мой ответ - я нигде не создавал таски, но они есть.
    – tym32167
    18 мая 2018 в 21:27
  • 1
    Я с вами полностью согласен, просто я хотел объяснить человеку, что использовать thread это не очень хорошо. И в его коде Thread.Start можно заменить на Task.Run 18 мая 2018 в 21:35
  • 1
    К сожаленнию, это тоже не поможет, так как поместив много задач пул потоков, пул не будет их выполнять все разом, а будет выполнять пачками по очереди, что тоже будет работать медленно. Самый оптимальный вариант - не использовать ни потоки, ни пул потоков, а ожидать ответов от сервера асинхронно.
    – tym32167
    18 мая 2018 в 21:43

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.