7

私の静的ハンドラにはWeakReferenceto myがありますActivity(これは、十分に文書化されたメモリ リークの問題を防ぐためです)。

長い遅延メッセージを投稿しましたが、このメッセージをアクティビティ (フォアグラウンドにある必要があります) に配信したいと考えています。

私の懸念は、向きが変わるとアクティビティが破棄され、ハンドラーが破棄されるべきであった古いアクティビティへの参照を持っていることです。

これを回避するために、onCreate私はこれを行います。

    if(mHandler == null)
        mHandler = new LoginHandler(this);
    else {
        mHandler.setTarget(this);
    }

そして、私のハンドラは静的グローバル変数として宣言されています:

private static LoginHandler     mHandler            = null;

また、実装クラスも以下のように静的です。

private static class LoginHandler extends Handler {

    private WeakReference<LoginActivity>    mTarget;

    LoginHandler(LoginActivity target) {
        mTarget = new WeakReference<LoginActivity>(target);
    }

    public void setTarget(LoginActivity target) {
        mTarget = new WeakReference<LoginActivity>(target);
    }

    @Override
    public void handleMessage(Message msg) {
        // process incoming messages here
        LoginActivity activity = mTarget.get();
        switch (msg.what) {
            case Constants.SUCCESS:
                activity.doSomething();
                break;

            default:
                activity.setStatusMessage("failed " + msg.obj, STATUS_TYPE_DONE);
        }
    }
}

私が知りたいのは、WeakReference の変更に問題があるかどうか、onCreateまたはこのアプローチに他に問題があるかどうかです。

ありがとう、

4

2 に答える 2

6

そこで、正しい考えを持っているかどうかを判断するために次のテストを書きましたが、 m アプローチが正しいようです。onCreateを変更すると、投稿されたメッセージはWeakReference常にフォアグラウンドにあるアクティビティに配信されます。常に新しい Handler を作成するようにこのコードを変更するとonCreate、更新メッセージが配信されないことに気付くでしょう。

public class MainActivity extends Activity {

    private static int COUNT = 0;

    static LoginHandler mHandler;

    private static class LoginHandler extends Handler {

        private WeakReference<MainActivity> mTarget;

        LoginHandler(MainActivity target) {
            mTarget = new WeakReference<MainActivity>(target);
        }

        public void setTarget(MainActivity target) {
            mTarget.clear();
            mTarget = new WeakReference<MainActivity>(target);
        }

        @Override
        public void handleMessage(Message msg) {
            // int duration = Toast.LENGTH_LONG;
            // process incoming messages here
            MainActivity activity = mTarget.get();
            activity.update(msg.arg1);
        }
    }

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);

        if(mHandler == null)
            mHandler = new LoginHandler(this);
        else
            mHandler.setTarget(this);

        ((Button)findViewById(R.id.button)).setOnClickListener(new View.OnClickListener() {

            @Override
            public void onClick(View v) {
                Message msg = new Message();
                msg.arg1 = COUNT++;
                mHandler.sendMessageDelayed(msg, 3000);

            }
        });

    }

    @Override
    public boolean onCreateOptionsMenu(Menu menu) {
        // Inflate the menu; this adds items to the action bar if it is present.
        getMenuInflater().inflate(R.menu.main, menu);
        return true;
    }

    private void update(int count) {
        ((TextView) findViewById(R.id.hello_world)).setText("Hello World @ "+ count);
    }

}
于 2013-08-15T06:51:14.693 に答える