同时从地图上书写和阅读会导致潜在的竞争条件

时间:2017-04-17 20:06:32

标签: http go concurrency race-condition

我有一个非常基本的Golang应用程序,可以创建并运行HTTP服务器。服务器有2个端点,一个用于发送数据,另一个用于接收。

每次向服务器发出POST请求时,都会解析来自正文的传入数据并将其推送到某个频道。然后我有一个从通道读取并将数据保存到地图的功能。

每次向服务器发出GET请求时,JSON都会封送地图并将其发送给客户端。

对HTTP服务器的每个请求都是异步运行的,而worker函数在它自己的goroutine中同步写入映射。

基本伪代码如下:

package main

import (
    "net/http"
)

type dataStore map[string]string

func listenHandler(stream chan string) http.HandlerFunc {
    return func(w http.ResponseWriter, r *http.Request) {
        // JSON unmarshal request body

        // push data onto stream
        stream <- data
    }
}

func serveHandler(store *dataStore) http.HandlerFunc {
    return func(w http.ResponseWriter, r *http.Request) {
        // JSON marshal data store
        // serve to client
    }
}

func worker(stream <-chan string) *dataStore {
    store := make(dataStore)
    go func() {
        for data := range stream {
            // process and write data to map
        }
    }()
    return &store
}

func main() {
    stream := make(chan string)
    store := worker(stream)

    http.HandleFunc("/listen", listenHandler(stream))
    http.HandleFunc("/serve", serveHandler(store))

    http.ListenAndServe(":8080", nil)
}

我已经测试并运行了应用程序没有任何问题,但我被告知它有潜在的竞争条件,我不知道为什么。这是真的?

1 个答案:

答案 0 :(得分:2)

在给出的代码中,您只是同步您的写入,而不是您的读取;这意味着你在写作时仍然可以从地图上读取,这将是一种竞争条件。

为了使其成为线程安全的,您需要在并发结构中包含读取和写入 - 使用通道或互斥锁 - 以确保在任何给定时间只有一个线程正在访问它(如果它是写入)操作。只要没有线程正在写入多个线程,就可以安全地同时读取多个线程。 .flex-center { align-items: center; display: flex; flex-wrap: wrap; } .text-container { flex-grow : 1; } 明确提供此功能。