public List<String> getContactsFromFirebase(){
FirebaseDatabase.getInstance().getReference().child("Users")
.addListenerForSingleValueEvent(new ValueEventListener() {
@Override
public void onDataChange(DataSnapshot dataSnapshot) {
for (DataSnapshot snapshot : dataSnapshot.getChildren()) {
Users user = snapshot.getValue(Users.class);
assert user != null;
String contact_found = user.getPhone_number();
mContactsFromFirebase.add(contact_found);
Log.i("Test", mContactsFromFirebase.toString());
}
}
@Override
public void onCancelled(DatabaseError databaseError) {
}
});
return mContactsFromFirebase;
}
我似乎无法找到错误。在上面的代码中,当我调用日志时,我从mContactsFromFirebase
获取值,但getContactsFromFirebase()
方法返回一个空列表。你能帮帮我吗?
答案 0 :(得分:7)
数据是从Firebase异步加载的。由于从服务器获取数据可能需要一些时间,因此主要Android代码会继续,Firebase会在数据可用时调用onDataChange
。
这意味着当你return mContactsFromFirebase
时它仍然是空的。最简单的方法是查看几个日志语句:
System.out.println("Before attaching listener");
FirebaseDatabase.getInstance().getReference().child("Users")
.addListenerForSingleValueEvent(new ValueEventListener() {
@Override
public void onDataChange(DataSnapshot dataSnapshot) {
System.out.println("In onDataChange");
}
@Override
public void onCancelled(DatabaseError databaseError) {
throw databaseError.toException(); // don't ignore errors
}
});
System.out.println("After attaching listener");
运行此代码时,将打印:
在附加监听器之前
附加监听器后
在onDataChange
这可能不是您期望输出的顺序。正如您在之后可以看到行之后,回调在onDataChange
之前被调用。这就解释了为什么你返回的列表是空的,或者(更正确地说)当你返回它时它是空的并且只是稍后填充。
有几种方法可以处理这种异步加载。
最简单的解释是将所有返回列表的代码放入 onDataChange
方法。这意味着此代码仅在加载数据后执行。最简单的形式:
public void onDataChange(DataSnapshot dataSnapshot) {
for (DataSnapshot snapshot : dataSnapshot.getChildren()) {
Users user = snapshot.getValue(Users.class);
assert user != null;
String contact_found = user.getPhone_number();
mContactsFromFirebase.add(contact_found);
System.out.println("Loaded "+mContactsFromFirebase.size()+" contacts");
}
}
但是有更多方法,包括使用自定义回调(类似于Firebase自己的ValueEventListener
):
public interface UserListCallback {
void onCallback(List<Users> value);
}
现在,您可以将此接口的实现传递给getContactsFromFirebase
方法:
public void getContactsFromFirebase(final UserListCallback myCallback) {
databaseReference.child(String.format("users/%s/name", uid)).addListenerForSingleValueEvent(new ValueEventListener() {
@Override
public void onDataChange(DataSnapshot dataSnapshot) {
for (DataSnapshot snapshot : dataSnapshot.getChildren()) {
Users user = snapshot.getValue(Users.class);
assert user != null;
String contact_found = user.getPhone_number();
mContactsFromFirebase.add(contact_found);
System.out.println("Loaded "+mContactsFromFirebase.size()+" contacts");
}
myCallback.onCallback(mContactsFromFirebase);
}
@Override
public void onCancelled(DatabaseError databaseError) {
throw databaseError.toException();
}
});
}
然后像这样称呼它:
getContactsFromFirebase(new UserListCallback() {
@Override
public void onCallback(List<Users> users) {
System.out.println("Loaded "+users.size()+" contacts")
}
});
这并不像同步加载数据那么简单,但这样做的好处是它可以在不阻塞主线程的情况下运行。
此主题之前已经讨论过很多,所以我建议您查看其中的一些问题:
Task
API和Firebase数据库的一种很酷但复杂的方式)