Questions tagged «code-reviews»

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

4
如何在请求请求中处理TODO？
该问题是从软件质量保证和测试堆栈交换迁移而来的，因为可以在软件工程堆栈交换中回答。 去年迁移 。 当我查看请求请求中的更改时，有时会偶然发现带有“ TODO”注释的注释，该注释可能有不同的原因，在我们的情况下，主要是因为： 用于解决问题的解决方案可以改进，但需要大量的时间投入。作者选择了一种更快的解决方案，但评论说可能有更好的选择 有一个临时代码可以解决现有的错误，应尽快修复 知道TODOs通常会在代码库的整个生命周期内停留在代码库中，那么我应该如何在请求请求中对它们做出反应？我如何礼貌地避免使用它，或者如果它确实是合理的，我如何确保PR的作者在以后的晚些时候进行跟进？
10
如何证明应用程序建立在错误的代码库上？
我目前正在审查由一些以前在我的工作中工作过的开发人员构建的系统。从用户的角度来看，该系统运行良好，但是当深入研究代码审查时，这简直是一团糟。我非常确信该应用程序的构建方式将无法承受未来的更新，更不用说使用率的高增长了。 问题是我知道情况有多严重，但我的上级却没有。我该如何向经理证明这一点，以便他真正地看到问题并可以说服他在当前代码库上进行最少的分类，并在不久的将来为该应用程序的下一版本启动新的开发线？
4
面向单独程序员的应用程序/代码审查？
是否有任何“以合理的价格”提供的服务，并提供有关应用程序的良好技术建议。在很多项目中，我通常是唯一的开发人员，有时，我认为我的一些工作需要进行改进以提高效率，改善MVC交互等。如果有一个专业的服务可以并且确实可以会做这样的评论
6
如何记录我的代码以进行最短时间检查？[关闭]
已关闭。这个问题需要更加集中。它当前不接受答案。 想改善这个问题吗？更新问题，使其仅通过编辑此帖子来关注一个问题。 去年关闭。 我想记录我的代码，以使几个月后再次阅读和浏览代码的需求降至最低。 我知道有不同类型的文档（在源代码中和外部，序列图等中）。 我只想知道什么是记录我的代码的有效方法，所以几个月后我想看我的代码时，我花更少的时间在阅读代码和理解代码流上。
3
从功能分支合并到母版之前进行代码审查的策略
我和我的团队使用功能分支（带有git）。我想知道哪种是合并到母版之前进行代码审查的最佳策略。 我从master结帐了一个新分支，我们称它为fb_＃1 我提交了几次，而不是想将其合并回母版 在合并之前，应该有人进行代码审查 现在有两种可能性： 第一 我将master合并到fb_＃1（不是将fb_＃1合并到master）以使其尽可能最新 队友查看了主管和fb_＃1主管之间的变化 如果fb_＃1可以，我们将fb_＃1合并到主服务器 优点：审查中没有过时的代码 缺点：如果其他人将“ 1”之间的内容合并。和“ 2”。他的更改将出现在评论中，尽管它们属于另一个评论。 第二名 队友查看结帐点（git merge-base master fb_＃1）和fb_＃1 head之间的更改 优点：我们可以清楚地看到在功能分支上所做的更改 缺点：评论中可能会出现一些过时的代码。 您认为哪种方法更好？为什么？也许还有另一种更合适的方法？
3
没有Github的类似Github的“拉取请求”
我是一家金融机构的分析师，由于数据敏感性，该机构不会在云中存储任何数据。但是，我取得了一些成功，使我的团队可以使用Git进行代码管理。我想知道是否有任何方法可以在我们自己的服务器中实现类似Github的请求。我感兴趣的特定功能是能够提交变更集以进行注释，而无需将其实际合并到给定的分支中。我喜欢（1）提交更改，（2）已对更改进行审查和评论，以及（3）接受提交或拒绝它的工作流程。是否可以在我们自己的服务器上实现（甚至更好，可以轻松实现）？
20 git  code-reviews 
17
如何说服程序员遵守基本规则
此问题是从Stack Overflow 迁移而来的，因为可以在Software Engineering Stack Exchange上回答。 迁移 8年前。 我必须一直要求程序员经常遵循几个规则。他们为他们编写代码，如果可行，那么工作就完成了。最基本的规则可能是： 进行变更 不在View或Controller中编写模型问题 避免硬编码 能告诉我您的经历吗？您如何处理？
9
关于说服老板代码审查是一件好事的提示[关闭]
已关闭。这个问题需要更加集中。它当前不接受答案。 想改善这个问题吗？更新问题，使其仅通过编辑此帖子来关注一个问题。 4年前关闭。 假设一个假设的公司中有一个工作，该公司中有几个开发人员很少在项目上一起工作，而Boss认为代码审查不值得花费时间和成本。 在此方案中可以提出哪些各种参数，这些参数将说明代码审查的好处？此外，这里反对代码审查的潜在论点是什么？如何应对？
4
我应该如何从经验不足的程序员那里修复代码？
一点背景：我是我们10人部门的两名程序员之一（其余是艺术家和管理人员）。我们两个人完成了使事情顺利进行所需的所有编码，并开发了所有即将出现的项目。我从事编程已有4年了，这是他的第一份“实际”工作（如他所说）。我们通常在任何时间点都在从事不同的项目。 几个月前，我开发了一组（绝不是完美的）类，这些类将用于以后的项目。该项目的很大一部分（出于计费原因）委托给他来设计和编程GUI界面。由于他是新手，所以我在设计上提供了一些帮助，并说如果他需要其他的帮助，请寻求帮助。几周前，他完成了该界面的演示，以演示该界面是否有效，尽管有点慢。 该项目的下一部分已经开始，我正在努力。我打开了界面，从下一步开始，然后立即遇到了问题（有点慢是有点轻描淡写，常见操作有错误等）。我查看了代码中的一些问题，并发现O(n^n)应该是的调用，O(n)没有错误检查的类型假设（在Python中），对原始代码添加的对GUI的引用等等。 现在，我绝对想教他什么地方出了问题以及如何解决它，但是他已经开始进行下一个项目，而这是在几周前。恐怕我说：“回去做对吧！” （当然有帮助）太苛刻了，与此同时，我们还有其他项目需要完成。我是否应该现在自己修复代码并在将来尝试捕获问题？
19 code-reviews  bug 
5
标准代码审查包含什么？
在我的公司中，这是一封电子邮件，讨论编写代码的人所实现的功能和修复的错误。收到邮件的审阅者将审阅代码并讨论其质量以及如何根据他的意见编辑代码。标准代码审查包含什么？
5
当代码“必须”不好时，我应该作为审核者做什么？
我在一个设计不良，设计过度的项目中工作，该项目几个月前已引入强制性代码审查。 我被要求检查大量实现新功能的代码。它具有与我们其他代码库相同的缺陷。我知道在很大程度上，这些缺陷会渗入新代码中，例如。因为必须继承设计不良的类或实现设计不良的接口，同时又要保持兼容性，因此，我无法提供不涉及重写一半代码库的更好的解决方案。但是我觉得从工程的角度来看，这对于已经损坏的代码库不利，因为我们还要对其进行更多破坏。我正在检查的代码肯定是不好的，但是必须要实现该功能。 对于此特定评论，我应该如何表现？我有办法保持诚信并保持建设性吗？ 请注意，我在此询问的是代码审查的范围。我确实意识到问题是由其他因素引起的，包括组织和工作文化，但是我想知道的是如何处理审查本身。
3
即使工作顺利，与上级和老板一起审查课程是否很好？
在我公司中，在交付任何项目之前，老板要求我的前辈审查由我或其他团队成员编写的程序，有时老板也会坐在一起与我们一起审查。 我认为这是获取知识的一种好方法，但是有时候程序运行良好时，经过审查它们就无法正常工作，因此我需要再次查看我的程序。 他们说，评审有助于优化程序和查询的执行，但是相对于程序的实际功能，我们能否更喜欢优化？
12
我们应该尝试审查所有代码吗？
我们目前正在修改开发流程，我想知道我们是否应该尝试对100％的提交同行进行审查。 您在代码审查方面有什么经验？ 您倾向于在它们上花费“大量”时间（例如每天1/2小时），还是只浏览最大5/10分钟？ 您每天/每周/每个冲刺/项目是否有固定的时间花费？ 最重要的是，您认为目标应该是对100％的代码进行同行评审还是不需要100％？
2
推荐的代码审查流程
我们通常在那时使用Perforce和SmartBear的Code Collaborator，Big Corp现在我们还将在某些项目中使用Mercurial。 Code Collaborator支持Mercurial（我们使用的是版本5），我正在尝试确定最佳时间（在提交/推送到服务器期间）是最佳/有效的代码审查时间。 谢谢
5
发烧期间如何进行有效的代码审查？
发布的最后期限是明天，您的同事最终完成了对于此发布至关重要的任务，项目经理站在您的肩膀上，并敦促您最终进行构建，并且您在审核过程中注意到同事的代码中存在缺陷。不是关键的东西，但是如果不是明天发布的东西，您将不会放手。更糟的是，您需要自己完成工作，并需要尽快完成。所以你会怎么做？尽管面临压力，您还是提出反对意见？ 我发现的一种方法是将该提交临时合并到另一个分支上，并留待以后检查。如果问题只是表面上的问题，并且是唯一仍在等待代码审查的问题，则它可以工作。但是，有没有更有效的方法来处理此问题？例如，您是否建议让一个人只进行代码审查和测试？
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.