腾讯云开发者社区-腾讯云

开发者社区

文档建议反馈控制台

文章/答案/技术大牛

发布

综合排序丨最热优先丨最新优先

代码审查那些事儿

代码审查那些事儿定义代码审查（英语：Code review）是指对计算机源代码系统化地审查，常用软件同行评审的方式进行，其目的是在找出及修正在软件开发初期未发现的错误，提升软件质量及开发者的技术。代码审查常以不同的形式进行，例如结对编程、非正式的看过整个代码，或是正式的软件检查 from wiki 大概流程时间: 最好每天或每周3 下班前1个小时主持: 会议室,由leader主持内容: 组内每人调400行代码的主逻辑出来查看每人简述完毕后,QA环节挑取有效的建议,标注到upsource的代码审查系统清单审查内容代码安全性代码性能代码注释单元测试代码优化格式化分类封装方法注意代码审查应该是日常工作审查内容不要过多,关注主逻辑,行数不要超过400行代码审查过程要保持心态平和,每个人都是值得尊重的补充 upsource 是jetbrain公司出的一个团队协作的代码审查系统,可以配合idea实时显示评论.安装时免费的,但是团队的成员账号不能超过5个.如果超过则需要给钱买授权;免费的适合小组内的代码审查.目前在用,感觉还行

2020-02-25

1.3K0

标签:

AI 代码审查（AICR）

虽然 openclaw/skills 仓库中可能没有名为 code_reviewer 的独立目录（截至查询时），但根据 OpenClaw 的设计范式和社区实践，一个典型的代码审查 Skill 通常具备以下能力维度一：注入测试上下文 —— 你的代码，测了吗？目标：让 AI 在审查时，能关联到相关的测试资产。 coverage.py：获取当前代码的测试覆盖率报告。工作流：当收到审查请求，Skill 首先定位被修改的函数/类。查询 coverage.xml，找出未被覆盖的行或分支。调用 LLM，结合代码变更与覆盖率缺口，生成审查意见：“新增的 validate_email_format() 函数未被任何测试覆盖。 diff "$CI_MERGE_REQUEST_DIFF" > review.json - python post_to_mr.py review.json # 将结果评论到 MR 结语：从代码审查到质量预言

2026-03-31

1.3K0

标签:

代码审查工具StyleCop

“代码审查”或是“代码评审”（Code Review），这是一个流程，当开发人员写好代码后，需要让别人来review一下他的代码，这是一种有效发现BUG的方法。由此，我们可以审查代码的风格、逻辑、思路……，找出问题，以及改进代码。因为这是代码刚刚出炉的时候，所以，这也是代码重构，代码调整，代码修改的最佳时候。长时间以来，Code Review需要有一些有效的工具来支持，这样我们就可以更容易，更有效率地来进行代码审查工作。 StyleCop是代码规范检查工具（Code Review 工具），它不仅仅检查代码格式，而是编码规范，包括命名和注释等。StyleCop可以帮助你更容易地进行这项活动。它会根据预定义的C#代码格式的最佳实践，对我们的源代码进行检查，并给出不符合编码风格的错误提示。

2018-01-19

2.7K0

标签:

提高代码质量之代码审查

所以代码审查(Code Review)就像是把玩鉴赏手工艺品一样，通过审查代码来体会编码者的思维逻辑，同时相互学习取长补短。代码审查是提高个人和团队的代码质量的一个很有用的方法。个人对自己的代码可以进行代码审查，因为今天的你已经不是昨天的你，你可以站在不同的角度和不同的层次来审查自己过去的代码。子曰：温故而知新。所以对于自己的代码应该定期做Code Review。相比之下团队的代码审查就不是那么容易实施了。但是代码审查在团队中能体现出更大的促进作用。如果是个人或者小团队，不想开源自己的代码的话，也可以利用Bitbucket来建立私有库托管代码和进行代码审查。总之 XXXX是一把双刃剑，代码审查也是如此。进行代码审查对于组织者的能力要求比较高，要多思考和调整，激发大家一起来做好代码审查，发挥其最有益的效用。

2019-08-19

1.7K0

标签:

云数据库 SQL Server

前端代码审查清单

前端代码审查清单是一个保证前端代码质量的审查清单。当我们在开发写代码的时候，总会各种各样的问题，自测的时候由于太熟悉自己的代码逻辑往往测试不够充分，无法发现问题。前端代码审查清单就是为了解决这个问题！清单存放了一些常见的问题，当我们开发完成之后，对照清单思考一下这些问题在代码中是否遇到或者妥善处理，从而提高代码质量。代码里获取 window.location 相关属性的地方，是否对里面的 XSS 字符做了过滤处理？前端性能 JS 代码是否尽量放在底部？CSS 代码是否尽量放在了顶部？代码质量你的代码是否遵循团队要求的代码规范？是否有冗余代码没有注释掉或者删掉？例如：删除或者注释 console.log 避免低端 IE 报错等。关键功能是否还有优化的空间？贡献由于本人才疏学浅，部分场景没有覆盖，欢迎大家补充更多审查点，提高前端代码质量！

2021-04-22

1.3K0

标签:

腾讯云测试服务

Jenkins+SonarQube代码审查

Jenkins+SonarQube代码审查引言安装PostgreSQL 安装SonarQube 实现代码审查概述 Jenkins安装SonarQube Scanner插件安装SonarQube Scanner Jenkins配置Sonar Server 非流水线项目添加SonarQube代码审查流水线项目添加SonarQube代码审查 ---- 引言 SonarQube是一个用于管理代码质量的开放平台 cd /usr/local/sonarqube-8.6.0/extensions/plugins ---- 生成令牌（需要将生成的令牌复制下来）： ---- 实现代码审查概述 Jenkins安装 - Jenkins配置Sonar Server Manage Jenkins->Conﬁgure System->SonarQube servers： ---- 非流水线项目添加SonarQube代码审查 system encoding sonar.sourceEncoding=UTF-8 sonar.java.binaries=**target/classes ---- 流水线项目添加SonarQube代码审查

大忽悠爱学习

2022-09-26

2.1K0

标签:

# 代码风格审查工具Cpplint

代码风格审查工具Cpplint 它山之石，可以攻玉。 --《诗经·小雅·鹤鸣》 1，简介 Cpplint是一个Python脚本，作为一款开源免费的代码静态检测工具，Google也使用它作为自己的C++代码检测工具，也就是说，只要你想代码遵从Google C++代码规范，那么Cpplint将会提供很好的代码静态检测支持。 Cpplint只是一个代码风格检测工具，其并不对代码逻辑、语法错误等进行检查。 2，安装 Cpplint在Window下可以如下几种方式使用 2.1运行命令行检测使用命令即可对代码文件进行检测。屏蔽代码不被检查假设代码中有些部分不希望被检查，或者觉得是cpplint产生的误报。仅仅须要在对应代码行尾加入凝视”// NOLINT”就可以，cpplint就会跳过这些行。

上善若水.夏

2019-08-27

6.2K0

标签:

代码审查之最佳实践

图片来自 https://xkcd.com/1513/ 本文谈论了以下话题：代码审查之为什么、查什么、何时查准备好被审查的代码代码审查的执行代码审查实例动机之所以要执行代码审查（code reviews 有研究显示，即便是简短、非正式的代码审查也能显著影响代码质量和 bug 的出现的频次合规合法的环境通常需要审查。在某些需要监管的环境中，即便是微小的调整也需要代码审查。代码审核不分尊卑长幼：作为团队中最资深的人也并不意味着其代码就不需要审查。何时审查代码审查应该晚于自动检查（自动测试、样式检查、持续集成）成功完成后，但要早于代码合并到仓库的主线分支之前。通常并不对最后一次发布前积累的总量改变执行正式的代码审查。假如代码作者已经遵循了上述原则（特别是自我审查和确保代码能运行了），这里还有一份审查者应该注意的清单：意图代码是否达成了作者的意图？

2020-06-15

2K0

标签:

腾讯云测试服务

代码审查拯救世界？

代码审查是指阅读代码来检查源代码与编码标准的符合性以及代码质量的活动。现在，越来越多的团队倡导要进行代码审查活动，而本文作者通过一幅漫画，来诠释其对代码审查的理解，观点可能不符合大家的常规心理。通过代码审查解决？我想问的主要问题：代码审查期间真的会发现问题或bug吗？ ➤静态代码分析这样的bug在我的代码中根本不会发生，因为我们在整个项目中都启用了静态代码分析。例如FindBugs这款软件，它会立即揪出这种bug。代码审查会发现这个bug吗？对还是不对呢？人类擅长推理、思考、富有创造性、并且发现错误，但语法编译对人类来说是一件可怕的事情。在代码审查期间，你可能仅仅发现一些拼写或基础错误（typo）但并未真正发现bug。难道第二双眼睛就可以发现了吗？

CSDN技术头条

2018-02-12

1.5K0

标签:

前端代码审查清单

前端代码审查清单是一个保证前端代码质量的审查清单。当我们在开发写代码的时候，总会各种各样的问题，自测的时候由于太熟悉自己的代码逻辑往往测试不够充分，无法发现问题。前端代码审查清单就是为了解决这个问题！清单存放了一些常见的问题，当我们开发完成之后，对照清单思考一下这些问题在代码中是否遇到或者妥善处理，从而提高代码质量。代码里获取 window.location 相关属性的地方，是否对里面的 XSS 字符做了过滤处理？前端性能 JS 代码是否尽量放在底部？CSS 代码是否尽量放在了顶部？代码质量你的代码是否遵循团队要求的代码规范？是否有冗余代码没有注释掉或者删掉？例如：删除或者注释 console.log 避免低端 IE 报错等。关键功能是否还有优化的空间？换位思考，你能根据你的注释推断出下面代码的功能吗？代码里是否还存在 TODO ？是否可以删掉或者完善功能？可能产生意外情况的地方是否留下说明？测试代码逻辑是否正确、可用、符合需求？

2020-07-13

1.9K0

标签: