查询时避免返回对象初始化及sql.ErrNoRows判断

执行SQL查询时,请避免提前将查询结果初始化,以避免结构对象默认值的影响,避免创建不必要的对象内存。通过返回对象指针nil判断避免sql.ErrNoRows使用,降低代码对error处理的复杂度、统一项目中对空查询结果处理逻辑。

一个反面例子:

  1. func (s *sTask) GetOne(ctx context.Context, id uint64) (out *entity.ResourceTask, err error) {
  2. out = new(model.TaskDetail)
  3. err = dao.ResourceTask.Ctx(ctx).WherePri(id).Scan(out)
  4. if err != nil {
  5. if err == sql.ErrNoRows {
  6. err = gerror.Newf(`record not found for "%d"`, id)
  7. }
  8. return
  9. }
  10. return
  11. }

在该例子中,实际返回的out对象由于对象初始化的缘故有了默认值(无论SQL是否查询到数据),并且sql.ErrNoRows的判断增加了代码逻辑中对error处理的复杂度。

建议改进如下:

  1. func (s *sTask) GetOne(ctx context.Context, id uint64) (out *entity.ResourceTask, err error) {
  2. err = dao.ResourceTask.Ctx(ctx).WherePri(id).Scan(&out)
  3. if err != nil {
  4. return
  5. }
  6. if out == nil {
  7. err = gerror.Newf(`record not found for "%d"`, id)
  8. }
  9. return
  10. }

注意代码中&out的使用。

更多的介绍请参考:ORM结果处理-为空判断