前言
看到这样的标题,第一反应是不是那个“渣渣辉”在电脑屏幕中挥舞着大砍刀,听着古天乐说着,“这是你从未玩过的全新版本”。当然,这并不是标题党,既然“这是你从未玩过的全新版本”,那么“是兄弟就来看我的文章”。(不是兄弟也请留步,嘤嘤嘤)
这可能是你从未见过如此优雅的写法背景
在对公司一个几百年没有更新维护旧项目进行功能开发的时候,由于当初开发这个项目的人早已各奔前程,也没有留下什么可视性的文档,所以只能从一些代码以及注释中尽可能地摸索这个项目的系统架构以及对应的业务功能,这无疑是一件十分头疼的事情(看别人的代码和看几个月前自己的代码就像是闻SHI一样)。
看了几遍下来,大多数的代码写的都不堪直视,甚至连基本的格式化代码也没有,但是其中有一段的代码却引起了我的注意,也是因为这段代码才引起了我的思考,也是就有了这篇文章。
到底是什么样的代码呢
话不多说,直接上代码:
new AbstractSpinBusiness()
{
@Override
protected boolean handle()
{
CompanyProfile updateProfile = getProfileForUpdateConf( staff, attrMap ); return(companyProfileDao.updateCompanyProfileConf( updateProfile ) > 0);
}
}.spin( 5 );
简单介绍下,这段的意思是执行handle()里面的方法,并且设置了5次重试限制。<br />
然后我们再来看看AbstractSpinBusiness这个抽象类<br />
AbstractSpinBusiness.class
java
import org.apache.log4j.Logger;
public abstract class AbstractSpinBusiness {
private Logger logger = Logger.getLogger( this.getClass() );
public void spin( int times )
{
for ( int i = 0; i < times; i++ )
{
if ( handle() )
{
return;
}
logger.debug( String.format( "spin重试%s", i ) );
}
} /**
* 执行主体
* @return true:执行成功,不需要重试 false:执行失败
*/
protected abstract boolean handle();
}
看到这里,好像也不过只是对AbstractSpinBusiness当中的handle的实现,并且运用在spin方法当中。可能细心的同学已经发现了这使用了模板的设计模式,如果能够发现,那么给你点个赞;如果没能发现,问题也不大,因为我在这里也并非是来讲模板设计模式。当然,模板设计模式也是十分重要且优秀的写法,在抽象业务、架构当中用的是遍地开花!
那么这个写法很优雅吗
回想一下我们最早学习JDBC的时候,我们需要手动获取Connection,需要将参数设置到PreparedStatement当中,执行之后需要将对象再包装成我们想要的数据格式,这一系列的操作下来,我们可以发现,真正跟业务相贴切的就只是那一条SQL而已,其他的工作都只是一个辅助工作。
这可能是你从未见过如此优雅的写法所以,像MyBatis、Hibernate等这类ORM框架才会孕育而生。开发人员在使用这些框架的时候,只需要关心我们自己的业务。例如MyBatis,我们只需要写好业务的SQL以及对应的Mapper,那么整合到业务的Service当中就可以,而其他的操作已经封装在框架中无感知执行。<br />
回归到我们刚开始的那段代码当中。不难发现这简单的代码中,其实蕴含着同样的思想,那就是将业务代码和非业务代码独立开来。试想,如果有其他地方也需要使用类似的重试逻辑,那么是否又需要写一套重试的代码呢?<br />
这可能是你从未见过如此优雅的写法扩展
沿着这样的思路,在最近一次优化中使用了相同的写法。<br />目前系统中存在一些慢SQL,SQL的本身是比较简单,EXPLAIN执行计划当中也没有什么问题,就只是单纯的rows比较多。问题在于业务中产生大量的参数,导致in里面的数据太多,造成SQL执行效率变低。那么比较直接的优化就是控制这里的参数,进行分批处理。(当然这里就不考虑网上说的子查询或者是eq_range_index_dive_limit参数之类)
要是你会怎么写
要是没有前面的引子,我想大家的代码大概会是这样
import com.google.common.collect.Lists;
import java.util.List;
public class Test {
public static final int PARTITION_SIZE = 1000;
public static void main( String[] args )
{
/* 这里就模拟是业务参数 */
List<Long> paramIds = Lists.newArrayList( 1L, 2L, 3L );
/* 进行分隔 */
List<List<Long> > partitionParamIds = Lists.partition( paramIds, PARTITION_SIZE );
List<Object> resultList = Lists.newArrayListWithExpectedSize( paramIds.size() );
partitionParamIds.forEach( partition - > {
/* 执行具体的DAO操作,当然这里也是模拟 */
resultList.addAll( new ObjectDao().getList( partition ) );
} );
System.out.println( resultList.size() );
}
}
class ObjectDao {
/* 都说了是模拟模拟,不要挑刺了 */
public List<Object> getList( List<Long> paramIds )
{
List<Object> resultList = Lists.newArrayList();
for ( Long paramId : paramIds )
{
resultList.add( paramId );
}
return(resultList);
}
}
其实这样写本身也是比较简单整洁的,但是有没有发现,这里切片的动作和具体的业务代码还是混杂在一起,做不到一定意义上的职责分明。那是不是可以提供一个辅助类来做这些事情呢,所以我改写下。
这可能是你从未见过如此优雅的写法import com.google.common.collect.Lists;
import java.util.List;
public class Test {
public static void main( String[] args )
{
/* 这里就模拟是业务参数 */
List<Long> paramIds = Lists.newArrayList( 1L, 2L, 3L );
List<Object> resultList = new CommonDoPartition<>().partitionToQuery( paramIds,
partition - > new ObjectDao().getList( partition ) );
System.out.println( resultList.size() );
}
}
class ObjectDao {
/* 都说了是模拟模拟,不要挑刺了 */
public List<Object> getList( List<Long> paramIds )
{
List<Object> resultList = Lists.newArrayList();
for ( Long paramId : paramIds )
{
resultList.add( paramId );
}
return(resultList);
}
}
这里用到了一个CommonDoPartition类,我们来看下它是怎么实现的<br />
CommonDoPartition.java
import com.google.common.collect.Lists;
import org.apache.commons.collections.CollectionUtils;
import org.apache.log4j.Logger;
import java.util.List;
import java.util.function.Function;
public class CommonDoPartition<T> {
private final static Logger logger = Logger.getLogger( CommonDoPartition.class );
public static final int PARTITION_SIZE = 1000;
public < T, R > List<R> partitionToQuery( int partitionSize, List<T> all, Function<List<T>, List<R> > function )
{
if ( CollectionUtils.isEmpty( all ) )
{
logger.warn( "no data to query" );
return(Lists.newArrayList() );
}
List<List<T> > partitions = Lists.partition( all, partitionSize ); List<R> result = Lists.newArrayList(); for ( List<T> list : partitions )
{
List<R> resultList = function.apply( list ); if ( !CollectionUtils.isEmpty( resultList ) )
{
result.addAll( resultList );
}
}
return(result);
}
public < T, R > List<R> partitionToQuery( List<T> all, Function<List<T>, List<R> > function )
{
return(this.partitionToQuery( PARTITION_SIZE, all, function ) );
}
}
可以看到,分片的操作放到了这个公共方法当中,于是业务方只需要用lambda表达式支持他想要的业务逻辑就可以了,非业务性质工作都可以由这个工具类来完成。
再来扩展一下
既然实现了查询操作,同样也可以搞一下执行操作
这可能是你从未见过如此优雅的写法public void partitionToDo( int partitionSize, List<T> all, Consumer<List<T> > consumer )
{
if ( CollectionUtils.isEmpty( all ) )
{
logger.warn( "no data to consume" );
return;
}
List<List<T> > partitions = Lists.partition( all, partitionSize );
for ( List<T> list : partitions )
{
consumer.accept( list );
}
}
public void partitionToDo( List<T> all, Consumer<List<T> > consumer )
{
this.partitionToDo( PARTITION_SIZE, all, consumer );
}
public < T, R > void partitionToQueryAndDo( int partitionSize, List<T> all, Function<List<T>, List<R> > function, Consumer<List<R> > consumer )
{
if ( CollectionUtils.isEmpty( all ) )
{
logger.warn( "no data to consume" );
return;
}
List<List<T> > partitions = Lists.partition( all, partitionSize );
List<R> resultList;
for ( List<T> list : partitions )
{
resultList = function.apply( list );
consumer.accept( resultList );
}
}
public < T, R > void partitionToQueryAndDo( List<T> all, Function<List<T>, List<R> > function, Consumer<List<R> > consumer )
{
this.partitionToQueryAndDo( PARTITION_SIZE, all, function, consumer );
}
这里的partitionToDo是分批去执行某些任务,partitionToQueryAndDo是结合了之前的分批查询某些数据,并且对这些数据进行操作。这些都是可以组合起来的例子。
结尾
当然,本人也能力有限,也不见得能将大多数的场景进行抽象、组合、归纳,只是觉得在平时的开发中,光写业务光写代码,缺少了自己的思考以及那些优秀的设计思路和理念的话,总归还是无法达到自我提升的程度。本文也只是我在平时工作中的一些小小思考,也还算是一点小小的总结,也仅供大家参考。
这可能是你从未见过如此优雅的写法
网友评论