notifyDataSetChanged后,Android ListView不刷新


116

我的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);
        }
    }
}

我的ListView

<?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对象,为什么要在下面的空行中对其进行测试?
2013年

代码不会引发异常,我使用调试进行了检查。所有方法执行均无错误。是的,这是一个愚蠢的错误。
编码员

Answers:


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 2014年

7
为什么必须要clear()并再次添加项目?那不是目的notifyDataSetChanged()吗?
Phil Ryan


1
感谢@tomsaz Gawel,您的swapItems确实对我有很大帮助,我不知道为什么我的adapter.notifydatasetchanged不起作用,因为我传递的“列表”也已更新,即使我已经通过打印日志对其进行了检查,请您解释一下我概念
Kimmi Dhingra

1
这个答案是正确的。问题是适配器的Item ArrayList没有被更新。这意味着您可以调用notifydatasetchanged,直到您的脸变成蓝色为止而没有任何效果。适配器使用相同的数据集更新您的数据集,因此没有更改。此答案中发布的解决方案的另一种选择可能是更干净的:adapter.items = items; adapter.notifyDataSetChanged();
李丽

23

您正在onResume()中将重载的项目分配给全局变量项,但这不会反映在ItemAdapter类中,因为它有自己的实例变量,称为“ items”。

为了进行刷新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);**
}

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

请注意,我在使用items.addAll()而不是adapter.addAll()。让适配器对更改做出反应的唯一事情是notifyDataSetChanged。适配器完全看不到更改的原因是该items列表与适配器使用的列表相同。
贾斯汀·布雷特弗勒

4

如果已经设置了适配器,则再次设置将不会刷新列表视图。相反,首先检查listview是否具有适配器,然后调用适当的方法。

我认为在设置列表视图时创建适配器的新实例不是一个好主意。而是创建一个对象。

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个片段(选项卡),问题中的代码与包含列表视图的片段之一有关。将项目传递到的原因ItemAdapter是我要为行着色,并且列表视图显示多个数据项目。我已经发布了适配器的代码。
编码员

您需要使用“ this”将填充列表的代码放入示例代码中。而不是“活动”
AlexGo 2013年

在某些情况下,当您在其他线程中运行notifyDataSetChanged()时,它不会更新,因此上述解决方案在某些情况下是正确的。
Ayman Al-Absi 2014年

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。在第一次适配器初始化中,将项目填充到列表中,然后将适配器设置为listview。之后,在每次更改项目时,您都必须清除主要值,List<Item> items然后再用新项目填充该值并调用notifySetDataChanged();

这就是它的工作方式:)。


谢谢回复。我做了您提到的更改。我已经发布了我的代码。它仍然不起作用。现在,当添加新项目时,它甚至不显示列表视图。
编码员

我已经更改了代码。观察到的奇怪现象是,DB中的项没有得到更新
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()  
    }
});

这确实使我有所作为,因为更新是通过不同的线程通过网络进行的。
查克(Chuck)

0

尝试这样:

this.notifyDataSetChanged();

代替:

adapter.notifyDataSetChanged();

你必须notifyDataSetChanged()ListView不给适配器类。


当然不会,如果活动通过列表视图进行扩展,这是唯一的机会
cmario
By using our site, you acknowledge that you have read and understand our Cookie Policy and Privacy Policy.
Licensed under cc by-sa 3.0 with attribution required.