背景
年初开发了一个项目，项目中有这样一个需求，需要每隔一秒钟向服务器发起一次状态信息查询指令，在这个过程中偶尔会主动发起一次状态信息修改的指令（PS：因为服务器的处理能力有限，必须间隔一秒才能发送命令，否则指令将不会被执行）。本来不是什么大问题，但是由于开发人员从事android开发时间较短，多线程用得也不多，所以看似简单的问题，最后使得整个项目出现了较大的性能问题。下面主要讲讲如何一步步的优化过程。（本文适合初学者，大牛可以绕道前行）。
优化前
以下是功能部分代码
public void sendOrderToServer(final byte[] data) {
Thread thread = new Thread(new Runnable()
{
@Override
public void run()
{
order_queue.offer(data);//将指令放入队列中
if (null != order_queue && order_queue.size() != 0) {
synchronized (lock) {
while (order_queue.size() != 0) {
try {
byte[] orderData = order_queue.poll();//取出队首指令
int dataLength = orderData.length;
if (dataLength > 0) {
//发送指令到服务器
}
try {
Thread.sleep(1000);//等待一秒钟，然后执行下一条指令
} catch (InterruptedException e) {
e.printStackTrace();
}
} catch (IOException e) {
e.printStackTrace();
}
}
}
}
}
});
thread.start();
}
代码很简单，相信大家都能看懂是什么意思，阅读下来好像也没什么太大问题。模拟器运行一遍，没有问题，各项数据显示正常。然后信心满满安装到真机上，拍拍手，欧✌️，搞定。可惜理想是美好的，现实是残酷的，分分钟啪啪啪打脸。app运行不到一分钟，整个界面就开始卡顿了，操作屏幕毫无反应。这是为啥呢 ？？？
然后认真阅读了一下代码，发现里面用到了synchronized (lock)同步锁，笔者为什么要这样写，估计是考虑到指令需要间隔一秒钟才能发送，又要保证指令是按序的。由于app每次发送命令都需要调用sendOrderToServer方法，并创建新的线程。这就出现了一个问题，除了第一个线程能获取到同步锁，其他的线程都无法获取到，因为order_queue队列永远不会为空。那么其他线程都相当于死锁，都无法执行后面的代码，也就不能释放占用的系统资源。这样线程不停的创建，系统资源被一点点消耗掉，最后造成界面卡顿。既然问题找到了，那就好办了，其实优化方法有很多，下面只是其中一种。
优化后
public void sendOrderToServer(final byte[] data) {
order_queue.offer(data);
}
private class WriteThread extends Thread {
@Override
public void run() {
super.run();
while (true) {
if (null != order_queue && order_queue.size() != 0) {
byte[] orderData = order_queue.poll();
if (null != orderData) {
int dataLength = orderData.length;
if (dataLength > 0) {
//发送指令到服务器
}
}
try {
Thread.sleep(1000);
} catch (InterruptedException e) {
e.printStackTrace();
}
} catch (IOException e) {
e.printStackTrace();
}
}
}
}
}
只创建一个WriteThread对象，里面运行一个while (true)的循环，发送指令时只需要调用sendOrderToServer方法，往order_queue队列中加入指令就好。
总结
1、项目中使用多线程加锁时，需要注意不要造成死锁。
2、如果需要创建大量的线程，请使用线程池来管理，减少线程创建的开销，以及资源的浪费。
