[Golang] context引发的一个小bug

asong · · 484 次点击 · · 开始浏览    
这是一个创建于 的文章,其中的信息可能已经有所发展或是发生改变。

背景

哈喽,everybody,小松子,再次回归,最近工作比较忙,好久都没有更新文章了,接下来会进行补更。今天与大家分享一个日常开发比较容易错误的点,那就是contxt误用导致的bug,我自己就因为误用导致异步更新缓存都失败了,究竟是因为什么呢?看这样一个例子,光看代码,你能看出来有什么bug吗?
func AsyncAdd(run func() error)  {
    //TODO: 扔进异步协程池
    go run()
}

func GetInstance(ctx context.Context,id uint64) (string, error) {
    data,err := GetFromRedis(ctx,id)
    if err != nil && err != redis.Nil{
        return "", err
    }
    // 没有找到数据
    if err == redis.Nil {
        data,err = GetFromDB(ctx,id)
        if err != nil{
            return "", err
        }
        AsyncAdd(func() error{
            return UpdateCache(ctx,id,data)
        })
    }
    return data,nil
}

func GetFromRedis(ctx context.Context,id uint64) (string,error) {
    // TODO: 从redis获取信息
    return "",nil
}

func GetFromDB(ctx context.Context,id uint64) (string,error) {
    // TODO: 从DB中获取信息
    return "",nil
}

func UpdateCache(ctx context.Context,id interface{},data string) error {
    // TODO:更新缓存信息
    return nil
}

func main()  {
    ctx,cancel := context.WithTimeout(context.Background(), 3 * time.Second)
    defer cancel()
    _,err := GetInstance(ctx,2021)
    if err != nil{
        return
    }
}

分析

我们先简单分析一下,这一段代码要干什么?其实很简单,我们想要获取一段信息,首先会从缓存中获取,如果缓存中获取不到,我们就从DB中获取,从DB中获取到信息后,在协程池中放入更新缓存的方法,异步去更新缓存。整个设计是不是很完美,但是在实际工作中,异步更新缓存就没有成功过?

导致失败的原因就在这一段代码:

    AsyncAdd(func() error{
            return UpdateCache(ctx,id,data)
        })

错误的原因只有一个,就是这个ctx,如果改成这样,就啥事没有了。

AsyncAdd(func() error{
            ctxAsync,cancel := context.WithTimeout(context.Background(),3 * time.Second)
            defer cancel()
            return UpdateCache(ctxAsync,id,data)
        })

看到这个,想必大家就已经知道为什么吧?

在这个ctx树中,根结点发生了cancel(),会将信号即时同步给下层,因为异步任务的ctx也在这棵树的节点上,所以当main goroutine取消了ctx时,异步任务也被取消了,导致了缓存更新一直失败。

因为我之前写过一篇关于详解Context包,看这一篇就够了!!!的文章,就不在这里细说其原理了,想知道其内部是怎么实现的,看以前这篇文章就可以了。在这里在与大家分享一下context的使用原则,避免踩坑。

  • context.Background 只应用在最高等级,作为所有派生 context 的根。
  • context 取消是建议性的,这些函数可能需要一些时间来清理和退出。
  • 不要把Context放在结构体中,要以参数的方式传递。
  • Context作为参数的函数方法,应该把Context作为第一个参数,放在第一位。
  • 给一个函数方法传递Context的时候,不要传递nil,如果不知道传递什么,就使用context.TODO
  • Context的Value相关方法应该传递必须的数据,不要什么数据都使用这个传递。context.Value 应该很少使用,它不应该被用来传递可选参数。这使得 API 隐式的并且可以引起错误。取而代之的是,这些值应该作为参数传递。
  • Context是线程安全的,可以放心的在多个goroutine中传递。同一个Context可以传给使用其的多个goroutine,且Context可被多个goroutine同时安全访问。
  • Context 结构没有取消方法,因为只有派生 context 的函数才应该取消 context。

Go 语言中的 context.Context 的主要作用还是在多个 Goroutine 组成的树中同步取消信号以减少对资源的消耗和占用,虽然它也有传值的功能,但是这个功能我们还是很少用到。在真正使用传值的功能时我们也应该非常谨慎,使用 context.Context 进行传递参数请求的所有参数一种非常差的设计,比较常见的使用场景是传递请求对应用户的认证令牌以及用于进行分布式追踪的请求 ID。

总结

写这篇文章的目的,就是把我日常写的bug分享出来,防止后人踩坑。已经踩过的坑就不要再踩了,把找bug的时间节省出来,多学点其他知识,他不香嘛~。

好啦,这篇文章就到这里啦,素质三连(分享、点赞、在看)都是笔者持续创作更多优质内容的动力!

创建了一个Golang学习交流群,欢迎各位大佬们踊跃入群,我们一起学习交流。入群方式:加我vx拉你入群,或者公众号获取入群二维码

结尾给大家发一个小福利吧,最近我在看[微服务架构设计模式]这一本书,讲的很好,自己也收集了一本PDF,有需要的小伙可以到自行下载。获取方式:关注公众号:[Golang梦工厂],后台回复:[微服务],即可获取。

我翻译了一份GIN中文文档,会定期进行维护,有需要的小伙伴后台回复[gin]即可下载。

翻译了一份Machinery中文文档,会定期进行维护,有需要的小伙伴们后台回复[machinery]即可获取。

我是asong,一名普普通通的程序猿,让我们一起慢慢变强吧。欢迎各位的关注,我们下期见~~~

推荐往期文章:


有疑问加站长微信联系(非本文作者)

本文来自:Segmentfault

感谢作者:asong

查看原文:[Golang] context引发的一个小bug

入群交流(和以上内容无关):加入Go大咖交流群,或添加微信:liuxiaoyan-s 备注:入群;或加QQ群:692541889

484 次点击  
加入收藏 微博
暂无回复
添加一条新回复 (您需要 登录 后才能回复 没有账号 ?)
  • 请尽量让自己的回复能够对别人有帮助
  • 支持 Markdown 格式, **粗体**、~~删除线~~、`单行代码`
  • 支持 @ 本站用户;支持表情(输入 : 提示),见 Emoji cheat sheet
  • 图片支持拖拽、截图粘贴等方式上传