Question

我在我创建的应用程序中遇到了大量内存泄漏。我创建了一个非常简单的应用程序来重现问题。此应用程序仅引用FirebaseDatabase并设置ChildEventListener。当用户单击该按钮时，它会向数据库添加一条记录，并启动一个与System.gc（）有关的新活动。

多次按下该按钮将导致Leak Canary生成转储。

MainActivity.java：

public class MainActivity extends AppCompatActivity {

private FirebaseDatabase firebaseDatabase;
private DatabaseReference dbRef;

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);


    firebaseDatabase = FirebaseDatabase.getInstance();
    dbRef = firebaseDatabase.getReference("leak");
    dbRef.addChildEventListener(new ChildEventListener() {
        @Override
        public void onChildAdded(DataSnapshot dataSnapshot, String previousChildName) {
        }

        @Override
        public void onChildChanged(DataSnapshot dataSnapshot, String previousChildName) {
        }

        @Override
        public void onChildRemoved(DataSnapshot dataSnapshot) {

        }

        @Override
        public void onChildMoved(DataSnapshot dataSnapshot, String previousChildName) {
        }

        @Override
        public void onCancelled(DatabaseError databaseError) {
        }
    });

    findViewById(R.id.btn_leak).setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View view) {
            dbRef.child(UUID.randomUUID().toString()).setValue("Yes");
            Intent leakIntent = new Intent(getApplicationContext(), LeakActivity.class);
            startActivity(leakIntent);
        }
    });

}

}

LeakActivity.java：

public class LeakActivity extends AppCompatActivity {

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.leak);

        System.gc();
    }
}

由于发布限制，泄漏金丝雀日志为here.

我在代码中做错了什么，或者这与Firebase有关？

编辑： @ qbix的答案似乎有效。对于其他人，这是MainActivity.java的工作版本：

public class MainActivity extends AppCompatActivity {

private FirebaseDatabase firebaseDatabase;
private DatabaseReference dbRef;
private ChildEventListener dbListener;

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);


    firebaseDatabase = FirebaseDatabase.getInstance();
    dbRef = firebaseDatabase.getReference("leak");
    dbListener = getDbListener();
    dbRef.addChildEventListener(dbListener);

    findViewById(R.id.btn_leak).setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View view) {
            dbRef.child(UUID.randomUUID().toString()).setValue("Yes");
            Intent leakIntent = new Intent(getApplicationContext(), LeakActivity.class);
            startActivity(leakIntent);
        }
    });
}

@Override
protected void onStop() {
    dbRef.removeEventListener(dbListener);
    super.onStop();
}

private ChildEventListener getDbListener(){
    return new ChildEventListener() {
        @Override
        public void onChildAdded(DataSnapshot dataSnapshot, String s) {

        }

        @Override
        public void onChildChanged(DataSnapshot dataSnapshot, String s) {

        }

        @Override
        public void onChildRemoved(DataSnapshot dataSnapshot) {

        }

        @Override
        public void onChildMoved(DataSnapshot dataSnapshot, String s) {

        }

        @Override
        public void onCancelled(DatabaseError databaseError) {

        }
    };
}

}

Answer 1

我没有使用LeakCanary，所以这只是一个有根据的猜测。

当不再需要时，

ChildEventListeners需要unregistered。通常在活动生命周期方法中添加和删除侦听器，例如onCreate()和onDestroy()。不再创建匿名侦听器，而是创建该类型的对象，并在不再需要时使用Query.removeEventListener()将其删除，以查看是否消除了泄漏报告。

Answer 2

我认为最好分别在onStart（）/ onStop（）或onCreate（）/ onDestroy（）回调中添加/删除侦听器。

如果在onCreate（）中添加了一个侦听器并在onStop（）中删除了某个侦听器，则可能会在没有调用onCreate（）的情况下恢复活动，但调用onStart（）并且不会设置侦听器。

https://twitter.com/dan_abramov/status/748969886433546240

这是我的代码或Firebase代码导致此泄漏吗？

2 个答案: