Как правильно реализовать AsyncTask? статический или нестатический вложенный класс?
Реализован "Логин" из примеров 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[] { "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?
4 ответа
В целом, я бы порекомендовал статическую реализацию (хотя обе приемлемы).
Подход Google потребует меньше кода, но ваша асинхронная задача будет тесно связана с вашей активностью (что означает, что ее нелегко использовать повторно). Но иногда этот подход более читабелен.
С подходом CommonsGuy потребуется больше усилий (и больше кода) для отделения активности и асинхронной задачи, но в итоге у вас будет более модульный, более многократно используемый код.
Там нет единого "правильного" способа реализации AsyncTask, Но вот мои два цента:
Этот класс предназначен для выполнения "легкой" работы в контексте действия. Вот почему у него есть методы onPreExecute, onProgressUpdate, onPostExecute работает в потоке пользовательского интерфейса, чтобы они могли получить доступ к полям и быстро обновлять графический интерфейс. Любую задачу, выполнение которой может занять больше времени и не предназначенную для обновления определенного действия, следует перенести в Службу.
Эти методы в основном используются для обновления графического интерфейса. Поскольку GUI связан с экземпляром Activity (поля, вероятно, объявлены как закрытые переменные-члены), более удобно реализовать AsyncTask как нестатический вложенный класс. Это также самый естественный способ на мой взгляд.
В случае, если задание будет повторно использовано в других видах деятельности, я думаю, что должно быть разрешено иметь свой собственный класс. Честно говоря, я не фанат статических вложенных классов, особенно внутренних представлений. Если это класс, это означает, что он концептуально отличается от деятельности. И если оно статично, это означает, что оно не связано с этим конкретным случаем действия. Но поскольку они являются вложенными, эти классы визуально находятся внутри родительского класса, что затрудняет его чтение, и может остаться незамеченным в проводнике пакетов проекта, поскольку он показывает только файлы. И несмотря на то, что он менее связан, чем внутренние классы, это не очень полезно: если класс меняется, вам нужно объединить / передать весь родительский файл в систему управления версиями. Если вы будете использовать его повторно, то вам нужно будет получить к нему доступ как Parent.Nested везде. Таким образом, чтобы не связывать другие виды деятельности с Parent класс, вы, вероятно, хотели бы реорганизовать его и извлечь вложенный класс в его собственный файл.
Так что для меня вопрос был бы Inner Class против класса верхнего уровня.
Связанная статья уже говорит это
Это, тем не менее, подчеркивает, что вы хотите, чтобы doInBackground() вашей AsyncTask была полностью отделена от Activity. Если вы коснетесь своей Активности только в главном потоке приложения, ваша AsyncTask может пережить изменение ориентации без изменений.
Не трогайте активность (например, ее участников) из AsyncTask, что соответствует статическим вложенным классам
Статические Вложенные Классы
Как и в случае методов и переменных класса, статический вложенный класс связан с его внешним классом. И, как и методы статического класса, статический вложенный класс не может напрямую ссылаться на переменные экземпляра или методы, определенные в включающем его классе - он может использовать их только через ссылку на объект.
Хотя в примерах из Android, справки AsyncTask и использования AsyncTask все еще используются нестатические вложенные классы.
И почему этот Java-класс вложен в Java? Я сначала пошел бы со статическим внутренним классом и прибегнул бы к нестатической версии только, если это действительно необходимо.
Я обнаружил, что нестатическое обновление вложенного пользовательского интерфейса Asynctask происходит быстрее, когда вам нужно часто обновлять пользовательский интерфейс, вызывая 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");
});
}
Это в 1000 раз быстрее, чем реализация слушателей для статического AsyncTask. Я могу ошибаться, но это был мой опыт.
static:
@Override
protected void onProgressUpdate(String... values) {
OnTaskStringUpdatedListener.OnTaskStringUpdated(task, values[0]);
}