While I understand that the effect is largely the same, it would seem to be more in li

GameHelper should be a retained Fragment about android-basic-samples HOT 4 CLOSED

playgameservices commented on August 28, 2024

GameHelper should be a retained Fragment

from android-basic-samples.

Comments (4)

cqr commented on August 28, 2024

Actually, it looks like onActivityResult is passed to fragments (who knew?) so your base activity could actually be very little.

@Override
public void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    mHelper = GameHelper.attach(this, <optional flags for clients here>);
}

// methods that wrap mHelper in a warm, loving embrace.

from android-basic-samples.

btco commented on August 28, 2024

Good idea. We'll look into it.

from android-basic-samples.

yhuiyang commented on August 28, 2024

Hi @chrisrhoden
This is real cool idea, I have tried it, and it works well. But I can not omit calling mHelper.onActvityResult() in my container activity. Can you show me what you did, so you can omit calling helper's onActivityResult() explicitly in container activity?

What I did is: (similar to your first comment)
In container activity:

@override
public void onCreate(Bundle savedInstanceState) {
    ...
    mHelper = (ApiHelper) getSupportFragmentManager().findFragmentByTag(TAG_API_HELPER);
    if (mHelper == null) {
        Bundle args = new Bundle();
        args.putInt(RC_RESOLVE_SIGNIN, 9999); /* tell helper to use which request code number to resolve sign in error */
        args.putXXX(...) /* any other thing, ex: flag for different clients */
        mHelper = (ApiHelper) Fragment.instantiate(this, ApiHelper.class.getName(), args);
        getSupportFragmentManager().beginTransaction().add(mHelper, TAG_API_HELPER).commit();
    }
}
@override
public void onActivityResult(int request, int result, Intent data) {
    super.onActiivtyResult(request, result, data);
    if (request == 9999)
        mHelper.onActvityResult(request, result, data);
}

In helper

@override
public void onConnectionFailed(ConnectResult result) {
    ...
    if (result.hasResolution()) {
        Bundle args = this.getArguments();
        int rc = args.getInt(RC_RESOLVE_SIGNIN);
        result.startResolutionForResult(getActivity(), rc); /* The first parameter needs activity, I can not use fragment here. So, I think currently the resolution activity is started from container activity instead of fragment itself */
    }
}

from android-basic-samples.