Ví dụ cho sync.WaitGroup đúng không?

Cách sử dụng ví dụ này có sync.WaitGroupđúng không? Nó mang lại kết quả mong đợi, nhưng tôi không chắc chắn về wg.Add(4)vị trí của wg.Done(). Có hợp lý khi thêm bốn goroutines cùng một lúc wg.Add()không?

http://play.golang.org/p/ecvYHiie0P

package main

import (
    "fmt"
    "sync"
    "time"
)

func dosomething(millisecs time.Duration, wg *sync.WaitGroup) {
    duration := millisecs * time.Millisecond
    time.Sleep(duration)
    fmt.Println("Function in background, duration:", duration)
    wg.Done()
}

func main() {
    var wg sync.WaitGroup
    wg.Add(4)
    go dosomething(200, &wg)
    go dosomething(400, &wg)
    go dosomething(150, &wg)
    go dosomething(600, &wg)

    wg.Wait()
    fmt.Println("Done")
}

Kết quả (như mong đợi):

Function in background, duration: 150ms
Function in background, duration: 200ms
Function in background, duration: 400ms
Function in background, duration: 600ms
Done

Vâng, ví dụ này là chính xác. Điều quan trọng là wg.Add()xảy ra trước khi gotuyên bố để ngăn chặn các điều kiện chủng tộc. Những điều sau đây cũng sẽ đúng:

func main() {
    var wg sync.WaitGroup
    wg.Add(1)
    go dosomething(200, &wg)
    wg.Add(1)
    go dosomething(400, &wg)
    wg.Add(1)
    go dosomething(150, &wg)
    wg.Add(1)
    go dosomething(600, &wg)

    wg.Wait()
    fmt.Println("Done")
}

Tuy nhiên, việc gọi đi gọi lại sẽ rất vô ích wg.Addkhi bạn đã biết nó sẽ được gọi bao nhiêu lần.

Waitgroupshoảng sợ nếu bộ đếm giảm xuống dưới không. Bộ đếm bắt đầu từ 0, mỗi cái Done()là a -1và mỗi cái Add()phụ thuộc vào tham số. Vì vậy, để đảm bảo rằng quầy không bao giờ giảm xuống dưới và tránh hoảng loạn, bạn cần phải Add()được đảm bảo để đến trước Done().

Trong cờ vây, những đảm bảo như vậy được đưa ra bởi mô hình bộ nhớ .

Mô hình bộ nhớ cho biết rằng tất cả các câu lệnh trong một chương trình goroutine duy nhất dường như được thực hiện theo thứ tự như khi chúng được viết. Có thể chúng sẽ không thực sự theo thứ tự đó, nhưng kết quả sẽ như thể. Nó cũng được đảm bảo rằng một quy trình sẽ không chạy cho đến sau gocâu lệnh gọi nó . Vì câu lệnh Add()xảy ra trước gocâu lệnh và gocâu lệnh xảy ra trước câu lệnh Done(), chúng ta biết điều đó Add()xảy ra trước câu lệnh Done().

Nếu bạn phải có gocâu lệnh trước Add(), chương trình có thể hoạt động chính xác. Tuy nhiên, nó sẽ là một điều kiện của cuộc đua vì nó sẽ không được đảm bảo.

Tôi khuyên bạn nên nhúng lệnh wg.Add()gọi vào doSomething()chính hàm, để nếu bạn điều chỉnh số lần nó được gọi, bạn không phải điều chỉnh riêng thông số thêm theo cách thủ công, điều này có thể dẫn đến lỗi nếu bạn cập nhật nhưng quên cập nhật khác (trong ví dụ nhỏ này là không chắc, nhưng cá nhân tôi vẫn tin rằng nó là thực tiễn tốt hơn để sử dụng lại mã).

Như Stephen Weinberg đã chỉ ra trong câu trả lời của mình cho câu hỏi này , bạn phải tăng nhóm chờ trước khi tạo ra gofunc, nhưng bạn có thể thực hiện điều này một cách dễ dàng bằng cách gói gofunc sinh ra bên trong doSomething()chính hàm, như sau:

func dosomething(millisecs time.Duration, wg *sync.WaitGroup) {
    wg.Add(1)
    go func() {
        duration := millisecs * time.Millisecond
        time.Sleep(duration)
        fmt.Println("Function in background, duration:", duration)
        wg.Done()
    }()
}

Sau đó, bạn có thể gọi nó mà không cần gogọi, ví dụ:

func main() {
    var wg sync.WaitGroup
    dosomething(200, &wg)
    dosomething(400, &wg)
    dosomething(150, &wg)
    dosomething(600, &wg)
    wg.Wait()
    fmt.Println("Done")
}

Là một sân chơi: http://play.golang.org/p/WZcprjpHa_

• cải tiến nhỏ dựa trên câu trả lời của Mroth
• sử dụng trì hoãn cho Xong sẽ an toàn hơn

  func dosomething(millisecs time.Duration, wg *sync.WaitGroup) {
  wg.Add(1)
  go func() {
      defer wg.Done()
      duration := millisecs * time.Millisecond
      time.Sleep(duration)
      fmt.Println("Function in background, duration:", duration)
  }()
}

func main() {
  var wg sync.WaitGroup
  dosomething(200, &wg)
  dosomething(400, &wg)
  dosomething(150, &wg)
  dosomething(600, &wg)
  wg.Wait()
  fmt.Println("Done")
}