大部分的公司,开发项目都是作坊式的,没有产品和项目的需求分析,进而做出技术架构和详细设计。
很多人,听到上级和老板的一个想法,就开始写代码,边写边改,甚至推倒重来。
最终,导致的常见结果之一,项目代码混乱,新员工甚至老员工,对项目理解比较吃力。
如果你去改造代码,改好了,没有任何功劳。改出问题了,领导、测试、产品,很可能会说你不行。
这一点,是让很多程序员纠结的地方。
我个人还是倾向重构的,先熟悉项目总体环境,从易到难。
项目开发,从外部看,就是一个个的功能。
从内部看,不就是一个个函数和API吗。
只要输入和输出是稳定的,就不会出现重大的失误,导致牵一发动全身的不良后果。
下面记录一些,我个人自己揣摩的一些重构方法,阅读《重构》这本书,也带给了我一些启发。
1.命名规范化
包名、类名、方法名、变量名,取合适的英文单词。
比如在一个电商项目中。
包名:com.company.shop.front.controller,前端系统的控制层
类名:OrderController,处理订单的控制器
方法名:findById,根据id查找对象,我个人更喜欢用get。
变量名:List<String> memberIdList = new ArrayList<String>();
会员id的list集合
比较坑的命名:Object tagcode = map.get("tagCode");
变量命名,没有按照Java驼峰式命名规范来写,“tagcode”和“tagCode”竟然同时存在。
在Java内部代码变量名,如果错了还有提示,在Freemarker和JSP等界面中,字符串的值,用错了,根本没有提示。
大小写不统一的bug,还很难发现。
命名要一致:Controller、Service、Dao等,同一个人和不同人的,尽可能遵循一定的标准,阅读和修改其他人代码也更顺利。
2.代码合理组织
controller:控制器,响应请求,路由控制
service:服务层,处理业务逻辑
dao:数据访问层
model:数据库模型
bean:内部用的实体类
util:工具代码
interceptor:拦截器
其它代码,可以按照功能等进行划分,让人一眼望去,就知道这个包下的代码,大致做了什么事情。
3.项目合理组织
合理拆分项目:
移动端,mobile项目
Web前端,front项目
后端管理:backend项目
合理服务拆分:
商品服务系统:ProductService项目
会员服务系统:UserService项目
登录服务系统:LoginService项目
订单服务系统:OrderService项目
公共代码:Model模型、Util工具类
听说淘宝和京东的电商网站,有几百上千个服务。
4.使用最小最恰当的作用域
类,大多数类用的是public,public class ProductService,如果只是在包内部使用,可以去掉public。
字段,尽可能用private, private ProductService productService。如果需要,通过get和set方法,来获得和修改值。
方法/函数,对外被调用的用public,只在类的内部使用的,尽可能用private。
public void add(){ doAdd(); } private void doAdd(){ }
很多人,内部代码都搞成public,乍一眼看上去,还以为外部有调用。
必须得看看依赖,才确定。
对外暴露了过多的接口,不该被调用的被调用了。
5.常量提取
"success",把作用一致并且相同的字符串,提取成常量,统一管理。
其中一种方式:
public class FrontConst { public static final String COOKIE_LOGINNAME = "loginNameCookie"; public static final String COOKIE_PASSWORD = "passWordCookie"; }
6.重复代码提取和封装
业务代码,提取成私有方法,内部重复使用。
工具代码,提取到工具类中,可以复用,比如日期处理、把list转换成map。
public class BizUtil { /** * 把1个集合,转换成Map。用法示例:Map<String, Dictionary> dictionaryMap = BizUtil.listToMap("BIANMA", dictionarys); * @param keyName 集合元素唯一字段的名称 * @param list 集合元素 * @return map */ public static <K, V> Map<K, V> listToMap(final String keyName, List<V> list) { if(CollectionUtils.isEmpty(list)){ return null; } return Maps.uniqueIndex(list, new Function<V, K>() { @Override public K apply(V v) { try { return (K)PropertyUtils.getSimpleProperty(v, keyName); } catch (IllegalAccessException | InvocationTargetException | NoSuchMethodException e) { e.printStackTrace(); return null; } } }); } /** * 把1个集合,转换成Map。用法示例:Map<String, Dictionary> dictionaryMap = * BizUtil.listToMap(String.class, Dictionary.class, "BIANMA", dictionarys); * * @param k * Map的key的class * @param v * Map的value的class,集合list元素的类型 * @param keyName * 集合元素唯一字段的名称 * @param list * 集合元素 * @return map */ //这种方式,废弃了,需要多传2个参数 public static <K, V> Map<K, V> listToMap(Class<K> k, Class<V> v, String keyName, List<V> list) { Map<K, V> map = Maps.newHashMap(); if (CollectionUtils.isNotEmpty(list)) { for (V val : list) { try { PropertyDescriptor pd = new PropertyDescriptor(keyName, v); Method getMethod = pd.getReadMethod();// 获得get方法 Object o = getMethod.invoke(val);// 执行get方法返回一个Object if (o != null && o.getClass().equals(k)) { map.put((K) o, val); } } catch (IllegalAccessException | IllegalArgumentException | InvocationTargetException | IntrospectionException e) { e.printStackTrace(); } } } return map; }
7.函数拆分
1个函数,有5个参数,有2种不同的使用场景。
同一个函数完成了2件不同的事情。
void add(int a,int b,int c);
换成
void addOrder(int a,int b);
void addProduct(int c)。
8.函数合并
2个函数,完成了类似的事情,简化成1个,
函数是拆分,还是合并,要具体分析。
9.单一职责原则
一个项目、一个模块、一个类、一个函数,完成一件事。
如果完成了多件事,需要进行拆分成多个独立的函数,至少内部需要进行拆分。
比如:
void add(int a,int b,int c)
可以拆分成
void add1(int a,int b)和void add2(int c)
也可以拆分成
void add(int a,int b,int c){
add1(a,b);
add2(c);
}
10.控制类和函数的行数
一个类,如果函数太多,往往是承载了太多的功能。
把不紧密相关的功能,堆积在了一起。
一个方法,如果代码超过了100行,存在问题的可能性就增大了很多。
在我自身的经历中,很多同事的函数,只要超过100行,一眼望去就能发现问题。
另外,函数代码过多,修改bug和新增业务逻辑的时候,很容易引入新的问题。
因此,函数拆分、降低作用域,可以保证过去稳定的代码逻辑,不会有改动。
11.提前返回
先检查错误,如果有问题,直接返回,以免嵌套过深。
public String add(){ Member member = this.getMember(); if(StringUtils.isBlank(...){ return error("地址信息有误!"); } }
经常出现下面的情况
if(...){
if(...){
if(...){
}
}
}
return ...;
12.定义枚举
枚举是常量的进一步封装。
public enum OrderPayStatusEnum { NO("0", "未支付"), YES("1", "已支付"); private String code; private String remark; OrderPayStatusEnum(String code, String remark) { this.code = code; this.remark = remark; } public String getCode() { return code; } public void setCode(String code) { this.code = code; } public String getRemark() { return remark; } public void setRemark(String remark) { this.remark = remark; } public static String getPayStatus(String code){ if(StringUtils.isEmpty(code)){ return null; } if(code.equals(NO.getCode())){ return NO.getRemark(); }else if(code.equals(YES.getCode())){ return YES.getRemark(); } return null; } }
13.统一API交互接口
后端和移动端,后端和Web前端的交互,都是Result的json字符串。
后端,所有请求,统一都是返回Result。
public class Result {
private Integer code;
private String desc;
private Object data;
}
多人开发的时候,如果没有架构师之类的角色,各自为战,真是乱套。
14.模块化和依赖
大多数项目,都会有诸如邮件、短信验证码、登录服务、图片云服务等。
把配置文件单独拿出来,可以手动通过Spring的配置文件,配置bean。
如果需要,引入配置文件,不引入,代码也不会报错。
spring-mail-config.xml
<context:property-placeholder location="file:${config_path}/config/mail.properties" ignore-unresolvable="true" /> <bean id="javaMailSender" class="org.springframework.mail.javamail.JavaMailSenderImpl"> <property name="host" value="${mailServerHost}" /> <property name="port" value="${mailServerPort}" /> <property name="javaMailProperties"> <props> <prop key="mail.smtp.auth">true</prop> <prop key="mail.smtp.timeout">25000</prop> </props> </property> <!-- 发送者用户名 --> <property name="username" value="${mailUserName}" /> <!-- 发送者密码 --> <property name="password" value="${mailPassword}" /> <property name="defaultEncoding" value="UTF-8" /> </bean> <bean id="mailClientServer" class="com.shop.common.mail.MailClientServer"> <property name="javaMailSender" ref="javaMailSender" /> <property name="configuration" ref="freeMarkerConfigurationFactory" /> <property name="mailFromAddress" value="${mailFromAddress}" /> </bean>
还比如,咱们的图片用的阿里云的OSS。
统一封装。
配置文件
oss.properties
oss.xml
<context:property-placeholder location="file:${config_path}/config/oss.properties" ignore-unresolvable="true" /> <bean id="ossConfig" class="com.shop.common.oss.OssConfig"> <property name="endpoint" value="${oss.endpoint}" /> <property name="bucketName" value="${oss.bucketName}" /> <property name="accessKeyId" value="${oss.accessKeyId}" /> <property name="accessKeySecret" value="${oss.accessKeySecret}" /> <property name="imgDomain" value="${oss.imgDomain}" /> <property name="fileDomain" value="${oss.fileDomain}" /> </bean>
Java代码
public class OssUtil {}
15.代码复用
代码复用,是一个广泛的话题。
前文提到的枚举、常量、函数拆分,目的之一就是为了方便代码复用。
一个优秀的程序员,抵得上十个差的程序员,其中一个原因就是“代码复用多”。
总是写重复的代码,把程序员这个智力密集型工作,干成了体力密集型,早晚被累死。
梳理业务需求、做好业务方面的架构设计。
另外,技术方面的架构设计,完全掌握在技术人员手中,对业务的依赖不大。
开发一个功能时,先想好思路,技术方案,再写代码。
架构复用、设计复用、工具类、枚举、常量。
另外还有一种,非常常见的,流程复用。
前端系统:
1.构造url和参数
2.执行请求,获得数据
3.根据状态码,执行不同的操作
4.正常状态,渲染数据,或执行动作
移动端:
流程类似
后端:
1.稳定并且统一的API接口
2.业务逻辑封装
无论内部怎么变,接口稳定,就不会影响Web前端和移动端。
在合适的时机,内部项目拆分,项目服务化。
重构的技巧,还是有很多的,更多还是看个人对编程的理解。
对于有经验的程序员来说,《重构》是一本不错的书,可以用来快速加深平时对项目代码的梳理。
对于新手来说,《重构》和设计模式,这种书,受益偏小。
小雷-一个有独立见解的年轻人
2016年4月16日~清明节刚过,五一又要来了
湖北-武汉~听说最近有几条公交专线要开通了
最近,有好多好多的想法和话题要写,时间不太够。
更准确来说, 时间是有的,写文章,也要看心情。
写一篇严肃一点的长篇大论,一般都需要1小时以上。
在写作之前,大脑里面一直在不断地“打草稿”,写的过程中,需要不断发散,要有条理,还要严谨,站得住脚。
一些问题的根源,也许是我的身价还是太低了,更多时间忙着搞钱,一些有价值的事情,只能慢慢地去做。