Это фича или баг?

Question

[kolo2012]

Всегда будет 2

package main

import (
	"fmt"
	"sync"
)

func main() {
	q := []string{"1", "2"}
	wg := new(sync.WaitGroup)
	for _, val := range q {
		wg.Add(1)
		go func(){
			fmt.Println(val)
			wg.Done()
		}()
	}
	wg.Wait()
}

https://play.golang.org/p/pSwQVmjsV-

Comments

[Никита]

https://golang.org/doc/effective_go.html#channels
Эта ситуация полностью описана в доках

[kolo2012]

Никита: я как-то упустил это, а мб. просто забыл.

Answer 1

[Даша Циклаури]

читай про замыкания (частые ошибки новичков), как происходит захват переменной, чтобы результат был ожидаемый вот пример кода

package main

import (
  "fmt"
  "sync"
)

func main() {
  q := []string{"1", "2"}
  wg := new(sync.WaitGroup)
  for _, val := range q {
    wg.Add(1)
    go func(tVal string){
      fmt.Println(tVal)
      wg.Done()
    }(val)
  }
  wg.Wait()
}

вот еще классная статья, прочитав которую не будешь допускать такие ошибки

Comments

[kolo2012]

я как-бы не новичёк. Это уже сделал как только наткнулся. Более интересно зачем так сделано в go.

[kolo2012]

по хорошему замыкание должно работать с копией данных, а не ссылкой на ячейку.

[Даша Циклаури]

kolo2012: stackoverflow.com/questions/25919213/why-does-gola...

[kolo2012]

Ivan Sokolov: напиши это в ответы, пометку поставлю.

[kolo2012]

Ivan Sokolov: я не про объяснение. Я про "так сделано по историческим причинам"