MiniGhost 最近的时间轴更新 MiniGhost 最近的时间轴更新 |
MiniGhost🏢 Golang V2EX 第 87408 号会员，加入于 2014-12-19 14:31:24 +08:00今日活跃度排名 1660 |
http://feeeei.com
北京
feeeei
| [北京] 奇安信 - 服务端中台部门 - Golang 后端开发工程师 酷工作 • MiniGhost • 33 天前 • 最后回复来自 FIllerFooo | 14 |
| 国庆回家的各位，你们会买点儿什么带回家吗？ 问与答 • MiniGhost • 231 天前 • 最后回复来自 EscYezi | 9 |
| 出库克送的 Airpods，已到手 750 包顺丰 二手交易 • MiniGhost • 297 天前 • 最后回复来自 MiniGhost | 1 |
| Gopher China 门票转让 二手交易 • MiniGhost • 330 天前 • 最后回复来自 MiniGhost | 2 |
| Macbook 在接显示器 USB 上行后，开机必卡白苹果 MacBook Pro • MiniGhost • 346 天前 • 最后回复来自 Gakki0118 | 6 |
| 有没有折腾耳机升级线的老哥啊，求推荐平衡线 问与答 • MiniGhost • 2019-02-19 14:18:19 PM • 最后回复来自 del1214 | 7 |
| 分享一下改的 Nginx 编译安装直接开启 TLS1.3 的脚本 程序员 • MiniGhost • 2019-02-11 20:34:22 PM • 最后回复来自 yzc27 | 6 |
| 关于使用 Intel Hades Canyon（冥王峡谷）搞黑苹果的可行性探讨
1 问与答 • MiniGhost • 2019-11-27 09:19:36 AM • 最后回复来自 justin2018
|
31 |
| 出一台 充新一加 3T 6+64G 枪灰色版 二手交易 • MiniGhost • 2017-03-25 19:33:23 PM • 最后回复来自 2642375043 | 2 |
MiniGhost 最近回复了
53 天前 回复了 RiceMarch 创建的主题 › 职场话题 › CodeReview？烂了算了 |
不要搞这种开会形式的 Code Review ，因为每个人对项目代码理解的程度大多都不同。
有可能一个新功能，就组内的 2 个同事比较熟悉，剩下的只知道个大概，那么在 Code Review 的时候，很容易就他们俩知道在聊什么，其他人根本听不懂，为了照顾其他同事大篇幅的讲上下文，也有可能一时半会儿消化理解不完。
最合适的是让你的 Reviewer 给你看代码，IM 里面把 PR/MR 丢过去，Reviewer 有空了帮你看一下，遇到问题在 PR/MR 中写评论。
如果觉得这个问题其他同事也需要注意，就把这个 PR/MR 丢在你们开发小群里，告知大家需要注意一下这里。
有可能一个新功能，就组内的 2 个同事比较熟悉，剩下的只知道个大概，那么在 Code Review 的时候，很容易就他们俩知道在聊什么，其他人根本听不懂，为了照顾其他同事大篇幅的讲上下文，也有可能一时半会儿消化理解不完。
最合适的是让你的 Reviewer 给你看代码，IM 里面把 PR/MR 丢过去，Reviewer 有空了帮你看一下，遇到问题在 PR/MR 中写评论。
如果觉得这个问题其他同事也需要注意，就把这个 PR/MR 丢在你们开发小群里，告知大家需要注意一下这里。
55 天前 回复了 fkdog 创建的主题 › Apple › Studio Display 用的 ems 发货，卡在杭州萧山处理中心已经四天了。 |
买了个擦脸的护肤品，香港发货，现在还在广东，已经一个月了...
60 天前 回复了 niceyuri 创建的主题 › 问与答 › 关于 CodeReview，和团队小伙伴产生了分歧。 |
@tool2d #29
第一句话我就不认同，我认为必然有简洁的代码...
举个例子，写个排序，我是自己手写个排序算法简洁，还是直接 array.sort(list) 简洁？
再聊随着需求增加，会不断熵增，这个我完全认同，重构我也能接受。
但是你认为，重构屎山简单，还是重构相对清晰简明的代码简单？
很有可能在熵增的过程中，把控不好，产品层面熵增了 O(n)，代码层面熵增了 O(n²)，Code Review 做得好可以尽可能让业务熵增与代码熵增呈现一个线性关系。
第一句话我就不认同，我认为必然有简洁的代码...
举个例子，写个排序，我是自己手写个排序算法简洁，还是直接 array.sort(list) 简洁？
再聊随着需求增加，会不断熵增，这个我完全认同，重构我也能接受。
但是你认为，重构屎山简单，还是重构相对清晰简明的代码简单？
很有可能在熵增的过程中，把控不好，产品层面熵增了 O(n)，代码层面熵增了 O(n²)，Code Review 做得好可以尽可能让业务熵增与代码熵增呈现一个线性关系。
60 天前 回复了 niceyuri 创建的主题 › 问与答 › 关于 CodeReview，和团队小伙伴产生了分歧。 |
@yzbythesea #28
我刚刚讲的只是举了几个在 Code Review 中关注的例子，并不全，我还是认为即使稍有一定工作经验的人代码也不一定就是 OK 的... 也许我们之间对代码 OK 的标准的理解存在差异。
比如要求实现一个订单支付的接口，不同的人也许会有不同的设计方式：
- POST /order/{order_id}/payment
- POST /order/{order_id}/pay
- POST /order/pay?order_id=xxx
也比如有的人曾经的工作经验是 HTTP Response 在程序中永远反 200 ，也有的人会相对更遵守 RESTful 。
这上面的取舍，在产品层面都无关产品质量，但是我认为这应该也是 Code Review 时应该把控的一部分。
也有可能一些团队对这些也有明确的约束，但是 Code Review 还是有很多东西值得关注的。
我日常中还有的一些 Code Review 纠正同事的案例还有：
- 这段写的多余了，我之前写过类似的，在 xxx ，你复用一下就可以了
- 这里可以使用语言的新特性、xxx 第三方库实现，更简洁
- 标准库里有标准实现，不用自己再写一遍，直接 xxxx 就完事儿了
- 这里用防御式写法，先把这几种异常情况先判断处理了，剩下的代码逻辑会更干净
还有很多是语言相关的，不清楚你主语言是什么，怕讲出来非这个语言的开发 Get 不到点就不写了。
我刚刚讲的只是举了几个在 Code Review 中关注的例子，并不全，我还是认为即使稍有一定工作经验的人代码也不一定就是 OK 的... 也许我们之间对代码 OK 的标准的理解存在差异。
比如要求实现一个订单支付的接口，不同的人也许会有不同的设计方式：
- POST /order/{order_id}/payment
- POST /order/{order_id}/pay
- POST /order/pay?order_id=xxx
也比如有的人曾经的工作经验是 HTTP Response 在程序中永远反 200 ，也有的人会相对更遵守 RESTful 。
这上面的取舍，在产品层面都无关产品质量，但是我认为这应该也是 Code Review 时应该把控的一部分。
也有可能一些团队对这些也有明确的约束，但是 Code Review 还是有很多东西值得关注的。
我日常中还有的一些 Code Review 纠正同事的案例还有：
- 这段写的多余了，我之前写过类似的，在 xxx ，你复用一下就可以了
- 这里可以使用语言的新特性、xxx 第三方库实现，更简洁
- 标准库里有标准实现，不用自己再写一遍，直接 xxxx 就完事儿了
- 这里用防御式写法，先把这几种异常情况先判断处理了，剩下的代码逻辑会更干净
还有很多是语言相关的，不清楚你主语言是什么，怕讲出来非这个语言的开发 Get 不到点就不写了。
60 天前 回复了 niceyuri 创建的主题 › 问与答 › 关于 CodeReview，和团队小伙伴产生了分歧。 |
@smilenceX #23 [握手]
60 天前 回复了 niceyuri 创建的主题 › 问与答 › 关于 CodeReview，和团队小伙伴产生了分歧。 |
@yzbythesea #12 @smilenceX #18
我对此持相反观点，不知道你听没听过 “代码是用来让人读的，只是顺便让机器执行而已”
CodeReview 不是为了保证 0bug 。比如一些边界值、异常情况没有考虑，直到测试阶段才暴露出来这太正常了。
CodeReview 最要求是更简洁、更已读、更适合的代码，比如是否落实了项目规范中的 MVC 、DDD 、比如是否 3 行代码就可以搞定的事情但是你不知道你写了 30 行、比如是否满足了 SOILD 原则等等。
简而言之：Code Review 是用来保证代码质量，测试是用来保证产品质量，这两者并不是一个东西。
我对此持相反观点，不知道你听没听过 “代码是用来让人读的，只是顺便让机器执行而已”
CodeReview 不是为了保证 0bug 。比如一些边界值、异常情况没有考虑，直到测试阶段才暴露出来这太正常了。
CodeReview 最要求是更简洁、更已读、更适合的代码，比如是否落实了项目规范中的 MVC 、DDD 、比如是否 3 行代码就可以搞定的事情但是你不知道你写了 30 行、比如是否满足了 SOILD 原则等等。
简而言之：Code Review 是用来保证代码质量，测试是用来保证产品质量，这两者并不是一个东西。
60 天前 回复了 niceyuri 创建的主题 › 问与答 › 关于 CodeReview，和团队小伙伴产生了分歧。 |
你的观点是正确的，少量多次肯定是好的
我之前遇到过一些同事，一口气提一两千行代码，这是指望 Reviewer 是个神仙吗？
错误的习惯就应该就改正，而不是迁就他
还有就是，这种事情给我的启发跟教训就是，如果不是很好的团队，就不要搞民主，搞专政搞一言堂。
先把自己的方案推下去，日常多留心一下大家的执行情况，团队里面人员有执行不到位的即时纠正，之后再看情况了解了解大家的反馈，是否要做一些调整
我之前遇到过一些同事，一口气提一两千行代码，这是指望 Reviewer 是个神仙吗？
错误的习惯就应该就改正，而不是迁就他
还有就是，这种事情给我的启发跟教训就是，如果不是很好的团队，就不要搞民主，搞专政搞一言堂。
先把自己的方案推下去，日常多留心一下大家的执行情况，团队里面人员有执行不到位的即时纠正，之后再看情况了解了解大家的反馈，是否要做一些调整
70 天前 回复了 yongboy 创建的主题 › NAS › 各位，选择东芝 MG 还是 MN 系列用于 NAS 存储呢 ？ |
企业盘吧，企业版都是各项参数拉满的
MG08 、MG09 听说声音相比较其西数希捷的声音会小一些
而且我觉得 NAS 盘没必要考虑那么一丢丢的极限性能，千兆 LAN 下众生平等...
自己搞了两块 MG08 ，顺序写确实很小，别的没明显对比过就不太好评价了
MG08 、MG09 听说声音相比较其西数希捷的声音会小一些
而且我觉得 NAS 盘没必要考虑那么一丢丢的极限性能，千兆 LAN 下众生平等...
自己搞了两块 MG08 ，顺序写确实很小，别的没明显对比过就不太好评价了
85 天前 回复了 qaqLjj 创建的主题 › VPS › 有两个闲置的甲骨文 vps 能用来干什么呢 |
梯子、挖矿、PT
Select Language