sockpuppet9527

sockpuppet9527

V2EX 第 164946 号会员,加入于 2016-03-26 13:17:19 +08:00
今日活跃度排名 17949
根据 sockpuppet9527 的设置,主题列表被隐藏
二手交易 相关的信息,包括已关闭的交易,不会被隐藏
sockpuppet9527 最近回复了
14 天前
回复了 conyliu 创建的主题 MySQL mysql 9 出来了?
@mark2025 这可能有点夸张,有前同事去 zilliz 说 Milvus + cuda 会比 pgvector 快的....

不过话说回来 PG 是我所看过的数据库中,个人认为写的最好的,代码质量相当高。
14 天前
回复了 conyliu 创建的主题 MySQL mysql 9 出来了?
@mark2025 PG hook 多,extension 好做,社区很壮大,即使 PG 本身不带的功能,社区总是能找到 extension 做过了的。
18 天前
回复了 KJH 创建的主题 生活 换了个房子,隔音超级不好
如果是二房东/公寓管理房租的,那么看看能不能换一套?
18 天前
回复了 huyangq 创建的主题 程序员 大家 code review 是如何做的?
@sockpuppet9527 #18 另外想补充一些,目前应该知名一些的开源项目都是需要每一个 PR 进行 review 的。
18 天前
回复了 huyangq 创建的主题 程序员 大家 code review 是如何做的?
@huyangq #13

如果读的顺畅了,那速度会很快。跑 case 的话,我有个环境专门来验 case ,一套脚本拉下来+编译+跑 case 10~20 分钟左右。

目前我工作上做 code review 时间大概是 2 ~ 3 成左右,我之前同事,某个开源的 maintainer ,他的 code review 时间大概要占在 5 成左右。基本年长一些的同事都是占这个值。

我个人是拥护 code review 的,code review 带来的好处是去追赶进度/盲目重构不可比的。
21 天前
回复了 huyangq 创建的主题 程序员 大家 code review 是如何做的?
1. code review 的流程

我本人做 code review ,得细分看什么类型的 pr ,如果是 fix 类的 pr ,那么只做逻辑上的验证即可。
但如果是 feature 类的 pr ,会先把 branch 拉下来,看本身测试 case 跑一下。然后找到"入口",一般来说都是接口,如果不是接口,那么回想着能不能改成接口?

有了"入口"之后,那么基本就是接口->实现->调用者这样去看,我会一行行读,主要看几个方面
- 当前接口在哪个层级?放得位置是否合理?抽象接口做的是否足够合理?
- 实现函数是否合理?注释/命名是否符合目前的 code style ?参数和返回是否能改的更合理?
- 当前实现逻辑是否正确?是否存在风险?参数有无验证?
- 是否存在极端 case 出现问题?

当然还有很多,一时半会可能总结不出来,但如果你让别人多 review 你的代码,你也能找到自己的经验

2. code review 的频率

每个 PR
@kinsou 好的,感谢,我再忍忍
关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   1114 人在线   最高记录 6679   ·     Select Language
创意工作者们的社区
World is powered by solitude
VERSION: 3.9.8.5 · 14ms · UTC 23:38 · PVG 07:38 · LAX 16:38 · JFK 19:38
Developed with CodeLauncher
♥ Do have faith in what you're doing.