Code Review
今天在圆老师的带领下,进行了一次Code Review。尽管之前也知道一些Code Smell,然鹅,在限时训练的时候,就顾不上那么多了,各种随意,直接导致代码中有很多需要重构的地方。
问题集中体现在以下几个方面:
- Test setup(在多个测试中都
new
了同一个对象,所以应该把这个对象提到@Before
中,减少冗余代码) - 移动方法到正确的类中(将方法根据对应的功能放在对应的类中,不要在一个类中写与该类无关的方法)
- 空行/格式化(写完代码记得要养成格式化的习惯,在适当的地方加上空行,帮助理清代码结构)
- 类名是名词,方法名用动词
- 重构一次运行一次测试(每次重构都要运行测试,来检验这次重构是否正确,如果正确,删除掉以前的代码)
- 消除重复代码
- 分离职责(每个方法都实现自己对应的职责,不要一个方法实现太多职责)
- 去除不必要临时变量
这么多问题的存在,证明,还是没有培养出好的编程习惯,以后一定要控制我自己~减少 Code Smell 的发生,培养良好的编程习惯,从我做起。
三层架构
三层架构是一个分层式的软件体系架构设计理念,适用于所有项目。
1. 表示层(表示层一般使用下面👇这几种语言框架)
- JSP
- JS
- Angular
- Reactive
作用:用于显示数据和接收用户输入的数据,为用户提供一种交互式操作的界面。主要对用户的请求接受,以及数据的返回,为客户端提供应用程序的访问。
2. 业务逻辑层(同层之间可以自己调用,这一层是一些服务,服务之间可以互相调用)
作用:主要负责对数据层的操作。也就是说把一些数据层的操作进行组合。用户输入的数据通过业务逻辑层的处理发给数据层,数据层返回的数据通过业务逻辑层发送给界面展示。
业务逻辑层无疑是系统架构中的核心部分。它处于表示层和数据访问层中间,起到了数据交换中承上启下的作用
3. 持久层(或数据访问层)
作用:主要完成各个对数据文件的操作,是对数据库的操作,具体为业务逻辑层或表示层提供数据服务。
三层架构的特点:
- 低层不能调用上层。
- 同层之前可以互相调用,但是,持久层和展示层之间不能互相调用。
三层架构的优点:
- 开发人员可以只关注整个结构中的某个一层。
- 实现高内聚低耦合的思想。
- 降低层与层之间的依赖。
- 结构更加明确。
- 方便后期维护。
MVC(Model-View-Controller)
MVC 把用户界面独立到一些文件中(Views),把一些和用户交互的程序逻辑(Controller)单独放在一些文件中,在 Views 和 Controller 中传递数据使用一些专门封装数据的实体对象,这些对象,统称为Models。
三层架构和 MVC 的区别
MVC 属于展示层的一种架构模式,和三层架构没有必然关系。
三层架构是系统架构,属于宏观的解决方案。可能会需要三种不同职能的程序员来维护,每个人可以专注于自己擅长的模块,分别进行开发。
MVC 是设计模式,属于微观的解决方案。是同一个程序员可以接触的东西,目的是为了让HTML代码和业务逻辑代码分开,让代码看起来更加清晰,便于开发。