[移动开发]没考虑多任务的Android程序

最近一直在忙着对团队的Android程序进行重构和升级，但发现原来的Android程序压根没有考虑多任务问题。典型代码如下：

上传照片代码（伪代码，说明问题为主）；原来代码的含义是让上传图片不超过4个，否则出错。

if(mPicList.size() != 4) {
    //上传代码函数
    httpPostPic(); //伪代码    
} else {
    Toast.showText(this, "最多上传四张"); //返回不能上传的对话框
}

原来的工程师这样想：如果图片上传小于4张，则继续上传；否则无法上传。

原来的工程师还沾沾自喜：大部分时候，这部分代码还正确。

但在复杂环境中，这部分代码肯定出错；因为，如果有网络延时，上传照片成为多任务，则照片可能会超过4个，然后，照片就可以永远上传。

修改方法：

1）加上synchronized等同步关键字；

2）size 条件改为 <= 4 。

修改了很多这样的代码，整个程序稳定性大规模提高。

PS：

? ? ? 工程师不仅仅要完成问题，而且要考虑自己的设计的适应性和兼容性，考虑风险的可能存在。不能考虑多种场景，只考虑任务完成的，不是一个合格的工程师。