notifyDataSetChangedの後でAndroid ListViewが更新されない


116

My ListFragmentコード

public class ItemFragment extends ListFragment {

    private DatabaseHandler dbHelper;
    private static final String TITLE = "Items";
    private static final String LOG_TAG = "debugger";
    private ItemAdapter adapter;
    private List<Item> items;


    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) {
        View view = inflater.inflate(R.layout.item_fragment_list, container, false);        
        return view;
    }

    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.setHasOptionsMenu(true);
        super.onCreate(savedInstanceState);
        getActivity().setTitle(TITLE);
        dbHelper = new DatabaseHandler(getActivity());
        items = dbHelper.getItems(); 
        adapter = new ItemAdapter(getActivity().getApplicationContext(), items);
        this.setListAdapter(adapter);

    }



    @Override
    public void onResume() {
        super.onResume();
        items.clear();
        items = dbHelper.getItems(); //reload the items from database
        adapter.notifyDataSetChanged();
    }

    @Override
    public void onListItemClick(ListView l, View v, int position, long id) {
        super.onListItemClick(l, v, position, id);
        if(dbHelper != null) { //item is edited
            Item item = (Item) this.getListAdapter().getItem(position);
            Intent intent = new Intent(getActivity(), AddItemActivity.class);
            intent.putExtra(IntentConstants.ITEM, item);
            startActivity(intent);
        }
    }
}

マイリストビュー

<?xml version="1.0" encoding="utf-8"?>
<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
    android:layout_width="fill_parent"
    android:layout_height="fill_parent"
    android:orientation="vertical" >

    <ListView
        android:id="@android:id/list"
        android:layout_width="fill_parent"
        android:layout_height="wrap_content" />

</LinearLayout>

しかし、これはを更新しませんListView。アプリを再起動した後でも、更新されたアイテムは表示されません。私のItemAdapter拡張BaseAdapter

public class ItemAdapter extends BaseAdapter{

    private LayoutInflater inflater;
    private List<Item> items;
    private Context context;

    public ProjectListItemAdapter(Context context, List<Item> items) {
        super();
        inflater = LayoutInflater.from(context);
        this.context = context;
        this.items = items;

    }

    @Override
    public int getCount() {
        return items.size();
    }

    @Override
    public Object getItem(int position) {
        return items.get(position);
    }

    @Override
    public long getItemId(int position) {
        return position;
    }

    @Override
    public View getView(int position, View convertView, ViewGroup parent) {
        ItemViewHolder holder = null;
        if(convertView == null) {
            holder = new ItemViewHolder();
            convertView = inflater.inflate(R.layout.list_item, parent,false);
            holder.itemName = (TextView) convertView.findViewById(R.id.topText);
            holder.itemLocation = (TextView) convertView.findViewById(R.id.bottomText);
            convertView.setTag(holder);
        } else {
            holder = (ItemViewHolder) convertView.getTag();
        }
        holder.itemName.setText("Name: " + items.get(position).getName());
        holder.itemLocation.setText("Location: " + items.get(position).getLocation());
        if(position % 2 == 0) {                                                                                 
            convertView.setBackgroundColor(context.getResources().getColor(R.color.evenRowColor));
        } else {    
            convertView.setBackgroundColor(context.getResources().getColor(R.color.oddRowColor));
        }
        return convertView;
    }

    private static class ItemViewHolder {
        TextView itemName;
        TextView itemLocation;
    }
}

誰かが助けてくれますか?


2
データベース操作が正しく機能しているかどうかを確認するためにテストしましたか?アダプターはどのように見えますか?また、adapter参照用のonオブジェクトを作成する場合、1行下のnullをテストするのはなぜですか?
Luksprog 2013年

コードが例外をスローしないため、デバッグを使用してチェックしました。すべてのメソッドはエラーなしで実行されました。ええ、それは愚かな間違いです。
Coder

回答:


229

あなたのonResume方法を見てくださいItemFragment

@Override
public void onResume() {
    super.onResume();
    items.clear();
    items = dbHelper.getItems(); // reload the items from database
    adapter.notifyDataSetChanged();
}

呼び出す前に更新したのnotifyDataSetChanged()は、アダプターのフィールドではなくprivate List<Item> items;、フラグメントの同じように宣言されたフィールドです。アダプタは、アダプタを作成したときに渡したアイテムのリストへの参照を(フラグメントのonCreateなどで)保存します。(変更の数の意味で)最短ですが、コードを期待どおりに動作させるエレガントな方法は、単に行を置き換えることです:

    items = dbHelper.getItems(); // reload the items from database

    items.addAll(dbHelper.getItems()); // reload the items from database

よりエレガントなソリューション:

1)アイテムprivate List<Item> items;を削除しますItemFragment-アダプターでのみアイテムへの参照を保持する必要があります

2)onCreateを次のように変更します。

@Override
public void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    super.setHasOptionsMenu(true);
    getActivity().setTitle(TITLE);
    dbHelper = new DatabaseHandler(getActivity());
    adapter = new ItemAdapter(getActivity(), dbHelper.getItems());
    setListAdapter(adapter);
}

3)ItemAdapterにメソッドを追加します。

public void swapItems(List<Item> items) {
    this.items = items;
    notifyDataSetChanged();
}

4)onResumeを次のように変更します。

@Override
public void onResume() {
    super.onResume();
    adapter.swapItems(dbHelper.getItems());
}

dbHelper全体をアダプタに移動する方がクリーンではないでしょうか?したがってadapter.swapItems();、呼び出すだけで、アダプターが処理を行いますdbHelper.getItems()。しかしとにかく答えをありがとう:)
Ansgar

7
なぜclear()してアイテムを再度追加する必要があるのですか?それはまさに目的ではありませんnotifyDataSetChanged()か?
フィルライアン


1
@tomsaz Gawelに感謝します。あなたのswapItemsは本当に私を助けてくれます。私が渡した「リスト」も更新されているので、なぜ私のadapter.notifydatasetchangedが機能しないのかわかりません。コンセプト
Kimmi Dhingra 2015

1
この答えは正しいです。問題は、アダプターのアイテムArrayListが更新されていなかったことです。つまり、顔が青くなるまでnotifydatasetchangedを呼び出すことができ、何の影響もありません。アダプターは同じデータセットでデータセットを更新しているため、変更はありません。この回答で投稿された解決策のもう1つの代替案は、次のとおりです。adapter.items = items; adapter.notifyDataSetChanged();
Ray Li

23

リロードされたアイテムをのグローバル変数アイテムに割り当てていますが、 'items'という独自のインスタンス変数があるためonResume()ItemAdapterクラスには反映されません。

リフレッシュするためListViewにリフレッシュを()を追加しItemAdapter、リストのデータは、IEの項目を受け付けクラス

class ItemAdapter
{
    .....

    public void refresh(List<Item> items)
    {
        this.items = items;
        notifyDataSetChanged();
    } 
}

onResume()次のコードで更新

@Override
public void onResume()
{
    super.onResume();
    items.clear();
    items = dbHelper.getItems(); //reload the items from database
    **adapter.refresh(items);**
}

1
これはまさに正しいです。アダプターのコンストラクターはアイテムが渡されることを期待しますが、外部クラスのフィールドを更新するだけです。
LuxuryMode

こんにちはサントッシュ。あなたは、同様の問題を見てみることができます:stackoverflow.com/questions/35850715/...

8

onResume()でこの行を変更します

items = dbHelper.getItems(); //reload the items from database

items.addAll(dbHelper.getItems()); //reload the items from database

問題は、アダプターに新しいアイテムのリストを決して伝えないことです。新しいリストをアダプターに渡したくない場合(そうでない場合があるようです)、の後に使用items.addAllしますclear()。これにより、アダプターが参照しているのと同じリストを確実に変更できます。


それは混乱だadapter.clear()ビューがリフレッシュする必要があることを実現するためにアダプタを強制しませんが、adapter.add()またはadapter.addAll()ありません。でも答えてくれてありがとう!
w3bshark 2015

私が使用items.addAll()していて、adapter.addAll()ではなかったことに注意してください。アダプターが変更に反応できるようにしているのは、だけですnotifyDataSetChanged。アダプターが変更をまったく認識する理由は、itemsリストがアダプターが使用しているリストと同じだからです。
Justin Breitfeller、2015

4

アダプターが既に設定されている場合、再度設定してもリストビューは更新されません。代わりに、最初にリストビューにアダプタがあるかどうかを確認してから、適切なメソッドを呼び出します。

リストビューの設定中にアダプターの新しいインスタンスを作成することは、あまり良い考えではないと思います。代わりに、オブジェクトを作成します。

BuildingAdapter adapter = new BuildingAdapter(context);

    if(getListView().getAdapter() == null){ //Adapter not set yet.
     setListAdapter(adapter);
    }
    else{ //Already has an adapter
    adapter.notifyDataSetChanged();
    }

また、UIスレッドで更新リストを実行してみてください。

activity.runOnUiThread(new Runnable() {         
        public void run() {
              //do your modifications here

              // for example    
              adapter.add(new Object());
              adapter.notifyDataSetChanged()  
        }
});

UIスレッドの実装方法がわかりません。私のメインアクティビティには3つのフラグメント(タブ)があり、質問のコードはリストビューを含むフラグメントの1つに関連しています。アイテムを渡す理由はItemAdapter、行に色を付けたいためであり、リストビューには複数のデータアイテムが表示されます。アダプターのコードを掲載しました。
コーダー

"this"を使用して、リストに入力するコードをサンプルコードに追加する必要があります。「アクティビティ」の代わり
AlexGo 2013年

別のスレッドでnotifyDataSetChanged()を実行しても更新されない場合があるため、上記の解決策が適切な場合があります。
Ayman Al-Absi 14

4

リストビューを更新するかどうかに関係なくonResume()onCreate()または他の関数でリストビューを更新するかどうかは関係ありません。最初に理解する必要があるのは、アダプターの新しいインスタンスを作成する必要がないことです。データを含む配列。アイデアはこれに似ています:

private ArrayList<String> titles;
private MyListAdapter adapter;
private ListView myListView;

@Override
public void onCreate(Bundle savedInstanceState){
    super.onCreate(savedInstanceState);
    setContentView(R.layout.main_activity);

    myListView = (ListView) findViewById(R.id.my_list);

    titles = new ArrayList<String>()

    for(int i =0; i<20;i++){
        titles.add("Title "+i);
    }

    adapter = new MyListAdapter(this, titles);
    myListView.setAdapter(adapter);
}


@Override
public void onResume(){
    super.onResume();
    // first clear the items and populate the new items
    titles.clear();
    for(int i =0; i<20;i++){
        titles.add("New Title "+i);
    }
    adapter.notifySetDataChanged();
}

したがって、その回答に応じて、で同じを使用する必要List<Item>がありますFragment。最初のアダプター初期化では、リストに項目を入力し、アダプターをリストビューに設定します。その後、アイテムを変更するたびに、メインから値をクリアしてから、List<Item> items新しいアイテムを再度入力してを呼び出す必要がありますnotifySetDataChanged();

それはそれがどのように機能するかです:)。


返信いただきありがとうございます。あなたが言ったように私は変更を行いました。コードを投稿しました。それでも動作しません。新しいアイテムが追加されたときにリストビューも表示されなくなりました。
コーダー、2013年

コードを変更しました。奇妙なことに、DBでアイテムが更新されない
Coder

このスレッドは、データベースのためであるstackoverflow.com/questions/14555332/...
Coderの

3

AlexGoからの返事が私にトリックをもたらしました:

getActivity().runOnUiThread(new Runnable() {
        @Override
        public void run() {
         messages.add(m);
         adapter.notifyDataSetChanged();
         getListView().setSelection(messages.size()-1);
        }
});

リストの更新は、更新がGUIイベントからトリガーされ、UIスレッド内に入る前に機能しました。

ただし、別のイベント/スレッドからのリストを更新すると(つまり、アプリの外部からの呼び出し)、更新はUIスレッドに含まれず、getListViewの呼び出しが無視されます。上記のようにrunOnUiThreadを使用して更新を呼び出すと、うまくいきました。ありがとう!!


3

これを試して

@Override
public void onResume() {
super.onResume();
items.clear();
items = dbHelper.getItems(); //reload the items from database
adapter = new ItemAdapter(getActivity(), items);//reload the items from database
adapter.notifyDataSetChanged();
}

3
adpter.notifyDataSetInvalidated();

onPause()Activityクラスのメソッドでこれを試してください。



1

リストがアダプタ自体に含まれている場合、リストを更新する関数を呼び出すと、 notifyDataSetChanged()

UIスレッドからこの関数を実行すると、私にとってはうまくいきました。

refresh()アダプター内の機能

public void refresh(){
    //manipulate list
    notifyDataSetChanged();
}

次に、UIスレッドからこの関数を実行します

getActivity().runOnUiThread(new Runnable() { 
    @Override
    public void run() {
          adapter.refresh()  
    }
});

更新が別のスレッドを介してネットワーク経由で行われたため、これは確かに私に違いをもたらしました。
チャック

0

このようにしてみてください:

this.notifyDataSetChanged();

の代わりに:

adapter.notifyDataSetChanged();

アダプタクラスでnotifyDataSetChanged()ListViewなく、する必要があります。


もちろん、そうではありません。アクティビティがリストビューによって拡張される唯一のチャンスです
cmario 2016
弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.