首页 > 编程语言 >java 高质量编程,正反例分析

java 高质量编程,正反例分析

时间:2023-01-30 11:35:52浏览次数:47  
标签:正例 java 正反 userDO 编程 List 反例 new public


导读

世界上只有两种物质:高效率和低效率;世界上只有两种人:高效率的人和低效率的人。----萧伯纳

常量&变量

直接赋值常量,禁止声明新对象

直接赋值常量值,只是创建了一个对象引用,而这个对象引用指向常量值。

反例

Long i=new Long(1L);
String s=new String("abc");

正例

Long i=1L;
String s="abc";

当成员变量值无需改变时,尽量定义为静态常量

在类的每个对象实例中,每个成员变量都有一份副本,而成员静态常量只有一份实例。

反例

public class HttpConnection{
private final long timeout=5L;
...
}

正例

public class HttpConnection{
private static final long timeout=5L;
...
}

尽量使用基本数据类型,避免自动装箱和拆箱

Java中的基本数据类型double、float、long、int、short、char、boolean,分别对应包装类Double、Float、Long、Integer、Short、Character、Boolean。

Jvm支持基本类型与对象包装类的自动转换,被称为自动装箱和拆箱。装箱和拆箱都是需要CPU和内存资源的,所以应尽量避免自动装箱和拆箱。

反例

Integer sum = 0;
int[] values = { 1, 2, 3, 4, 5 };
for (int value : values) {
sum+=value;
}

正例

int sum = 0;
int[] values = { 1, 2, 3, 4, 5 };
for (int value : values) {
sum+=value;
}

如果变量的初值会被覆盖,就没有必要给变量赋初值

反例

public static void main(String[] args) {
boolean isAll = false;
List<Users> userList = new ArrayList<Users>();
if (isAll) {
userList = userDAO.queryAll();
} else {
userList=userDAO.queryActive();
}
}

public class Users {

}

public static class userDAO {
public static List<Users> queryAll() {
return null;
}

public static List<Users> queryActive() {
return null;
}
}

正例

public static void main(String[] args) {
boolean isAll = false;
List<Users> userList;
if (isAll) {
userList = userDAO.queryAll();
} else {
userList=userDAO.queryActive();
}
}

public class Users {

}

public static class userDAO {
public static List<Users> queryAll() {
return null;
}

public static List<Users> queryActive() {
return null;
}
}

尽量使用函数内的基本类型临时变量

在函数内,基本类型的参数和临时变量都保存在栈(Stack)中,访问速度较快;对象类型的参数和临时变量的引用都保存在栈(Stack)中,内容都保存在堆(Heap)中,访问速度较慢。在类中,任何类型的成员变量都保存在堆(Heap)中,访问速度较慢。

反例

public final class Accumulator {
private double result = 0.0D;

public void addAll(@NonNull double[] values) {
for (double value : values) {
result += value;
}
}
}

正例

public final class Accumulator {
private double result = 0.0D;

public void addAll(@NonNull double[] values) {
double sum = 0.0D;
for (double value : values) {
sum += value;
}
result += sum;
}
}

尽量不要在循环体外定义变量

在老版本JDK中,建议“尽量不要循环体内定义变量”,但是在新版的JDK中已经做了优化。通过对编译后的字节码分析,变量定义在循环体外和循环体内没有本质的区别,运行效率基本上是一样的。反而,根据“局部变量作用域最小化”原则,变量定义在循环体内更科学更便于维护,避免了延长对象生命周期导致延缓回收问题。

反例

UserVO userVo;
List<UserVO> userDOList=new ArrayList<>(5);
for(UserVO vo:userDOList) {
userVo=new UserVO();
...
}

正例

List<UserVO> userDOList=new ArrayList<>(5);
for(UserVO vo:userDOList) {
UserVO userVo=new UserVO();
...
}

不可变的静态常量,尽量使用非线程安全类

不可变的静态常量,虽然需要支持多线程访问,也可以使用非线程安全类。

反例

public static final Map<String, Class> CLASS_MAP;
static {
Map<String, Class> classMap=new ConcurrentHashMap<>(16);
classMap.put("VARCHAR", java.lang.String.class);
...
CLASS_MAP=Collections.unmodifiableMap(classMap);
}

正例

public static final Map<String, Class> CLASS_MAP;
static {
Map<String, Class> classMap=new HashMap<>(16);
classMap.put("VARCHAR", java.lang.String.class);
...
CLASS_MAP=Collections.unmodifiableMap(classMap);
}

不可变的成员变量,尽量使用非线程安全类

不可变的成员变量,虽然需要支持多线程访问,也可以使用非线程安全类。

反例

private List<Strategy> strategyList;
private Map<String, Strategy> strategyMap;

public void afterPropertiesSet() {
if (CollectionUtils.isNotEmpty(strategyList)) {
int size=(int)Math.ceil(strategyList.size()*4.0/3);
Map<String, Strategy> map=new ConcurrentHashMap<>(size);
strategyMap=Collections.unmodifiableMap(map);
}
}

正例

private List<Strategy> strategyList;
private Map<String, Strategy> strategyMap;

public void afterPropertiesSet() {
if (CollectionUtils.isNotEmpty(strategyList)) {
int size=(int)Math.ceil(strategyList.size()*4.0/3);
Map<String, Strategy> map=new HashMap<>(size);
strategyMap=Collections.unmodifiableMap(map);
}
}

对象&类

禁止使用JSON转换对象

JSON提供把对象转换为JSON字符串、把JSON字符串转为对象的功能,于是被某些人用来转换为对象。这种对象转换方式,虽然在功能上没有问题,但是在性能上却存在问题。

反例

List<UserDO> userDOList=new ArrayList<>();
List<UserVO> userVOList=JSON.parseArray(JSON.toJSONString(userDOList), UserVO.class);

正例

List<UserDO> userDOList=new ArrayList<>();
List<UserVO> userVOList=new ArrayList<>();
for(UserDO userDO:userDOList) {
UserVO vo=new UserVO();
...
}

尽量不使用反射赋值对象

用反射赋值对象,主要优点是节省了代码量,主要缺点却是性能有所下降。

反例

List<UserDO> userDOList=new ArrayList<>();
List<UserVO> userVOList=new ArrayList<>();
for(UserDO userDO:userDOList) {
UserVO vo=new UserVO();
BeanUtils.copyProperties(userDO,vo);
userVOList.add(vo);
}

正例

List<UserDO> userDOList=new ArrayList<>();
List<UserVO> userVOList=new ArrayList<>();
for(UserDO userDO:userDOList) {
UserVO vo=new UserVO();
vo.setId(userDO.getId());
...
userVOList.add(vo);
}

采用Lambda表达式替换内部匿名类

对于大多数刚接触JDK8的同学来说,都会认为Lambda表达式就是匿名内部类的语法糖。实际上,Lambda表达式在大多数虚拟机中采用invokeDynamic指令实现,相对于匿名内部类在效率上会更高一些。

反例

List<User> userList=new ArrayList<>();
Collections.sort(userList,new Comparator<User>() {
@Override
public int compare(User o1, User o2) {
Long userId1=o1.getId();
Long userId2=o2.getId();
return userId1.compareTo(userId2);
}
});

正例

List<User> userList=new ArrayList<>();
Collections.sort(userList,(User o1,user o2)->{
Long userId1=o1.getId();
Long userId2=o2.getId();
return userId1.compareTo(userId2);
});

尽量避免定义不必要的子类

多一个类就需要多一份类加载,所以尽量避免定义不必要的子类。

反例

public static final Map<String, Class> CLASS_MAP=Collections.unmodifiableMap(new HashMap<String, Class>(16){
private static final long serialVersionUID=1L;
{
put("VARCHAR", java.lang.String.class);
}
});

正例

public static final Map<String, Class> CLASS_MAP;
static {
Map<String, Class> classMap=new HashMap<>();
classMap.put("VARCHAR", java.lang.String.class);
CLASS_MAP=Collections.unmodifiableMap(classMap);
}

尽量指定类的final修饰符

为类指定final修饰符,可以让该类不可以被继承。如果指定了一个类为final,则该类所有的方法都是final的,Java编译器会寻找机会内敛所有的final方法。内敛对于提升Java运行效率作用重大,具体可参见Java运行期优化,能够使性能平均提高50%。

反例

public class DateHelper{
...
}

正例

public final class DateHelper{
...
}

注:使用Spring的AOP特性时,需要对Bean进行动态代理,如果Bean类添加了final修饰,会导致异常。

方法

把跟类成员变量无关的方法声明成静态方法

静态方法的好处就是不用生成类的实例就可以直接调用。静态方法不再属于某个对象,而是属于它所在的类。只需要通过其类名就可以访问,不需要再消耗资源去反复创建对象。即便在类内部的私有方法,如果没有使用到类成员变量,也应该声明为静态方法。

反例

public int getMonth(Date date) {
Calendar calendar=Calendar.getInstance();
calendar.setTime(date);
return calendar.get(calendar.MONTH)+1;
}

正例

public static int getMonth(Date date) {
Calendar calendar=Calendar.getInstance();
calendar.setTime(date);
return calendar.get(calendar.MONTH)+1;
}

尽量使用基本数据类型作为方法参数类型,避免不必要的装箱、拆箱和空指针判断

反例

public static double sum(double value1,double value2) {
double double1=Objects.isNull(value1)?0.0D:value1;
double double2=Objects.isNull(value2)?0.0D:value2;
return double1+double2;
}

正例

public static double sum(double value1,double value2) {    
return value1+value2;
}

尽量使用基本数据类型作为方法返回值类型,避免不必要的装箱、拆箱和空指针判断

在JDK类库的方法中,很多方法返回值都采用了基本数据类型,首先是为了避免不必要的装箱和拆箱,其次是为了避免返回值的空指针判断。比如:

  • Collection.isEmpty()
  • Map.size()

反例

public static void main(String[] args) {
UserDO userDO=new UserDO();
boolean isValid=isValid(userDO);
if (Objects.isNull(isValid)&&Objects.isNull(isValid)) {

}
}
public static Boolean isValid(UserDO userDO) {
if (Objects.isNull(userDO)) {
return false;
}
return Boolean.TRUE.equals(userDO.getIsValid());
}

正例

public static void main(String[] args) {
UserDO userDO=new UserDO();
if (isValid(userDO)) {

}
}
public static Boolean isValid(UserDO userDO) {
if (Objects.isNull(userDO)) {
return false;
}
return Boolean.TRUE.equals(userDO.getIsValid());
}

协议方法参数值非空,避免不必要的空指针判断

协议编程,可以@NonNull和@Nullable标注参数,是否遵循全凭调用者自觉。

反例

public static Boolean isValid(UserDO userDO) {
if (Objects.isNull(userDO)) {
return false;
}
return Boolean.TRUE.equals(userDO.getIsValid());
}

正例

public static Boolean isValid(@NonNull UserDO userDO) {
if (Objects.isNull(userDO)) {
return false;
}
return Boolean.TRUE.equals(userDO.getIsValid());
}

协议方法返回值非空,避免不必要的空指针判断

协议编程,可以@NonNull和@Nullable标注参数,是否遵循全凭实现者自觉。

反例

public static void main(String[] args) {
OrderService orderService=null;
List<OrderVO> orderList=orderService.queryUserOrder((long) 5);
}
public interface OrderService{
public List<OrderVO> queryUserOrder(Long userId);
}

正例

public static void main(String[] args) {
OrderService orderService=null;
List<OrderVO> orderList=orderService.queryUserOrder((long) 5);
}
public interface OrderService{
@NonNull
public List<OrderVO> queryUserOrder(Long userId);
}

被调用方法已支持判空处理,调用方法无需再进行判空处理

反例

UserDO userDO = null;
if (StringUtils.isNotBlnk(values)) {
userDO = JSON.parseObject(values, UserDO.class);
}

正例

UserDO userDO = JSON.parseObject(values, UserDO.class);

尽量避免不必要的函数封装

方法调用会引起入栈和出栈,导致消耗更多的CPU和内存,应当尽量避免不必要的函数封装。当然,为了使代码更简洁、更清晰、更易维护,增加一定的方法调用所带来的性能损耗是值得的。

反例

public static void main(String[] args) {
boolean isVip=isVip(User.getVip());
}
public static boolean isVip(boolean isVip) {
return Boolean.TRUE.equals(isVip);
}

正例

public static void main(String[] args) {
boolean isVip=Boolean.TRUE.equals(User.getVip());
}

尽量指定方法的final修饰符

方法指定final修饰符,可以让方法不可以被重写,Java编译器会寻找机会内敛所有final方法。内敛对于提升Java运行效率作用重大,具体可参见Java运行期优化,能够使性能平均提高50%。

注:所有的private方法会隐士地被指定final修饰符,所以无需再为其指定final修饰符。

反例

public class User
{
public int getAge()
{
return 10;
}
}

正例

public class User
{
public final int getAge()
{
return 10;
}
}

注:使用Spring的AOP特性时,需要对Bean进行动态代理,如果方法添加了final修饰,将不会被代理。

表达式

尽量减少方法的重复调用

反例

List<User> userList=new ArrayList<>();
for (int i = 0; i < userList.size(); i++) {
...
}

正例

List<User> userList=new ArrayList<>();
int userLength=userList.size();
for (int i = 0; i < userLength; i++) {
...
}

尽量避免不必要的方法调用

反例

List<User> userList=userDAO.queryActive();
if (isAll) {
userList=userDAO.queryAll();
}

正例

List<User> userList;
if (isAll) {
userList=userDAO.queryAll();
}else {
userList=userDAO.queryActive();
}

尽量使用移位来代替正整数乘除

用移位操作可以极大地提高性能。对于乘除2^n(n为正整数)的正整数计算,可以用移位操作来代替。

反例

int num1=a*4;
int num2=a/4;

正例

int num1=a<<2;
int num2=a>>2;

提取公共表达式,避免重复计算

提取公共表达式,只计算一次值,然后重复利用值。

反例

double distance=Math.sqrt((x2-x1)*(x2-x1)+(y2-y1)*(y2-y1));

正例

double dx=x2-x1;
double dy=y2-y1;
double distance=Math.sqrt(dx*dx+dy*dy);

double distance=Math.sqrt(Math.pow(x2-x1,2)+Math.pow(y2-y1,2));

尽量不在条件表达式中用!取反

使用!取反会多一次计算,如果没有必要则优化掉。

反例

if(!(a>=10)){
....
}else{
....
}

正例

if(a<10){
...
}else{
...
}

对于多常量选择分支,尽量使用switch语句而不是if-else语句

if-else语句,每个if条件语句都要加装计算,知道if条件语句为true为止。switch语句进行了跳转优化,Java采用tableswitch或lookupswitch指令实现,对于多常量选择分支处理效率更高。

经过试验证明:在每个分支出现概率相同的情况下,低于5个分支时if-else语句效率更高,高于5个分支时switch语句效率更高。

反例

if(i==1){
....
}else if(i==2){
...
}else if(i==...){
...
}else{
...
}

正例

switch (i) {
case 1:
...
break;
case 2:
...
break;
case 3:
...
break;
default:
...
break;
}

备注:如果业务复杂,可以采用Map实现策略模式

字符串

尽量不要使用正则表达式匹配

正则表达式匹配效率较低,尽量使用字符串匹配操作。

反例

String source="a::1,b::2,c::3";
String target=source.replaceAll("::", "=");
String[] targets=source.split("::");

正例

String source="a::1,b::2,c::3";
String target=source.replaceAll("::", "=");
String[] targets=StringUtils.split(source,"::");

尽量使用字符替换字符串

字符串的长度不确定,而字符的长度固定为1,查找和匹配的效率自然提高了。

反例

String source="a:1,b:2,c:3";
int index=source.indexOf(":");
String target=source.replace(":", "=");

正例

String source="a:1,b:2,c:3";
int index=source.indexOf(':');
String target=source.replace(':', '=');

尽量使用StringBuilder进行字符串拼接

String是final类,内容不可修改,所以每次字符串拼接都会生成一个新对象。

StringBuilder在初始化时申请了一块内存,以后的字符串拼接都在这块内存中执行,不会申请新内存和生成新对象。

反例

String s = "";
for (int i = 0; i < 10; i++) {
s += i + ",";
}

正例

StringBuilder s = new StringBuilder();
for (int i = 0; i < 10; i++) {
s.append(i).append(",");
}

不要使用""+转换字符串

使用""+进行字符串转换,使用方便但是效率低,建议使用String.valueOf。

反例

int i = 123;
String s = "" + i;

正例

int i = 123;
String s = String.valueOf(i);

数组

不要使用循环拷贝数组,尽量使用System.arraycopy拷贝数组

推荐使用System.arraycopy拷贝数组,也可以使用Arrays.copyOf拷贝数组。

反例

int[] source = new int[] { 1, 2, 3, 4, 5 };
int[] targets = new int[source.length];
for (int i = 0; i < source.length; i++) {
targets[i] = source[i];
}

正例

int[] source = new int[] { 1, 2, 3, 4, 5 };
int[] targets = new int[source.length];
System.arraycopy(source, 0, targets, 0, targets.length);

集合转换为类型T数组时,尽量传入空数组T[0]

将集合转换为数组有2种方式:toArray(new T[n])和toArray(new T[0])。在旧的Java版本中,建议使用toArray(new T[n]),因为创建数组时锁需的反射调用非常慢。在OpenJDK6后,反射调用是内在的,使得性能得以提高,toArray(new T[0])比toArray(new T[n])效率更高。

此外,toArray(new T[n])比toArray(new T[0])多获取一次列表大小,如果计算列表大小耗时过长,也会导致toArray(new T[n])效率降低。

反例

List<Integer> integerList=Arrays.asList(1,2,3,4,5);
Integer[] integers=integerList.toArray(new Integer[integerList.size()]);

正例

List<Integer> integerList=Arrays.asList(1,2,3,4,5);
Integer[] integers=integerList.toArray(new Integer[0]); //勿用new Integer[]{}

建议:集合应该提供一个toArray(Class<T> clazz)方法,避免无用的空数组初始化(new T[0]);

集合转换为Object数组时,尽量使用toArray()方法

转换Object数组时,没有必要使用toArray[new Object[0]],可以直接使用toArray()。避免了类型的判断,也避免了空数组的申请,所以效率会更高。

反例

List<? extends Object> objectList=Arrays.asList(1,"2",3);
Object[] objects=objectList.toArray(new Object[0]);

正例

List<? extends Object> objectList=Arrays.asList(1,"2",3);
Object[] objects=objectList.toArray();

集合

初始化集合时,尽量指定集合大小

Java集合初始化时都会指定一个默认大小,当默认大小不再满足数据需求时就会扩容,每次扩容的时间复杂度有可能是0(n)。所以,尽量指定预知的集合大小,就能避免或减少集合的扩容次数。

反例

List<UserDO> userdoList=new ArrayList<UserDO>();
Set<Long> userSet=new HashSet<Long>();
Map<Long, UserDO> userMap=new HashMap<Long, UserDO>();
List<UserVO> userList=new ArrayList<UserVO>();
for (UserDO userDO:userdoList) {
userSet.add(userDO.getId());
userMap.put(userDO.getId(), userDO);
userList.add(transUser(userDO));
}

正例

List<UserDO> userdoList = new ArrayList<UserDO>();
int userSize = userdoList.size();
Set<Long> userSet = new HashSet<Long>(userSize);
Map<Long, UserDO> userMap = new HashMap<Long, UserDO>((int) Math.ceil(userSize * 4.0 / 3));
List<UserVO> userList = new ArrayList<UserVO>(userSize);
for (UserDO userDO : userdoList) {
userSet.add(userDO.getId());
userMap.put(userDO.getId(), userDO);
userList.add(transUser(userDO));
}

不要使用循环拷贝集合,尽量使用JDK提供的方法拷贝集合

JDK提供的方法可以一步指定集合的容量,避免多次扩容浪费时间和空间。同时,这些方法的底层也是调用System.arraycopy方法实现,进行数据的批量拷贝效率更高。

反例

List<UserDO> user1List=new ArrayList<UserDO>();
List<UserDO> user2List=new ArrayList<UserDO>();
List<UserDO> userList=new ArrayList<UserDO>(user1List.size()+user2List.size());
for (UserDO user1:user1List) {
userList.add(user1);
}
for (UserDO user2:user2List) {
userList.add(user2);
}

正例

List<UserDO> user1List=new ArrayList<UserDO>();
List<UserDO> user2List=new ArrayList<UserDO>();
List<UserDO> userList=new ArrayList<UserDO>(user1List.size()+user2List.size());
userList.addAll(user1List);
userList.addAll(user2List);

尽量使用Arrays.asList转化数组为列表

原理与“不要使用循环拷贝集合,尽量使用JDK提供的方法拷贝集合”类似。

反例

List<String> typeList=new ArrayList<String>(8);
typeList.add("Short");
typeList.add("Integer");
typeList.add("Long");
String[] names=new String[] {};
List<String> nameList=new ArrayList<String>();
for (String name:names) {
nameList.add(name);
}

正例

List<String> typeList=Arrays.asList("Short","Integer","Long");
String[] names=new String[] {};
List<String> nameList=Arrays.asList();
nameList.addAll(Arrays.asList(names));

直接迭代需要使用的集合

直接迭代需要使用的集合,无需通过其他操作获取数据。

反例

Map<Long, UserDO> userMap=new HashMap<Long, UserDO>();
for (long userId:userMap.keySet()) {
UserDO userDO=userMap.get(userId);
}

正例

Map<Long, UserDO> userMap=new HashMap<Long, UserDO>();
for (Map.Entry<Long, UserDO> userEntry:userMap.entrySet()) {
Long userId=userEntry.getKey();
UserDO userDO=userEntry.getValue();
}

不要使用size方法检测空,必须使用isEmpty方法检测空

使用size方法来检测空逻辑上没有问题,但使用isEmpty方法使得代码更易读,并且可以获得更好的性能。任何isEmpty方法实现的时间复杂度都是0(1),但是某些size方法实现的时间复杂度有可能是0(n)。

反例

List<UserDO> userList=new ArrayList<UserDO>();
if (userList.size()==0) {

}
Map<Long, UserDO> userMap=new HashMap<Long, UserDO>();
if (userMap.size()==0) {

}

正例

List<UserDO> userList=new ArrayList<UserDO>();
if (userList.isEmpty()) {

}
Map<Long, UserDO> userMap=new HashMap<Long, UserDO>();
if (userMap.isEmpty()) {

}

非随机访问的List,尽量使用迭代代替随机访问

对于列表,可分为随机访问和非随机访问两类,可以用是否实现RandomAccess接口判断。随机访问列表,直接通过get获取数据不影响效率。而非随机访问列表,通过get获取数据效率极低。

反例

List<UserDO> userList=new ArrayList<UserDO>();
int size=userList.size();
for (int i = 0; i < size; i++) {

}

正例

List<UserDO> userList=new ArrayList<UserDO>();
for (UserDO userDO:userList) {

}

其实,不管列表支不支持随机访问,都应该使用迭代进行遍历。

尽管使用HashSet判断值存在

在Java集合类库中,List的contains方法普通时间复杂度是0(n),而HashSet的时间复杂度为0(1)。如果需要频繁调用contains方法查找数据,可以先将List转换成HashSet。

反例

List<Long> userIdList=new ArrayList<Long>();
List<UserDO> userList=new ArrayList<UserDO>();
for (UserDO userDO:userList) {
if (userIdList.contains(userDO.getId())) {

}
}

正例

Set<Long> userIdSet=new HashSet<Long>();
List<UserDO> userList=new ArrayList<UserDO>();
for (UserDO userDO:userList) {
if (userIdSet.contains(userDO.getId())) {

}
}

避免先判断存在再进行获取

如果需要先判断存在再进行获取,可以直接获取并判断空,从而避免了二次查找操作。

反例

public UserVO transUser(UserDO userDO,Map<Long, RoleDo> roleMap) {
UserVO userVO=new UserVO();
userVO.setId(userDO.getId());
if (roleMap.containsKey(userDO.getId())) {

}
return null;
}

正例

public UserVO transUser(UserDO userDO,Map<Long, RoleDo> roleMap) {
UserVO userVO=new UserVO();
userVO.setId(userDO.getId());
RoleDo role=roleMap.get(userDO.getId());
if (Objects.nonNull(role)) {

}
return null;
}

异常

直接捕获对应的异常

直接捕获对应的异常,避免用instanceof判断,效率更高代码更简洁。

反例

try {

} catch (Exception e) {
if (e instanceof IIOException) {
System.out.println("保存数据IO异常");
}else {
System.out.println("保存数据其他异常");
}
}

正例

try {

} catch (IIOException e) {
System.out.println("保存数据IO异常");
} catch (Exception e) {
System.out.println("保存数据其他异常");
}

尽量避免在循环中捕获异常

当循环体抛出异常后,无需循环继续执行时,没有必要在循环体中捕获异常。因为,过多的捕获异常会降低程序执行效率。

反例

public Double sum(List<String> valueList) {
double sum=0.0D;
for (String value:valueList) {
try {
sum+=Double.parseDouble(value);
} catch (Exception e) {
return null;
}
}
return sum;
}

正例

public Double sum(List<String> valueList) {
double sum = 0.0D;
try {
for (String value : valueList) {
sum += Double.parseDouble(value);
}
} catch (Exception e) {
return null;
}
return sum;
}

禁止使用异常控制业务流程

相对于条件表达式,异常的处理效率更低。

反例

public static boolean isValid(UserDO user) {
try {
return Boolean.TRUE.equals(user.getId());
} catch (Exception e) {
return false;
}
}

正例

public static boolean isValid(UserDO user) {
if (Objects.isNull(user)) {
return false;
}
return Boolean.TRUE.equals(user.getId());
}

缓冲区

初始化时尽量指定缓冲区大小

初始化时,指定缓冲区的预期容器大小,避免多次扩容浪费时间和空间。

反例

StringBuffer buffer=new StringBuffer();
StringBuilder buider=new StringBuilder();

正例

StringBuffer buffer=new StringBuffer(1024);
StringBuilder buider=new StringBuilder(1024);

尽量重复使用同一缓冲区

针对缓冲区,Java虚拟机需要花时间生成对象,还要花时间进行垃圾回收处理。所以,尽量重复利用缓冲区。

反例

StringBuffer buider1=new StringBuffer(128);
buider1.append("abcdef");
StringBuffer buider2=new StringBuffer(128);
buider2.append("abcdef");

正例

StringBuffer buider1=new StringBuffer(128);
buider1.append("abcdef");
buider1.setLength(0);
buider1.append("abcdef");

注:其中,使用setLength方法让缓冲区重新从0开始。

尽量设计使用同一缓冲区

为了提高程序运行效率,在设计上尽量使用同一缓冲区。

反例

public static String toXml(UserDO user) {
StringBuilder buider=new StringBuilder(128);
buider.append("<UserDO>");
buider.append(toXml(user.getId()));
buider.append("</UserDO>");
return buider.toString();
}

public static String toXml(Long value) {
StringBuilder builder=new StringBuilder(128);
builder.append("<Long>");
builder.append(value);
builder.append("</Long>");
return builder.toString();
}
//调用
UserDO user=new UserDO();
String xml=toXml(user);

正例

public static String toXml(StringBuilder buider,UserDO user) {
buider.append("<UserDO>");
buider.append(toXml(buider,user.getId()));
buider.append("</UserDO>");
return buider.toString();
}

public static String toXml(StringBuilder builder,Long value) {
builder.append("<Long>");
builder.append(value);
builder.append("</Long>");
return builder.toString();
}
//调用
StringBuilder builder=new StringBuilder(128);
UserDO user=new UserDO();
String xml=toXml(builder,user);

去掉每个转化方法中缓冲区申请,申请一个缓冲区给每个转换方法使用。从时间上来说,节约了大量缓冲区的申请释放时间;从空间上来说,节约了大量缓冲区的临时存储空间。

尽量使用缓冲流减少IO操作

  • BufferedReader
  • BufferedWriter
  • BufferedInputStream
  • BufferedOutputStream
  • ....

可以大幅减少IO次数并提升IO速度。

反例

try {
FileInputStream inputStream = new FileInputStream("a.txt");
FileOutputStream outputStream = new FileOutputStream("b.txt");
int size = 0;
byte[] temp = new byte[1024];
while ((size = inputStream.read(temp)) != -1) {
outputStream.write(temp, 0, size);
}
} catch (IIOException e) {
System.out.println(e.getMessage());
}

正例

try {
BufferedInputStream inputStream = new BufferedInputStream(new FileInputStream("a.txt"));
BufferedOutputStream outputStream = new BufferedOutputStream(new FileOutputStream("b.txt"));
int size = 0;
byte[] temp = new byte[1024];
while ((size = inputStream.read(temp)) != -1) {
outputStream.write(temp, 0, size);
}
} catch (IIOException e) {
System.out.println(e.getMessage());
}

其中,可以根据实际情况手动指定缓冲流的大小,把缓冲流的缓冲作用发挥到最大。

线程

在单线程中,尽量使用非线程安全类

使用非线程安全类,避免了不必要的同步开销。

反例

StringBuffer buffer=new StringBuffer(128);
buffer.append("abcd");

正例

StringBuilder buffer=new StringBuilder(128);
buffer.append("abcd");

在多线程中,尽量使用线程安全类

使用线程安全类,比自己实现的同步代码更简洁更高效。

反例

private volatile int count=0;
public void access() {
synchronized (this) {
count++;
}
}

正例

private final AtomicInteger countInteger=new AtomicInteger(0);
public void access() {
countInteger.incrementAndGet();
}

尽量减少同步代码块范围

在一个方法中,可能只有一小部分的逻辑是需要同步控制的,如果同步控制了整个方法会影响执行效率。所以,尽量减少同步代码块的范围,只对需要进行同步的代码进行同步。

反例

private volatile int count=0;
public synchronized void access() {
count++;
//...非同步操作
}

正例

private volatile int count=0;
public void access() {
synchronized (this) {
count++;
}
//...非同步操作
}

尽量合并为同一同步代码块

同步代码块是有性能开销的,如果确定可以合并为同一同步代码块,就应该尽量合并同一同步代码快。

反例

//处理单一订单
public synchronized void handleOrder(OrderDO order) {

}
//处理所有订单
public void handleOrder(List<OrderDO> orderList) {
for (OrderDO order:orderList) {
handleOrder(order);
}
}

正例

// 处理单一订单
public void handleOrder(OrderDO order) {

}

// 处理所有订单
public synchronized void handleOrder(List<OrderDO> orderList) {
for (OrderDO order : orderList) {
handleOrder(order);
}
}

尽量使用线程池减少线程开销

多线程中两个必要的开销:线程的创建和上下文切换。采用线程池,可以尽量地避免这些开销。

反例

public void executeTask(Runnable runnable) {
new Thread(runnable).start();
}

正例

private static final ExecutorService EXECUTOR_SERVICE=Executors.newFixedThreadPool(10);
public static void executeTask(Runnable runnable) {
EXECUTOR_SERVICE.execute(runnable);
}

标签:正例,java,正反,userDO,编程,List,反例,new,public
From: https://blog.51cto.com/u_13288309/6026145

相关文章

  • 【Java项目推荐】值得写到简历上的项目--黑马点评
    优惠卷秒杀​​前言​​​​优惠券秒杀​​​​实现优惠券秒杀下单​​​​超卖问题​​​​一人一单​​​​分布式锁​​​​redis中加锁的一些特殊情况​​​​手动实现......
  • 云原生干货|一文读懂函数编程及其工作原理
    啥是函数编程我先用通俗的大白话给大家解释一下函数(Functions,FunctionasaService,FaaS)的几个要点,这样看后面示例时才不会容易懵。函数就是在云平台体系内运行的、与......
  • Java锁
    Java锁悲观锁概述认为自己在使用数据的时候一定有别的线程跟修改数据,因此在获取数据的时候会先加锁,确保数据不会被别的线程修改。synchronized关键字和Lock的实现类都......
  • Java I/O经典代码实例
    packageIO;importjava.io.*;publicclassFileDirectoryDemo{publicstaticvoidmain(String[]args){//如果没有指定参数,则缺省为当前目录。if(args.length==......
  • java整合hudi-client 0.11.1
    1.Linux部署hudi环境(1)安装maven-3.5.4、jdk1.8环境#解压maven,重命名tar-xfapache-maven-3.5.4-bin.tar.gz-C/usr/local/mvapache-maven-3.5.4maven#解压jdk,重命名ta......
  • java中map根据value排序并返回有序的key集合
    /***map根据value倒序排**@parammap*@return*/privateList<String>sortMap(Map<String,Integer>map){List<Map.......
  • java学习之—方法
    一、方法的概念方法:一段特定功能的代码。需要多次执行的代码,可以使用一对大括号将其包裹,给这段代码一个名字,每次使用这段代码的时候,就可以使用该名字进行访问。二、......
  • 注释 标识符和关键字 java
    单行注释// 多行注释/*......*/ 文件注释/**......*/ 关键字  标识符●所有的标识符都应该以字母(A-Z或者a-z),美元符($)、或者下划线()开始●首字符......
  • JavaScript 前端大文件上传
    ​对于大文件的处理,无论是用户端还是服务端,如果一次性进行读取发送、接收都是不可取,很容易导致内存问题。所以对于大文件上传,采用切块分段上传,从上传的效率来看,利用多线程......
  • Java继承
    extends的意思是“扩展”,子类是父类的扩展Java中只有单继承,没有多继承继承是类和类之间的关系继承关系的两个类,一个是子类,一个是父类子类继承父类的所有方......