Android Java Firebase Querying objects into ArrayList

Question

So I have been trying to download data from Firebase into an ArrayList:

 public void updateOnce(){
    animalList = new ArrayList<>();
    Query query = mDatabase.orderByChild("id");
    query.addListenerForSingleValueEvent(new ValueEventListener() {
        @Override
        public void onDataChange(DataSnapshot dataSnapshot) {
            if (dataSnapshot.exists()) {
                for (DataSnapshot messageSnapshot: dataSnapshot.getChildren()) {
                    SpottedAnimal pet = messageSnapshot.getValue(SpottedAnimal.class);
                    animalList.add(pet);
                }
            }
        }

        @Override
        public void onCancelled(DatabaseError databaseError) {

        }
    });
}

Debugger shows that the Object SpottedAnimal pet does get created and then inserted into the animalList that is a global ArrayList. But once I call the method:

public ArrayList<SpottedAnimal> getList(){
    return animalList;
}

The animalList comes out empty on the other side.

 FirebaseGetList() animalListHook = new FirebaseGetList(); 
 animalListHook.updateOnce();
 ArrayList<SpottedAnimal> animalList = animalListHook.getList();

Firebase API are asynchronous. Please read this blog to understand more about why this is. medium.com/google-developers/… — Doug Stevenson
– Doug Stevenson, Commented Apr 16, 2018 at 23:21
Possible duplicate of How to return dataSnapshot value as a result of a method? — Alex Mamo
– Alex Mamo, Commented Apr 23, 2018 at 16:34
Please see the duplicate to understand why you have this behaviour and how you can solve it. — Alex Mamo
– Alex Mamo, Commented Apr 23, 2018 at 16:35

davidxxx · Accepted Answer · 2018-04-16 19:17:21Z

1

onDataChange() is an asynchronous call.
So you have to consume the list as the event that adds an Animal object is triggered.
The client class should implement a Observer interface or a similar thing and as you instantiate FirebaseGetList, you should pass this (the client object):

In the client class :

public interface Notifiable {
   void update();
}

public class ClientClass implements Notifiable {

    public void foo(){
      FirebaseGetList animalListHook = new FirebaseGetList(); 
      animalListHook.updateOnce(this);
      ArrayList<SpottedAnimal> animalList = animalListHook.getList();
   }

  @Override
  public void update(){
      // use the animalList here
  }
}

in FirebaseGetList :

 @Override
    public void onDataChange(DataSnapshot dataSnapshot) {
        if (dataSnapshot.exists()) {
            for (DataSnapshot messageSnapshot: dataSnapshot.getChildren()) {
                SpottedAnimal pet = messageSnapshot.getValue(SpottedAnimal.class);
                animalList.add(pet);                    
            }
            this.update(); // method to notify 
        }
    }

Note that using global variables is probably a design issue and should be corrected.
For example the update() method could pass the List of Animal as a parameter.

edited Apr 16, 2018 at 19:17

answered Apr 16, 2018 at 19:12

davidxxx

132k23 gold badges231 silver badges228 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

BryceSoker Over a year ago

Thank you, this got me an idea to what I should do, I'll try and get it to work and if it does work I'll mark it as done, it might take a while tho.

Collectives™ on Stack Overflow

Android Java Firebase Querying objects into ArrayList

1 Answer 1

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related