Android पर AsyncTask और एरर हैंडलिंग


147

मैं अपने कोड को उपयोग Handlerकरने से परिवर्तित कर रहा हूं AsyncTask। उत्तरार्द्ध यह क्या करता है पर महान है - मुख्य UI थ्रेड में अतुल्यकालिक अद्यतन और परिणामों की हैंडलिंग। मेरे लिए क्या अस्पष्ट है अपवादों को कैसे संभालना है अगर कोई चीज़ अंदर जाती है AsyncTask#doInBackground

जिस तरह से मैं कर रहा हूं वह एक त्रुटि हैंडलर है और इसे संदेश भेजने के लिए है। यह ठीक काम करता है, लेकिन क्या यह "सही" दृष्टिकोण है या बेहतर विकल्प है?

यह भी मैं समझता हूं कि यदि मैं त्रुटि हैंडलर को गतिविधि क्षेत्र के रूप में परिभाषित करता हूं, तो इसे UI थ्रेड में निष्पादित करना चाहिए। हालांकि, कभी-कभी (बहुत अप्रत्याशित रूप से) मुझे एक अपवाद मिलेगा, जो यह कहता है कि इससे उत्पन्न कोड Handler#handleMessageगलत थ्रेड पर निष्पादित हो रहा है। क्या मुझे Activity#onCreateइसके बजाय त्रुटि हैंडलर को इनिशियलाइज़ करना चाहिए ? जगह runOnUiThreadमें Handler#handleMessageबेमानी लगता है, लेकिन यह बहुत मज़बूती से निष्पादित करता है।


आप अपना कोड क्यों बदलना चाहते हैं? क्या कोई अच्छा कारण था?
एचजीपीबी

4
@ हेराल्डो यह एक बेहतर कोडिंग अभ्यास है, कम से कम मुझे कैसा लगता है
बोसोन

जवाबों:


178

यह ठीक काम करता है लेकिन क्या यह "सही" दृष्टिकोण है और क्या बेहतर विकल्प है?

मैं पर पकड़ Throwableया Exceptionमें AsyncTaskउदाहरण ही है और उसके बाद में इसके साथ कुछ करना onPostExecute()है, तो मेरी त्रुटि से निपटने के लिए स्क्रीन पर एक संवाद प्रदर्शित का विकल्प है।


8
प्रतिभाशाली! अब हैंडलर के साथ बंदर को कोई ज़रूरत नहीं है
बॉस्सोन

5
क्या इस तरह मुझे थ्रोबेबल या अपवाद पर पकड़ चाहिए? "अपने खुद के AsyncTask उपवर्ग में एक इंस्टेंस-वैरिएबल जोड़ें जो आपके बैकग्राउंड प्रोसेसिंग का परिणाम देगा।" जब आपको कोई अपवाद मिलता है, तो इस चर में अपवाद (या कुछ अन्य त्रुटि-स्ट्रिंग / कोड) को संग्रहीत करें। जब onPostExecute कहा जाता है, तो देखें कि क्या यह उदाहरण-चर कुछ त्रुटि पर सेट है। यदि ऐसा है, तो एक त्रुटि संदेश दिखाएं। "(उपयोगकर्ता" बोस्टन की सड़कें " समूहों से। googlegroup
android

1
@OneWorld: हाँ, यह ठीक होना चाहिए।
कॉमन्सवेयर

2
हाय सीडब्ल्यू, क्या आप कृपया इसे और अधिक विस्तार से करने के अपने तरीके को समझा सकते हैं - शायद एक संक्षिप्त कोड उदाहरण के साथ? बहुत बहुत धन्यवाद!!
ब्रुइसर

18
@ क्रूजर: github.com/commonsguy/cw-lunchlist/tree/master/15-Internet/… में एक AsyncTaskपैटर्न है जिसका मैं वर्णन करता हूं।
कॉमन्सवेयर

140

एक AsyncResult ऑब्जेक्ट बनाएँ (जिसे आप अन्य प्रोजेक्ट्स में भी उपयोग कर सकते हैं)

public class AsyncTaskResult<T> {
    private T result;
    private Exception error;

    public T getResult() {
        return result;
    }

    public Exception getError() {
        return error;
    }

    public AsyncTaskResult(T result) {
        super();
        this.result = result;
    }

    public AsyncTaskResult(Exception error) {
        super();
        this.error = error;
    }
}

इस ऑब्जेक्ट को अपने AsyncTask doInBackground तरीकों से लौटाएं और पोस्टएक्स्यूट में चेक करें। (आप अपने अन्य एसिंक्स कार्यों के लिए इस वर्ग को आधार वर्ग के रूप में उपयोग कर सकते हैं)

नीचे एक कार्य का मॉकअप है जो वेब सर्वर से JSON प्रतिक्रिया प्राप्त करता है।

AsyncTask<Object,String,AsyncTaskResult<JSONObject>> jsonLoader = new AsyncTask<Object, String, AsyncTaskResult<JSONObject>>() {

        @Override
        protected AsyncTaskResult<JSONObject> doInBackground(
                Object... params) {
            try {
                // get your JSONObject from the server
                return new AsyncTaskResult<JSONObject>(your json object);
            } catch ( Exception anyError) {
                return new AsyncTaskResult<JSONObject>(anyError);
            }
        }

        protected void onPostExecute(AsyncTaskResult<JSONObject> result) {
            if ( result.getError() != null ) {
                // error handling here
            }  else if ( isCancelled()) {
                // cancel handling here
            } else {

                JSONObject realResult = result.getResult();
                // result handling here
            }
        };

    }

1
मुझें यह पसंद है। अच्छा एनकैप्सुलेशन। चूँकि यह मूल उत्तर का एक दृष्टांत है, उत्तर ठहरता है, लेकिन यह निश्चित रूप से एक बिंदु के योग्य है
बोसोन

जेनरिक कितना उपयोगी हो सकता है, यह एक बहुत अच्छा प्रदर्शन है। यह जटिलता के संदर्भ में एक अजीब गंध फेंक रहा है, लेकिन एक तरह से मैं वास्तव में स्पष्ट नहीं कर सकता।
num1

4
अच्छा विचार है, केवल एक प्रश्न: तुम क्यों कहते हैं super()में AsyncTaskResultजब वर्ग कुछ भी विस्तार नहीं करता है?
डेटोर्नर

7
"कोई नुकसान नहीं" - अनावश्यक कोड पठनीयता और रखरखाव के लिए हमेशा हानिकारक होता है। वहां से निकल जाओ! :)
10

2
वास्तव में समाधान पसंद आया ... इसके बारे में सोचने के लिए आ रहा है - सी # लोगों ने सी # संगत बैकग्राउंड मूल कार्यान्वयन में बिल्कुल एक ही विधि का उपयोग किया ...
Vova

11

जब मुझे अपवादों को AsyncTaskठीक से संभालने की आवश्यकता महसूस होती है , तो मैं इसे सुपर क्लास के रूप में उपयोग करता हूं:

public abstract class ExceptionAsyncTask<Params, Progress, Result> extends AsyncTask<Params, Progress, Result> {

    private Exception exception=null;
    private Params[] params;

    @Override
    final protected Result doInBackground(Params... params) {
        try {
            this.params = params; 
            return doInBackground();
        }
        catch (Exception e) {
            exception = e;
            return null;
        }
    }

    abstract protected Result doInBackground() throws Exception;

    @Override
    final protected void onPostExecute(Result result) {
        super.onPostExecute(result);
        onPostExecute(exception, result);
    }

    abstract protected void onPostExecute(Exception exception, Result result);

    public Params[] getParams() {
        return params;
    }

}

सामान्य रूप से, आप doInBackgroundपृष्ठभूमि के काम करने के लिए अपने उपवर्ग में ओवरराइड करते हैं, जहां जरूरत है वहां खुशी से फेंकना। फिर आपको लागू करने के लिए मजबूर किया जाता है onPostExecute(क्योंकि यह सार है) और यह धीरे-धीरे आपको सभी प्रकार के हैंडल करने के लिए याद दिलाता है Exception, जो पैरामीटर के रूप में पारित किए जाते हैं। ज्यादातर मामलों में, अपवाद कुछ प्रकार के यूआई आउटपुट का नेतृत्व करते हैं, इसलिए ऐसा करने के onPostExecuteलिए एक सही जगह है।


1
वाह, क्यों न केवल paramsआगे पास किया जाए, इसलिए यह मूल के समान है और माइग्रेट करना आसान है?
टीडब्लूआरआरओबी

@TWiStErRob उस विचार के साथ कुछ भी गलत नहीं है। यह मेरी व्यक्तिगत पसंद की बात है, मुझे लगता है कि मैं परम का उपयोग नहीं करता हूं। मैं पसंद new Task("Param").execute()से अधिक new Task().execute("Param")
सुलई

5

यदि आप RoboGuice फ्रेमवर्क का उपयोग करना चाहते हैं जो आपके लिए अन्य लाभ लाता है तो आप RoboAsyncTask को आज़मा सकते हैं जिसमें अतिरिक्त कॉलबैक onException () है। वास्तविक अच्छा काम करता है और मैं इसका उपयोग करता हूं। http://code.google.com/p/roboguice/wiki/RoboAsyncTask


इस बारे में आपका क्या अनुभव है? बहुत स्थिर?
निकुन्नसन

है RoboGuiceअब भी जिंदा? 2012 के बाद से अपडेट नहीं किया गया है?
दिमित्री के

नहीं, RoboGuice मृत और पदावनत है। डैगर 2 अनुशंसित प्रतिस्थापन है, लेकिन यह केवल नंगे हड्डियों वाला डीआई लाइब्रेरी है।
एवी चेरी

3

मैंने अपनी खुद की AsyncTask को एक ऐसे इंटरफेस के साथ उपवर्ग बना दिया जो सफलता और असफलता के लिए कॉलबैक को परिभाषित करता है। इसलिए यदि आपके AsyncTask में एक अपवाद रखा गया है, तो OnFailure फ़ंक्शन अपवाद को पास कर देता है, अन्यथा onSuccess callback आपके परिणाम को पारित कर देता है। Android के पास कुछ बेहतर क्यों उपलब्ध नहीं है, यह मेरे से परे है।

public class SafeAsyncTask<inBackgroundType, progressType, resultType>
extends AsyncTask<inBackgroundType, progressType, resultType>  {
    protected Exception cancelledForEx = null;
    protected SafeAsyncTaskInterface callbackInterface;

    public interface SafeAsyncTaskInterface <cbInBackgroundType, cbResultType> {
        public Object backgroundTask(cbInBackgroundType[] params) throws Exception;
        public void onCancel(cbResultType result);
        public void onFailure(Exception ex);
        public void onSuccess(cbResultType result);
    }

    @Override
    protected void onPreExecute() {
        this.callbackInterface = (SafeAsyncTaskInterface) this;
    }

    @Override
    protected resultType doInBackground(inBackgroundType... params) {
        try {
            return (resultType) this.callbackInterface.backgroundTask(params);
        } catch (Exception ex) {
            this.cancelledForEx = ex;
            this.cancel(false);
            return null;
        }
    }

    @Override
    protected void onCancelled(resultType result) {
        if(this.cancelledForEx != null) {
            this.callbackInterface.onFailure(this.cancelledForEx);
        } else {
            this.callbackInterface.onCancel(result);
        }
    }

    @Override
    protected void onPostExecute(resultType result) {
        this.callbackInterface.onSuccess(result);
    }
}

3

Cagatay Kalan के समाधान का अधिक व्यापक समाधान नीचे दिखाया गया है:

AsyncTaskResult

public class AsyncTaskResult<T> 
{
    private T result;
    private Exception error;

    public T getResult() 
    {
        return result;
    }

    public Exception getError() 
    {
        return error;
    }

    public AsyncTaskResult(T result) 
    {
        super();
        this.result = result;
    }

    public AsyncTaskResult(Exception error) {
        super();
        this.error = error;
    }
}

ExceptionHandlingAsyncTask

public abstract class ExceptionHandlingAsyncTask<Params, Progress, Result> extends AsyncTask<Params, Progress, AsyncTaskResult<Result>>
{
    private Context context;

    public ExceptionHandlingAsyncTask(Context context)
    {
        this.context = context;
    }

    public Context getContext()
    {
        return context;
    }

    @Override
    protected AsyncTaskResult<Result> doInBackground(Params... params)
    {
        try
        {
            return new AsyncTaskResult<Result>(doInBackground2(params));
        }
        catch (Exception e)
        {
            return new AsyncTaskResult<Result>(e);
        }
    }

    @Override
    protected void onPostExecute(AsyncTaskResult<Result> result)
    {
        if (result.getError() != null)
        {
            onPostException(result.getError());
        }
        else
        {
            onPostExecute2(result.getResult());
        }
        super.onPostExecute(result);
    }

    protected abstract Result doInBackground2(Params... params);

    protected abstract void onPostExecute2(Result result);

    protected void onPostException(Exception exception)
    {
                        new AlertDialog.Builder(context).setTitle(R.string.dialog_title_generic_error).setMessage(exception.getMessage())
                .setIcon(android.R.drawable.ic_dialog_alert).setPositiveButton(R.string.alert_dialog_ok, new DialogInterface.OnClickListener()
                {
                    public void onClick(DialogInterface dialog, int which)
                    {
                        //Nothing to do
                    }
                }).show();
    }
}

उदाहरण कार्य

public class ExampleTask extends ExceptionHandlingAsyncTask<String, Void, Result>
{
    private ProgressDialog  dialog;

    public ExampleTask(Context ctx)
    {
        super(ctx);
        dialog = new ProgressDialog(ctx);
    }

    @Override
    protected void onPreExecute()
    {
        dialog.setMessage(getResources().getString(R.string.dialog_logging_in));
        dialog.show();
    }

    @Override
    protected Result doInBackground2(String... params)
    {
        return new Result();
    }

    @Override
    protected void onPostExecute2(Result result)
    {
        if (dialog.isShowing())
            dialog.dismiss();
        //handle result
    }

    @Override
    protected void onPostException(Exception exception)
    {
        if (dialog.isShowing())
            dialog.dismiss();
        super.onPostException(exception);
    }
}

मुझे myActivity.getApplicationContext () के रूप में getResources () विधि मिल गई। getResources ()
स्टीफन

2

यह साधारण वर्ग आपकी मदद कर सकता है

public abstract class ExceptionAsyncTask<Param, Progress, Result, Except extends Throwable> extends AsyncTask<Param, Progress, Result> {
    private Except thrown;

    @SuppressWarnings("unchecked")
    @Override
    /**
     * Do not override this method, override doInBackgroundWithException instead
     */
    protected Result doInBackground(Param... params) {
        Result res = null;
        try {
            res = doInBackgroundWithException(params);
        } catch (Throwable e) {
            thrown = (Except) e;
        }
        return res;
    }

    protected abstract Result doInBackgroundWithException(Param... params) throws Except;

    @Override
    /**
     * Don not override this method, override void onPostExecute(Result result, Except exception) instead
     */
    protected void onPostExecute(Result result) {
        onPostExecute(result, thrown);
        super.onPostExecute(result);
    }

    protected abstract void onPostExecute(Result result, Except exception);
}

2

एक अन्य तरीका जो चर सदस्य साझाकरण पर निर्भर नहीं करता है वह है रद्द करना।

यह android डॉक्स से है:

सार्वजनिक अंतिम बूलियन रद्द (बूलियन mayInterruptIfRunning)

इस कार्य का निष्पादन रद्द करने का प्रयास। यह प्रयास विफल हो जाएगा यदि कार्य पहले ही पूरा हो चुका है, पहले से ही रद्द हो चुका है, या किसी अन्य कारण से रद्द नहीं किया जा सकता है। यदि सफल हो, और रद्द किए जाने पर यह कार्य प्रारंभ नहीं हुआ है, तो यह कार्य कभी भी नहीं चलना चाहिए। यदि कार्य पहले ही शुरू हो चुका है, तो mayInterruptIfRunning पैरामीटर निर्धारित करता है कि क्या कार्य को रोकने के प्रयास में इस कार्य को निष्पादित करने वाले थ्रेड को बाधित किया जाना चाहिए।

इस विधि को कॉल करने से onInCancelled (ऑब्जेक्ट) UI थ्रेड पर doInBackground (ऑब्जेक्ट []] रिटर्न के बाद आह्वान किया जाएगा। इस पद्धति को कॉल करना यह गारंटी देता है कि onPostExecute (ऑब्जेक्ट) कभी भी लागू नहीं है। इस विधि को लागू करने के बाद, आपको कार्य को जल्द से जल्द पूरा करने के लिए समय-समय पर doInBackground (ऑब्जेक्ट []) से isCancelled () द्वारा लौटाए गए मान की जांच करनी चाहिए।

तो आप कैच स्टेटमेंट में कॉल को रद्द कर सकते हैं और यह सुनिश्चित कर सकते हैं कि onPostExcute को कभी भी कॉल न किया जाए, लेकिन इसके बजाय UI थ्रेड पर onCancelled है। तो आप त्रुटि संदेश दिखा सकते हैं।


आप त्रुटि संदेश को ठीक से नहीं दिखा सकते हैं, क्योंकि आप समस्या (अपवाद) नहीं जानते हैं, फिर भी आपको एक AsyncTechResult को पकड़ना और वापस करना होगा। इसके अलावा, रद्द करने वाला उपयोगकर्ता कोई त्रुटि नहीं है, यह एक अपेक्षित सहभागिता है: आप इनमें से कैसे अंतर करते हैं?
ट्वीस्टरेब

cancel(boolean)onCancelled()शुरुआत से ही एक कॉल मौजूद onCancelled(Result)था , लेकिन एपीआई 11 में जोड़ा गया था
टीडब्लूआरआरओबी

0

वास्तव में, AsyncTask FutureTask और Executor का उपयोग करते हैं, FutureTask समर्थन अपवाद-श्रृंखला पहले हम एक सहायक वर्ग को परिभाषित करते हैं

public static class AsyncFutureTask<T> extends FutureTask<T> {

    public AsyncFutureTask(@NonNull Callable<T> callable) {
        super(callable);
    }

    public AsyncFutureTask<T> execute(@NonNull Executor executor) {
        executor.execute(this);
        return this;
    }

    public AsyncFutureTask<T> execute() {
        return execute(AsyncTask.THREAD_POOL_EXECUTOR);
    }

    @Override
    protected void done() {
        super.done();
        //work done, complete or abort or any exception happen
    }
}

दूसरा, उपयोग करते हैं

    try {
        Log.d(TAG, new AsyncFutureTask<String>(new Callable<String>() {
            @Override
            public String call() throws Exception {
                //throw Exception in worker thread
                throw new Exception("TEST");
            }
        }).execute().get());
    } catch (InterruptedException e) {
        e.printStackTrace();
    } catch (ExecutionException e) {
        //catch the exception throw by worker thread in main thread
        e.printStackTrace();
    }

-2

व्यक्तिगत रूप से, मैं इस दृष्टिकोण का उपयोग करूंगा। यदि आपको जानकारी की आवश्यकता है तो आप केवल अपवादों को पकड़ सकते हैं और स्टैक ट्रेस का प्रिंट आउट ले सकते हैं।

पृष्ठभूमि में अपना काम करें एक बूलियन मान लौटाएं।

ऐसी बात हे:

    @Override
                protected Boolean doInBackground(String... params) {
                    return readXmlFromWeb(params[0]);
         }

        @Override
                protected void onPostExecute(Boolean result) {

              if(result){
              // no error
               }
              else{
                // error handling
               }
}

-2

एक अन्य संभावना Objectरिटर्न प्रकार के रूप में उपयोग करने के लिए होगी , और onPostExecute()ऑब्जेक्ट प्रकार के लिए जांच में । यह छोटा है।

class MyAsyncTask extends AsyncTask<MyInObject, Void, Object> {

    @Override
    protected AsyncTaskResult<JSONObject> doInBackground(MyInObject... myInObjects) {
        try {
            MyOutObject result;
            // ... do something that produces the result
            return result;
        } catch (Exception e) {
            return e;
        }
    }

    protected void onPostExecute(AsyncTaskResult<JSONObject> outcome) {
        if (outcome instanceof MyOutObject) {
            MyOutObject result = (MyOutObject) outcome;
            // use the result
        } else if (outcome instanceof Exception) {
            Exception e = (Exception) outcome;
            // show error message
        } else throw new IllegalStateException();
    }
}

1
पूरी तरह से अप्रासंगिक
दीनू

-2

यदि आप सही अपवाद जानते हैं तो आप कॉल कर सकते हैं

Exception e = null;

publishProgress(int ...);

उदाहरण के लिए:

@Override
protected Object doInBackground(final String... params) {

    // TODO Auto-generated method stub
    try {
        return mClient.call(params[0], params[1]);
    } catch(final XMLRPCException e) {

        // TODO Auto-generated catch block
        this.e = e;
        publishProgress(0);
        return null;
    }
}

और "onProgressUpdate" पर जाएं और फॉलिंग करें

@Override
protected void onProgressUpdate(final Integer... values) {

    // TODO Auto-generated method stub
    super.onProgressUpdate(values);
    mDialog.dismiss();
    OptionPane.showMessage(mActivity, "Connection error", e.getMessage());
}

यह कुछ मामलों में ही मददगार होगा। इसके अलावा, आप एक Global Exceptionचर रख सकते हैं और अपवाद तक पहुंच सकते हैं।


1
कृपया, यह मत करो। यह वास्तव में, वास्तव में, खराब शैली है!
जिमीबी
हमारी साइट का प्रयोग करके, आप स्वीकार करते हैं कि आपने हमारी Cookie Policy और निजता नीति को पढ़ और समझा लिया है।
Licensed under cc by-sa 3.0 with attribution required.