Android Array Adapter with ArrayList and ListView not updating when the arraylist is changed

Question

I have an android app with a screen that comprises of a ListView, which I am using to display a list of devices. These devices are held in an array.

I am trying to use

Answer 1

While the accepted answer solves the problem, the explanation of why is incorrect, and since this is an important concept I thought I'd attempt to clarify.

Slartibartfast's explanation that notifyDataSetChanged() only works when add, insert,remove, or clear is called on the adapter is incorrect.

That explanation is true of the setNotifyOnChange() method, which if set to true (as it is by default) will automatically call notifyDataSetChanged() when any of those four actions occur.

I think the poster confused the two methods. notifyDatasetChanged() itself does not have those restrictions. It just tells the adapter that the list it is looking at has changed, and it does not matter how the change to the list actually happened.

While I can't see the source code for your createNewDeviceList(), I would guess your problem came from the fact that you had the adapter referencing the original list you created, and then you created a new list in createNewDeviceList(), and since the adapter was still pointing to the old list it could not see the changes.

The solution slartibartfast mentioned works because it clears the adapter and specifically adds the updated list to that adapter. Thus you don't have the problem of your adapter pointing to the wrong place.

Hope this helps someone!

Answer 2

your method addDevice is causing an endless loop. Dont call a method from itself like you are doing here :

deviceList = addDevice();

Answer 3

For an ArrayAdapter, notifyDataSetChanged only works if you use the add, insert, remove, and clear functions on the Adapter.

1. Use clear to clear the adapter - arrayAdapter.clear()
2. Use Adapter.addAll and add the newly formed list - arrayAdapter.addAll(deviceList)
3. Call notifyDataSetChanged

Alternatives:

1. Repeat this step after new devicelist is formed - but this is redundant

   arrayAdapter = new ArrayAdapter<Device>(this, android.R.layout.simple_list_item_1, deviceList);
   

2. Create your own class derived from BaseAdapter and ListAdapter that gives you more flexibility. This is most recommended.