关于最近一次Code Review 的一些想法@TOC

前言

团队内部会经常进行code review ,以前并没review 自己的代码,一直作为一个旁观者的角度去看问题。这次review 自己的代码之后,让自己有意思到很多的不足。

Code Review 有感

  • 关于服务的边界?

    • 之前自己从来没有思考过这个问题,都是从以满足业务为基本要求去写代码。举一个简单的例子,之前如果要提供一个修改状态的接口,我可能只会对外暴露一个updateStatus 的接口,他们可以把状态(例如:INIT,FAIL,ERROR,SUCCESS)作为一个参数传进来,调用者想要更新为什么状态,都可以(当然,不是所有的状态修改都会成功,在服务内可以做一状态的校验)。
    • 在code review的时候,他们认为应该对外提供多个接口,比如updateStatus2SUC/updateStatus2Fail,在自己服务内部的时候,可以重用大部分的代码。这样只给接口调用者最小的权限。
  • 关于接口中参数的定义?
    • 之前自己定义接口中的dto 的时候,属性基本上和domain是一样的。所以像updateTime,createTime 这样的属性都是由接口调用者设置好之后传到服务端,然后设置/保存到数据库的。其实自己也知道,想updateTime 这种字段是不需要传的,可以直接使用数据的时间。这次code review的时候,被同事明确指出来,以后这种字段能不传就不传,一些可以确定的字段不需要传,比如创建时间,初始化状态是INIT……

希望自己在下次code review的时候有更好的表现_

Code Review 有感相关推荐

  1. 如何在团队中做好Code Review

    一.Code Review的好处 想要做好Code Review,必须让参与的工程师充分认识到Code Review的好处 1.互相学习,彼此成就 无论是高手云集的架构师团队,还是以CURD为主的业务 ...

  2. Google是如何做Code Review的?| CSDN原力计划

    作者 | 帅昕 xindoo 编辑 | 屠敏 出品 | CSDN 博客 我和几个小伙伴一起翻译了Google前一段时间放出来的Google's Engineering Practices docume ...

  3. 刚进美团,就被各种Code Review,真的有必要吗?

    点击关注公众号,Java干货及时送达 众所周知,Code Review是开发过程中一个非常重要的环节,但是很多公司或者团队是没有这一环节的,今天笔者结合自己所在团队,浅谈Code Review的价值及 ...

  4. 刚入职,就被各种 Code Review,真的有必要吗?

    点击上方蓝色"方志朋",选择"设为星标" 回复"666"获取独家整理的学习资料! 来源:juejin.cn/post/ 6882333635 ...

  5. 有必要做 Code Review 吗???

    点击上方蓝色"方志朋",选择"设为星标" 回复"666"获取独家整理的学习资料!作者:梨香 链接:https://juejin.im/pos ...

  6. 你太菜了,竟然不知道Code Review...

    点击上方"方志朋",选择"设为星标" 回复"666"获取新整理的面试资料 作者:宝玉   来源:http://1t.click/aA4h 我 ...

  7. 从零开始 Code Review,两年实战经验分享!

    点击上方"方志朋",选择"置顶公众号" 技术文章第一时间送达! 来源:http://t.cn/RtHE14S 前几天看了<Code Review 程序员的 ...

  8. 作为CTO,我为什么必须要求代码进行Code Review!

    来源:宝玉 链接:https://cnblogs.com/dotey/p/11216430.html 我一直认为Code Review(代码审查)是软件开发中的最佳实践之一,可以有效提高整体代码质量, ...

  9. 万字详文告诉你如何做 Code Review

    点击上方"小白学视觉",选择加"星标"或"置顶" 重磅干货,第一时间送达本文转自|机器学习实验室 前言 作为公司代码委员会 golang 分 ...

最新文章

  1. 你想了解的「SpringCloud」都在这里
  2. 怎么打散铺铜_装修辅材有哪些?怎么选?元老级工头:照这样去买你家多住50年...
  3. python爬app_Python爬虫抓取手机APP的传输数据
  4. oracle if/else功能的实现的3种写法
  5. 训练和测试自己的图像集
  6. winform combobox 单机不关闭_单机脉冲除尘器铜仁单机脉冲除尘器参数
  7. Spring依赖注入流程【完整版】(populateBean方法内部执行流程)
  8. mac 爱普生打印机驱动_爱普生l301打印机驱动Mac版-爱普生L301驱动Mac版下载 V9.31-PC6苹果网...
  9. 水果销售管理系统课程设计报告
  10. 【机器学习算法】线性回归算法
  11. 【论文笔记-5】Spatial Transformer Networks(STN)
  12. 联想ThinkBook解锁FN键
  13. 电磁兼容(EMC)测试
  14. 2021届 美的笔试 C++开发工程师
  15. Mal-PEG3-acid,518044-40-1亲水性PEG间隔物增加了在水介质中的溶解度
  16. 关联性——灰色关联分析
  17. 《有效成交量分析法》(威廉•帕斯卡尔)一、有效成交量概念的提出
  18. 【曹工杂谈】Maven底层容器Plexus Container的前世今生,一代芳华终落幕
  19. 简单shell命令学习(一)
  20. Android 实现URL生成二维码

热门文章

  1. 无炫技:纯粹的Bert和Roberta
  2. 2021华为软件精英挑战赛(杭厦第20名)
  3. Cross-scene Crowd Counting via Deep Convolutional Neural Networks2015论文笔记
  4. 打包开发板根文件系统,并制作成img镜像
  5. Java开源 Web开发框架 (一)
  6. ETC收费交易流程规范
  7. 聚英国际|【燎原计划启动周第二期AMA】Filecoin经济模型及热点争议解读
  8. swipecell样式_反馈组件 SwipeCell 滑动单元格 - 闪电教程JSRUN
  9. 数据网站 免费数据网站
  10. Dubbo解析及原理浅析