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

[anonymous2351d00]

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.请大佬帮忙看看代码，优化优化

[anonymous2351d00]

看帖子格式时候又看了一遍代码，好多 nil 没有处理，我好难受

[anonymous2351d00]

大家看的时候就看个格式吧，别细究业务了，想知道这种 err 和 nil 怎么处理好看

[rimutuyuan]

Open 获取 Client 后还能使用吗，defer 里不是 Close 了，我都是在 ExistClientID 中 defer client.close ，错误一般不关心

[webcape233]

这个 err 不会外传，难道 ide 没报错？应该 return "" err , 前面那两个可以将 err 写在第一个=语句后用分号分隔

[dobelee]

打开的方法里怎么还关闭了？

[none]

Open 里打开的 client 在返回之前 close 了，外面根本用不了

[lessMonologue]

最内层只负责返回数据或者 error ，调用者自行处理错误（打印日志...）。

[anonymous2351d00]

@dobelee
@rimutuyuan

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


@lessMonologue 好的大佬，我重构一下

[anonymous2351d00]

@webcape233 还没有试过，理解着是函数后面返回值也可以用？给他们赋值 return 中断函数外面就能收到值了？