Android AsyncTask中的ConcurrentModificationException

时间:2016-04-08 12:46:13

标签: java android arraylist android-asynctask concurrentmodification

我有一个奇怪的问题,我希望我能解释得很好。我的应用程序有两个活动 - MainActivity和SearchActivity。我在MainActivity上有一个按钮,它触发从设备上的数据库上传到我的Web服务器上的远程数据库。如果我在第一次启动应用程序时单击按钮,没问题,工作正常。如果我切换到SearchActivity,不做任何事情,然后切换回来,然后尝试按钮,应用程序崩溃并发生ConcurrentModificationException。 我有一个AsyncTask,它发送本地数据库的内容(已经从数据库中拉出并通过参数作为ArrayList发送到线程)。我花了几个小时调试这个,但仍无法解决它的问题。任何建议都将不胜感激。

这是按下按钮时触发的代码,从单独的Databaser线程请求数据库的内容

Button btnRemoteSync = (Button)findViewById(R.id.btnSync);
    btnRemoteSync.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View v) {
            Intent startUpload = new Intent(getString(R.string.broadcast_search_database));
            startUpload.putExtra("type-id",1);
            LocalBroadcastManager.getInstance(getApplicationContext()).sendBroadcast(startUpload);
        }
    });

这是BroadcastReceiver中的代码,它从数据库中获取每个响应,并将它们添加到自定义ResponseObjects的ArrayList中。当数据库线程发送bssid值为DONE时,将启动AsyncTask,并将ArrayList作为参数传入。

@Override
    public void onReceive(Context context, Intent intent) {
        String bssid = intent.getStringExtra(getString(R.string.data_bssid));
       if (bssid.equals("DONE")) {
            RemoteDatabaseUploader rdb = new RemoteDatabaseUploader(getApplicationContext());
            rdb.execute(databases);
       } else {
           databases.add(new ResponseObject(getApplicationContext(),
                   bssid,
                   intent.getStringExtra(getString(R.string.data_ssid)),
                   intent.getStringExtra(getString(R.string.data_capabilities)),
                   intent.getIntExtra(getString(R.string.data_level), 0),
                   intent.getIntExtra(getString(R.string.data_frequency), 0),
                   intent.getStringExtra(getString(R.string.data_timestamp)),
                   intent.getDoubleExtra(getString(R.string.data_latitude), 0),
                   intent.getDoubleExtra(getString(R.string.data_longitude), 0)));

       }
    }

下面是AsyncTask的doInBackground代码

@Override
protected Integer doInBackground(ArrayList<ResponseObject>... params) {
    ArrayList<ResponseObject> entries = params[0];
    try {
        URL url = new URL(insertURL);
        for (Iterator<ResponseObject> it = entries.iterator(); it.hasNext();) {
            ResponseObject ro = it.next(); // THIS IS WHERE THE EXCEPTION REFERENCES IN THE DEBUG OUTPUT
            HttpURLConnection urlConnection = (HttpURLConnection)url.openConnection();

            urlConnection.setRequestMethod("POST");
            urlConnection.setRequestProperty("USER-AGENT", "Mozilla/5.0");
            urlConnection.setRequestProperty("ACCEPT-LANGUAGE", "en-US,en;0.5");
            urlConnection.setDoOutput(true);
            String postParams = "bssid=" + ro.BSSID
                    + "&ssid=" + ro.SSID
                    + "&capabilities=" + ro.CAPABILITIES
                    + "&level=" + String.valueOf(ro.LEVEL)
                    + "&frequency=" + String.valueOf(ro.FREQUENCY)
                    + "&timestamp=" + ro.TIMESTAMP
                    + "&lat=" + String.valueOf(ro.LAT)
                    + "&long=" + String.valueOf(ro.LON);
            DataOutputStream wr = new DataOutputStream(urlConnection.getOutputStream());
            wr.writeBytes(postParams);
            wr.flush();
            wr.close();
            Log.d("RemoteDatabase : ", "Post sent " + ro.BSSID + " || " + String.valueOf(urlConnection.getResponseCode()));
        }
    } catch (MalformedURLException e) {
        e.printStackTrace();
    } catch (ProtocolException e) {
        e.printStackTrace();
    } catch (IOException e) {
        e.printStackTrace();
    }

    entries.clear();

    return null;
}

<小时/> 编辑 - 我似乎已经将问题追溯到另一​​部分代码,其中点击按钮发送广播。该按钮肯定只发送一次(使用Log.d检查),但数据库中的接收是两次接收。现在试着解决这个问题。

2 个答案:

答案 0 :(得分:2)

已经有一段时间了,但我意识到我忘了添加我是如何解决这个问题的,以防万一这是对其他人的帮助。

我发现这里的问题实际上是某些Android设备发送了多个广播副本。我正在使用HTC手机进行测试,显然由于某种原因他们发送了2份所有广播。我的代码工作的方式是我在广播中产生一个线程,这导致2个相同的线程在相同的数据上运行。当这些完成后,他们每个人都发送他们的&#34;完成&#34;广播,导致其中4个被主线程接收。完全乱七八糟 我最终不得不为每个广播添加一个唯一的ID令牌并在接收端记录这些值,因此如果两次收到相同的ID,则第二次不会采取任何操作。

答案 1 :(得分:1)

请在异步任务的onPreExecute()方法中插入一个进度条,并在onPostExecute()中将其关闭。我认为完成异步任务需要花费太多时间,并且在完成之前再次点击按钮异步任务。