6

カスタムリストアダプタを持つListActivityを作成しました。onCreateの実行時に、リストはContentProviderから更新されます。また、アプリを実行すると開始されるサービスがあり、最初にContentProviderを更新してから、コンテンツが更新されたことをブロードキャストで送信します。
私のListActivityはブロードキャストを受信し、私のListViewを更新しようとします。私の問題は、ListViewに通知されずにListViewアダプタのデータが変更されるという断続的なエラーが発生することです。私はnotifyDataSetChanged()更新した直後のリストアダプタのメソッド。発生しているように見えるのは、更新するサービスからブロードキャストを受信したときにonCreateを最初に呼び出した後、リストがまだ更新中であるため、最初の実行から更新が完了する前にListViewを更新しようとします。これは意味がありますか?これが私のコードの一部です。

注:サービスは正常に機能しており、新しいデータを取得してContentProviderを更新します。また、更新されると、アクティビティでブロードキャストを取得します。

@Override
public void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    ctx = this;
    getPrefs();
    setContentView(R.layout.main);

    // Setup preference listener
    preferences = PreferenceManager.getDefaultSharedPreferences(this);
    preferences.registerOnSharedPreferenceChangeListener(listener);


    // Setup report list adapter
    ListView nzbLv = (ListView) findViewById(R.id.report_list);
    nzbla = new NZBReportListAdaptor(ctx);
    getReports();
    nzbla.setListItems(report_list);            
    nzbLv.setAdapter(nzbla);        
    // Broadcast receiver to get notification from NZBService to update ReportList
    registerReceiver(receiver,
            new IntentFilter(NZBService.BROADCAST_ACTION));

    startService(new Intent(ctx, NZBService.class));
}

@Override
public void onResume() {
    super.onResume();
    timerHandler.resume();      
new updateSabQueue().execute();
    //updateList();
}

@Override
public void onPause() {
    super.onPause();
    timerHandler.pause();
    unregisterReceiver(receiver);
}


private BroadcastReceiver receiver = new BroadcastReceiver() {
    public void onReceive(Context context, Intent intent) {
        Toast.makeText(ctx, "NZBService broadcast recieved", Toast.LENGTH_SHORT).show();
        updateReportList();
    }
};


private void updateReportList() {
    new updateReportList().execute();
}



private class updateReportList extends AsyncTask<Void, Void, Boolean> {

    /* (non-Javadoc)
     * @see android.os.AsyncTask#onPreExecute()
     * Show progress dialog
     */
    protected void onPreExecute() {
    }

    /* (non-Javadoc)
     * @see android.os.AsyncTask#doInBackground(Params[])
     * Get new articles from the internet
     */
    protected Boolean doInBackground(Void...unused) {
        getReports();
        return true;
    }

    /**
     * On post execute.
     * Close the progress dialog
     */
    @Override
    protected void onPostExecute(Boolean updated) {
        if (updated) {
            Log.d(TAG, "NZB report list adapter updated");
            synchronized(this) {
                nzbla.setListItems(report_list);            
            }
            Log.d(TAG, "NZB report list notified of change");
            nzbla.notifyDataSetChanged();                           
        }
    }
}

この質問に答えたので、更新されたコードを投稿して、遭遇する可能性のある他の人を支援します。

@Override
  public void onCreate(Bundle savedInstanceState) {
  super.onCreate(savedInstanceState);
  ctx = this;
    getPrefs();
setContentView(R.layout.main);

    // Setup preference listener
    preferences = PreferenceManager.getDefaultSharedPreferences(this);
    preferences.registerOnSharedPreferenceChangeListener(listener);

    // Setup report list adapter
    ListView nzbLv = (ListView) findViewById(R.id.report_list);
    nzbla = new NZBReportListAdaptor(ctx);
    report_list.addAll(getReports());
    nzbla.setListItems(report_list);            
    nzbLv.setAdapter(nzbla);        
    // Broadcast receiver to get notification from NZBService to update ReportList
    registerReceiver(receiver,
            new IntentFilter(NZBService.BROADCAST_ACTION));

    startService(new Intent(ctx, NZBService.class));
}


private class updateReportList extends AsyncTask<Void, Void, ArrayList<Report>> {

    /* (non-Javadoc)
     * @see android.os.AsyncTask#onPreExecute()
     * Show progress dialog
     */
    protected void onPreExecute() {
    }

    /* (non-Javadoc)
     * @see android.os.AsyncTask#doInBackground(Params[])
     * Get new articles from the internet
     */
    protected ArrayList<Report> doInBackground(Void...unused) {
        return getReports();
    }

    /**
     * On post execute.
     * Close the progress dialog
     */
    @Override
    protected void onPostExecute(ArrayList<Report> updated) {
        nzbla.setListItems(updated);            
        nzbla.notifyDataSetChanged();                           
    }
}


private ArrayList<Report> getReports() {
    ArrayList<Report> reports = new ArrayList<Report>();
    ContentResolver r = getContentResolver();
    Cursor c = r.query(NZBReportProvider.CONTENT_URI, null, null, null, NZBReportProvider.ARTICLE_KEY_ROWID + " DESC");
    startManagingCursor(c);
    Log.d(TAG, "NZBReport cursor.getCount=" + c.getCount());
    int title = c.getColumnIndex(NZBReportProvider.ARTICLE_KEY_TITLE);
    int desc = c.getColumnIndex(NZBReportProvider.ARTICLE_KEY_DESCRIPTION);
    int cat = c.getColumnIndex(NZBReportProvider.ARTICLE_KEY_CAT);
    int size = c.getColumnIndex(NZBReportProvider.ARTICLE_KEY_SIZE);
    int link = c.getColumnIndex(NZBReportProvider.ARTICLE_KEY_LINK);
    int catid = c.getColumnIndex(NZBReportProvider.ARTICLE_KEY_CATID);
    int date = c.getColumnIndex(NZBReportProvider.ARTICLE_KEY_DATE_ADDED);
    int group = c.getColumnIndex(NZBReportProvider.ARTICLE_KEY_GROUP);

    if (c.getCount() > 0) {
        c.moveToFirst();
        do {
            URL url = null;
            try {
                url = new URL(c.getString(link));
            } catch (MalformedURLException e) {
                // TODO Auto-generated catch block
                e.printStackTrace();
            }
            reports.add(new Report(c.getString(title), url, c.getString(desc), c.getString(cat), c.getString(date), c.getString(size), c.getInt(catid), c.getString(group)));               
        } while (c.moveToNext());                   
    }
    return reports;
}
4

2 に答える 2

11

同期ブロックが不要になるように、UIスレッドでアダプターデータのすべての更新を行う必要があります。また、AsyncTask実行するたびに新しく作成される同期は役に立たないためです。

もう1つの問題は、notifyDataSetChangedの外部を呼び出していることですAdapter。メソッドの最後で呼び出す必要がありますsetListItems。UIスレッドで実行されているため、エラーが発生することはありませんが、そのように呼び出すことはできません。

getReportsメソッドがのバッキングストアを変更していないことを確認する必要がありますAdapter。別のスレッドで実行されているため、Adapterアクセスできるものを変更することもできません。ロックで保護されていても。メソッドで行う必要があるdoInBackgroundのは、更新のリストや新しいリストなどを生成し、それに渡して、UIスレッドonPostExecuteのに新しいデータをコミットすることです。Adapterしたがって、getReports関数が変更report_listされてAdapterいて、参照があるreport_list場合は、それが間違っています。getReports新しいものを作成し、UIスレッドでの作成が完了report_listしたら、それをに戻す必要があります。Adapter

繰り返しになりますが、変更できるのはデータのみでAdapterあり、その後ListViewはUIスレッドにもアクセスできます。同期/ロックを使用しても、この要件は変わりません。

于 2011-01-08T23:55:12.560 に答える
-1

サービスからUIリストビューを更新する場合は、サービスのonDestroyでnotifyDataSetChanged()を呼び出す必要があります

メインアクティビティからアダプタを静的にし、そのアダプタ名を呼び出します。notifyDataSetChanged()

このような

@Override
       public void onDestroy() {

               if (MainActivity.isInFront == true) {
                       if (MainActivity.adapter != null)
                               MainActivity.adapter.notifyDataSetChanged();
                       MainActivity.listView.setAdapter(MainActivity.adapter);
               }
}               
于 2013-06-21T13:49:23.373 に答える