1

这是不好的做法吗？

import android.app.Activity;
import android.os.AsyncTask;

public abstract class Asynch extends Activity {
    public abstract void doInBackgroundAbstract();
    public abstract void onPostExecuteAbstract();
    protected class AsynchtaskSubClass extends AsyncTask {
        @Override
        protected Object doInBackground(Object... arg0) {
            doInBackgroundAbstract();
            return null;

        }
        protected void onPostExecute(Object result){
            onPostExecuteAbstract();
        }
    }

}

public class SomeActivity extends Asynch {

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_show_classes);
        new AsynchtaskSubClass().execute();
    }
            private Document doc;
        @Override
        public void doInBackgroundAbstract() {
            // NETWORK OPERATION GOES HERE!
            try {
                doc = Jsoup.connect("mySite").get();
            } catch (IOException e) {
                // TODO Auto-generated catch block
                e.printStackTrace();
            }

        }
        @Override
        public void onPostExecuteAbstract() {
            // Update UI Thread Here
            TextView myTextView = new TextView(SomeActivity.this);
            myTextView.setText(doc.select("a[href]").attr("abs:href").toString());
            setContentView(myTextView);

        }

这对我来说看起来更干净。我不必继续创建内部异步任务或创建单独的类来运行异步任务。如果我想拥有一个在许多活动中重用的异步任务，那么我只需扩展异步并覆盖抽象方法并将其保留为单独的类。他们这样做有什么问题吗？我看到的唯一问题是，如果我需要在一个活动中使用两个或多个异步任务。

4

1 回答 1

0

正如您已经指出的那样，您的活动中不能有多个异步任务。除此之外，您的班级只能扩展一个其他班级，而不是更多。因此，如果您想要一个扩展 ListActivity 并具有异步任务的活动，则必须创建另一个抽象异步（“listActivityAsynch”）类，依此类推。
另一件事是，仅通过查看 SomeActivity，其他程序员不容易理解您的代码。他们首先必须查看异步并了解您使用的不常见设计。如果您是唯一使用此代码的人，那可能没问题。但如果其他人必须使用它，最好遵循常见的 android 实践，即在内部类中进行。

于 2014-03-14T11:20:22.197 回答