82

程序员你为什么这么累【续】:编码习惯之工具类规范

 6 years ago
source link: http://blog.didispace.com/cxy-wsm-zml-10/?amp%3Butm_medium=referral
Go to the source link to view the article. You can view the picture content, updated content and better typesetting reading experience. If the link is broken, please click the button below to view the snapshot at that time.

程序员你为什么这么累【续】:编码习惯之工具类规范

导读: 程序员你为什么这么累?

一个项目不可能没有工具类,工具类的初衷是良好的,代码重用,但到了后面工具类越来越乱,有些项目工具类有几十个,看的眼花缭乱,还有不少重复。如何编写出好的工具类,我有几点建议:

就是要定义自己的工具类,尽量不要在业务代码里面直接调用第三方的工具类。这也是解耦的一种体现。如果我们不定义自己的工具类而是直接使用第三方的工具类有2个不好的地方

  1. 不同的人会使用不同的第三方工具库,会比较乱。
  2. 将来万一要修改工具类的实现逻辑会很痛苦。

以最简单的字符串判空为例,很多工具库都有 StringUtils工具类,如果我们使用commons的工具类,一开始我们直接使用 StringUtils.isEmpty ,字符串为空或者空串的时候会返回为true,后面业务改动,需要改成如果全部是空格的时候也会返回true,怎么办?我们可以改成使用 StringUtils.isBlank 。看上去很简单,对吧? 如果你有几十个文件都调用了,那我们要改几十个文件,是不是有点恶心?再后面发现,不只是英文空格,如果是全角的空格,也要返回为true,怎么办?StringUtils上的方法已经不能满足我们的需求了,真不好改了。。。

所以我的建议是,一开始就自己定义一个自己项目的StringUtil,里面如果不想自己写实现,可以直接调用commons的方法,如下:

public static boolean isEmpty(String str) {
return org.apache.commons.lang3.StringUtils.isEmpty(str);
}

后面全部空格也返回true的时候,我们只需要把isEmpty改成isBlank;再后面全部全角空格的时候也返回true的话,我们增加自己的逻辑即可。我们只需要改动和测试一个地方。

在举一个真实一点的例子,如复制对象的属性方法。

一开始,如果我们自己不定义工具类方法,那么我们可以使用 org.springframework.beans.BeanUtils.copyProperties(source, dest)这个工具类来实现,就一行代码,和调用自己的工具类没有什么区别。看上去很OK,对吧?

随着业务发展,我们发现这个方式的性能或者某些特性不符合我们要求,我们需要修改改成 commons-beanutils包里面的方法,org.apache.commons.beanutils.BeanUtils.copyProperties(dest, source),这个时候问题来了,第一个问题,它的方法的参数顺序和之前spring的工具类是相反的,改起来非常容易出错!第二个问题,这个方法有异常抛出,必须声明,这个改起来可要命了!结果你发现,一个看上去很小的改动,改了几十个文件,每个改动还得测试一次,风险不是那么得小。有一点小奔溃了,是不是?

等你改完之后测试完了,突然有一天需要改成,复制参数的时候,有些特殊字段需要保留(如对象id)或者需要过滤掉(如密码)不复制,怎么办?这个时候我估计你要崩溃了吧?不要觉得我是凭空想象,编程活久见,你总会遇到的一天!

所以,我们需要定义自己的工具类函数,一开始我定义成这样子。

public void copyAttribute(Object source, Object dest) {

org.springframework.beans.BeanUtils.copyProperties(source, dest);

}

后面需要修改为commons-beanutis的时候,我们改成这样即可,把参数顺序掉过来,然后处理了一下异常,我使用的是Lombok的SneakyThrows来处理异常,你也可以捕获掉抛出运行时异常,个人喜好。

@SneakyThrows
public void copyAttribute(Object source, Object dest) {
org.apache.commons.beanutils.BeanUtils.copyProperties(dest, source);
}

再后面,复制属性的时候需要保留某些字段或者过滤掉某些字段,我们自己参考其他库实现一次即可,只改动价格和测试一个文件一个方法,风险非常可控。

还记得我之前的帖子里说的需求变更吗?你可以认为这算需求变更,但同样的需求变更,我一个小时改完测试,没有任何风险轻轻松松上线,你可能满头大汗加班加点还担心出问题。。。

使用父类/接口

上面那点隐藏实现,说到底是封装/解耦的思想,而现在说的这点是抽象的思想,做好了这点,我们就能编写出看上去很专业的工具类。这点很好理解,但是我们容易忽略。

举例,假设我们写了一个判断arraylist是否为空的函数,一开始是这样的。

public static boolean isEmpty(ArrayList<?> list) {
return list == null || list.size() == 0;
}

这个时候,我们需要思考一下参数的类型能不能使用父类。我们看到我们只用了size方法,我们可以知道size方法再list接口上有,于是我们修改成这样。

public static boolean isEmpty(List<?> list) {
return list == null || list.size() == 0;
}

后面发现,size方法再list的父类/接口Collection上也有,那么我们可以修改为最终这样。

public static boolean isEmpty(Collection<?> list) {
return list == null || list.size() == 0;
}

到了这部,Collection没有父类/接口有size方法了,修改就结束了。最后我们需要把参数名字改一下,不要再使用list。改完后,所有实现了Collection都对象都可以用,最终版本如下:

public static boolean isEmpty(Collection<?> collection) {
return collection == null || collection.size() == 0;
}

是不是看上去通用多了 ,看上去也专业多了?上面的string相关的工具类方法,使用相同的思路,我们最终修改一下,把参数类类型由String修改为CharSequence,参数名str修改为cs。如下:

public static boolean isEmpty(CharSequence cs) {
return org.apache.commons.lang3.StringUtils.isEmpty(cs);
}

思路和方法很简单,但效果很好,写出来的工具类也显得很专业!总结一下,思路是抽象的思想,主要是修改参数类型,方法就是往上找父类/接口,一直找到顶为止,记得修改参数名。

使用重载编写衍生函数组

开发过的兄弟都知道,有一些工具库,有一堆的重载函数,调用起来非常方便,经常能直接调用,不需要做参数转换。这些是怎么样编写出来的呢?我们举例说明。

现在需要编写一个方法,输入是一个utf-8格式的文件的文件名,把里面内容输出到一个list。我们刚刚开始编写的时候,是这个样子的

public static List<String> readFile2List(String filename) throws IOException {
List<String> list = new ArrayList<String>();

File file = new File(filename);

FileInputStream fileInputStream = new FileInputStream(file);

BufferedReader br = new BufferedReader(new InputStreamReader(fileInputStream,
"UTF-8"));

// XXX操作

return list;
}

我们先实现,实现完之后我们做第一个修改,很明显,utf-8格式是很可能要改的,所以我们先把它做为参数提取出去,方法一拆为二,就变成这样。

public static List<String> readFile2List(String filename) throws IOException {
return readFile2List(filename, "UTF-8");
}

public static List<String> readFile2List(String filename, String charset)
throws IOException {
List<String> list = new ArrayList<String>();

File file = new File(filename);
FileInputStream fileInputStream = new FileInputStream(file);

BufferedReader br = new BufferedReader(new InputStreamReader(fileInputStream,
charset));

// XXX操作

return list;
}

多了一个方法,直接调用之前的方法主体,主要的代码还是只有一份,之前的调用地方不需要做任何修改!可以放心修改。

然后我们在看里面的实现,下面这2行代码里面,String类型的filename会变化为File类型,然后在变化为FileInputStream 类型之后才使用

File file = new File(filename);
FileInputStream fileInputStream = new FileInputStream(file);

这里我们就应该想到,用户可能直接传如File类型,也可能直接传入FileInputStream类型,我们应该都需要支持,而不需要用户自己做类型的处理!在结合上一点的使用父类,把FileInputStream改成父类InputStream,我们最终的方法组如下:

package plm.common.utils;

import java.io.BufferedReader;
import java.io.File;
import java.io.FileInputStream;
import java.io.IOException;
import java.io.InputStream;
import java.io.InputStreamReader;
import java.util.ArrayList;
import java.util.List;

import org.apache.commons.io.IOUtils;

/**
* 工具类编写范例,使用重载编写不同参数类型的函数组
*
* @author 晓风轻 https://github.com/xwjie/PLMCodeTemplate
*
*/
public class FileUtil {

private static final String DEFAULT_CHARSET = "UTF-8";

public static List<String> readFile2List(String filename) throws IOException {
return readFile2List(filename, DEFAULT_CHARSET);
}

public static List<String> readFile2List(String filename, String charset)
throws IOException {
FileInputStream fileInputStream = new FileInputStream(filename);
return readFile2List(fileInputStream, charset);
}

public static List<String> readFile2List(File file) throws IOException {
return readFile2List(file, DEFAULT_CHARSET);
}

public static List<String> readFile2List(File file, String charset)
throws IOException {
FileInputStream fileInputStream = new FileInputStream(file);
return readFile2List(fileInputStream, charset);
}

public static List<String> readFile2List(InputStream fileInputStream)
throws IOException {
return readFile2List(fileInputStream, DEFAULT_CHARSET);
}

public static List<String> readFile2List(InputStream inputStream, String charset)
throws IOException {
List<String> list = new ArrayList<String>();

BufferedReader br = null;
try {
br = new BufferedReader(new InputStreamReader(inputStream, charset));

String s = null;
while ((s = br.readLine()) != null) {
list.add(s);
}
} finally {
IOUtils.closeQuietly(br);
}

return list;
}

}

怎么样?6个方法,实际上代码主体只有一份,但提供各种类型的入参,调用起来很方便。开发组长编写的时候,多费一点点时间,就能写来看上去很专业调用起来很方便的代码。如果开发组长不写好,开发人员发现现有的方法只能传String,她要传的是InputStream,她又不敢改原来的代码,就会copy一份然后修改一下,就多了一份重复代码。代码就是这样烂下去了。。。

关键点,多想一步,根据参数变化编写各种类型的入参函数,需要保证函数主要代码只有一份。

使用静态引入

工具类的一个问题就是容易泛滥,主要原因是开发人员找不到自己要用的方法,就自己写一个,开发人员很难记住类名,你也不可能天天代码评审。

所以要让开发人员容易找到,我们可以使用静态引入,在Eclipse里面这样导入:

img

img

这样,任何地方开发人员只要一敲就可以出来,然后再约定一下项目组方法名规范,这样工具类的使用就会简单很多!

物理上独立存放

这点是我的习惯,我习惯把和业务无关的代码放到独立的工程或者目录,在物理上要分开,专人维护。不是所有人都有能力写工具类,独立存放专门维护,专门的权限控制有助于保证代码的纯洁和质量。这样普通的开发人员就不会随意修改。

例如我的范例工程里面,专门建立了一个source目录存放框架代码,工具类也在里面,这里的代码,只有我一个人会去修改:

img

img

几乎所有人都知道面向对象的思想有抽象封装,但几个人真正能做到,其实有心的话,处处都能体现出这些思想。编写工具类的时候需要注意参数的优化,而且大型项目里面不要在业务代码里面直接调用第三方的工具类,然后就是多想一步多走一步,考虑各种类型的入参,这样你也能编写出专业灵活的工具类!

Github:晓风轻的代码模板:https://github.com/xwjie/PLMCodeTemplate ,欢迎加星fork。

本文作者:晓风轻,
原文链接:https://zhuanlan.zhihu.com/p/29199049
版权归作者所有,转载请注明作者、原文、译者等出处信息

About Joyk


Aggregate valuable and interesting links.
Joyk means Joy of geeK