doushi1964 2017-06-23 03:13
浏览 16
已采纳

返回结构或错误的函数

I have method that looks like this:

var eventNotDetected = errors.New("Event not detected")

type VoucherUsageEvent struct {
    M models.M
}

func detectVoucherUsageEvent(_ uint64, changeset changelog.Changeset) (VoucherUsageEvent, error) {
    var event VoucherUsageEvent

    if changeset.GetHeader().Table != db.TABLE_NAME {
        return event, eventNotDetected
    }

    switch changeset.(type) {
    case changelog.InsertChangeset, changelog.UpdateChangeset:
        m, err := models.Map(changeset)

        if err != nil {
            return event, err
        }

        return VoucherUsageEvent{m}, nil

    default:
        return event, eventNotDetected
    }
}

The idea is that it can return an object instance or not, depending on the input. If the object instance can't be returned, the function should indicate why it could not be returned (error occurred during mapping, or just not found)

The return signature is a bit weird when the function doesn't return the instance, for example:

return event, eventNotDetected
return event, err

Basically I just use event because I can't return just nil.

Or is the better way to just return a reference in this case, so I could change the return cases to:

return nil, eventNotDetected
return nil, err
  • 写回答

1条回答 默认 最新

  • dtc99987 2017-06-23 03:42
    关注

    Yes you can, define the out parameter as pointer *VoucherUsageEvent.

    func detectVoucherUsageEvent(_ uint64, changeset changelog.Changeset) (*VoucherUsageEvent, error)
    

    Return as reference or nil.

    return &VoucherUsageEvent{m}, nil
    

    OR

    return &event, eventNotDetected
    

    OR

    return nil, err
    
    本回答被题主选为最佳回答 , 对您是否有帮助呢?
    评论

报告相同问题?

悬赏问题

  • ¥15 一直显示正在等待HID—ISP
  • ¥15 Python turtle 画图
  • ¥15 关于大棚监测的pcb板设计
  • ¥15 stm32开发clion时遇到的编译问题
  • ¥15 lna设计 源简并电感型共源放大器
  • ¥15 如何用Labview在myRIO上做LCD显示?(语言-开发语言)
  • ¥15 Vue3地图和异步函数使用
  • ¥15 C++ yoloV5改写遇到的问题
  • ¥20 win11修改中文用户名路径
  • ¥15 win2012磁盘空间不足,c盘正常,d盘无法写入