去比赛检测器错误



这是我的并发缓存代码:

package cache
import (
"sync"
)
// Func represents a memoizable function, operating on a string key, to use with a Cache
type Func func(key string) (interface{}, error)
// FuncResult stores the value of a Func call
type FuncResult struct {
val interface{}
err error
}
// Cache is a cache that memoizes results of an expensive computation
//
// It has a traditional implementation using mutexes.
type Cache struct {
// guards done
mu   sync.RWMutex
done map[string]chan bool
memo map[string]*FuncResult
f    Func
}
// New creates a new Cache and returns its pointer
func New(f Func) *Cache {
return &Cache{
memo: make(map[string]*FuncResult),
done: make(map[string]chan bool),
f:    f,
}
}
// Get a string key if it exists, otherwise computes the value and caches it.
//
// Returns the value and whether or not the key existed.
func (c *Cache) Get(key string) (*FuncResult, bool) {
c.mu.RLock()
_, ok := c.done[key]
c.mu.RUnlock()
if ok {
return c.get(key), true
}
c.mu.Lock()
_, ok = c.done[key]
if ok {
c.mu.Unlock()
} else {
c.done[key] = make(chan bool)
c.mu.Unlock()
v, err := c.f(key)
c.memo[key] = &FuncResult{v, err}
close(c.done[key])
}
return c.get(key), ok
}
// get returns the value of key, blocking on an existing computation
func (c *Cache) get(key string) *FuncResult {
<-c.done[key]
fresult, _ := c.memo[key]
return fresult
}

当我使用种族检测器运行此程序时,我没有收到任何错误:

package main
import (
"fmt"
"log"
"sync"
"time"
"github.com/yangmillstheory/go-cache/cache"
)
var f = func(key string) (interface{}, error) {
log.Printf("Computing value for key %sn", key)
time.Sleep(1000 * time.Millisecond)
return fmt.Sprintf("value for %s", key), nil
}
func main() {
var wg sync.WaitGroup
c := cache.New(f)
n := 10
k := "key1"
start := time.Now()
for i := 0; i < n; i++ {
wg.Add(1)
go func() {
defer wg.Done()
c.Get(k)
}()
}
wg.Wait()
log.Printf("Elapsed: %sn", time.Since(start))
}

但是,当我在循环体中启动两个不同的 goroutines 时,每个程序都获得不同的键,我收到一个错误:

解决此问题的方法是添加另一个互斥c.nu来保护memo,但它会使程序更慢,更复杂

func (c *Cache) Get(key string) (*FuncResult, bool) {
c.mu.RLock()
_, ok := c.done[key]
c.mu.RUnlock()
if ok {
return c.get(key), true
}
c.mu.Lock()
_, ok = c.done[key]
if ok {
c.mu.Unlock()
} else {
c.done[key] = make(chan bool)
c.mu.Unlock()
v, err := c.f(key)
c.nu.Lock()
c.memo[key] = &FuncResult{v, err}
c.nu.Unlock()
close(c.done[key])
}
return c.get(key), ok
}
// get returns the value of key, blocking on an existing computation
func (c *Cache) get(key string) *FuncResult {
<-c.done[key]
c.nu.RLock()
fresult, _ := c.memo[key]
c.nu.RUnlock()
return fresult
}

这里实际上有竞争条件需要担心吗?如果不同的 goroutines 同时访问同一数据结构中的不同键,那么只要同步发生在对给定键的访问中,这似乎不应该是一个问题?

换句话说,您是必须跨所有键同步,还是仅跨同一键同步?并发备忘录的用例似乎表明后者就足够了?

地图需要同步,特别是你不能在写入它时读取它,无论它是相同还是不同的键,所以你需要在meno地图周围锁定。

相关内容

  • 没有找到相关文章

最新更新