Communication between TabActivity and the embedded activity

Question

I am trying to figure out the best practice of communication between a TabActivity and the child activity embedded in this TabActivity.

In my TabActivity, there is a button. When the button is clicked, I want the child activity embedded in this TabActivity to be updated. I wrote the code like below, and just wonder whether it is a good practice. Thanks.

MyTabActivity.java

public class MyTabActivity extends TabActivity implements OnClickListener {
    private TabHost m_tabHost;

  public void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.ff_tab_activity);

    LayoutInflater inflater = (LayoutInflater) getSystemService(Context.LAYOUT_INFLATER_SERVICE);

    m_tabHost = getTabHost(); 
    TabHost.TabSpec spec; 
    Intent intent; 

    intent = new Intent().setClass(this, ChildActivity.class);
    spec = m_tabHost.newTabSpec("Tab 1");
    spec.setContent(intent);
    tabView = (TextView) inflater.inflate(R.layout.tab_indicator, null);
    spec.setIndicator(tabView);
    m_tabHost.addTab(spec);

    m_tabHost.setCurrentTab(0);
    ImageView nextButtonIv = (ImageView) findViewById(R.id.next_button);
    nextButtonIv.setOnClickListener(this);
  }

  @Override
  public void onClick(View v) {
    switch (v.getId()) {
    case R.id.next_button:
        synchronized (ChildActivity.class) {
            if (null != ChildActivity.s_childActivity) {
                ChildActivity.s_childActivity.changeUI();
            }
        }
        break;
    }
}

ChildActivity.java

public class ChildActivity extends Activity {
    public static ChildActivity s_childActivity;

    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);

        synchronized (MatchupsActivity.class) {
            s_childActivity = this;
        }
        setContentView(R.layout.child_activity);
    }

    public void changeUi() {
        code that changes UI
    }

    protected void onDestroy() {
        super.onDestroy();

        synchronized (MatchupsActivity.class) {
            s_childActivity = null;
        }
}

score 7 · Answer 1 · answered Sep 27 '10 at 20:23

Since TabActivity is an ActivityGroup, I would use one of the following:

getCurrentActivity()

Returns the child tab activity being displayed. In your case, this method will return the instance of ChildActivity being used.

ChildActivity childActivity = (ChildActivity) getCurrentActivity();

getLocalActivityManager().getActivity(String)

Returns the child tab activity given its ID/tab spec name, whatever activity being displayed.

ChildActivity childActivity = (ChildActivity) getLocalActivityManager().getActivity("Tab 1");

I suggest overriding onNewIntent(Intent) in your ChildActivity:

Intent intent = new Intent();
intent.putExtra("xyz", "whatever"); // or a serializable

ChildActivity childActivity = (ChildActivity) getLocalActivityManager().getActivity("Tab 1");
childActivity.onNewIntent(intent);

Let me know if it works!

score 2 · Accepted Answer · answered Jun 28 '10 at 18:06

2

Seems fine. A couple of notes:
- I see no reason for synchronization.
- I'd replace

ChildActivity.s_childActivity.changeUI();

with

if(ChildActivity.s_childActivity != null){
    ChildActivity.s_childActivity.changeUI();
}

or even

try{
    ChildActivity.s_childActivity.changeUI();
} catch(Exception e){
    //log
}

for added paranoid safety. :)

answered Jun 28 '10 at 18:06

yanchenko

56,576
33
147
165

I see. They are in the same thread. I prefer if(ChildActivity.s_childActivity != null){ ChildActivity.s_childActivity.changeUI(); } Thanks. – user256239 Jun 28 '10 at 18:16

score 2 · Answer 3 · answered May 20 '11 at 09:45

The way above with

ChildActivity childActivity = (ChildActivity) getLocalActivityManager().getActivity("Tab 1");
childActivity.onNewIntent(intent);

is not very nice. Instead of invoking your activity method directly (it can be null!!!) better do it this way:

Intent intent = new Intent(this, ChildActivity.class);
intent.addFlags(Intent.FLAG_ACTIVITY_SINGLE_TOP);
intent.putExtra(AlbumBrowser.INTENT_EXTRA_FILTER, mediaTitle);
getLocalActivityManager().startActivity("activityIdHere", intent);

score 1 · Answer 4 · answered Jun 28 '10 at 18:31

Couple of design issues with this, but overall it seems reasonable.

I would forgo the static instance in the ChildActivity class. Why? Well, think about the relationship you're modeling. A TabActivity has a ChildActivity. This is textbook composition, and would be accomplished by adding a ChildActivity field to the TabActivity class, like so:

 public class TabActivity {

    private ChildActivity child;

    //remember to initialize child in onCreate

    //then, call methods using child.changeUI();, for example

}

This is better, because A) now I can have multiple instances of TabActivity and ChildActivity that won't interfere with each other (before, it was just a static variable, so only one ChildActivity could be used), and B) the ChildActivity is encapsulated inside the TabActivity class... before, it was a public field, meaning anything can use and modify it (might not be desirable; can often lead to some strange runtime bugs as well as messy, tied-together code) - we changed it to a private field, because we don't really want other classes accessing it in unexpected ways.

The only thing you may need access to from the ChildActivity is the parent (TabActivity). To do this, add the following methods and field to the ChildActivity class, and call the registerParent() method after constructing the ChildActivity:

public class ChildActivity ...{

private TabActivity parent;

public void registerParent(TabActivity newParent){
    if (newParent != null){
        parent = newParent;
    }
}
}

So, if you need access to the parent TabActivity from the child, just call parent.someMethod();

It would also be wise to access fields like parent and child through getters and setters; it might save you some time in the long run.

How do you get the activity if you create the tab like this? `tabHost.addTab(tabHost.newTabSpec("foo") .setIndicator(indicator1) .setContent(new Intent(this, Foo.class)));` — Artem Russakovskii, Aug 08 '11 at 22:26
Turns out Android already gives us Activity.getParent() which returns exactly the parent TabActivity if you're embedded in one, so none of this is needed. — Artem Russakovskii, Aug 08 '11 at 22:44

score 1 · Answer 5 · answered Aug 20 '11 at 04:56

You can use getParent() to obviate the need to do any of this.

Here's my launcher child class with buttons that switch between activities handled by the tabHost:

public class LaunchPadActivity extends Activity implements OnClickListener {

private static final int ICON_PROFILE = 0;

private static final int ICON_SEARCH = 1;

private static final int ICON_MAP = 2;

private static final int FAVOURITES = 3;

public void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);

    setContentView(R.layout.launchpad);

    GridView launchPad = (GridView) findViewById(R.id.launchpad);
    launchPad.setAdapter(new LaunchIconAdapter(this));
}

public class LaunchIconAdapter extends BaseAdapter {
    private Context mContext;

    // references to our images
    private Integer[] mThumbIds = { R.drawable.user, R.drawable.find,
            R.drawable.map, R.drawable.favourites, R.drawable.reviews,
            R.drawable.news, R.drawable.tutorial, R.drawable.info,
            R.drawable.options, };

    public String[] texts = { "Profile", "Search", "Map", "Favourites",
            "Reviews", "News", "Tutorial", "Info", "Options" };

    public LaunchIconAdapter(Context c) {
        mContext = c;
    }

    // Number of thumbs determines number of GridView items
    public int getCount() {
        return mThumbIds.length;
    }

    public Object getItem(int position) {
        return null;
    }

    public long getItemId(int position) {
        return 0;
    }

    // create a new ImageView for each item referenced by the Adapter
    public View getView(int position, View convertView, ViewGroup parent) {
        // Icon elements
        LinearLayout launchIcon;
        ImageView launchImage;
        TextView launchText;

        if (convertView == null) {
            launchIcon = (LinearLayout) ((LayoutInflater) mContext
                    .getSystemService(LAYOUT_INFLATER_SERVICE)).inflate(
                    R.layout.launchicon, null);

        } else {
            launchIcon = (LinearLayout) convertView;
        }

        // Add ClickListener with metadata
        launchIcon.setTag(new Integer(position));
        launchIcon.setOnClickListener(LaunchPadActivity.this);

        // Get subviews
        launchImage = (ImageView) launchIcon
                .findViewById(R.id.launch_image);
        launchText = (TextView) launchIcon.findViewById(R.id.launch_text);

        // Configure subviews
        launchImage.setImageResource(mThumbIds[position]);
        launchText.setText(texts[position]);

        return launchIcon;
    }

}

@Override
public void onClick(View v) {
    int position = ((Integer) v.getTag()).intValue();
    switch (position) {
    case ICON_PROFILE:
        Toast.makeText(this, "Profile", Toast.LENGTH_LONG).show();
        break;

    case ICON_SEARCH:
        Toast.makeText(this, "Search", Toast.LENGTH_LONG).show();
        ((TabActivity) getParent()).getTabHost().setCurrentTab(1);
        break;

    case ICON_MAP:
        Toast.makeText(this, "Map", Toast.LENGTH_LONG).show();
        ((TabActivity) getParent()).getTabHost().setCurrentTab(2);
        break;

    case FAVOURITES:
        Toast.makeText(this, "Map", Toast.LENGTH_LONG).show();
        ((TabActivity) getParent()).getTabHost().setCurrentTab(3);
        break;

    }

}

}

Works like a charm.

Communication between TabActivity and the embedded activity

5 Answers5

Linked