Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

7 个优化建议 #16

Open
guowei-gong opened this issue Nov 7, 2023 · 3 comments
Open

7 个优化建议 #16

guowei-gong opened this issue Nov 7, 2023 · 3 comments

Comments

@guowei-gong
Copy link

嗨。这是我在学习 due 的过程中发现的一些小问题(有一些可能并不是问题)

截图可能比较大,需要点开来查看,因为我想提供更完整的追踪线索。(🤔 我怎么不使用 github 的代码链接?因为我截图完了才想起这个功能更方便您查看对应的代码,下次一定)

1. cluster/node/context.go 中的 Context 注释应该是:获取上下文

image

2. cluster/node/node.go 中的 setState,n.instance 的判断可以提前返回

修改前:
image

修改后:

if n.instance == nil {
  return
}
...

3. transport/grpc/node/server.go 中的 NewServer 错误判断是否是多余的?

修改前:
image

修改后:
image

4. locate/redis/watcher 中的 newWatcherMgr 函数中,go 协程中的错误并没有做任何处理

image

5. locate/redis/watcher 的 Next 方法中的 state 添加枚举值或注释

image

6. internal/dispatcher/dispatcher.go 中的 ReplaceServices 方法,释放锁可以使用 defer。但我不确定这么做的目的是否是为了缩小锁的临界区?

image

7. transport/grpc/internal/resolver/direct/builder.go 中的 Build 方法中,err 可以缩小作用域

修改前:
image

修改后:
image

@guowei-gong
Copy link
Author

network/ws/client_options 中的结构体 clientOptions 字段 msgType 没有任何地方使用

image

@guowei-gong
Copy link
Author

/due/transport/internal/pb/gate.proto 中,service Gate 下的 rpc Stat 的备注错误。应该是 统计会话总数

image

@guowei-gong
Copy link
Author

due/registry/etcd/registry_test.go 文件中,92 行代码应该是 reg.Close()

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant