web-dev-qa-db-ja.com

notifyDataSetChanged後にAndroid ListViewが更新されない

私の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を更新しません。アプリを再起動した後でも、更新されたアイテムは表示されません。 ItemAdapterBaseAdapterを拡張します

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;
    }
}

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

110
Coder

onResumeItemFragmentメソッドを見てください:

@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)ItemFragmentからアイテムprivate List<Item> items;を削除します-アダプター内でのみ参照する必要があります

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());
}
219
Tomasz Gawel

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

ListViewを更新するには、リストデータ、つまり項目を受け入れるItemAdapterクラスにrefresh()を追加します

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);**
}
22
Santhosh

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

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

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

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

8

リストビューを更新する場合、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();
}

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

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

4
hardartcore

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

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

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()  
        }
});
4
AlexGo

これを試して

@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
Gautami

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
user2996950
adpter.notifyDataSetInvalidated();

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

2
Som
adapter.setNotifyDataChanged()

トリックを行う必要があります。

1
Hitman

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

this.notifyDataSetChanged();

の代わりに:

adapter.notifyDataSetChanged();

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

0
Jachu

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

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

アダプター内のrefresh()関数

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

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

getActivity().runOnUiThread(new Runnable() { 
    @Override
    public void run() {
          adapter.refresh()  
    }
});
0
Dévan Coetzee