- 主题:各位是从哪里获取code review的动力的？
那这么说，全球被review最多的可是Windows 98。Linus和Raymond说众人之眼的时候是在夸Windows 98质量比Linux优秀
【 在 RuralHunter 的大作中提到: 】
: 你要这么想，漏洞被爆出来本身就是一种review
:
--
FROM 123.116.198.*
内核又没法正常使用，说的肯定是发行版，挑个质量最好的讨论就行
【 在 nikezhang 的大作中提到: 】
: heartbleed那是openssl的，又不是内核的
: 【 在 leadu (leadu) 的大作中提到: 】
: : 查出来的可以说是review有效的，也可以说还有海恩法则，后面还有很多未暴露的。
: ...................
--
FROM 123.116.198.*
不一样，开源的人家直接告诉开发者哪一行代码出问题了，所以很快就改掉。
【 在 leadu (leadu) 的大作中提到: 】
: 那这么说，全球被review最多的可是Windows 98。Linus和Raymond说众人之眼的时候是在夸Windows 98质量比Linux优秀
--
FROM 116.233.76.*
这个想法和众人之眼一样，只是糊弄天真的程序员的
之前有安全公司出过报告，Linux的漏洞修复速度在所有主流os中是最慢的
【 在 RuralHunter 的大作中提到: 】
: 不一样，开源的人家直接告诉开发者哪一行代码出问题了，所以很快就改掉。
:
--
FROM 123.116.198.*
很好奇是怎么做到的？
code review 是 law of triviality（自行车棚效应）最好的体现：reddit 上面也经常有关于这方面的 meme：一个 PR 只改了2行代码，各种改动意见；另外一个改了500行，于是所有人都 LGTM
【 在 chopin1019 (chopin) 的大作中提到: 】
: 有质量的code review是我司升职的重要考核指标之一
--
FROM 203.211.110.*
而且众人之眼的描述可没有漏洞修复速度，别跑的太远了
【 在 RuralHunter 的大作中提到: 】
: 不一样，开源的人家直接告诉开发者哪一行代码出问题了，所以很快就改掉。
:
--
FROM 123.116.198.*
能理解。改了500行那个太复杂，看不懂，没法review
我现在review那种大的PR，都要下载到本地，实际跑起来debug跟一下，才能稍微摸到里面的逻辑，很费时间
问题就在于，manager往往看不到你花的这些时间，他只会看到你做了几个item
【 在 eGust 的大作中提到: 】
: 很好奇是怎么做到的？
: code review 是 law of triviality（自行车棚效应）最好的体现：reddit 上面也经常有关于这方面的 meme：一个 PR 只改了2行代码，各种改动意见；另外一个改了500行，于是所有人都 LGTM
:
--
FROM 120.21.57.*
未经code review批准的代码无法合并到主分支，系统设定，人工合并可以，找其他部门的人有权限的人帮你操作。
然后很快就会出现，你不review别人的代码，等你要review的时候，你就找不到人。就算你是超级大牛，别人不diao你，你也没招。
【 在 dpblue 的大作中提到: 】
: 我们组里谁都可以review代码，头儿貌似也不太关心你是否review同事们的代码，所以我平时都只顾着自己手上的活，懒得去review。但感觉这样不太好，请问各位是从哪里获取code review的动力的？
--
FROM 219.142.54.*
C++不如C#效率高啊
【 在 here080 (hero080) 的大作中提到: 】
: 理论上大工程C++效率比C高，因为模板比void*快
--
FROM 27.91.71.*
虽然我不懂，但是感觉你这个法子不靠谱啊，万一这个id以后被人弄出来呢？
【 在 dpblue 的大作中提到: 】
: 关于你说的“code review是个挑错性质的东西，不怎么受人待见，而且容易起争执”，这个我也想知道大家是怎么解决的。如果代码真的错了还好说，那种改善型的意见，真不知道是该给还是不该给。
: 举一个例子，我同事写了一个client，需要实现isAvailable()，就是去连一下对端服务器，连上了就返回true。
: 他的方法是调用一个取groups的API，这个API会返回属于某个ID的一大堆groups，但问题是这堆groups根本用不上。
: ...................
--
FROM 219.142.54.*