需要有关 Go 并发编程的帮助

“纵有疾风来，人生不言弃”，这句话送给正在学习Golang的朋友们，也希望在阅读本文《需要有关 Go 并发编程的帮助》后，能够真的帮助到大家。我也会在后续的文章中，陆续更新Golang相关的技术文章，有好的建议欢迎大家在评论留言，非常感谢！

对于这个任务，我需要在数字列表中找到最小和。然后我必须打印具有最小总和的数字。这必须通过 mutex 和 waitgroups 来完成。我找不到错误在哪里或者为什么输出不同。 逻辑： scanf n 并用 len(n) 创建向量。然后创建数字总和的函数并将该函数转发到第二个函数，我们在一个 for 循环中将 goroutine 函数传递给第二个函数。

我运行此代码几次，有时对相同的输入给出不同的答案。 输入： 3 13 12 11 输出： 有时 12 有时11

package main

import (
    "fmt"
    "math"
    "runtime"
    "sync"
)

var wg sync.WaitGroup
var mutex sync.Mutex
var vector []int
var i int
var n int
var firstsum int
var p int //Temp sum
var index_result int

func sumanajmanjih(broj int) int {
    var br int
    var suma int
    br = int(math.Abs(float64(broj)))
    suma = 0
    for {
        suma += br % 10
        br = br / 10

        if br <= 0 {
            break
        }
    }
    return suma
}
func glavna(rg int) {
    var index int
    firstsum = sumanajmanjih(vector[0])
    for {
        mutex.Lock()
        if i == n {
            mutex.Unlock()
            break
        } else {
            index = i
            i += 1
            mutex.Unlock()
        }

        fmt.Printf("Procesor %d radni indeks %d\n", rg, index)
        p = sumanajmanjih(vector[index])
        if p < firstsum {
            firstsum = p
            index_result = index
        }
    }
    wg.Done()
}
func main() {
    fmt.Scanf("%d", &n)
    vector = make([]int, n)
    for i := 0; i < n; i++ {
        fmt.Scanf("%d", &vector[i])
    }
    fmt.Println(vector)
    brojGR := runtime.NumCPU()
    wg.Add(brojGR)
    for rg := 0; rg < brojGR; rg++ {
        go glavna(rg)
    }
    wg.Wait()
    fmt.Println(vector[index_result])
}

正确答案

不是您问题的完整答案，而是一些使代码更具可读性和稳定性的建议：

1. 使用英语命名 - glavna、brojGR 很难理解
2. 向代码添加注释以解释意图
3. 尽量避免共享/全局变量，尤其是对于并发代码。 glavna(rg) 是并发执行的，并且您在该函数内分配全局 i 和 p ，这是一个竞争条件。将所有数据显式作为参数或函数结果传入和传出到函数中。
4. 互斥锁很容易锁定代码，并且调试起来很复杂。简化其使用。通常 defer mutex.Unlock() 在 Lock() 之后的下一行就足够了。

到这里，我们也就讲完了《需要有关 Go 并发编程的帮助》的内容了。个人认为，基础知识的学习和巩固，是为了更好的将其运用到项目中，欢迎关注golang学习网公众号，带你了解更多关于的知识点！