AsyncTaskを実装する正しい方法は何ですか？静的または非静的ネストクラス？

Question

Androidの例の「ログイン」は、非静的内部クラスとして実装されていますAsyncTask。ただし、Commonsguysによると、このクラスは静的であり、外部への弱参照を使用する必要がありますアクティビティ これを参照 。

では、AsyncTaskを実装する正しい方法は何ですか？静的または非静的？

Commonsguyの実装
https://github.com/commonsguy/cw-Android/tree/master/Rotation/RotationAsync/

Googleからのログイン例

package com.example.asynctaskdemo;

import Android.animation.Animator;
import Android.animation.AnimatorListenerAdapter;
import Android.annotation.TargetApi;
import Android.app.Activity;
import Android.os.AsyncTask;
import Android.os.Build;
import Android.os.Bundle;
import Android.text.TextUtils;
import Android.view.KeyEvent;
import Android.view.Menu;
import Android.view.View;
import Android.view.inputmethod.EditorInfo;
import Android.widget.EditText;
import Android.widget.TextView;

/**
 * Activity which displays a login screen to the user, offering registration as
 * well.
 */
public class LoginActivity extends Activity {
    /**
     * A dummy authentication store containing known user names and passwords.
     * TODO: remove after connecting to a real authentication system.
     */
    private static final String[] DUMMY_CREDENTIALS = new String[] { "[email protected]:hello", "[email protected]:world" };

    /**
     * The default email to populate the email field with.
     */
    public static final String EXTRA_EMAIL = "com.example.Android.authenticatordemo.extra.EMAIL";

    /**
     * Keep track of the login task to ensure we can cancel it if requested.
     */
    private UserLoginTask mAuthTask = null;

    // Values for email and password at the time of the login attempt.
    private String mEmail;
    private String mPassword;

    // UI references.
    private EditText mEmailView;
    private EditText mPasswordView;
    private View mLoginFormView;
    private View mLoginStatusView;
    private TextView mLoginStatusMessageView;

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

        setContentView(R.layout.activity_login);

        // Set up the login form.
        mEmail = getIntent().getStringExtra(EXTRA_EMAIL);
        mEmailView = (EditText) findViewById(R.id.email);
        mEmailView.setText(mEmail);

        mPasswordView = (EditText) findViewById(R.id.password);
        mPasswordView.setOnEditorActionListener(new TextView.OnEditorActionListener() {
            @Override
            public boolean onEditorAction(TextView textView, int id, KeyEvent keyEvent) {
                if (id == R.id.login || id == EditorInfo.IME_NULL) {
                    attemptLogin();
                    return true;
                }
                return false;
            }
        });

        mLoginFormView = findViewById(R.id.login_form);
        mLoginStatusView = findViewById(R.id.login_status);
        mLoginStatusMessageView = (TextView) findViewById(R.id.login_status_message);

        findViewById(R.id.sign_in_button).setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                attemptLogin();
            }
        });
    }

    @Override
    public boolean onCreateOptionsMenu(Menu menu) {
        super.onCreateOptionsMenu(menu);
        getMenuInflater().inflate(R.menu.activity_login, menu);
        return true;
    }

    /**
     * Attempts to sign in or register the account specified by the login form.
     * If there are form errors (invalid email, missing fields, etc.), the
     * errors are presented and no actual login attempt is made.
     */
    public void attemptLogin() {
        if (mAuthTask != null) {
            return;
        }

        // Reset errors.
        mEmailView.setError(null);
        mPasswordView.setError(null);

        // Store values at the time of the login attempt.
        mEmail = mEmailView.getText().toString();
        mPassword = mPasswordView.getText().toString();

        boolean cancel = false;
        View focusView = null;

        // Check for a valid password.
        if (TextUtils.isEmpty(mPassword)) {
            mPasswordView.setError(getString(R.string.error_field_required));
            focusView = mPasswordView;
            cancel = true;
        }
        else if (mPassword.length() < 4) {
            mPasswordView.setError(getString(R.string.error_invalid_password));
            focusView = mPasswordView;
            cancel = true;
        }

        // Check for a valid email address.
        if (TextUtils.isEmpty(mEmail)) {
            mEmailView.setError(getString(R.string.error_field_required));
            focusView = mEmailView;
            cancel = true;
        }
        else if (!mEmail.contains("@")) {
            mEmailView.setError(getString(R.string.error_invalid_email));
            focusView = mEmailView;
            cancel = true;
        }

        if (cancel) {
            // There was an error; don't attempt login and focus the first
            // form field with an error.
            focusView.requestFocus();
        }
        else {
            // Show a progress spinner, and kick off a background task to
            // perform the user login attempt.
            mLoginStatusMessageView.setText(R.string.login_progress_signing_in);
            showProgress(true);
            mAuthTask = new UserLoginTask();
            mAuthTask.execute((Void) null);
        }
    }

    /**
     * Shows the progress UI and hides the login form.
     */
    @TargetApi(Build.VERSION_CODES.HONEYCOMB_MR2)
    private void showProgress(final boolean show) {
        // On Honeycomb MR2 we have the ViewPropertyAnimator APIs, which allow
        // for very easy animations. If available, use these APIs to fade-in
        // the progress spinner.
        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.HONEYCOMB_MR2) {
            int shortAnimTime = getResources().getInteger(Android.R.integer.config_shortAnimTime);

            mLoginStatusView.setVisibility(View.VISIBLE);
            mLoginStatusView.animate().setDuration(shortAnimTime).alpha(show ? 1 : 0).setListener(new AnimatorListenerAdapter() {
                @Override
                public void onAnimationEnd(Animator animation) {
                    mLoginStatusView.setVisibility(show ? View.VISIBLE : View.GONE);
                }
            });

            mLoginFormView.setVisibility(View.VISIBLE);
            mLoginFormView.animate().setDuration(shortAnimTime).alpha(show ? 0 : 1).setListener(new AnimatorListenerAdapter() {
                @Override
                public void onAnimationEnd(Animator animation) {
                    mLoginFormView.setVisibility(show ? View.GONE : View.VISIBLE);
                }
            });
        }
        else {
            // The ViewPropertyAnimator APIs are not available, so simply show
            // and hide the relevant UI components.
            mLoginStatusView.setVisibility(show ? View.VISIBLE : View.GONE);
            mLoginFormView.setVisibility(show ? View.GONE : View.VISIBLE);
        }
    }

    /**
     * Represents an asynchronous login/registration task used to authenticate
     * the user.
     */
    public class UserLoginTask extends AsyncTask<Void, Void, Boolean> {
        @Override
        protected Boolean doInBackground(Void... params) {
            // TODO: attempt authentication against a network service.

            try {
                // Simulate network access.
                Thread.sleep(2000);
            }
            catch (InterruptedException e) {
                return false;
            }

            for (String credential : DUMMY_CREDENTIALS) {
                String[] pieces = credential.split(":");
                if (pieces[0].equals(mEmail)) {
                    // Account exists, return true if the password matches.
                    return pieces[1].equals(mPassword);
                }
            }

            // TODO: register the new account here.
            return true;
        }

        @Override
        protected void onPostExecute(final Boolean success) {
            mAuthTask = null;
            showProgress(false);

            if (success) {
                finish();
            }
            else {
                mPasswordView.setError(getString(R.string.error_incorrect_password));
                mPasswordView.requestFocus();
            }
        }

        @Override
        protected void onCancelled() {
            mAuthTask = null;
            showProgress(false);
        }
    }
}

特定の状況に依存する場合、ListViewアイテム（テキスト+ビットマップ）をHttpClientを使用してインターネットからロードすると、どのようにAsyncTaskを実装する必要がありますか？

Answer 1

一般に、静的実装をお勧めします（両方とも許容されます）。

Googleのアプローチでは必要なコードは少なくなりますが、非同期タスクは活動と密接に結びついています（つまり、再利用は容易ではありません）。ただし、このアプローチの方が読みやすい場合があります。

CommonsGuyのアプローチでは、アクティビティとasynctaskを分離するためにより多くの労力（およびより多くのコード）が必要になりますが、最終的には、よりモジュール化された、再利用可能なコードになります。

Answer 2

AsyncTaskを実装する単一の「正しい」方法はありません。しかし、ここに私の2セントがあります。

このクラスは、アクティビティのコンテキストで「軽い」作業を実行することを目的としています。そのため、UIスレッドでメソッドonPreExecute、onProgressUpdate、onPostExecuteを実行しているため、フィールドにアクセスしてGUIをすばやく更新できます。完了するまでに時間がかかる可能性があり、特定のアクティビティを更新することを目的としていないタスクは、サービスに移動する必要があります。

これらのメソッドは、主にGUIを更新するために使用されます。 GUIはActivityインスタンスに関連するものであるため（フィールドはプライベートメンバー変数として宣言される可能性が高い）、AsyncTaskを非静的なネストされたクラスとして実装する方が便利です。それは私の意見では最も自然な方法でもあります。

タスクが他のアクティビティで再利用される場合は、独自のクラスを許可する必要があると思います。正直なところ、私は静的なネストされたクラス、特にビューの内部は好きではありません。クラスの場合は、アクティビティとは概念的に異なることを意味します。そして、それが静的である場合、それはアクティビティのこの具体的なインスタンスに関連していないことを意味します。しかし、それらがネストされているため、これらのクラスは視覚的に親クラスの内部にあり、読みにくくなり、プロジェクトパッケージエクスプローラーではファイルしか表示されないため、気付かれない可能性があります。また、内部クラスよりも結合が少ないにもかかわらず、これはそれほど有用ではありません。クラスが変更された場合、親ファイル全体をバージョン管理にマージ/コミットする必要があります。再利用する場合は、どこからでもParent.Nestedとしてアクセスする必要があります。したがって、他のアクティビティをParentクラスに結合しないようにするために、おそらくそれをリファクタリングして、ネストされたクラスを独自のファイルに抽出する必要があります。

だから私にとっての質問はインナークラス対トップレベルクラスでしょう。

Answer 3

リンクされた記事はすでにそれを言っています

ただし、これは、AsyncTaskのdoInBackground（）をアクティビティから完全に切り離したいことを強調しています。メインアプリケーションスレッドでアクティビティに触れるだけの場合、AsyncTaskは向きの変更をそのまま維持できます。

Static Nested Classes と一致するAsyncTaskのアクティビティ（そのメンバーなど）には触れないでください。

静的なネストされたクラス
クラスのメソッドと変数と同様に、静的なネストされたクラスはその外部クラスに関連付けられています。また、静的クラスメソッドと同様に、静的ネストクラスは、その包含クラスで定義されているインスタンス変数またはメソッドを直接参照することはできません。オブジェクト参照を通じてのみ使用できます。

ただし、Androidの例である AsyncTaskリファレンス および sing AsyncTask は、静的でないネストされたクラスをまだ使用しています。

そして、これに従って Javaの静的なネストされたクラス、なぜ？ 、最初にstatic内部クラスを使用して、非静的バージョンのみに頼ります、それが本当に必要な場合。

Answer 4

OnProgressUpdateでrunOnUiThreadを呼び出してUIを頻繁に更新する必要がある場合は、静的でないネストされたAsynctask UIの更新の方が速いことがわかりました。たとえば、TextViewに行を追加する必要がある場合。

non-static:
    @Override
    protected void onProgressUpdate(String... values) {
        runOnUiThread(() -> {
            TextView tv_results = findViewById(R.id.tv_results);
            tv_results.append(values[0] + "\n");
        });
    }

静的AsyncTaskのリスナーを実装するよりも1000倍高速です。私は間違っているかもしれませんが、それは私の経験でした。

static:
        @Override
        protected void onProgressUpdate(String... values) {
            OnTaskStringUpdatedListener.OnTaskStringUpdated(task, values[0]);
        }