В "Вход" из 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?
В целом я бы рекомендовал статическую реализацию (хотя оба они приемлемы).
В подходе Google потребуется меньше кода, но ваша асинтеза будет тесно связана с вашим актитивом (что означает, что его нельзя легко повторно использовать). Но иногда этот подход более читабельен.
При использовании подхода CommonsGuy потребуется больше усилий (и больше кода), чтобы разделить активность и асинтезу, но в итоге у вас будет более модульный и многоразовый код.
Нет единого "правильного" способа реализации AsyncTask. Но вот мои два цента:
Этот класс предназначен для выполнения "легкой" работы в контексте Activity. Поэтому он использует методы onPreExecute, onProgressUpdate, onPostExecute, запущенные в потоке пользовательского интерфейса, чтобы они могли быстро получать доступ к полям и быстро обновлять графический интерфейс. Любая задача, которая может занять больше времени и не предназначена для обновления определенного действия, должна быть перенесена в службу.
Эти методы в основном используются для обновления GUI. Поскольку графический интерфейс является чем-то связанным с экземпляром Activity (поля, скорее всего, объявлены как частные переменные-члены), более удобно реализовать AsyncTask как нестатический вложенный класс. Это также самый естественный способ, на мой взгляд.
В случае, если задача будет повторно использоваться в других действиях, я думаю, что ей должно быть позволено иметь свой собственный класс. Честно говоря, я не поклонник статических вложенных классов, особенно внутри представлений. Если это класс, это означает, что он концептуально отличается от активности. И если он является статичным, это означает, что он не связан с этим конкретным примером деятельности. Но поскольку они вложены, эти классы визуально находятся внутри родительского класса, что затрудняет чтение и может оставаться незамеченным в проводнике пакета проекта, поскольку он показывает только файлы. И, несмотря на то, что они менее связаны с внутренними классами, это не так уж и полезно: если класс изменяется, вы должны объединить/перенести весь родительский файл в элемент управления версиями. Если вы хотите его повторно использовать, вам придется обращаться к нему как Parent.Nested. Поэтому, чтобы не связывать другие действия с классом Parent, вы, вероятно, захотите реорганизовать его и извлечь вложенный класс в свой собственный файл.
Итак, для меня вопрос будет состоять из класса Inner Class vs Top-Level Class.
Связанная статья уже говорит об этом
Это подчеркивает, однако, что вы хотите, чтобы doInBackground() вашей AsyncTask полностью отделилась от Activity. Если вы касаетесь только своей активности в основном потоке приложения, ваша AsyncTask может пережить изменение ориентации.
Не прикасайтесь к Activity (например, его членам) из AsyncTask, что соответствует Static Nested Classes
Статические вложенные классы
Как и в методах и переменных класса, статический вложенный класс связан с его внешним классом. Как и статические методы класса, статический вложенный класс не может напрямую ссылаться на переменные экземпляра или методы, определенные в его охватывающем классе, - он может использовать их только через ссылку на объект.
Хотя, примеры из Android, ссылка AsyncTask и Использование AsyncTask по-прежнему используют нестатические вложенные классы.
И в соответствии с этим статическим вложенным классом в Java, почему?, я бы сначала пошел со статическим внутренним классом и применил только к нестатической версии, если это действительно необходимо.