百度360必应搜狗淘宝本站头条
docker systemdzabbix微信报警脚本loadjsqt5.5pagination.js
当前位置：网站首页 > 技术教程 > 正文

如何做好Code Review 如何做好一个管理者

suiw9 2024-12-19 16:25 16 浏览 0 评论

让Google程序员如此优秀的一件重要但非常简单的事情：Code Review

1、Code Review的好处

  • 对代码保持敬畏
  • 学习他人的优点
  • 团队间传播知识

对于Coder来说，当你知道有同事会检查你的代码时，你的编程态度就会认真起来，写出的代码会更加整洁、符合规范，程序的结构也会更加合理。如果没有严肃的代码审查，缺乏他人检查带给你的紧迫感，你的代码提交就会逐渐随意起来。

对于Reviewer来说，检查同事的代码是一个很好的学习机会，对于一个问题或功能，一定是有多种修复或解决方案的，理解其他人的实现方式，避免局限在自我的思维方式，是非常有益的。

另外，在很多开发团队中，经常是一个人负责一个模块，每个人只关注自己的那个模块，假如这个人休假或离职，就会出现其他人无法维护的情况，这对整个团队来说是非常不利的。团队的其他成员也许做不到像代码作者一样对程序的熟悉程度，但熟悉程序的设计和架构是十分必要的。

2、营造健康的Code Review文化

  • Code Review不是批评和指责
  • Code Review不是展示技能的地方
  • 互相尊重，换位思考
  • 鼓励很重要

在做代码审查时，缺乏审查经验的Reviewer经常会犯一些错误，如批评多于鼓励、炫技多于查错，从而造成糟糕的代码审查体验，成为团队成员接受代码审查的障碍。

在团队中营造健康的Code Review文化，一是要互相尊重，懂得换位思考，Coder要主动向Reviewer解说代码的修改目的和内容，Reviewer则重点在于理解作者的实现思路，查看是否有明显错误；二是避免总是试图找出错误，要多一些鼓励的评论，如 代码逻辑清晰、实现很巧妙。

3、Code Review三步走

第一遍：浏览代码，了解作者的预期目标和实现思路。

第二遍：精读代码，理解作者实现方式，检查是否有明显错误。

第三遍：和自己脑海的实现方式对比，提出建议或赞赏。

4、Code Review看什么

  • 是否符合编码规范（借助自动化工具）
  • 是否实现预期目的
  • 变量命名是否合理
  • 函数长度是否合适
  • 是否存在重复代码
  • 边界条件是否正确
  • 参数取值是否合理
  • 关键函数是否有UT
  • 是否具备可扩展性
  • 是否容易维护
  • 。。。。。。

相关推荐

nginx的反向代理（Nginx的反向代理和负载均衡）

nginxProxy代理1、代理原理反向代理服务的实现：需要有一个负载均衡设备（即反向代理服务器）来分发用户请求，将用户请求分发到后端正真提供服务的服务器上。服务器返回自己的服务到负载均衡设备。负...

Nginx UI: 更好用更现代化的Nginx 管理面板

各位铲屎官大家好，我是喵~...

性能测试之tomcat+nginx负载均衡（nginxtcp负载均衡）

nginxtomcat配置准备工作：两个tomcat执行命令cp-rapache-tomcat-8.5.56apache-tomcat-8.5.56_2修改被复制的tomcat2下con...

nginx upstream节点健康检查（nginx tcp 健康检查）

1、前提条件编译nginx时增加nginx_upstream_check_module模板git地址：https://github.com/yaoweibin/nginx_upstream_check...

Nginx 的高并发处理能力（nginx支持高并发原理）

为了实现Nginx的高并发处理能力，需要从**硬件资源**、**操作系统**、**Nginx配置**等多个方面进行优化。以下是详细的配置和示例：---...

Nginx最全详解(万字图文总结)（nginxs）

大家好，我是mikechen。Nginx是非常重要的负载均衡中间件，被广泛应用于大型网站架构，下面我就全面来详解Nginx@mikechen本篇已收于mikechen原创超30万字《阿里架构师进阶专题...

如何用 Nginx 实现前端灰度发布（nginx 灰度测试规则）

前言在前端开发中，灰度发布是一种重要的策略，它允许我们在不影响所有用户的情况下，逐步推出新功能或更新。通过灰度发布，我们可以测试新版本的稳定性和性能，同时收集用户反馈。今天，我们将探讨如何使用Ngi...

nginx配置优化场景-直接套用so happy!

前言(叠甲在先)Nginx是一款高性能的Web服务器，广泛应用于互联网领域。...

Nginx配置前后端服务（nginx前后端分离部署）

nginx安装完成后，可以通过命令查看配置文件nginx-t配置文件nginx.conf，是总的配置，有的人会把配置全部配置到这个文件中，如果服务很多，这个文件变得非常庞大，我见过一个配置很大的，在...

使用Nginx配置TCP负载均衡（nginx如何配置负载均衡）

假设Kubernetes集群已经配置好，我们将基于CentOS为Nginx创建一个虚拟机。...

Nginx服务器深度指南:安装、配置、优化指令超详解

在当今数字化时代，Web服务器是支撑互联网应用的关键基础设施。Nginx作为一款高性能的开源Web服务器，凭借卓越的性能、丰富的功能和出色的稳定性，在Web服务器领域占据了重要地位。无论是大型互联网公...

Nginx的配置详解(附代码)（nginx基本配置）

本篇文章给大家带来的内容是关于Nginx的配置详解（附代码），有一定的参考价值，有需要的朋友可以参考一下，希望对你有所帮助。常用配置项在工作中，我们与Nginx打交道更多的是通过其配置文件来进行。...

Nginx配置文件详解（nginx配置文件详解带实例）

Nginx配置文件详解Nginx是一款面向性能设计的HTTP服务器，相较于Apache、lighttpd具有占有内存少，稳定性高等优势。...

从 0 到 1:构建高可用 Linux 负载均衡集群(基于 Nginx + Keepalived)

在高并发业务场景下，单台服务器往往无法支撑大量请求，因此需要使用**负载均衡（LoadBalancing）**技术来提升系统的稳定性和可用性。Nginx+Keepalived是常见的开源负载均...

配置Nginx TCP转发（nginx 接口转发）

Nginx一般用在HTTP的转发，TCP的转发大都会使用HAProxy。工作中遇到一个需求，用到了Nginx服务作为TCP转发。场景是这样，数据采集设备通过公网将数据推送到后端应用服务，服务部署在业主...

取消回复欢迎 你 发表评论:

一周热门
最近发表
标签列表
Copyright Your WebSite.Some Rights Reserved.