修改一处代码，都要通篇进行review

最新推荐文章于 2024-07-18 21:37:27 发布
最新推荐文章于 2024-07-18 21:37:27 发布
阅读量1.9k 收藏
点赞数 1
分类专栏： 思想总结
版权声明：本文为博主原创文章，遵循 CC 4.0 BY-SA 版权协议，转载请附上原文出处链接和本声明。
本文链接：https://blog.csdn.net/KuaiLeShiFu/article/details/79475247
版权

在一个方法中，增加一行代码，觉得没问题吗？

原则：

修改一处代码，都要通篇进行review；
重点看下修改的代码对原有代码的影响是什么？新增的代码是干啥的？

对流程有什么影响；
对逻辑有什么影响？

例子：

在一个执行过滤功能的方法中，新增一个过滤代码。
可能造成的影响以及review的重点是：该行代码放的位置对吗？是放在哪个过滤前，还是哪个过滤后，还是放到哪些过滤之间？

bigzhangbig
关注
专栏目录
【改善既有代码的设计】结合真实的代码案例场景，谈谈我的一些实施方法（RT-Thread技术论坛优秀文章）
一个专注于嵌入式IoT领域的架构师，深耕IoT领域多年，深度掌握IoT领域的相关技术栈，包括但不限于RTOS内核的实现及其移植、硬件驱动移植开发、网络通讯协议开发、编译构建原理及其实现、底层汇编及编译原理、编译优化及代码重构、嵌入式IoT系统的架构设计等。
10-25 1万+
本文结合真实的代码案例场景，聊到了一些旧代码，顺便谈一谈我的一些实施方法，希望对大家有所启发。
AI上推荐 之 DIEN模型(序列模型与推荐系统的花火碰撞)
Miracle8070
03-06 7464
1. 前言 随着信息技术和互联网的发展， 我们已经步入了一个信息过载的时代，这个时代，无论是信息消费者还是信息生产者都遇到了很大的挑战： 信息消费者：如何从大量的信息中找到自己感兴趣的信息？ 信息生产者：如何让自己生产的信息脱颖而出， 受到广大用户的关注？ 为了解决这个矛盾， 推荐系统应时而生， 并飞速前进，在用户和信息之间架起了一道桥梁，一方面帮助用户发现对自己有价值的信息， 一方面让信息能够展现在对它感兴趣的用户前面。 推荐系统近几年有了深度学习的助推发展之势迅猛， 从前深度学习的传统推荐模型(协
部署代码review和CI
dabbler_zhu的专栏
04-23 620
公司原先搭了一个代码Review的服务器，由于历史原因，装的是一个32bit的Ubuntu系统，后来由于需要，需要安装gitlab，由于gitlab需要64位系统，所以临时凑合了个vagrant，本质就是一个纯粹的虚拟机，感觉不爽，这两天终于抽出时间来重新整理了一下。基于Ubuntu 18.04 x64版本和Docker来部署，减少后面换机器换系统可能导致的重复安装工作。 Docker安装 D...
code review 看什么
程序员写的技术 FAQ 和杂文
03-18 1014
1，大家都是人出错难免，不要笑话他人。 2，初级团队看格式，注释，风格等，当然有自动化工具检查最好。 3，代码的可读性优先于代码语法糖。 4，有单元测试最好。要是能强制写最好。但是否百分百覆盖，这个认为不要求。 5，有代码规范、review 指南、总结。 6，开发时注释的别人代码，是否开启，不应通过注释调节开发流程。至少是不能提交。 7，开发时加的日志输出是否关闭，加的日志就不能去除，否则就不应加。通过设置日志级别控制。 8，不要写固定数字，字符串，尽量少写，多用常量代替或配置文件。 先写到
服务器代码review要点
Ftworld21的专栏
01-09 271
1. 数据存档检查 (1) 检查跨区是否会因活动细节不一致导致数据异常或数据丢失，一般来说跨区是不应该修改活动数据的，所以会在活动的定时器、事件触发入口以及消息触发入口屏蔽活动的开启，并且进入战区或者旅游区时会备份一下数据，这样回原区的时候，直接把备份的数据带回。 (2) 检查跨区是否能获取某些活动的奖励，如果能的话，根据(1)的规则，已经获取奖励的标志并不能带回，从而导致奖励被刷。 (3)...
代码 review 注意事项 check list
鲸鱼写程序的专栏
07-15 750
代码review checklist
代码检测：Code Review与CheckStyle
weixin_34351321的博客
02-14 126
      本文向大家介绍Code Review的主要内容，以及流行的检查Code Conventions的工具。同时，对于目前应用最为广泛的CheckStyle的应用给出详细的介绍，也列举了很多使用CheckStyle的最佳实践。 　　一、Code Review & Code Conventions 　　质量是衡量一个软件是否成功的关键要素。而对于商业软件系统，尤其是企业应...
从零开始Code Review
热门推荐
晓月的专栏
11-11 2万+
从零开始Code Review这篇帖子不是介绍Code Review的方法论, 而是记录了我们团队怎么从没有这个习惯到每天都进行review的过程. 希望能对诸位的团队有所帮助. 最初来到这个新组建的团队是木有code review的. 头说, 这个月你来搞吧.当我第一次知道必须得搞review的时候, 其实我是拒绝的! 因为我觉得…呀…你不能叫我马上搞立马搞, 第一,
Go语言高效率Web开发五：改造面向过程代码，步步惊心（步一）
q1472750149的博客
01-08 471
如图所示，返回当前周签到记录： { "data": { "2022-01-02": true, "2022-01-03": false, "2022-01-04": false, "2022-01-05": false, "2022-01-06": false, "2022-01-07": false, "2022-01-08": false } } true代表签到 false代表未签到 复制代码 面向过程代码实现 源码链接 f...
Android应用优化之代码检测优化
qq_435559203的博客
05-24 3033
前言最近换了新的公司，面对新的代码大家都有不同的熟悉过程和方法。在我的角度来说，利用代码检测工具，可以更直接地去熟悉代码逻辑和业务逻辑，表现得自己去代码质量很有追求，最重要当然是在公司的任务管理工时上面显得自己积极向上啦。不过在修改代码之前，你要根据项目的分工、明确在公司的定位，不然会造成一些不愉快的事情，但是总的来说我们还是对代码质量有追求的！
代码Review的必要性和考虑要素
Merry Zhang的专栏
01-06 5186
 极限编程里提到结对编程和代码Review,凡是稍微懂编程的人看了都会赞成。这也体现了代码Review的重要性和必要性。但是，在实际的执行过程中，代码Review往往很难得到很好的执行。主要原因可能包含以下几点： （1）对自己编写的代码的不够自信，害怕别人找到问题； (2) 对于自己的编写的代码过于自信，不觉得需要代码Review；(3) 对于自己的代码过于封闭，不愿意与别人分
质量保证考试题目、答案及详解
qq_45499206的博客
10-22 2万+
B. 质量策划就是制定质量目标，并策划符合项目特点、切中痛点的、给力的定制化流程等质量保证和控制措施的质量活动。B. 质量与每个人息息相关（作业员，管理人员，技术人员等），不只是质量部的责任（全员参与）B. 质量与每个人息息相关（作业员，管理人员，技术人员等），不只是质量部的责任（全员参与）C. 质量策划不仅仅是对质量目标的制定，更应该看到具体的质量保证措施和质量控制措施。D. 提交给客户的版本必须来自配置库，且必须是转测试后的版本。D. 提交给客户的版本必须来自配置库，且必须是转测试后的版本。
质量管理
xiaohuige123的专栏
02-08 5310
1、了解公司的质量方针、目标及基本质量要求；了解公司相关流程；了解公司关键质量活动。2、质量知识简介、讨论什么是质量、软件质量的定义、质量方针&质量目标&、基本质量要求、公司相关流程介绍、公司质量活动&规范。3、讨论：什么是质量？什么是质量好？4、软件质量的定义第一次就把正确的事情做正确软件质量是：软件与明确的和隐含的需求相一致的程度。具体地说，软件质量是软件符合明确叙述的...
质量测试---
清心的博客
03-10 3454
47. 新七大质量工具：鱼骨图、柏拉图、查检表、层别法、散布图、直方图、管制图。老七大质量工具：关系图、系统图法、KJ法、箭头图法、矩阵图法、PDPC法、矩阵数据解析法。39. AAR必须提供足够的时间进行完整的讨论，比如半天、一天时间，直到讨论出有效的改进措施为止。44. 目标是指管理活动所要求达到的成果，是企业在一定时期内努力奋斗的方向和希望达到的具体指标。B. 质量与每个人息息相关（作业员，管理人员，技术人员等），不只是质量部的责任（全员参与）A. 衡量质量的指标如果只有1个，那就是客户满意度。
质量保证考试题目答案及其分析
最新发布
m0_58435031的博客
07-18 2691
B. 管理根因分析-技术根因分析-措施拟定-对策实施-标准化总结。A. 把握现状-根因分析-措施拟定-对策实施-标准化总结。B. 流程顺序为计划——需求——设计——编码——测试。你的答案: A,B,C,D正确答案 A,B,C,D。你的答案: A,B,C,D正确答案 A,B,C,D。你的答案: A,B,C,D正确答案 A,C,D。A. 所制定的措施需要措施实施人的确认与承诺。你的答案: A,C,D正确答案 A,C,D。你的答案: A,B,C正确答案 A,B,C。你的答案: A,B,D正确答案 A,B,D。
质量考试质量保证
lidadang的博客
12-08 4346
43. 交付项目管理，L3流程统一，按4种项目类型确定在不同阶段（活动）的管控和服务要素（质量要求），但不含具体交付场景涉及的要求。41. AAR必须提供足够的时间进行完整的讨论，比如半天、一天时间，直到讨论出有效的改进措施为止。B. 帮助成员对团队发生的事情及意义形成统一的观点 或理解，这是团队达成进一步共识的基础。D. 团队将能更熟练的设定实际、可达成的目标，但无法帮助建立团队坦诚、开放的氛围。C. 通过集体的学习，丰富团队成员的经验和知识，促进团队的成长。
前端代码质量保障之代码review
weixin_30478619的博客
12-28 821
经验丰富的程序员和一般程序员之间的最大区别，不仅体现在解决问题的能力上， 还体现在日常代码的风格上。掌握一门技术可能需要几月，甚至几周就够了。 好的习惯风格养成却需数年。 团队成员之间需要合作，代码需要日后可维护，个体的能力和习惯存在差异。 故保证代码质量及风格，就需要制定一定的规则，按项目周期(最好是在上预发之前)组织进行集体代码revi...
万字详文告诉你如何做 Code Review
腾讯技术工程
06-17 1822
作者：cheaterlin，腾讯 PCG 后台开发工程师前言作为公司代码委员会 golang 分会的理事，我 review 了很多代码，看了很多别人的 review 评论。发现不少同学 ...
代码规范评审
lucky的专栏
10-22 1300
责任心（Owner心态） 以业务产出价值为目标，实现个人价值成长，要求每位同事都拥有主人翁意识对待工作中的所有问题。 意识上坚信自己，态度上富有责任心。 编码规范 因团队成员原来背景各不相同，风格迥异，为了提升整个团队的效率，提升团队凝聚力，在此约定相应的规范让大家遵守，一定程度上减少沟通成本，减少各种不必要的出错概率 Java语言规范 1、代码格式化，统一使用IDEA初始安装后默认的格式，避免使用自定义格式插件，避免代码合并时diff过多，代码提交到Git仓库之前，都需要格式好，记住格式化
ThreadPoolExecutor 创建线程池，设置 最大线程数量之类的，那段经典代码怎么写来着，给我写出详细的示例代码，通篇用 markdown格式，代码不要和 说明文字 混在一起 好不好，谢谢，
02-20
from concurrent.futures import ThreadPoolExecutor# 创建一个最大线程数为4的线程池 executor = ThreadPoolExecutor(max_workers=4)# 定义一个函数 def func(x): return x * x# 将函数提交到线程池，获取一个 ...
评论
添加红包

请填写红包祝福语或标题

个

红包个数最小为10个

元

红包金额最低5元

当前余额3.43元 前往充值 >
需支付：10.00元
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付元
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明：

1.余额是钱包充值的虚拟货币，按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载，可以购买VIP、付费专栏及课程。

余额充值