MiniGhost 最近的时间轴更新
MiniGhost

MiniGhost

🏢  Golang
V2EX 第 87408 号会员,加入于 2014-12-19 14:31:24 +08:00
国庆回家的各位,你们会买点儿什么带回家吗?
问与答  •  MiniGhost  •  235 天前  •  最后回复来自 EscYezi
9
出库克送的 Airpods,已到手 750 包顺丰
二手交易  •  MiniGhost  •  301 天前  •  最后回复来自 MiniGhost
1
Gopher China 门票转让
二手交易  •  MiniGhost  •  335 天前  •  最后回复来自 MiniGhost
2
有没有折腾耳机升级线的老哥啊,求推荐平衡线
问与答  •  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 最近回复了
    58 天前
    回复了 RiceMarch 创建的主题 职场话题 CodeReview?烂了算了
    不要搞这种开会形式的 Code Review ,因为每个人对项目代码理解的程度大多都不同。

    有可能一个新功能,就组内的 2 个同事比较熟悉,剩下的只知道个大概,那么在 Code Review 的时候,很容易就他们俩知道在聊什么,其他人根本听不懂,为了照顾其他同事大篇幅的讲上下文,也有可能一时半会儿消化理解不完。


    最合适的是让你的 Reviewer 给你看代码,IM 里面把 PR/MR 丢过去,Reviewer 有空了帮你看一下,遇到问题在 PR/MR 中写评论。
    如果觉得这个问题其他同事也需要注意,就把这个 PR/MR 丢在你们开发小群里,告知大家需要注意一下这里。
    买了个擦脸的护肤品,香港发货,现在还在广东,已经一个月了...
    @tool2d #29

    第一句话我就不认同,我认为必然有简洁的代码...

    举个例子,写个排序,我是自己手写个排序算法简洁,还是直接 array.sort(list) 简洁?


    再聊随着需求增加,会不断熵增,这个我完全认同,重构我也能接受。

    但是你认为,重构屎山简单,还是重构相对清晰简明的代码简单?

    很有可能在熵增的过程中,把控不好,产品层面熵增了 O(n),代码层面熵增了 O(n²),Code Review 做得好可以尽可能让业务熵增与代码熵增呈现一个线性关系。
    @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 不到点就不写了。
    @smilenceX #23 [握手]
    @yzbythesea #12 @smilenceX #18

    我对此持相反观点,不知道你听没听过 “代码是用来让人读的,只是顺便让机器执行而已”

    CodeReview 不是为了保证 0bug 。比如一些边界值、异常情况没有考虑,直到测试阶段才暴露出来这太正常了。

    CodeReview 最要求是更简洁、更已读、更适合的代码,比如是否落实了项目规范中的 MVC 、DDD 、比如是否 3 行代码就可以搞定的事情但是你不知道你写了 30 行、比如是否满足了 SOILD 原则等等。


    简而言之:Code Review 是用来保证代码质量,测试是用来保证产品质量,这两者并不是一个东西。
    你的观点是正确的,少量多次肯定是好的

    我之前遇到过一些同事,一口气提一两千行代码,这是指望 Reviewer 是个神仙吗?
    错误的习惯就应该就改正,而不是迁就他

    还有就是,这种事情给我的启发跟教训就是,如果不是很好的团队,就不要搞民主,搞专政搞一言堂。

    先把自己的方案推下去,日常多留心一下大家的执行情况,团队里面人员有执行不到位的即时纠正,之后再看情况了解了解大家的反馈,是否要做一些调整
    企业盘吧,企业版都是各项参数拉满的

    MG08 、MG09 听说声音相比较其西数希捷的声音会小一些

    而且我觉得 NAS 盘没必要考虑那么一丢丢的极限性能,千兆 LAN 下众生平等...

    自己搞了两块 MG08 ,顺序写确实很小,别的没明显对比过就不太好评价了
    90 天前
    回复了 qaqLjj 创建的主题 VPS 有两个闲置的甲骨文 vps 能用来干什么呢
    梯子、挖矿、PT
    关于   ·   帮助文档   ·   API   ·   FAQ   ·   我们的愿景   ·   广告投放   ·   感谢   ·   实用小工具   ·   2194 人在线   最高记录 5497   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 35ms · UTC 01:59 · PVG 09:59 · LAX 18:59 · JFK 21:59
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.