一道关于优化的面试题，一家互联网公司的面试题

近期遇到一个有意思的面试题，关于优化的。

系统中有一个方法func，每10秒钟就会调用一次，性能有问题，请优化一下。主要考虑执行效率。

方法例如以下。

LinkedList<Date> dates = new LinkedList<Date>();

……

public String func(String bits)
{
Random random = new Random(System.currentTimeMillis());

String str = "";
for(int i = 0; i < (bits.length() > 1000 ? bits.length() : 1000); ++i)
{
SimpleDateFormat sdf = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss");
str += sdf.format(dates.get(i));
}
return str;

}

能够看到，这种方法的作用就是将dates的元素依照List的顺序以一定的格式拼接在一起，拼接的元素总数以bits.length() > 1000 ? bits.length() : 1000得到。

我的优化方案：因为for循环里总是在做String的链接，这样会产生非常多小的对象，而循环次数也不少，easy导致垃圾回收。String改为StringBuffer，这样就能避免产生小对象。看代码。

public String func1(String bits)
{
StringBuffer sb = new StringBuffer();
SimpleDateFormat sdf = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss");
for(int i = 0; i < (bits.length() > 1000 ?

bits.length() : 1000); ++i)
{
sb.append(sdf.format(dates.get(i)));
}
return sb.toString();
}

上面的就是我的面试答案。哎，仅仅有更好，没有最好啊。

进一步，循环至少迭代1000次。那么，在某个步骤准备好一个字符串（datemat1000）。

可假设dates发生变化，比方remove前面的对象。就会导致结果不对。

再进一步，假设考虑到了remove的时候，变化这个datamat1000。那结果就正确。

有没有更好的方案呢，假设bits非常长（对应的。dates肯定也长），还是会迭代非常多次，效率还会有问题。

啊哈，灵光一闪。是否能预先将dates全部的元素预先拼接在一起。在dates add和remove的时候将拼接的StringBuffer做出对应变化，在func里依据bits的长度从StringBuffer里复制字符串来。

这样就不会再每10秒执行的func里有太多动作。

缺点：remove（object o）这种方法里须要又一次构建字符串。

请看代码：

MyLinkedList dates = new MyLinkedList();

//      ……
public String func2(String bits)
{
int bitlen = bits.length() > 1000 ? bits.length() : 1000;
return dates.getDatesSb().substring(0, bitlen * dates.SDF_LEN);
}

class MyLinkedList<Date> extends LinkedList<Date>{

private StringBuffer datesSb = new StringBuffer();
private SimpleDateFormat sdf = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss");
public final int SDF_LEN = 19;
@Override
public boolean add(Date date)
{
boolean isok = super.add(date);
if(!isok)
return isok;
dateSb.append(sdf.format(date));
return isok;
}
private void reInit()
{
StringBuffer sb = new StringBuffer();
for(int i = 0; i < super.size(); i++)
{
sb.append(sdf.format(super.get(i)));
}
dateSb = sb;
}
@Override
public boolean remove(Object o)
{
boolean isok = super.remove(o);
if(!isok)
return isok;
reInit();
return isok;
}
@Override
public Date remove(int index)
{
Date date = super.remove(index);
if(date != null)
{
dateSb.delete(index* SDF_LEN, (index + 1) * SDF_LEN);
}
return date;
}
//……
public StringBuffer getDatesSb() {
return datesSb;
}

}