2015-09-18 2 views
0

У меня есть фрагмент. В этом фрагменте я запускаю HTTP-запрос на json-rpc. Чтобы обработать результат, у меня есть что-то подобное в моем обратном вызове.Как class.this в обратном вызове может быть нулевым?

FragmentClass.this.getActivity().runOnUiThread(new Runnable() { 
    @Override 
    public void run() { 
     // Do something 
    } 
}); 

Проблема, иногда я получаю NullPointerException на первой линии ... Мой первый намереваюсь был, что фрагмент отделились поститься, может быть, потому что пользователь выбирает другой фрагмент в то время как запрос работает и так

FragmentClass.this.getActivity(); 

не имеет деятельности и возвращает null. Я прилагаю все это с, если так:

// New if: 
if (FragmentClass.this.getActivity() != null) { 

    FragmentClass.this.getActivity().runOnUiThread(new Runnable() { 
     @Override 
     public void run() { 
      // Do something 
     } 
    }); 

} 

Но ... Ничего ... Сейчас я получаю NullPointerException на если заявление. представляется, что

FragmentClass.this 

есть null.

Как это возможно. Я думал, что экземпляр будет удерживаться до тех пор, пока часть кода не будет нуждаться в нем, и gc соберет его ...

Вот стек, который дает мне logcat. Я изменил имя пакета и имена классов. Строка 192 - это строка оператора if.

09-18 10:49:36.915 3860-3860/de.unkown.app E/AndroidRuntime﹕ FATAL EXCEPTION: main 
    java.lang.NullPointerException 
      at de.unkown.app.camera.#FragmentClass$6.onError(FragmentClass.java:192) 
      at de.unkown.app.webservice.JsonRpcService$10.onError(JsonRpcService.java:497) 
      at de.unkown.app.webservice.JsonRpcService$DefaultErrorListener.onErrorResponse(JsonRpcService.java:107) 
       at com.android.volley.Request.deliverError(Request.java:577) 
      at com.android.volley.ExecutorDelivery$ResponseDeliveryRunnable.run(ExecutorDelivery.java:101) 
      at android.os.Handler.handleCallback(Handler.java:725) 
      at android.os.Handler.dispatchMessage(Handler.java:92) 
      at android.os.Looper.loop(Looper.java:137) 
      at android.app.ActivityThread.main(ActivityThread.java:5041) 
      at java.lang.reflect.Method.invokeNative(Native Method) 
      at java.lang.reflect.Method.invoke(Method.java:511) 
      at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:793) 
      at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:560) 
      at dalvik.system.NativeStart.main(Native Method) 

Благодарим за помощь!

Artur

+0

Где находится трассировка стека? –

+0

Попробуйте 'getActivity()' вместо 'FragmentClass.this.getActivity()' –

+0

Почему бы просто не использовать getActivity()? –

ответ

-1

Ok,

я изменил во всех асинхронных преграждает

FragmentClass.this.getActivity() 

в

getActivity() 

Это швы, чтобы помочь. Пожалуйста, не спрашивайте меня, почему ... Я думал, что это должно быть то же самое ...

0

создать глобальную переменную Контекстный контекст;

чем инициализировать переменную на Create метод:

контекст = получить активность();

чем использовать его как:

context. runOnUiThread (new Runnable() { 
    @Override 
    public void run() { 
     // Do something 
    } 
}); 

надеюсь, это поможет вам.

+0

Прежде всего создание глобальной переменной Контекстный контекст - плохая идея. Это может привести к утечкам памяти и ошибкам времени выполнения, особенно когда у вас есть несколько фрагментов в представлении пейджера. Во-вторых, для контекста всегда используйте '' 'getContext()' '' Если вы используете AsyncTask и хотите использовать контекст в методах AsyncTask, передайте контекст как конечную переменную параметра метода методу, охватывающему AsyncTask, и использовать в переопределенных методах AsyncTask. Если вы используете getContext() в методах AsyncTask, это может привести к NullPointerException – developer1011

+0

context.runOnUiThread даже не компилируется. Это требует getActivity. Поэтому ниже код является безопасным для использования '' ', если (isAdded()) getActivity() runOnUiThread (новый Runnable() { @Override общественности недействительным запуска() { // Сделайте что-нибудь } }). '' ' – developer1011

0

getActivity() возвращает null в onCreate и onCreateView методов.
Использовать getActivity() внутри onViewCreated. Хорошо использовать код, используя getActivity с чеком - if (isAdded()) { }.

В вашем случае вы получаете нуль, потому что к тому времени, как вы вызываете FragmentClass.this.getActivity(), ваш фрагмент больше не отображается пользователю, по существу, onStop уже был вызван.

Для дальнейшего добавить к моему ответу -
Предположим, вы вызываете метод DoSomething() на фрагменте, в котором вы выполняете AsyncTask. Затем передайте параметр YourFragmentClass.this в качестве параметра метода в doSomething(YourFragmentClass thiz) и используйте этот объект в своих методах doInBackground() и onPostExecute() вместо использования YourFragmentClass.this, поскольку YourFragmentClass.this возвращает значение null, поскольку к моменту запуска AsynTask ваш фрагмент переместился бы на onPause или OnStop.

Смежные вопросы