我的查找功能是否遵循最佳实践

最近发现不少小伙伴都对Golang很感兴趣，所以今天继续给大家介绍Golang相关的知识，本文《我的查找功能是否遵循最佳实践》主要内容涉及到等等知识点，希望能帮到你！当然如果阅读本文时存在不同想法，可以在评论中表达，但是请勿使用过激的措辞~

我想确保通过 id 查找模型的查询是正确的，因为我将对所有结构/模型使用相同的模式。

func (dbs *dbservice) getuserlocationid(locationid int) (user, error) {
    var model user
    if dbs.deps.db.where("location_id = ?", locationid).find(&model).recordnotfound() {
        return model, errors.new("user not found")
    }
    return model, nil
}

因此，web 应用程序中的一个常见用例是查找模型，如果模型不存在，我将插入一条新记录。

使用上面我会做的：

user, err := GetUserLocationById(123)
if err != nil {
  err := InsertNewUser(user)
}

现在我正在努力解决的问题是这个。如果错误不为零，那么我应该插入一个新用户。但是，如果错误是因为函数 getuserlocationbyid 内的查询具有错误的列名称怎么办？ 我不想在行确实存在时开始插入行。

寻求一些建议以确保我正确地执行此操作。

解决方案

关于“最佳实践”，最好发在the dedicated SO website下。

var errusernotfound = errors.new("user not found")

func (dbs *dbservice) getuserlocationid(locationid int) (user user, err error)
    record := dbs.deps.db.where("location_id = ?", locationid).find(&user)
    if record.recordnotfound() {
        err = errusernotfound
    }
    return
}

好吧，恕我直言，这更接近 go“最佳实践”标准。为常见返回的错误创建包错误值被认为是很好的做法。它可以帮助调用者正确处理每种情况（应该有），但这里的情况并非如此。

对于另一个片段，您真的不想这样做。首先，您省略了 getuserlocationbyid 和 insertnewuser 方法的指针接收器。然后，您通常不想隐藏父级的作用域 err 变量，但这里可以...

user, err := dbs.getuserlocationbyid(123)
if err != nil {
   err = dbs.insertnewuser(user)
   // handle err here
}

无论如何，我认为这样做有错误太糟糕了，因为除了“errusernotfound”之外，没有其他可能性。我建议改用 ok 布尔值。

func (dbs *dbservice) getuserlocationid(locationid int) (user user, ok bool)
    record := dbs.deps.db.where("location_id = ?", locationid).find(&user)
    ok = !record.recordnotfound()
    return
}

然后

var user User
if user, ok := dbs.GetUserLocationId(123); !ok {
    if err := dbs.InsertNewUser(user); err != nil {
        panic(err)
    }
}

理论要掌握，实操不能落！以上关于《我的查找功能是否遵循最佳实践》的详细介绍，大家都掌握了吧！如果想要继续提升自己的能力，那么就来关注golang学习网公众号吧！