Q: golang 错误处理,我总觉得我写的很怪,不知道怎么改 - V2EX
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
anonymous2351d00
V2EX    问与答

Q: golang 错误处理,我总觉得我写的很怪,不知道怎么改

  •  
  •   anonymous2351d00 2023 年 3 月 6 日 1885 次点击
    这是一个创建于 1133 天前的主题,其中的信息可能已经有所发展或是发生改变。

    Intro

    • 萌新刚刚开始写 golang
    • 总觉得哪里怪怪的

    如下代码

    • 数据源
    func Open() (*ent.Client, error) { dsn := fmt.Sprintf("%s:%s@tcp(%s)/ncepu", Option.Username, Option.Password, Option.Address) client, err := ent.Open("mysql", dsn) if err != nil { return nil, err } defer func(client *ent.Client) { err := client.Close() if err != nil { log.Fatal(err) } }(client) return client, nil } 
    • 检测 id 是否存在
    func ExistClientID(clientID string) (bool, error) { client, err := mysql.Open() if err != nil { return false, err } return client.Authorization.Query().Where(authorization.ClientID(clientID)).Exist(context.Background()) } 
    • 最外层函数
    func PasswordAuthorizationHandler(ctx context.Context, clientID, username, password string) (userID string, err error) { exist, e := service.ExistClientID(clientID) if e != nil { err = e return } if !exist { err = errors.ErrInvalidClient return } uid, e := service.Authorization(ctx, username, password) if e != nil { err = e return } return uid, nil } 

    qeustion

    • 1.这几段代码里的 error 会不会像我想的那样传递到最外层
    • 2.Open()函数中的 defer 这样写很难受,但又不想在每一处都写 defer close
    • 3.请大佬帮忙看看代码,优化优化
    9 条回复    2023-03-07 09:40:45 +08:00
    anonymous2351d00
        1
    anonymous2351d00  
    OP
       2023 年 3 月 6 日
    看帖子格式时候又看了一遍代码,好多 nil 没有处理,我好难受
    anonymous2351d00
        2
    anonymous2351d00  
    OP
       2023 年 3 月 6 日
    大家看的时候就看个格式吧,别细究业务了,想知道这种 err 和 nil 怎么处理好看
    14v45mJPBYJW8dT7
        3
    14v45mJPBYJW8dT7  
       2023 年 3 月 6 日
    Open 获取 Client 后还能使用吗,defer 里不是 Close 了,我都是在 ExistClientID 中 defer client.close ,错误一般不关心
    webcape233
        4
    webcape233  
       2023 年 3 月 6 日 via iPhone
    这个 err 不会外传,难道 ide 没报错?应该 return "" err , 前面那两个可以将 err 写在第一个=语句后用分号分隔
    dobelee
        5
    dobelee  
       2023 年 3 月 6 日 via iPhone
    打开的方法里怎么还关闭了?
    none
        6
    none  
       2023 年 3 月 6 日
    Open 里打开的 client 在返回之前 close 了,外面根本用不了
    lessMonologue
        7
    lessMonologue  
       2023 年 3 月 7 日
    最内层只负责返回数据或者 error ,调用者自行处理错误(打印日志...)。
    anonymous2351d00
        8
    anonymous2351d00  
    OP
       2023 年 3 月 7 日
    @dobelee
    @rimutuyuan

    我应该再写一个函数叫 Destroy 在调用层关闭连接,然后再 处理完数据以后在处理数据的函数里关闭链接


    @lessMonologue 好的大佬,我重构一下
    anonymous2351d00
        9
    anonymous2351d00  
    OP
       2023 年 3 月 7 日
    @webcape233 还没有试过,理解着是函数后面返回值也可以用?给他们赋值 return 中断函数外面就能收到值了?
    关于     帮助文档     自助推广系统     博客     API     FAQ     Solana     988 人在线   最高记录 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 24ms UTC 19:22 PVG 03:22 LAX 12:22 JFK 15:22
    Do have faith in what you're doing.
    ubao msn snddm index pchome yahoo rakuten mypaper meadowduck bidyahoo youbao zxmzxm asda bnvcg cvbfg dfscv mmhjk xxddc yybgb zznbn ccubao uaitu acv GXCV ET GDG YH FG BCVB FJFH CBRE CBC GDG ET54 WRWR RWER WREW WRWER RWER SDG EW SF DSFSF fbbs ubao fhd dfg ewr dg df ewwr ewwr et ruyut utut dfg fgd gdfgt etg dfgt dfgd ert4 gd fgg wr 235 wer3 we vsdf sdf gdf ert xcv sdf rwer hfd dfg cvb rwf afb dfh jgh bmn lgh rty gfds cxv xcv xcs vdas fdf fgd cv sdf tert sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf shasha9178 shasha9178 shasha9178 shasha9178 shasha9178 liflif2 liflif2 liflif2 liflif2 liflif2 liblib3 liblib3 liblib3 liblib3 liblib3 zhazha444 zhazha444 zhazha444 zhazha444 zhazha444 dende5 dende denden denden2 denden21 fenfen9 fenf619 fen619 fenfe9 fe619 sdf sdf sdf sdf sdf zhazh90 zhazh0 zhaa50 zha90 zh590 zho zhoz zhozh zhozho zhozho2 lislis lls95 lili95 lils5 liss9 sdf0ty987 sdft876 sdft9876 sdf09876 sd0t9876 sdf0ty98 sdf0976 sdf0ty986 sdf0ty96 sdf0t76 sdf0876 df0ty98 sf0t876 sd0ty76 sdy76 sdf76 sdf0t76 sdf0ty9 sdf0ty98 sdf0ty987 sdf0ty98 sdf6676 sdf876 sd876 sd876 sdf6 sdf6 sdf9876 sdf0t sdf06 sdf0ty9776 sdf0ty9776 sdf0ty76 sdf8876 sdf0t sd6 sdf06 s688876 sd688 sdf86