Tag: 代码审查
评论被删除的代码
评论被删除的代码是否是一个好习惯？ 例如： // Code to do {task} was removed by Ajahn on 10/10/08 because {reason}. 我的开发人员在同行评审中发现有人注意到我们应该对要删除的代码行进行评论。 我认为这是一个可怕的build议，因为它用无用的评论混淆了代码。 我们哪一个是对的？
用于Eclipse / Java / Flex开发的最佳免费代码审查工具
我正在使用Eclipse，我想知道什么是最好的Eclipse / Java / Flexbuilder代码审查工具。 如果有关系，我们使用SVN作为SCM。 以下是我遇到的以下内容： 木星 – 好像已经有一段时间了，看起来只有一个开发人员在项目中，文档中提到了Eclipse的老版本 IBM的eclipse协作代码审查插件 – 看起来像是一个专有的代码审查工具，所以不能确定它将来会得到支持（或者有社区协助开发的能力）。 Eclipse COLA实时共享编辑 – 不是真正的代码审查工具，而是更多的查看/编辑文件的方式点对点样式允许代码审查，如协作。 看看这个video演示的COLA 。
Git合并提交
我是新来的git（并享受很多！）。 在一个新的分支发展，我不断承诺我的应用程序的各种发展“国家”。 现在我必须检查它的审查，但不希望一切都在不同的提交（不同的评论和ID）。 我怎么能像所有的改变一样推动所有的变化呢？
你如何使错误的代码看起来不对？ 你使用什么模式来避免语义错误？
自从我第一次犯了错误， if我总是写这样的ifs： if (CONST == variable) { 避免常见（至less对我来说）这样做的错误： if (variable = CONST) { //WRONG, assigning 0 to variable 而且自从我读了乔尔·斯波尔斯基（Joel Spolsky）的文章“ 使错误的代码看起来错了”以来，我一直在试图将他的build议付诸实践。 那么，如果你犯了一个语义错误，你还会使用其他什么模式来使错误的代码看起来错误，或者强制语法错误？
如何让FxCop自定义词典工作？
如何才能使FxCop自定义字典正常工作？ 我已经尝试添加要识别的文字“CustomDictionary.xml”，该文件保存在与FxCop项目文件相同的文件夹中。 这似乎不起作用，因为即使在重新加载并重新运行FxCop之后，我仍然得到“标识符应该拼写正确”的FxCop消息。 使用版本1.36。
如何比较Bitbucket中的两个修订版本？
我的团队正在使用Bitbucket作为我们的git存储库，最近我们开始使用pull请求function进行代码评论。 它在第一次审查时工作正常，但是如果它经历了多次迭代（即，更改和更新请求），我希望看到自上一次代码审查后发生的新更改的链接。 我研究了“比较”function，但UI看起来只能在分支之间进行比较。 是否有一个简单的方法来获得两个提交之间的差异？
GIT的最佳代码审查工具
是否有一个开源或免费的代码审查工具，与GIT最好的工作？ 已经尝试过gerrit，但有没有更好的开源或免费的代码审查工具？ 在用户提交本地克隆之前，是否有能够在Web上显示更改的代码审查？ ReviewBoard或任何其他工具是否有这个function？
“黑客”是什么意思在黑客说话？
当有人在提交时写下“nit：removed whitespace”，“nit”是什么意思？ 我也看到它的大写，就像它是一个缩写（即NIT）。 有关示例用法，请参阅以下文章 ： 当然，有一个评论说：“Nit：尾随空白”和“根据编码手册第VII部分的第五部分，你不应该添加尾随空白，请看你不这样做”。 或者像这样的东西。 后者是一种被动攻击性的炮火，前者只是一个提醒。 “Chromium Code Reviews”的其他例子： 问题9662：修复小样式nit（已closures） 编辑：一个答案来自Bugzilla的审查页面 ： 有时评论者会在“Nit：”前添加他的评论。 这意味着他只是“挑剔” – 你不必去解决这些问题，但是我们希望你能做到。
“可以包装本地”是什么意思？ （IDEA检查）
我使用IntelliJ作为“检查代码”，其结果之一是： 问题简介可以在本地打包（在第18行(public class HeartBeat) ） 这是什么意思，我该如何解决？ 它全class是这样的： package com.xxxxxxxxxxx.app.xxxx; public class HeartBeat { private static final Logger LOG = LoggerFactory.getLogger( HeartBeat.class ); private final File heartBeatFile; public HeartBeat( File heartBeatFile ) { this.heartBeatFile = heartBeatFile; } public void beat() { try { FileUtils.writeStringToFile( heartBeatFile, String.valueOf( System.currentTimeMillis() ) ); } catch( IOException e ) { LOG.error( […]
基于GitHub的代码审查的工作stream程
我正在考虑使用GitHub作为我们的代码审查的主要工具。 像在线评论和比较视图这样的function，似乎有许多像Gerrit这样的工具提供的function。 有其他人使用过GitHub吗？ 如果是这样，你的工作stream程是什么？ 你有什么经验这样做，正面和负面？ 当我对此有所了解，并理清什么最适合我们时，我将编辑我的问题，分享我自己的build议工作stream程。 编辑build议的工作stream程 步骤0.使用awesome reviewth.is 设置一个post-receive hook 。 然后： 像平常一样commit -a -s ，但在提交消息中追加#reviewthis @username 。 如果构build失败，则审阅将被跳过，直到构build被恢复。 审阅者对逐行或在文件级别进行提交的评论。 GitHub会自动通知评论的评论者。 审阅者通过电子邮件通知审阅者，当评论与审阅摘要一起完成时。 Reviewee回复GitHub中的评论者评论，允许项目访问代码评论的历史logging。 我最大的问题是步骤2和步骤4/5。 Gerrit很好地工作，除非构build成功，否则不要求评论; 我想在GitHub中做到这一点。 步骤4/5也有可能让人讨厌（多封电子邮件），并减less审查过程的自动性（需要通过电子邮件发送摘要）。 如果有帮助，我们使用Hudson作为我们的构build服务器。 对这些问题的任何想法也会有所帮助。
