Questions tagged «code-reviews»

该标签用于有关代码审查和代码演练的问题。有关现有有效代码的评论,请访问http://codereview.stackexchange.com

4
我应该坚持要求我们在合并回主干之前执行代码审查吗?
从StackOverflow请求重新发布: 我的工作时间很短,开发时间非常有限。我们开发了一种对我们的工作成果很重要的工具,但每天都不使用。我是团队中唯一拥有程序员背景的人。 我的问题是,在合并回主干一年多之前,我一直在推动代码审查。每个人都对此表示同意,但是仍然只有我的代码被审查过。从长假回来后,我回到了代码注释的主干,因为“这是一个丑陋的解决方案-尽快删除”和“快速修复”。另一个新功能是任命了一个人负责该工具。(最初提供给我一个职位,但由于与工作无关的原因而拒绝了我。)他认为这是一种可行的工作方式:由于我们的开发时间有限,因此我们应该走捷径。 我担心的是其他开发人员编写的代码很丑陋:经常破坏封装,编写巨大的类,在陌生的地方添加内部类,几乎没有单元测试等等。最终将不可能进一步开发该工具。 我应该坚持要求我们在合并到主干之前执行代码审查,还是我只是一个代码质量卑鄙的人?

2
哪些工具或技术可用于描述不熟悉的代码库?[关闭]
关闭。这个问题是题外话。它当前不接受答案。 想改善这个问题吗? 更新问题,使它成为软件工程堆栈交换的主题。 5年前关闭。 手动检查不熟悉的代码(以进行检查或修改)时,我似乎有三个选择。 一个自上而下的阅读的代码,通过选择文件名如何根本都好像下一个源文件。 我通常最终会阅读几乎所有内容。一些文件两次。 一个广度优先读取,在那里我找到并用最少的理解阅读所有的调用方法。然后阅读该函数调用的所有函数,依此类推。 如果我深入几次,我的思想栈就会溢出。 甲深度优先读取,其中I通过所有的代码在调试器步骤,不确定是否这将需要8分钟8小时。 一旦我阅读了足够多的代码以对它的工作有一个相当扎实的理解,我通常会反映出我已经阅读了80%或更多的代码库,而基本代码则是20%或更少。我浪费了很多时间。 哪些工具可用于快速掌握陌生的代码?是否有任何工具可以对关键代码路径有一个“大印象”,并允许我深入了解任何一部分的细节?


4
从公共方法返回哈希表有什么问题,什么时候才有意义?
当您要返回多个项目而不是创建一个类并返回该对象时,从公共方法返回哈希表有哪些设计问题? 如果确实有问题,那么在什么情况下才有意义? 这个问题的答案如何根据语言是否动态而变化? 编辑:这是为了澄清键将是常量,并且是代码而不是数据的一部分。我们通常为之创建类的东西。问题是,如果创建类确实确实是正确的选择,那么为什么使用哈希表却会出错呢?

6
CSS,JS和HTML的代码审查指南
我被要求为审查CSS,JS和HTML创建准则。我知道有针对JS的编码指南,但对HTML和CSS一无所知。要复习JS,我一定会遵循这些准则并提及它们。但是CSS和HTML呢?除了逻辑错误和缩进问题之外,在查看标记和CSS时是否还需要检查某些特定事项?

4
为什么不建议使用仅设置属性?
今天,在工作中,我的一位同事审查了我的代码,并建议我删除仅设置属性,而改用一种方法。 当我们俩都忙于其他事情时,他告诉我看Property Design《框架设计指南》一书中的部分。在书中,作者只是说要避免: setter的属性比getter具有更广泛的可访问性 现在,我想知道为什么不建议使用仅设置属性?有人可以帮我澄清一下吗?
By using our site, you acknowledge that you have read and understand our Cookie Policy and Privacy Policy.
Licensed under cc by-sa 3.0 with attribution required.