Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Passing Activity or Context to other instance

Can you please tell me whats the best practice to pass activity or context to other instance(;)

But...,

  • What is better to pass? Activity or Context
  • Is good idea to have the Activity as Global (public static Activity activity)

here is my code. What would you change? (based on good android practices)

like image 645
George J Avatar asked Oct 16 '15 03:10

George J


1 Answers

Passing an Activity into another object that does not specifically require the Activity object is usually a bad idea. Activity extends Context so it works to satisfy a method that requires Context.

In your case, you can get the Context from the View that is passed into the method. However, if you need the Context for other purposes, you should avoid passing Activity and do something like Activity.getApplicationContext().

The reason for this is if the object you pass an Activity into lives longer than the Activity, the reference to the Activity will prevent Android from doing proper Garbage Collection (GC) and consume unnecessary memory. This is called a "memory leak".

EDIT:

To handle situations where you need to call Activity.findViewById() keep these things in mind. First, that is a View method (http://developer.android.com/reference/android/view/View.html#findViewById(int)) so be sure to call it from the correct view.

Second, if you need a View from an Activity you should pass it in as a WeakReference

like image 92
Jim Avatar answered Oct 15 '22 18:10

Jim