实现 AsyncTask 的正确方法是啥?静态或非静态嵌套类?

Posted

技术标签:

【中文标题】实现 AsyncTask 的正确方法是啥?静态或非静态嵌套类?【英文标题】:What's the correct way to implement AsyncTask? static or non static nested class?实现 AsyncTask 的正确方法是什么?静态或非静态嵌套类? 【发布时间】:2012-12-31 17:34:46 【问题描述】:

android 示例中的“登录”将AsyncTask 实现为非静态内部类。然而,根据 Commonsguys 的说法,这个类应该是静态的,并且使用对外部活动 see this 的弱引用。

那么实现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[]  "foo@example.com:hello", "bar@example.com: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);
        
    

如果取决于具体情况,那么使用HttpClient从互联网加载ListView项目(文本+加上位图),我应该如何实现我的AsyncTask?

【问题讨论】:

【参考方案1】:

没有单一的“正确”方式来实现AsyncTask。但这是我的两分钱:

此类旨在在 Activity 的上下文中执行“轻量级”工作。这就是为什么它在 UI 线程中运行 onPreExecuteonProgressUpdateonPostExecute 方法,以便他们可以快速访问字段并更新 GUI。任何可能需要较长时间才能完成且并非旨在更新特定活动的任务都应移至服务。

这些方法主要用于更新 GUI。由于 GUI 与 Activity 实例相关(这些字段可能声明为私有成员变量),因此将 AsyncTask 实现为非静态嵌套类更方便。这也是我认为最自然的方式。

如果任务要在其他活动中重用,我认为应该允许它有自己的类。老实说,我不喜欢静态嵌套类,尤其是内部视图。如果它是一个类,则意味着它在概念上与活动不同。如果它是静态的,则意味着它与该活动的具体实例无关。但是由于它们是嵌套的,这些类在视觉上位于父类中,使其更难阅读,并且在项目包资源管理器中可能不会被注意到,因为它只显示文件。尽管比内部类耦合少,但这并不是那么有用:如果类发生变化,您必须将整个父文件合并/提交给版本控制。如果您在哪里重复使用它,那么您必须在任何地方以Parent.Nested 的身份访问它。因此,为了不将其他活动耦合到 Parent 类,您可能希望重构它并将嵌套类提取到它自己的文件中。

所以对我来说,问题是内部类与***类。

【讨论】:

【参考方案2】:

一般来说,我会推荐静态实现(尽管两者都可以接受)。

Google 方法需要的代码更少,但您的 asynctask 将与您的活动紧密耦合(这意味着不容易重用)。但有时这种方法更具可读性。

使用 CommonsGuy 方法,将需要更多努力(和更多代码)来解耦活动和异步任务,但最终您将拥有更模块化、更可重用的代码。

【讨论】:

据我了解,非静态嵌套类在类外保留对它的引用。如果用户突然取消当前活动(点击后退按钮),而线程池中仍有多个任务排队。那么这种方法(非静态)是否会潜在地造成内存泄漏,因为 GC 将无法为该活动回收内存。我是否正确理解这一点?顺便说一句,非常感谢。 @Chan AsyncTasks 容易发生泄漏,包括内部和静态嵌套的。如果设备更改配置并重新创建活动,很容易忘记取消旧任务,然后它会在 bg 中运行。 @MisterSmith:谢谢。那么还有其他替代方法吗?【参考方案3】:

链接的文章已经说过了

不过,这确实强调了您希望 AsyncTask 的 doInBackground() 与 Activity 完全分离。如果您只在主应用程序线程上触摸您的 Activity,您的 AsyncTask 可以在方向更改后完好无损。

不要触摸来自AsyncTask 的Activity(例如其成员),这与Static Nested Classes 一致

静态嵌套类 与类方法和变量一样,静态嵌套类与其外部类相关联。和静态类方法一样,静态嵌套类不能直接引用在其封闭类中定义的实例变量或方法——它只能通过对象引用来使用它们。

尽管来自 Android 的示例 AsyncTask reference 和 Using AsyncTask 仍在使用非静态嵌套类。

根据Static nested class in Java, why?,我会先使用 static 内部类,如果真的有必要,只使用非静态版本。

【讨论】:

【参考方案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]);
        

【讨论】:

onProgressUpdate 仅在 UI 线程中运行。 runOnUiThread() 需要什么?

以上是关于实现 AsyncTask 的正确方法是啥?静态或非静态嵌套类?的主要内容,如果未能解决你的问题,请参考以下文章

iPhone iOS 实例化静态 NSString 单元重用标识符的正确方法是啥?

c2797 未实现成员初始化器列表或非静态数据成员初始化器内的列表初始化

在 iOS 应用程序中制作选项/设置视图的正确方法是啥?

如何在静态方法或非Spring Bean中注入Spring Bean

深入理解AsyncTask

PHP中的线程安全或非线程安全是啥?