Интервью - Android-код

Question

Iv'e был в интервью, что должно быть неправильно со следующим?

Я могу предположить, что проблема заключается в том, что вы не можете проверить, является ли класс нулевым?! Спасибо!

public class NiceActivity extends Activity { 

    private static AmazingClass a; 

    class AmazingClass { 
     int x; 
     int y; 
    } 

    @Override 
    protected void onCreate(Bundle savedInstanceState) { 
     super.onCreate(savedInstanceState); 
     setContentView(R.layout.activity_nice); 
     if (a == null) { 
      a = new AmazingClass(); 
     } 
    } 
} 

Answer 1

Внутренний класс не является статичным, что может привести к его оживлению активности, все еще имеющему ссылку на него. Это может привести к утечке активности.

В этой статье объясняется эта проблема более подробно: http://www.androiddesignpatterns.com/2013/01/inner-class-handler-memory-leak.html

Answer 2

В качестве внутреннего AmazingClass класса не является статическим, каждый экземпляр имеет неявную ссылку на внешний класс (NiceActivity).

Класс NiceActivity имеет ссылку на AmazingClass, и эта ссылка никогда не очищается.

Следовательно, активность никогда не будет собираться с мусором, и произойдет утечка памяти.

В схеме, которая будет примерно выглядеть следующим образом:

root GC -1-> Activity class -2-> Amazing object -3-> Activity object 

номер 2 или 3 ссылки должны быть очищены, чтобы избежать такой утечки.