首页

/

归档

/

友链

/

Github

/

模拟面试

/

独立黑客

/

资料

/

订阅

/

RSS

/

关于我


避免全局变量

睡觉前好像确实不应该想问题,大半夜的思维活跃睡不着了。当然,那只臭蚊子也有功劳,要是被我发现了我要灭了它。 不过,既然睡不着,那就起来写篇博客。

最近突然想到以前的一个项目,一个用来做token认证的微服务,当时为了快速实现,没有严格遵守MVC,很多controller里就有类似的 代码:

func XXXHandler() {
    db.Where("xxx = ?", 123).Find(&User)
}

后来同事接入opentracing的时候,就很痛苦。当然了,来新公司之后也写过类似的代码,主要是之前没有想到特别好的解决方案,以及 严格遵守MVC的必要性。Go的ORM实在是太难用了,以至于无法完全的将对象和数据库表解耦,如你所见,代码里还到处都是SQL的影子。 如果是SQLAlchemy还真的很难看出这样做有什么不好。像上面的代码,至少有这么几个坏处:

所以正确的方法应该是,遵循MVC。把数据库操作封装到M里,例如,model层这样写:

var db sql.DB // 不暴露db出去,把db限制在model这个包里

type User struct{}

func GetUserByID(id uint32) (*User, error) {
    user := User{}
    if err := db.Where("id = ?", id).Find(&user).Error; err != nil {
        return nil, err
    } else {
        return user, nil
    }
}

然后,controller里这样写:

func XXXHandler() {
    user, err := GetUserByID(user_id)
    xxxxxx
}

所以说,有些懒,偷不得。为了不让同事想开车从你身上碾几遍,还是好好设计,好好想好少挖坑吧😁