Android示例中的"Login"将AsyncTask
实现为非静态内部类。然而,根据Commonsguys的说法,这个类应该是静态的,并且对外部活动使用弱引用。
那么实现AsyncTask
的正确方法是什么呢?静态还是非静态?
Commonsguy实施
https://github.com/commonsguy/cw-android/tree/master/Rotation/RotationAsync/
谷歌登录示例
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?
没有单一的"正确"AsyncTask
的实现方式。但这是我的两点意见:
这个类的目的是执行"轻"在活动的上下文中工作。这就是为什么它在UI线程中运行onPreExecute
, onProgressUpdate
, onPostExecute
方法,以便它们可以访问字段并快速更新GUI。任何可能需要较长时间才能完成的任务,以及不需要更新特定活动的任务,都应该转移到Service。
这些方法主要用于更新GUI。由于GUI与Activity实例相关(字段可能被声明为私有成员变量),因此将AsyncTask
实现为非静态嵌套类更方便。在我看来这也是最自然的方式。
如果任务要在其他活动中重用,我认为应该允许它有自己的类。老实说,我不喜欢静态嵌套类,尤其是视图内部类。如果它是一个类,这意味着它在概念上不同于活动。如果它是静态的,则意味着它与活动的具体实例无关。但是由于它们是嵌套的,这些类在视觉上位于父类中,这使得它更难以阅读,并且在项目包资源管理器中可能不会被注意到,因为它只显示文件。尽管与内部类相比耦合度更低,但这并不是真的那么有用:如果类发生变化,您必须合并/提交整个父文件到版本控制。如果您要重用它,那么您必须在任何地方访问它作为Parent.Nested
。因此,为了不将其他活动耦合到Parent
类,您可能希望重构它并将嵌套类提取到自己的文件中。
所以对我来说问题是内部类vs顶层类
一般来说,我建议使用静态实现(尽管两者都是可以接受的)。
Google的方法将需要更少的代码,但你的asynctask将与你的活动紧密耦合(这意味着不容易重用)。但有时这种方法更容易读懂。
使用CommonsGuy方法,将需要更多的努力(和更多的代码)来解耦活动和异步任务,但最终您将拥有更模块化,更可重用的代码。
链接的文章已经说了
这确实强调,虽然,你希望你的AsyncTask的doInBackground()与Activity完全解耦。如果你只在主应用线程上触摸你的Activity,你的AsyncTask可以在方向改变后完好无损地存活。
不要从AsyncTask
中触摸Activity(例如它的成员),这是符合静态嵌套类
静态嵌套类
与类方法和变量一样,静态嵌套类与其外部类相关联。与静态类方法一样,静态嵌套类不能直接引用在其封闭类中定义的实例变量或方法——它只能通过对象引用来使用它们。
虽然,来自Android的例子,AsyncTask引用和使用AsyncTask仍然使用非静态嵌套类。
根据这个静态嵌套类在Java中,为什么?如果真的有必要,我会首先使用静态内部类,然后再使用非静态版本。
我发现,一个非静态嵌套Asynctask UI更新是更快的,当你需要更新你的UI频繁调用runOnUiThread在onProgressUpdate。例如,当你必须为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]);
}