एंड्रॉयड। विखंडन getActivity () कभी-कभी अशक्त लौटता है

Question

डेवलपर कंसोल त्रुटि रिपोर्ट में कभी-कभी मुझे NPE समस्या वाली रिपोर्ट दिखाई देती है। मुझे समझ नहीं आ रहा है कि मेरे कोड में क्या गलत है। एमुलेटर पर और मेरा डिवाइस एप्लिकेशन बिना संदंश के अच्छा काम करता है, हालांकि गेटअक्टिविटी () विधि कहा जाता है, तो कुछ उपयोगकर्ताओं को खंड वर्ग में NullPointerException मिलती है।

गतिविधि

pulic class MyActivity extends FragmentActivity{

    private ViewPager pager; 
    private TitlePageIndicator indicator;
    private TabsAdapter adapter;

    @Override
    public void onCreate(Bundle savedInstanceState) {
        pager = (ViewPager) findViewById(R.id.pager);
        indicator = (TitlePageIndicator) findViewById(R.id.indicator);
        adapter = new TabsAdapter(getSupportFragmentManager(), false);

        adapter.addFragment(new FirstFragment());
        adapter.addFragment(new SecondFragment());
        indicator.notifyDataSetChanged();
        adapter.notifyDataSetChanged();

        // push first task
        FirstTask firstTask = new FirstTask(MyActivity.this);
        // set first fragment as listener
        firstTask.setTaskListener((TaskListener) adapter.getItem(0));
        firstTask.execute();
    }

    indicator.setOnPageChangeListener(new ViewPager.OnPageChangeListener()  {
        @Override
        public void onPageSelected(int position) {
            Fragment currentFragment = adapter.getItem(position);
            ((Taskable) currentFragment).executeTask();
        }

        @Override
        public void onPageScrolled(int i, float v, int i1) {}

        @Override
        public void onPageScrollStateChanged(int i) {}
    });
}

AsyncTask क्लास

public class FirstTask extends AsyncTask{

    private TaskListener taskListener;

    ...

    @Override
    protected void onPostExecute(T result) {
        ... 
        taskListener.onTaskComplete(result);
    }   
}

फ्रैगमेंट क्लास

public class FirstFragment extends Fragment immplements Taskable, TaskListener{

    public FirstFragment() {
    }

    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) {
        return inflater.inflate(R.layout.first_view, container, false);
    }

    @Override
    public void executeTask() {
        FirstTask firstTask = new FirstTask(MyActivity.this);
        firstTask.setTaskListener(this);
        firstTask.execute();
    }

    @Override
    public void onTaskComplete(T result) {
        // NPE is here 
        Resources res = getActivity().getResources();
        ...
    }
}

हो सकता है कि यह त्रुटि तब होती है जब अनुप्रयोग पृष्ठभूमि से फिर से शुरू होता है। इस मामले में मुझे इस स्थिति को कैसे ठीक से संभालना चाहिए?

Comments

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

— जॉर्जी गोबोजोव

जब मैं बैकग्राउंड फ्रेग्मेंट से अपना आवेदन फिर से शुरू करता हूं, तो एक onResume को एक्टिविटी onCreate / onResume विधि से पहले बुलाया जाता है। ऐसा लगता है कि कुछ अलग किए गए टुकड़े अभी भी जीवित हैं और फिर से शुरू करने की कोशिश कर रहे हैं।

— जार्ज गोबोजोव

इस स्ट्रिंग में firstTask.setTaskListener ((TaskListener) अनुकूलक.getItem (0)); अनुकूलक ।getItem (0) पुराने टुकड़े को लौटाएं, एडॉप्टर सही तरीके से टुकड़ों को नहीं हटाता है

— Georgy Gobozov

जिस तरह से महान गतिविधि :) सवाल पूछा, टिप्पणी छोड़ दिया और जवाब दिया - सभी एक ही व्यक्ति द्वारा किया जाता है! इनके लिए +1।

— प्राइजऑफ

ConCxt (getActivity ()) को onCreateView () में सेव करें क्योंकि जब बैकग्राउंड केस में व्यू रिक्रिएट किया जाता है तो इसे ही कहते हैं।

— शा

Answer 1

ऐसा लगता है कि मुझे अपनी समस्या का हल मिल गया। बहुत अच्छी व्याख्याएँ यहाँ और यहाँ दी गई हैं । यहाँ मेरा उदाहरण है:

pulic class MyActivity extends FragmentActivity{

private ViewPager pager; 
private TitlePageIndicator indicator;
private TabsAdapter adapter;
private Bundle savedInstanceState;

 @Override
public void onCreate(Bundle savedInstanceState) {

    .... 
    this.savedInstanceState = savedInstanceState;
    pager = (ViewPager) findViewById(R.id.pager);;
    indicator = (TitlePageIndicator) findViewById(R.id.indicator);
    adapter = new TabsAdapter(getSupportFragmentManager(), false);

    if (savedInstanceState == null){    
        adapter.addFragment(new FirstFragment());
        adapter.addFragment(new SecondFragment());
    }else{
        Integer  count  = savedInstanceState.getInt("tabsCount");
        String[] titles = savedInstanceState.getStringArray("titles");
        for (int i = 0; i < count; i++){
            adapter.addFragment(getFragment(i), titles[i]);
        }
    }


    indicator.notifyDataSetChanged();
    adapter.notifyDataSetChanged();

    // push first task
    FirstTask firstTask = new FirstTask(MyActivity.this);
    // set first fragment as listener
    firstTask.setTaskListener((TaskListener) getFragment(0));
    firstTask.execute();

}

private Fragment getFragment(int position){
     return savedInstanceState == null ? adapter.getItem(position) : getSupportFragmentManager().findFragmentByTag(getFragmentTag(position));
}

private String getFragmentTag(int position) {
    return "android:switcher:" + R.id.pager + ":" + position;
}

 @Override
protected void onSaveInstanceState(Bundle outState) {
    super.onSaveInstanceState(outState);
    outState.putInt("tabsCount",      adapter.getCount());
    outState.putStringArray("titles", adapter.getTitles().toArray(new String[0]));
}

 indicator.setOnPageChangeListener(new ViewPager.OnPageChangeListener() {
        @Override
        public void onPageSelected(int position) {
            Fragment currentFragment = adapter.getItem(position);
            ((Taskable) currentFragment).executeTask();
        }

        @Override
        public void onPageScrolled(int i, float v, int i1) {}

        @Override
        public void onPageScrollStateChanged(int i) {}
 });

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

अपडेट करें

इसके अलावा, यह एक अच्छा अभ्यास है जब getActivity () कहा जाता है इससे पहले टुकड़ों को जांचने के लिए उपयोग किया जाता है। यह एक अशक्त सूचक अपवाद से बचने में मदद करता है जब टुकड़ा गतिविधि से अलग हो जाता है। उदाहरण के लिए, एक गतिविधि में एक टुकड़ा हो सकता है जो एक async कार्य को धकेलता है। जब कार्य समाप्त हो जाता है, तो onTaskComplete श्रोता को बुलाया जाता है।

@Override
public void onTaskComplete(List<Feed> result) {

    progress.setVisibility(View.GONE);
    progress.setIndeterminate(false);
    list.setVisibility(View.VISIBLE);

    if (isAdded()) {

        adapter = new FeedAdapter(getActivity(), R.layout.feed_item, result);
        list.setAdapter(adapter);
        adapter.notifyDataSetChanged();
    }

}

यदि हम विखंडन को खोलते हैं, एक कार्य को धक्का देते हैं, और फिर जल्दी से एक पिछली गतिविधि पर लौटने के लिए वापस दबाते हैं, जब कार्य समाप्त हो जाता है, तो यह getActivity () विधि को कॉल करके onPostExecute () में गतिविधि तक पहुंचने का प्रयास करेगा। यदि गतिविधि पहले से ही अलग है और यह जाँच नहीं है:

if (isAdded()) 

तब अनुप्रयोग क्रैश हो जाता है।

Comments

यद्यपि यह कष्टप्रद है, isAdded()प्रत्येक एक्सेस से पहले कॉल करना ... कोड को बदसूरत बनाता है।

— Ixx

वहाँ होने के बीच कोई अंतर नहीं लगता हैif(isAdded())if(getActivity() != null)

— StackOverflowed

Answer 2

ठीक है, मुझे पता है कि यह सवाल वास्तव में हल है लेकिन मैंने इसके लिए अपना समाधान साझा करने का फैसला किया है। मैंने अपने लिए अमूर्त अभिभावक वर्ग बनाया है Fragment:

public abstract class ABaseFragment extends Fragment{

    protected IActivityEnabledListener aeListener;

    protected interface IActivityEnabledListener{
        void onActivityEnabled(FragmentActivity activity);
    }

    protected void getAvailableActivity(IActivityEnabledListener listener){
        if (getActivity() == null){
            aeListener = listener;

        } else {
            listener.onActivityEnabled(getActivity());
        }
    }

    @Override
    public void onAttach(Activity activity) {
        super.onAttach(activity);

        if (aeListener != null){
            aeListener.onActivityEnabled((FragmentActivity) activity);
            aeListener = null;
        }
    }

    @Override
    public void onAttach(Context context) {
        super.onAttach(context);

        if (aeListener != null){
            aeListener.onActivityEnabled((FragmentActivity) context);
            aeListener = null;
        }
    }
}

जैसा कि आप देख सकते हैं, मैंने एक श्रोता जोड़ा है, इसलिए जब भी मुझे Fragments Activityमानक के बजाय प्राप्त करने की getActivity()आवश्यकता होगी, मुझे कॉल करना होगा

 getAvailableActivity(new IActivityEnabledListener() {
        @Override
        public void onActivityEnabled(FragmentActivity activity) {
            // Do manipulations with your activity
        }
    });

Comments

बहुत बढ़िया जवाब! वास्तविक समस्या हल करने के बाद से इसे सही के रूप में चिह्नित किया जाना चाहिए: मेरे मामले में यह जांचने के लिए पर्याप्त नहीं है कि getActivity () शून्य नहीं है क्योंकि मुझे अपना काम पूरा करना होगा चाहे कोई भी हो। यह प्रयोग कर रहा हूँ और यह पूरी तरह से काम करता है।

— हादस कमिंसकी

Answer 3

इससे छुटकारा पाने के लिए सबसे अच्छा है कि जब onAttachकॉल किया जाए तो गतिविधि का संदर्भ रखें और उदाहरण के लिए जहाँ भी ज़रूरत हो गतिविधि गतिविधि का उपयोग करें

@Override
public void onAttach(Context context) {
    super.onAttach(context);
    mContext = context;
}

@Override
public void onDetach() {
    super.onDetach();
    mContext = null;
}

संपादित, चूंकि onAttach(Activity)मूल्यह्रास किया गया है और अब onAttach(Context)इसका उपयोग किया जा रहा है

Comments

टुकड़े हमेशा अपनी मूल गतिविधि का संदर्भ रखते हैं और आपको getActivity () विधि के साथ उपलब्ध कराते हैं, यहां हम उसी संदर्भ को रख रहे हैं।

— पवन माहेश्वरी

यदि वास्तव में गतिविधि के साथ घटनाओं को साझा करने के लिए आपके टुकड़े की आवश्यकता हो तो Google इसकी अनुशंसा करता है। developer.android.com/guide/compenders/fragments.html ("गतिविधि के लिए ईवेंट कॉलबैक बनाना" के लिए देखें)

— Vering

आप onDetach मेथड जोड़ना चाह सकते हैं, जो गतिविधि के संदर्भ को कम करता है

— आधी रात

हाँ, गतिविधि गतिविधि संदर्भ को कम करने के लिए onDetach विधि पर mActivity = null को प्रारंभ करें।

— पवन माहेश्वरी

कभी ऐसा मत करो। आप अपनी पूरी गतिविधि लीक कर रहे हैं (और इसके साथ पूरे लेआउट ट्री, ड्रॉबल्स और इस तरह के साथ)। यदि getActivity()यह शून्य है, तो यह इसलिए है क्योंकि आप अब किसी गतिविधि में नहीं हैं। यह एक गंदा काम है।

— njzk2 19

Answer 4

मूल गतिविधि में onStart तक Fragment में getActivity () की आवश्यकता होने पर कॉल न करें।

private MyFragment myFragment;


public void onCreate(Bundle savedInstanceState)
{
    super.onCreate(savedInstanceState);

    FragmentTransaction ft = getSupportFragmentManager().beginTransaction();
    myFragment = new MyFragment();

    ft.add(android.R.id.content, youtubeListFragment).commit();

    //Other init calls
    //...
}


@Override
public void onStart()
{
    super.onStart();

    //Call your Fragment functions that uses getActivity()
    myFragment.onPageSelected();
}

Comments

वास्तव में, मुझे एक समान समस्या थी क्योंकि मैं खंड के निर्माण में कार्य शुरू कर रहा था। बहुत बहुत धन्यवाद।

— सुप्रीम डॉल्फिन

Answer 5

मैं थोड़ी देर के लिए इस तरह की समस्या से जूझ रहा हूं, और मुझे लगता है कि मैं एक विश्वसनीय समाधान के साथ आया हूं।

यह जानना बहुत मुश्किल है कि this.getActivity()किसी के लिए वापस नहीं जाना nullहै Fragment, खासकर यदि आप किसी भी तरह के नेटवर्क व्यवहार से निपट रहे हैं जो आपके कोड को Activityसंदर्भ वापस लेने के लिए पर्याप्त समय देता है ।

नीचे दिए गए समाधान में, मैं एक छोटे प्रबंधन वर्ग की घोषणा करता हूं जिसे बुलाया जाता है ActivityBuffer। अनिवार्य रूप से, यह classएक विश्वसनीय संदर्भ को बनाए रखने के साथ संबंधित है Activity, और जब भी कोई वैध संदर्भ उपलब्ध होता है, तो Runnableएक मान्य Activityसंदर्भ में निष्पादित करने का वादा करता है। Runnableरों यूआई धागा तुरंत अगर पर निष्पादन के लिए निर्धारित हैं Context, उपलब्ध है कि जब तक अन्यथा निष्पादन आस्थगित Contextतैयार है।

/** A class which maintains a list of transactions to occur when Context becomes available. */
public final class ActivityBuffer {

    /** A class which defines operations to execute once there's an available Context. */
    public interface IRunnable {
        /** Executes when there's an available Context. Ideally, will it operate immediately. */
        void run(final Activity pActivity);
    }

    /* Member Variables. */
    private       Activity        mActivity;
    private final List<IRunnable> mRunnables;

    /** Constructor. */
    public ActivityBuffer() {
        // Initialize Member Variables.
        this.mActivity  = null;
        this.mRunnables = new ArrayList<IRunnable>();
    }

    /** Executes the Runnable if there's an available Context. Otherwise, defers execution until it becomes available. */
    public final void safely(final IRunnable pRunnable) {
        // Synchronize along the current instance.
        synchronized(this) {
            // Do we have a context available?
            if(this.isContextAvailable()) {
                // Fetch the Activity.
                final Activity lActivity = this.getActivity();
                // Execute the Runnable along the Activity.
                lActivity.runOnUiThread(new Runnable() { @Override public final void run() { pRunnable.run(lActivity); } });
            }
            else {
                // Buffer the Runnable so that it's ready to receive a valid reference.
                this.getRunnables().add(pRunnable);
            }
        }
    }

    /** Called to inform the ActivityBuffer that there's an available Activity reference. */
    public final void onContextGained(final Activity pActivity) {
        // Synchronize along ourself.
        synchronized(this) {
            // Update the Activity reference.
            this.setActivity(pActivity);
            // Are there any Runnables awaiting execution?
            if(!this.getRunnables().isEmpty()) {
                // Iterate the Runnables.
                for(final IRunnable lRunnable : this.getRunnables()) {
                    // Execute the Runnable on the UI Thread.
                    pActivity.runOnUiThread(new Runnable() { @Override public final void run() {
                        // Execute the Runnable.
                        lRunnable.run(pActivity);
                    } });
                }
                // Empty the Runnables.
                this.getRunnables().clear();
            }
        }
    }

    /** Called to inform the ActivityBuffer that the Context has been lost. */
    public final void onContextLost() {
        // Synchronize along ourself.
        synchronized(this) {
            // Remove the Context reference.
            this.setActivity(null);
        }
    }

    /** Defines whether there's a safe Context available for the ActivityBuffer. */
    public final boolean isContextAvailable() {
        // Synchronize upon ourself.
        synchronized(this) {
            // Return the state of the Activity reference.
            return (this.getActivity() != null);
        }
    }

    /* Getters and Setters. */
    private final void setActivity(final Activity pActivity) {
        this.mActivity = pActivity;
    }

    private final Activity getActivity() {
        return this.mActivity;
    }

    private final List<IRunnable> getRunnables() {
        return this.mRunnables;
    }

}

इसके कार्यान्वयन के संदर्भ में, हमें पवन एम द्वारा वर्णित व्यवहार के साथ मेल खाने के लिए जीवन चक्र विधियों को लागू करने का ध्यान रखना चाहिए :

public class BaseFragment extends Fragment {

    /* Member Variables. */
    private ActivityBuffer mActivityBuffer;

    public BaseFragment() {
        // Implement the Parent.
        super();
        // Allocate the ActivityBuffer.
        this.mActivityBuffer = new ActivityBuffer();
    }

    @Override
    public final void onAttach(final Context pContext) {
        // Handle as usual.
        super.onAttach(pContext);
        // Is the Context an Activity?
        if(pContext instanceof Activity) {
            // Cast Accordingly.
            final Activity lActivity = (Activity)pContext;
            // Inform the ActivityBuffer.
            this.getActivityBuffer().onContextGained(lActivity);
        }
    }

    @Deprecated @Override
    public final void onAttach(final Activity pActivity) {
        // Handle as usual.
        super.onAttach(pActivity);
        // Inform the ActivityBuffer.
        this.getActivityBuffer().onContextGained(pActivity);
    }

    @Override
    public final void onDetach() {
        // Handle as usual.
        super.onDetach();
        // Inform the ActivityBuffer.
        this.getActivityBuffer().onContextLost();
    }

    /* Getters. */
    public final ActivityBuffer getActivityBuffer() {
        return this.mActivityBuffer;
    }

}

अंत में, आपके भीतर के किसी भी क्षेत्र में Fragmentयह फैली हुई है BaseFragmentकि आप कॉल के बारे में अविश्वसनीय हैं getActivity(), बस एक कॉल करें this.getActivityBuffer().safely(...)और ActivityBuffer.IRunnableकार्य के लिए घोषणा करें !

आपकी सामग्री void run(final Activity pActivity)को तब UI थ्रेड के साथ निष्पादित करने की गारंटी दी जाती है।

ActivityBufferतो इस प्रकार के रूप में इस्तेमाल किया जा सकता है:

this.getActivityBuffer().safely(
  new ActivityBuffer.IRunnable() {
    @Override public final void run(final Activity pActivity) {
       // Do something with guaranteed Context.
    }
  }
);

Comments

क्या आप इस .getActivityBuffer () का उपयोग करके एक उदाहरण जोड़ सकते हैं। सुरक्षित रूप से (...) विधि।

— fahad_sust

Answer 6

@Override
public void onActivityCreated(Bundle savedInstanceState) {
    super.onActivityCreated(savedInstanceState);
    // run the code making use of getActivity() from here
}

Comments

क्या आप अपने समाधान के बारे में थोड़ा और विवरण जोड़कर अपने जवाब को विस्तृत कर सकते हैं?

— अबरीसन

Answer 7

मुझे पता है कि यह एक पुराना सवाल है, लेकिन मुझे लगता है कि मुझे इसका जवाब देना होगा क्योंकि मेरी समस्या दूसरों द्वारा हल नहीं की गई थी।

सबसे पहले: मैं टुकड़े टुकड़े का उपयोग करके गतिशील रूप से टुकड़े जोड़ रहा था। दूसरा: मेरे टुकड़े को AsyncTasks (एक सर्वर पर DB प्रश्नों) का उपयोग करके संशोधित किया गया था। तीसरा: गतिविधि शुरू होने पर मेरे टुकड़े को तुरंत नहीं लगाया गया था: मैंने खंड चर प्राप्त करने के लिए एक कस्टम टुकड़ा इंस्टेंटेशन "इसे बनाएं या लोड करें" का उपयोग किया। चौथा: अभिविन्यास परिवर्तन के कारण गतिविधि को फिर से बनाया गया था

समस्या यह थी कि मैं क्वेरी उत्तर के कारण टुकड़े को "निकालना" चाहता था, लेकिन टुकड़ा गलत तरीके से पहले ही बनाया गया था। मुझे नहीं पता कि, शायद "कमिट" के कारण बाद में किया गया था, जब इसे हटाने का समय था, तो टुकड़ा अभी तक जोड़ा नहीं गया था। इसलिए getActivity () अशक्त लौट रही थी।

समाधान: 1) मुझे यह जांचना था कि मैं सही ढंग से एक नया बनाने से पहले टुकड़े के पहले उदाहरण को खोजने की कोशिश कर रहा था 2) मुझे उस खंड पर serRetainInstance (सच्चा) लगाना था ताकि इसे अभिविन्यास परिवर्तन (बैकस्टैक नहीं) के माध्यम से रखा जा सके इसलिए कोई समस्या नहीं है) 3) "इसे हटाने" से ठीक पहले "पुराने खंडन" को पुन: प्राप्त करने या प्राप्त करने के बजाय, मैंने सीधे गतिविधि प्रारंभ में टुकड़ा डाल दिया। गतिविधि पर इसे तुरंत शुरू करने से पहले इसे लोड करने से पहले "लोड" (या तत्काल) करने के बजाय खंड परिवर्तन को रोका जाता है।

Answer 8

Kotlin में आप getActivity () अशक्त स्थिति को संभालने के लिए इस तरह की कोशिश कर सकते हैं।

   activity.let { // activity == getActivity() in java

        //your code here

   }

यह जाँच करेगा कि गतिविधि शून्य है या नहीं और यदि अशक्त नहीं है तो आंतरिक कोड निष्पादित करें।