Middle+

// Kui tellimus uuendatakse, peame saatma tellimuse andmed mitmele teenusele (kolmanda osapoole API-d) // teenuste arv kasvab (võib olla tuhanded+) // kirjutasime koodi, alguses läks kõik hästi, kuid aja jooksul hakkas meie teenus palju mälu kasutama func (s *orderService) SendOrder(ctx context.Context, hosts []string, order Order) { for i := 0; i < len(hosts); i++ { go func() { // Kujutage ette, et see on pikk võrgukutse response, err := s.httpClient.Send(ctx, hosts[i], order) if err != nil { s.logger.Error(ctx, "saata ebaõnnestus", err) return } s.logger.Info(ctx, "edu", response) }() } }

sobes.tech AI

Vastus AI-lt

В представленном коде есть проблема с использованием переменной цикла внутри горутины — переменная hosts[i] может измениться к моменту выполнения горутины, что приведёт к неправильным вызовам.

Кроме того, при большом количестве сервисов (тысячи и более) запуск тысяч горутин одновременно приведёт к высокому потреблению памяти и ресурсов.

Рекомендации по исправлению:

Передавать переменную в замыкание, чтобы избежать захвата изменяющейся переменной:

for i := 0; i < len(hosts); i++ {
    host := hosts[i]
    go func() {
        response, err := s.httpClient.Send(ctx, host, order)
        if err != nil {
            s.logger.Error(ctx, "failed to send", err)
            return
        }
        s.logger.Info(ctx, "success", response)
    }()
}

Ограничить количество одновременно работающих горутин с помощью семафора или worker pool, чтобы не создавать тысячи горутин одновременно:

sem := make(chan struct{}, 100) // ограничение до 100 параллельных вызовов
for _, host := range hosts {
    sem <- struct{}{}
    go func(h string) {
        defer func() { <-sem }()
        response, err := s.httpClient.Send(ctx, h, order)
        if err != nil {
            s.logger.Error(ctx, "failed to send", err)
            return
        }
        s.logger.Info(ctx, "success", response)
    }(host)
}
// дождаться завершения всех горутин
for i := 0; i < cap(sem); i++ {
    sem <- struct{}{}
}

Таким образом, мы избегаем утечек памяти и контролируем нагрузку на систему.

Eelmine Edasi