关于最近一次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的时候有更好的表现_