使用另一个活动的静态方法开始活动是一种好习惯吗？

Question

我认为这是使用像这样的静态方法启动活动的一种常见做法

class HomeScreenActivity{
  ....
  ....

  public static void startHomeActivity(Context context){
    Intent intent = new Intent(context,HomeScreenActivity.class);
    activity.startActivity();
  }
  ....
  ....

}


class LoginActivity{
  ....
  ....

  public void startActivity(){
    HomeScreenActivity.start(this);
  }    
  ....
  ....
}

这是一个好习惯吗，它会造成内存泄漏，会造成什么问题？

Answer 1

如果这是您要问的问题，这不是一个坏习惯。

一种不好的做法是，例如，在静态字段中有一个上下文变量，这会造成泄漏。例如：

public class App extends Application {
    private static Context mContext;

    public static Context getContext() {
        return mContext;
    }


    @Override
    public void onCreate() {
        super.onCreate();
        mContext = this

    }
}

这是一种不好的做法。 android studio 应该警告你（静态上下文字段）。例如，这可以通过使用WeakReference 来解决。

但是由于您示例的方法是静态的，并且上下文是作为参数提供的，所以它没有任何问题。如果您希望方法是静态的，最好将活动/上下文对象作为参数传递。

如果您怀疑存在泄漏，您可以覆盖该上下文的 onDestroy 方法以查明何时存在泄漏（不会调用 onDestroy，因为仍然存在对它的引用）。此外，可以通过使用 Memory Profiler 和/或 LeakCanary 库来发现泄漏。