代码审查(Code Review)
代码审查(Code Review)清单
英文原文：oschina代码审查可以帮助提高代码质量，避免由于代码习惯而造成的 bug。
下面列出的这些要点因该可以作为大部分代码审查的指导，如果是 Java 应用的话，这些建议应该被视作最佳实践。
文档1. Javadoc 应该在每一个类和方法中添加。
2. 如果是修复某个 bug，应该添加 bug ID。
3. 走捷径的方法或者复杂的逻辑要有解释。
4. 如果代码会被公开，每个文件头都要标注版权信息。
5. 复杂的 HTML，JavaScript，CSS 应该包含文档。
功能1. 如果类似的逻辑被使用了多次，应该把它写成一个帮助类，然后在多出调用。
2. 鼓励使用 API 而不是重复编写代码解决相同的问题。
3. 要强调代码的单元测试。
4. 任何新加的代码不应该破坏已有的代码。
5. 假如是 Web 应用，JSP 不应该包含 Java 代码。
安全1. 任何代码都不能执行用户的输入，除非转义过了。
这个常常包含 JavaScript 的 eval 函数和 SQL 语句。
2. 禁止那些在短时间内提交非常多请求的 IP。
3. 任何类，变量，还有方法都应该有正确的访问域。
4. 尽量避免使用 iframe。
性能1. 所有数据库和文件操句柄在不需要的时候都应该被关闭。
2. SQL 语句的写法会导致性能千差万别。
3. 鼓励创建不可变（immutable）的类。
4. 类似的逻辑代码，尽量通过 if else 语句来实现更多的重用。
5. 尽量避免使用重对象（heavy objects）。
6. 如果是 Web 项目，请检查是否使用了合适的图片尺寸，CSS sprites 和浏览器缓存等技术。
7. 全局都需要的信息保存在 application context 中。
编码习惯1. 没有被使用的变量要删除。
2. 针对不同的 Exception 要用不同的 catch 语句，而不是一个 Exception 解决所有问题。
3. 针对变量，方法和类要用相同的命名方法。
什么是代码审查
什么是代码审查？代码审查（Code Review）是一种软件开发过程中的质量保证活动，通过审查程序代码的质量和一致性，以发现潜在的错误、改进代码结构、提高可维护性和可读性。
代码审查通常由开发团队中的其他成员或专门的代码审查人员完成，以确保代码的质量和符合团队的编码标准。
代码审查的目的是多方面的：1. 发现错误和问题：代码审查通过仔细检查代码中的逻辑错误、潜在的安全问题、性能问题和其他潜在的缺陷，以及与规范和最佳实践不一致的部分。
通过早期的发现和修复，可以减少后期的修复成本和风险。
2. 提高代码质量：代码审查有助于提高代码的质量和可靠性。
通过审查代码，可以发现并纠正低质量的代码，改进代码的结构和设计，以及提高代码的可读性和可维护性。
3. 促进知识共享和团队合作：代码审查是团队合作的重要环节。
通过审查代码，团队成员可以相互学习和分享知识，了解项目中其他人的工作和设计思路，促进团队合作和沟通。
4. 遵循编码标准和最佳实践：代码审查有助于确保代码符合团队的编码标准和最佳实践。
这包括代码的命名规范、注释规范、代码风格、错误处理和异常处理等方面。
通过审查代码，可以确保整个项目的代码风格和质量保持一致。
代码审查的过程通常包括以下几个步骤：1. 提交代码：开发人员将自己编写的代码提交到代码审查工具或版本控制系统中。
代码审查工具可以帮助记录审查过程和审查结果。
2. 分配审查人员：代码审查由其他团队成员或专门的代码审查人员完成。
审查人员可以根据项目和团队的需要进行分配。
3. 审查代码：审查人员对提交的代码进行审查。
他们仔细检查代码的质量、结构、逻辑、性能等方面，并与编码标准和最佳实践进行比较。
4. 提出问题和建议：审查人员对代码中发现的问题和改进提出评论、建议和问题。
这可以是关于代码逻辑的问题、潜在的错误、可读性和可维护性的改进等。
5. 讨论和解决问题：开发人员和审查人员之间进行讨论，解决代码审查中发现的问题和疑问。
idea中code review使用
在IntelliJ IDEA中，可以使用以下步骤进行代码审查（Code Review）：1. **打开代码审查工具**：在IntelliJ IDEA中，你可以通过以下步骤打开代码审查工具：* 在菜单栏中选择`Analyze -> Inspect Code...`* 或者使用快捷键`Ctrl+Alt+Shift+I`（Windows/Linux）或`Cmd+Option+Shift+I`（Mac）。
2. **选择要审查的代码**：在弹出的对话框中，你可以选择要审查的代码范围。
如果你想审查整个项目，可以选择`Whole project`。
如果你只想审查某个文件或某个特定的代码段，可以选择相应的范围。
3. **设置审查选项**：在弹出的`Inspection Profile` 对话框中，你可以设置代码审查的选项。
例如，你可以选择检查未使用的导入、未使用的变量、无效的捕获块等。
这些选项可以根据你的需要进行调整。
4. **开始审查**：点击`OK` 按钮后，IntelliJ IDEA将开始对选定的代码进行审查。
审查完成后，它将显示审查结果。
5. **查看和解决审查问题**：在审查结果中，你可以看到所有发现的问题，包括问题的类型、发生的位置以及问题的描述。
你可以逐个查看这些问题，并根据需要解决它们。
在解决问题时，IntelliJ IDEA将自动进行代码重构，以帮助你更轻松地解决问题。
6. **提交审查结果**：如果你解决了所有问题并满意于代码审查的结果，你可以提交审查结果。
在IntelliJ IDEA中，你可以通过以下步骤提交审查结果：* 在菜单栏中选择`VCS -> Git -> Commit...`* 在弹出的对话框中，选择你想要提交的文件或代码段。
* 输入提交信息，描述你提交的内容。
* 点击`Commit` 按钮提交你的更改。
如果你想将更改推送到远程仓库，你可以选择`Push` 按钮并输入相应的信息。
cr问题回答
CR（代码审查）什么是代码审查代码审查（Code Review）是指通过检查、分析和评估代码来找出其中的错误、缺陷和改进点的过程。
它是软件开发过程中非常重要的一环，可以帮助开发团队提高代码质量、降低维护成本，并加强团队合作和知识分享。
为什么进行代码审查进行代码审查有以下几个主要原因：1. 提高代码质量通过代码审查，可以找出潜在的错误、缺陷和不规范的编码风格。
及早发现和修复这些问题，可以避免它们在后续的开发和测试阶段引入更大的问题。
同时，通过与其他团队成员共同讨论和评估，可以提供更好的解决方案，并促使开发人员思考更合理、更高效的实现方式。
2. 知识分享与技能提升在代码审查过程中，不仅是被审查者能够从其他团队成员中学习到新知识和技巧，而且对于参与审查的人来说也是如此。
通过参与他人的代码审查，可以学习到其他人在设计、实现和优化方面的经验，并将其应用到自己的开发工作中，从而提升自己的技能水平。
3. 加强团队合作与沟通代码审查是一个团队协作的过程，通过共同讨论和评估代码，可以促进团队成员之间的交流和合作。
通过互相学习、指导和支持，团队成员能够更好地理解彼此的工作，并共同努力提高整体的代码质量。
此外，代码审查还可以帮助团队建立一致的编码规范和最佳实践，提高开发效率和协同工作能力。
如何进行代码审查要进行有效的代码审查，需要遵循以下步骤：1. 确定审查目标在开始代码审查之前，需要明确审查的目标和范围。
这可以包括特定功能模块、新添加或修改的代码段等。
明确审查目标有助于提高审查效率，并确保关注重点。
2. 审查准备在开始实际的代码审查之前，需要对待审查的代码进行准备。
这包括了解相关需求、设计文档和测试用例等。
通过对上下文信息的了解，可以更好地理解待审查代码，并找出其中可能存在问题或改进点。
3. 代码审查在进行代码审查时，可以考虑以下几个方面：•代码结构和组织：检查代码的结构是否清晰、模块划分是否合理，并确保代码符合团队的编码规范和最佳实践。
软件评审的方法
软件评审的方法
软件评审是一种对软件进行全面评价和审查的过程，旨在确保软件的质量和可靠性。
以下是一些常用的软件评审方法：
1. 代码审查（Code Review）：通过仔细检查源代码，评估其
质量、可读性、一致性和安全性等方面。
可以使用静态代码分析工具来辅助代码审查过程。
2. 设计评审（Design Review）：评估软件设计的合理性、可
扩展性、模块化和结构化程度等方面。
主要关注软件架构、模式和接口设计等。
3. 功能评审（Functional Review）：评估软件的功能是否满足
用户需求、是否符合规范和设计规范等。
可以通过测试用例和场景来验证软件的功能。
4. 性能评审（Performance Review）：评估软件在各种负载和
压力下的性能表现，包括响应时间、并发处理能力、资源利用率等。
5. 安全评审（Security Review）：评估软件的安全性，包括对
潜在漏洞和安全风险的识别和评估。
可以使用安全测试工具和技术来帮助评审。
6. 用户界面评审（User Interface Review）：评估软件的用户界面设计，包括用户友好性、可用性、一致性和可访问性等方面。
7. 文档评审（Documentation Review）：评估软件的相关文档，包括需求文档、设计文档、用户手册和帮助文档等，确保其准确、完整和易于理解。
8. 测试评审（Test Review）：评估软件的测试策略、测试计划、测试用例和测试结果等，确保软件的测试覆盖率和质量。
以上评审方法可以根据具体情况和需求进行组合和定制，以确保对软件进行全面的评价和审查。
软件工程中的代码审查与测试
软件工程中的代码审查与测试在软件开发过程中，代码审查（code review）和代码测试（code testing）是两个非常重要的环节。
代码审查是通过对代码的检查和审查来发现潜在的问题和错误，以确保代码的质量和可维护性。
代码测试则是通过运行测试用例来验证代码在各种情况下的正确性和稳定性。
本文将对软件工程中的代码审查与测试进行探讨和论述。
一、代码审查代码审查是在软件开发过程中对代码进行检查和评审的一项活动。
其目的是通过仔细检查代码来发现潜在的问题和错误，以提高代码的质量和可读性。
代码审查可以分为静态代码审查和动态代码审查两种方式。
静态代码审查是通过直接检查代码的语法、结构和规范性来进行评审。
开发人员可以通过手动检查代码来发现潜在的问题，也可以使用一些静态代码分析工具来辅助进行代码审查。
这些工具可以帮助检测代码中的逻辑错误、潜在的安全漏洞以及不符合规范的编码风格等问题。
动态代码审查则是通过运行代码并观察其执行情况来进行评审。
开发人员可以使用调试工具来逐行跟踪代码的执行流程和变量的取值情况，以发现代码中的问题和错误。
动态代码审查更加注重代码的正确性和可靠性，能够发现一些静态代码审查无法发现的问题。
二、代码测试代码测试是验证代码在各种情况下的正确性和稳定性的一项活动。
通过编写测试用例并运行代码，开发人员可以检测代码在各种输入情况下的输出是否符合预期，以及代码在各种异常情况下的鲁棒性。
代码测试可以分为单元测试、集成测试和系统测试等多个层次。
单元测试是对代码中最小的可测试单元进行测试，通常是对函数或方法进行测试。
集成测试则是测试多个模块或组件之间的交互和整合情况。
系统测试则是对整个系统进行测试，验证其是否满足用户需求和设计规格。
在进行代码测试时，可以使用各种测试技术和工具来辅助进行测试。
例如，常用的测试技术有黑盒测试和白盒测试。
黑盒测试是根据需求规格和功能规范来设计测试用例并执行测试，不关注代码的内部实现细节。
code review方案
代码审查（Code Review）是一种软件开发过程中的质量控制环节，旨在通过检查代码以确保其质量、可维护性和符合团队规范。
以下是一个简单的代码审查方案：1. 明确代码审查的目的和目标：- 提高代码质量- 提高团队协作和沟通- 统一编码规范- 预防潜在的代码问题和安全风险2. 制定代码审查流程：- 提交代码：开发人员在完成代码后，将其提交至代码库或版本控制系统。
- 发起审查：提交人可以选择一个或多个审查者，邀请他们进行代码审查。
- 审查代码：审查者按照预定的编码规范和团队标准，对代码进行逐行、逐函数的审查。
- 提交反馈：审查者在发现问题时，向提交人提供详细的问题说明和修改建议。
- 修改代码：提交人根据审查者的反馈，对代码进行修改。
- 重新审查：审查者对修改后的代码进行再次审查，确保问题已得到解决。
- 关闭审查：当审查者认为代码质量达到要求时，关闭审查。
3. 确立代码审查规则和标准：- 统一编码规范：制定清晰的编码规范，确保团队成员遵循相同的规范。
- 命名规范：统一命名规则，使代码易于阅读和理解。
- 代码风格：规范代码排版、缩进、空格等，提高代码可读性。
- 代码质量：关注代码的可维护性、可扩展性、性能优化等方面。
- 安全性：检查代码中是否存在潜在的安全风险和漏洞。
4. 培训和推广：- 对团队成员进行代码审查培训，确保他们熟悉代码审查流程、规则和标准。
- 鼓励团队成员积极参与代码审查，提高自己的编码能力和团队协作水平。
5. 持续改进：- 定期收集代码审查过程中的问题和建议，对审查流程和规则进行优化。
- 分析代码审查数据，了解团队成员的编码状况，为提高代码质量提供支持。
软件开发中的代码审查与审核
软件开发中的代码审查与审核在软件开发过程中，代码审查和审核是必不可少的环节。
通过对代码进行审查和审核，团队成员可以发现并纠正潜在的代码缺陷，提高代码质量，减少错误。
本文将介绍代码审查和审核的基本概念、作用以及最佳实践。
一、代码审查和审核的定义代码审查（code review）是一种通过检查获取代码来识别错误、优化代码和提高代码质量的过程。
它通常由团队成员或专家开发者进行，他们会定期地检查代码并提出建议改进。
代码审查可以在不同的阶段进行，例如在开发中、测试前或发布前。
代码审核（code inspection）通常是在开发过程的早期阶段进行的一种活动。
代码审核是一项严格的、程序化的过程，它旨在为软件产品的开发提供质量保证，并确保代码符合规范、标准和设计要求。
二、代码审查和审核的作用1. 提高代码的质量代码审查和审核可以帮助团队及时发现和纠正代码中的错误。
它可以帮助团队避免在开发过程中增加bug，降低代码质量。
2. 减少错误和延误通过代码审查和审核，团队可以发现潜在的缺陷并及时修复，这样可以减少错误和延误。
3. 提高团队的技能通过观察其他人的代码、看到其他开发者的解决问题的方法，团队成员可以对自己口袋中的技能进行提高。
三、代码审查和审核的最佳实践1. 审核和审查应该是有计划的团队应该有一个明确的计划，规定何时进行审查和审核。
例如，在完成某个功能时，应该对代码进行一次审查；在保持代码更新时，应该进行定期的审核。
2. 团队成员应该具备针对性的技能团队需要一个具有针对性的技能集来执行审查和审核。
过程中，正在被审查或审核的代码需要专业人员的对其进行评估。
3. 审核和审查的量要适度团队应该在一定的时间内处理适量的代码，以便在短时间内实现审查和审核的目标。
如果关注的代码量过大，代码审核和审查可能会超时导致随意的结果丢失，更损失人员的已有知识和技能。
4. 使用代码审查和审核工具使用代码审查和审核工具可以大大简化过程，并提高效率。
- 1、下载文档前请自行甄别文档内容的完整性，平台不提供额外的编辑、内容补充、找答案等附加服务。
- 2、"仅部分预览"的文档,不可在线预览部分如存在完整性等问题,可反馈申请退款(可完整预览的文档不适用该条件!)。
- 3、如文档侵犯您的权益，请联系客服反馈,我们会尽快为您处理(人工客服工作时间：9:00-18:30)。
代码审查(Code Review)一、概述代码审查（Code Review)是软件开发中常用的手段，和QA测试相比，它更容易发现和架构以及时序相关等较难发现的问题，还可以帮助团队成员提高编程技能，统一编程风格等，目前监控团队虽然提倡代码审查，也有相关的辅助工具，但是一直没有真正的推行起来，这半年的时间里，一些线上的bug如果经过代码审查，基本上可以避免的，大家也逐渐认识到代码审查可以有效地提高代码质量。
二、代码审查的作用1、提高代码质量。
通过代码审查来发现bug及代码中的不规范，这是不容置疑的，通过代码审查，代码将更加整洁，有更好的注释，更好的程序结构。
2、提高开发者开发水平。
开发者知道自己编写的代码会被同事审查，将会更加认真的编写代码，也将会督促开者不断地学习、向有经验的同事请教。
3、提高程序的可维护性。
一份程序代码将会有更多的同事熟悉，更好的代码质量，自然地也增加程序的可维护性。
4、提高开发者的对编码的责任感。
如果你在编程，而且知道将会有同事检查你的代码，你编程态度就完全不一样了。
你写出的代码将更加整洁，有更好的注释，更好的程序结构——因为你知道，那个你很在意的人将会查看你的程序。
没有代码审查，你知道人们最终还是会看你的程序。
但这种事情不是立即发生的事，它不会给你带来同等的紧迫感，它不会给你相同的个人评判的那种感受。
5、传播知识在很多的开发团队里，经常每一个人负责一个核心模块，每个人都只关注他自己的那个模块。
除非是同事的模块影响了自己的程序，他们从不相互交流。
这种情况的后果是，每个模块只有一个人熟悉里面的代码。
如果这个人休假或——但愿不是——辞职了，其他人则束手无策。
通过代码审查，至少会有两个人熟悉这些程序——作者，以及审查者。
审查者并不能像程序的作者一样对程序十分了解——但他会熟悉程序的设计和架构，这是极其重要的。
三、代码审查的执行障碍1、缺少代码审查的标准缺少代码审查的标准，往往审查人员习惯性地根据自身开发经验去进行代码审查，容易变成去挑毛病，找bug，容易产生不良地影响。
2、资深开发人员的较少，工作过多，缺少代码审查的时间安排在一个小规模团队里，资深的同事总是忙不过来，因为核心的工作都在等着他做，资深的开发人员少，代码审查的质量自然上不去。
3、团队成员的技术差距过大一个团队中往往有不少经验较浅的开发人员，当然编码水平也就相对较差，如果缺少有效的激励手段，往往资深的开发人员是比较讨厌经常去审查经验很浅的开发人员的代码。
四、代码审查的实践要素。
虽然代码审查是一个被广泛采纳的实践，但是由于存在很多的实践方式，所以开发团队经常搞不清楚它的最佳实践是什么样的，正确的代码审查应该重点考虑的几个因素有以下几点。
1、人员结构在你的团队中有多少同事拥有足够的专业技能来担当合格的代码审查者？作者曾经和多个开发团队沟通过，这些团队都声称本身只有一个资深的开发人员，如果让他来负责所有的代码审查工作，那么团队的核心开发就无法开展了。
作者也遇到过类似的情况。
在一个小规模团队里，资深的同事总是忙不过来，因为核心的工作都在等着他做。
2、地理位置你的团队成员是如何分布的？他们是否是分散在世界各地还是坐在同一个敞亮的办公室里？代码审查需要精力的专注和反馈，如果开发人员能够面对面的沟通，那么审查工作会便于实施。
而物理隔离的远程团队很难达到代码审查的满意结果。
3、所在领域你所在的IT领域需要遵守怎样的规则和约束呢？如果你在一个受到严格监管的行业，那么你必须遵循有关审计和报告的规则，这意味着你必须想办法跟踪代码审查的频率和质量。
如果所在的领域把安全性作为重点，那么可能在代码审查过程中要引入安全审计。
4、复杂性你的代码复杂性如何？在代码审查时，需要多个不同专业技能的审查者吗？例如，游戏开发可能会引入非常复杂的业务逻辑来处理文字交互和场景跟踪，同时还需要特定的动画处理和内存管理技术。
在审查如此复杂的代码时，应该引入多个审查者从不同角度来检阅代码的质量。
而且在许多情况下，不同的项目有不同的需求——存在多个团队构建应用，需要不同的配置文件。
这里有一些技巧可以帮助你改进代码审查过程的工具并解决一些可能会面临的问题：A.促进沟通任何审查过程，不论是代码还是文档还是绩效，最好是面对面进行。
在你使用工具做完代码审查之后，与相应的开发人员约个时间做口头的反馈。
如果你们不在同一地区，可以利用视频或者电话来进行沟通，但是这种沟通是必要的，你可以以此作为协作和指导的一种手段。
B.利用专业技能如果被审查的代码与其他代码领域存在关联，或者对安全、性能、可扩展性等方面存在一定的影响，你可能无法只安排一个人来完成审查工作。
你需要多个人来查看代码以确保所有可能的后果都被考虑在内。
在这种情况下，最好找到一个工具来协助多个人完成审查工作。
该工具的作用包括指定哪个人审查哪个角度并把相应的代码展现出来，而且还可以保存审查人员的注解。
C. 公开审查意见审查工具的优点之一是多人可以集中地审查相同的代码。
把所有人叫到一个屋子里做团队审查非常不利于大家考虑各个不同的方面。
但是工具可以帮助你让大家分别查看代码并把所有的意见集中起来：•避免意见重复，如果别人已经发表了类似的看法，那么后面的人就不用记录了。
•团队学习，通过查看别人的意见能够学习其他人的思维角度和专业领域。
•监督审查过程。
五、代码审查的一些常用经验与注意事项。
1.代码审查要求团队有良好的文化团队需要认识到代码审查是为了提高整个团队的能力，而不是针对个体设置的检查“关卡”。
“A的代码有个bug被B发现，所以A能力不行，B能力更好”，这一类的陷阱很容易被扩散从而影响团队内部的协作，因此需要避免。
另外，代码审查本身可以提高开发者的能力，让其从自身犯过的错误中学习，从他人的思路中学习。
如果开发者对这个流程有抵触或者反感，这个目的就达不到。
2.谨慎的使用审查中问题的发现率作为考评标准在代码审查中如果发现问题，对于问题的发现者来说这是好事，应该予以鼓励。
但对于被发现者，我们不主张使用这个方式予以惩罚。
软件开发中bug在所难免，过度苛求本身有悖常理。
更糟的是，如果造成参与者怕承担责任，不愿意在审查中指出问题，代码审查就没有任何的价值和意义。
3.控制每次审查的代码数量根据smartbear在思科所作的调查，每次审查200行-400行的代码效果最好。
每次试图审查的代码过多，发现问题的能力就会下降，我们在实践中发现，随着开发平台和开发语言的不同，最优的代码审查量有所不同。
但是限制每次审查的数量确实非常必要，因为这个过程是高强度的脑力密集型活动。
时间一长，代码在审查者眼里只是字母，无任何逻辑联系，自然不会有太多的产出。
4.带着问题去进行审查我们在每次代码审查中，要求审查者利用自身的经验先思考可能会碰到的问题，然后通过审查工作验证这些问题是否已经解决。
一个窍门是，从用户可见的功能出发，假设一个比较复杂的使用场景，在代码阅读中验证这个使用场景是否能够正确工作。
使用这个技巧，可以让审查者有代入感，真正的沉浸入代码中，提高效率。
大家都知道看武侠小说不容易瞌睡，而看专业书容易瞌睡，原因就是武侠小说更容易产生代入感。
有的研究建议每次树立目标，控制单位时间内审核的代码数量。
这个方法在我们的实践中显得很机械和流程化，不如上面的方法效果好。
5.所有的问题和修改，必须由原作者进行确认如果在审查中发现问题，务必由原作者进行确认。
这样做有两个目的：(1)确认问题确实存在，保证问题被解决(2)让原作者了解问题和不足，帮助其成长有些时候为了追求效率，有经验的审查者更倾向于直接修改代码乃至重构所有代码，但这样不利于提高团队效率，并且会增加因为重构引入新bug的几率，通常情况下我们不予鼓励。
6.利用代码审查激活个体“能动性" 即使项目进度比较紧张，无法完全的进行代码审查，至少也要进行部分代码的审查，此时随即抽取一些关键部分是个不错的办法。
背后的逻辑是，软件开发是非常有创造性的工作，开发者都有强烈的自我驱动性和自我实现的要求。
让开发者知道他写的任何代码都可能被其他人阅读和审察，可以促使开发者集中注意力，尤其是避免将质量糟糕，乃至有低级错误的代码提交给同伴审查。
开源软件也很好的利用了这种心态来提高代码质量。
7.在非正式，轻松的环境下进行代码审查如前所述，代码审查是一个脑力密集型的工作。
参与者需要在比较轻松的环境下进行该工作。
因此，我们认为像某些实践中建议的那样，以会议的形式进行代码审查效果并不好，不仅因为长时间的会议容易让效率低下，更因为会议上可能出现的争议和思考不利于进行如此复杂的工作。
8.提交代码前自我审查，添加对代码的说明所有团队成员在提交代码给其他成员审查前，必须先进行一次审查。
这次自我修正形式的审查除了检查代码的正确性以外，还可以完成如下的工作：(1)对代码添加注释，说明本次修改背后的原因，方便其他人进行审查。
(2)修正编码风格，尤其是一些关键数据结构和方法的命名，提高代码的可读性。
(3)从全局审视设计，是否完整的考虑了所有情景。
在实现之前做的设计如果存在考虑不周的情况，这个阶段可以很好的进行补救。
我们在实践中发现，即使只有原作者进行代码审查，仍然可以很好的提高代码质量。
9.实现中记录笔记可以很好的提高问题发现率成员在编码的时候应做随手记录，包括在代码中用注释的方式表示，或者记录简单的个人文档，这样做有几个好处：(1)避免遗漏。
在编码时将考虑到的任何问题都记录下来，在审查阶段再次检查这些问题都确认解决。
(2)根据研究，每个人都习惯犯一些重复性的错误。
这类问题在编码是记录下来，可以在审查的时候用作检查的依据。
(3)在反复记录笔记并在审查中发现类似的问题后，该类问题出现率会显著下降10.使用好的工具进行轻量级的代码审查比如利用findbug 进行代码扫描。
六、实现代码审查的前提条件（一）、审查制度的执行前提1、代码审查必须是基本制度，是工作中一部分。
2、在团队中需要有一份大家接受的代码规范文档。
3、与业务需求相关的代码修改，需要审查代码的同事熟悉业务需求，参与设计评审，不熟悉需求是不能有效地审查代码。
4、代码审查是工作中的一部分，需要安排一定的时间来进行，这需要领导在工作安排中合理安排时间。
（二）、代码审查的执行前提1、Code Review人员是否理解了Code Review的概念和Code Review将做什么如果做Code Review的人员不能理解Code Review对项目成败和代码质量的重要程度，他们的做法可能就会是应付了事。
2、代码是否已经正确的build，build的目的使得代码已经不存在基本语法错误我们总不希望高级开发人员或是主管将时间浪费在检查连编译都通不过的代码上吧。
3、代码执行时功能是否正确Code Review人员也不负责检查代码的功能是否正确，也就是说，需要复查的代码必须由开发人员或质量人员负责该代码的功能的正确性。