V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
V2EX  ›  feeeei  ›  全部回复第 2 页 / 共 13 页
回复总数  244
1  2  3  4  5  6  7  8  9  10 ... 13  
2022 年 3 月 18 日
回复了 niceyuri 创建的主题 问与答 关于 CodeReview,和团队小伙伴产生了分歧。
@tool2d #29

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

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


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

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

很有可能在熵增的过程中,把控不好,产品层面熵增了 O(n),代码层面熵增了 O(n²),Code Review 做得好可以尽可能让业务熵增与代码熵增呈现一个线性关系。
2022 年 3 月 18 日
回复了 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 不到点就不写了。
2022 年 3 月 18 日
回复了 niceyuri 创建的主题 问与答 关于 CodeReview,和团队小伙伴产生了分歧。
@smilenceX #23 [握手]
2022 年 3 月 18 日
回复了 niceyuri 创建的主题 问与答 关于 CodeReview,和团队小伙伴产生了分歧。
@yzbythesea #12 @smilenceX #18

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

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

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


简而言之:Code Review 是用来保证代码质量,测试是用来保证产品质量,这两者并不是一个东西。
2022 年 3 月 18 日
回复了 niceyuri 创建的主题 问与答 关于 CodeReview,和团队小伙伴产生了分歧。
你的观点是正确的,少量多次肯定是好的

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

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

先把自己的方案推下去,日常多留心一下大家的执行情况,团队里面人员有执行不到位的即时纠正,之后再看情况了解了解大家的反馈,是否要做一些调整
2022 年 3 月 8 日
回复了 yongboy 创建的主题 NAS 各位,选择东芝 MG 还是 MN 系列用于 NAS 存储呢 ?
企业盘吧,企业版都是各项参数拉满的

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

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

自己搞了两块 MG08 ,顺序写确实很小,别的没明显对比过就不太好评价了
2022 年 2 月 21 日
回复了 MarkLazy 创建的主题 VPS 有两个闲置的甲骨文 vps 能用来干什么呢
梯子、挖矿、PT
2022 年 2 月 19 日
回复了 uudj 创建的主题 剧集 有好看的剧,蹲家里刷刷剧了。
富豪谷底求翻身,第二季在 B 站刚上,正在追
2022 年 2 月 14 日
回复了 vzyw 创建的主题 程序员 给一个 rgb 数值,怎么知道这个颜色的名称?
RGB 三原色可以映射成 X 、Y 、Z 三维正方体,每个轴都是 0-255 的正整数

维护一个确定的颜色名称表,比如#FF0000 红色,等于( 255 ,0 ,0 )是三维正方体上的一个坐标。

每一个随机的颜色点,可以遍历一遍颜色表,求两点之间的相对距离
公式是:d=sqrt(x1-x2)^+(y1-y2)^+(z1-z2)^)

如果两点之间距离某个标准颜色小于一个值,比如 #FF0000 与 #FE0000 距离为 1 ,那么就可以认为是红色



如果颜色表过大,可以剪枝一下,三维中的任意维度差大于某个值就直接 skip
2022 年 2 月 8 日
回复了 leemubai1995 创建的主题 问与答 求推荐适合熬夜女生的护肤品
雅顿橘灿精华,拯救熬夜皮肤暗黄

不过我觉得更好的是送褪黑素?可以强改作息
2022 年 1 月 24 日
回复了 Taurus12C 创建的主题 问与答 过年回家,想送点什么给爸妈,来给点意见
玉镯子、吊坠、金银首饰
XDR 或者 UltraFine 日常能开 HDR 么?

如果跟其他显示器一样,日常开 HDR 画面会严重过亮过艳,看 HDR 电影也很少的话,感觉价值也不大?
2022 年 1 月 21 日
回复了 moonchild 创建的主题 数据库 操作数据库 update 忘了加 where
记得是有个配置可以配的,限制 update 必须带 where ,可以避免这种疏忽大意的问题
2021 年 12 月 27 日
回复了 nbweb 创建的主题 问与答 我就想知道这个 4T 红盘啥时坏
我也有 2 块 4T 的红盘,在群晖里组了个 Raid0

上上个周突然提示了个坏扇区 300 个...

吓的赶紧把重要的数据拷出来,机器关机了
2021 年 12 月 23 日
回复了 iovekkk 创建的主题 剧集 《雪中悍刀行》小说党们,电视剧你们看了吗?
小说没看,只看了电视剧

龙傲天的味道太浓了,动作特效、武术指导 0 分... 其他的还成,正常剧的水平吧
定一个 12:29 的闹钟呗? 公司内部的,应该不至于真的秒没吧...
2021 年 11 月 17 日
回复了 feeeei 创建的主题 酷工作 [北京] 奇安信 - 服务端中台部门 - Golang 后端开发工程师
@lance116 #11 哈哈哈,文案而已,不要太计较啦


@lanhao344 #12 校招前段时间已经完事儿了... sorry
2021 年 11 月 15 日
回复了 feeeei 创建的主题 酷工作 [北京] 奇安信 - 服务端中台部门 - Golang 后端开发工程师
@SlipStupig #9 啊哈哈哈哈,可是我现在感觉干的还挺好的...
2021 年 11 月 15 日
回复了 feeeei 创建的主题 酷工作 [北京] 奇安信 - 服务端中台部门 - Golang 后端开发工程师
@qq976739120 #6 哈哈哈,描述我是复制粘贴的,不过确实是这样的


@Kilerd #7 没有啊,浴室洗衣房跟健身房不是很搭嘛,跟福报没关系[doge]
1  2  3  4  5  6  7  8  9  10 ... 13  
关于   ·   帮助文档   ·   自助推广系统   ·   博客   ·   API   ·   FAQ   ·   Solana   ·   969 人在线   最高记录 6679   ·     Select Language
创意工作者们的社区
World is powered by solitude
VERSION: 3.9.8.5 · 32ms · UTC 20:32 · PVG 04:32 · LAX 12:32 · JFK 15:32
♥ Do have faith in what you're doing.