Android示例中的“登录”将AsyncTask实现为非静态内部类。但是，根据Commonsguys的说法，这个类应该是静态的，并且对外部活动使用弱引用see this。
那么实施AsyncTask的正确方法是什么？静态还是非静态？
Commonsguy Implementation
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);
}
}
}
如果它取决于具体情况，那么使用ListView从互联网加载HttpClient项（文本+加位图），我该如何实现我的AsyncTask？
答案 0 :(得分：17)
没有一种“正确”的方式来实施AsyncTask。但这是我的两分钱：
此类旨在在Activity的上下文中执行“轻量级”工作。这就是为什么它在UI线程中运行方法onPreExecute，onProgressUpdate，onPostExecute，以便他们可以快速访问字段和更新GUI。任何可能需要较长时间才能完成并且无意更新特定活动的任务都应移至服务中。
这些方法主要用于更新GUI。由于GUI与Activity实例相关（字段可能被声明为私有成员变量），因此将AsyncTask实现为非静态嵌套类更为方便。在我看来，这也是最自然的方式。
如果任务将在其他活动中重复使用，我认为应该允许它拥有自己的类。说实话，我不喜欢静态嵌套类，尤其是内部视图。如果它是一个类，则意味着它在概念上与活动不同。如果它是静态的，则意味着它与活动的这个具体实例无关。但是当它们嵌套时，这些类在视觉上位于父类中，使得它更难以阅读，并且在项目包资源管理器中可以不被注意，因为它只显示文件。尽管耦合程度低于内部类，但这并不是很有用：如果类发生更改，则必须将整个父文件合并/提交到版本控制。如果您在哪里重复使用它，那么您必须在任何地方以Parent.Nested的形式访问它。因此，为了不将其他活动耦合到Parent类，您可能希望重构它并将嵌套类提取到它自己的文件中。
所以对我来说，问题是Inner Class vs Top-Level Class。
答案 1 :(得分：14)
一般情况下，我会推荐静态实现（虽然两者都可以接受）。
Google方法将需要更少的代码，但您的asynctask将与您的actitivy紧密结合（这意味着不容易重复使用）。但有时这种方法更具可读性。
使用CommonsGuy方法，需要更多努力（和更多代码）来分离活动和asynctask，但最终你将拥有更模块化，更可重用的代码。
答案 2 :(得分：3)
链接文章已经说明了
但是，这确实强调了您希望AsyncTask的doInBackground（）与Activity完全分离。如果您只触摸主应用程序线程上的Activity，则AsyncTask可以在方向更改完好后继续存在。
请勿触摸AsyncTask上的活动（例如其成员），该活动符合Static Nested Classes
静态嵌套类
与类方法和变量一样，静态嵌套类与其外部类相关联。和静态类方法一样，静态嵌套类不能直接引用其封闭类中定义的实例变量或方法 - 它只能通过对象引用来使用它们。
尽管Android中的示例AsyncTask reference和Using AsyncTask仍在使用非静态嵌套类。
根据这个Static nested class in Java, why?，我首先使用 static 内部类，并且如果真的有必要，只使用非静态版本。
答案 3 :(得分：1)
我发现，当您需要通过在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]);
}