Eventbus от Greenrobot: два фрагмента, вызывающие задания асинхронных задач, запутали Eventbus

В настоящее время я работаю над Fragments, который был создан под Activities с FragmentPagerAdapter. Я использовал EventBus 3.0 GreenRobot для возврата некоторых AsyncTasks, которые я создал из класса обслуживания. Однако, поскольку эти два фрагмента создаются один за другим, возврат прикрепленного сообщения шины событий приводит к перепутыванию подписанных фрагментов. Я просмотрел stackoverflow, сделал то же, что и другие (например, поместил регистр eventBus в onStart и OnStop и т. д.), и я до сих пор не вижу даже подобной проблемы. Надеюсь, вы можете мне помочь. Спасибо!

Вот мои два фрагмента: (я пропустил некоторые ненужные коды)

<сильный>1. Фрагмент статуса

@Override
public View onCreateView(LayoutInflater inflater, ViewGroup container,
                         Bundle savedInstanceState) {
    Log.e(TAG, "StatusFragment onCreateView");
    eventBus = EventBus.getDefault();
    eventBus.register(this);
    StatusService statusService = StatusService.newInstance(getContext());
    statusService.getStatusList(); //Async Task to call onReturnAdapter once finished
    return mRecyclerView;
}

@Subscribe(sticky = true, threadMode = ThreadMode.MAIN)
public void onReturnList(List<Status> statuses) {
    Log.e(TAG, "onReturnList");
    mAdapter = new StatusRecyclerAdapter(statuses);
    mRecyclerView.setAdapter(mAdapter);
    eventBus.removeStickyEvent(statuses);
}

@Override
public void onStop() {
    Log.e(TAG, "onStop");
    eventBus.unregister(this);
    super.onStop();
}

<сильный>2. Служба статуса — класс службы для фрагмента статуса

public void getStatusList () {
    Log.e(LOG_TAG, "getStatusList");
    //do some async tasks here
    returnList(statuses);

}

private void returnList(List<Status> statuses) {
    Log.e(LOG_TAG, "returnList");
    eventBus.postSticky(statuses);
}

<сильный>3. Фрагмент диаграммы

@Override
public View onCreateView(LayoutInflater inflater, ViewGroup container,
                         Bundle savedInstanceState) {

    Log.e(LOG_TAG, "onCreateView");

    eventBus = EventBus.getDefault();
    eventBus.register(this);

    chartService = ChartService.newInstance(getContext());
    chartService.getDateResult(new DbRequestFeelings());
    return view;
}

@Subscribe(sticky = true, threadMode = ThreadMode.MAIN)
public void onReturnResults(List<Result> results) {
    Log.e(LOG_TAG, "onReturnResults");

    chartService.setupPieChart(results, feelingsPieChart);
}

@Override
public void onStop() {
    eventBus.unregister(this);
    super.onStop();
}

<сильный>4. Служба карт

public void getDateResult(final DbRequest dbRequest) {
    Log.e(LOG_TAG, "getDateResult");
    //do some async tasks
    returnResults(results);
}

private void returnResults(List<Result> results) {
    Log.e(LOG_TAG, "returnResults");
    eventBus.postSticky(results);
}

<сильный>5. Класс результата

public class Result {
      ....
}

<сильный>6. Класс статуса

public class Status {
    ...
}

Таким образом, каждый раз, когда создаются фрагменты, они вызывают свой сервисный класс, который, в свою очередь, публикует List<Status> и List<Results> практически один за другим. Из-за этого возникла путаница, где говорится, что

03-11 11:48:19.685  15148-15148/com.paularagones.moode E/EventBus﹕ Could not dispatch event: class java.util.ArrayList to subscribing class class com.paularagones.moode.Fragments.ChartFragment
    java.lang.ClassCastException: com.paularagones.moode.Models.Status cannot be cast to com.paularagones.moode.Models.Result

Вот полная трассировка стека:

03-11 11:48:19.589  15148-15148/com.paularagones.moode E/Moode-StatusFragment﹕ StatusFragment onCreate
03-11 11:48:19.589  15148-15148/com.paularagones.moode E/Moode-StatusFragment﹕ StatusFragment onCreateView
03-11 11:48:19.605  15148-15148/com.paularagones.moode E/StatusService﹕ getStatusList
03-11 11:48:19.609  15148-15148/com.paularagones.moode E/ChartFragment﹕ onCreateView
03-11 11:48:19.613  15148-15148/com.paularagones.moode E/ChartService﹕ newInstance
03-11 11:48:19.621  15148-15148/com.paularagones.moode E/ChartService﹕ getDateResult
03-11 11:48:19.681  15148-15148/com.paularagones.moode E/RecyclerView﹕ No adapter attached; skipping layout
03-11 11:48:19.681  15148-15148/com.paularagones.moode E/StatusService﹕ returnList
03-11 11:48:19.681  15148-15148/com.paularagones.moode E/Moode-StatusFragment﹕ onReturnList
03-11 11:48:19.681  15148-15148/com.paularagones.moode E/ChartFragment﹕ onReturnResults
03-11 11:48:19.685  15148-15148/com.paularagones.moode E/EventBus﹕ Could not dispatch event: class java.util.ArrayList to subscribing class class com.paularagones.moode.Fragments.ChartFragment
    java.lang.ClassCastException: com.paularagones.moode.Models.Status cannot be cast to com.paularagones.moode.Models.Result
            at com.paularagones.moode.Services.ChartService.getData(ChartService.java:147)
            at com.paularagones.moode.Services.ChartService.setupPieChart(ChartService.java:129)
            at com.paularagones.moode.Fragments.ChartFragment.onReturnResults(ChartFragment.java:116)
            at java.lang.reflect.Method.invokeNative(Native Method)
            at java.lang.reflect.Method.invoke(Method.java:511)
            at org.greenrobot.eventbus.EventBus.invokeSubscriber(EventBus.java:485)
            at org.greenrobot.eventbus.EventBus.postToSubscription(EventBus.java:420)
            at org.greenrobot.eventbus.EventBus.postSingleEventForEventType(EventBus.java:397)
            at org.greenrobot.eventbus.EventBus.postSingleEvent(EventBus.java:370)
            at org.greenrobot.eventbus.EventBus.post(EventBus.java:251)
            at org.greenrobot.eventbus.EventBus.postSticky(EventBus.java:292)
            at com.paularagones.moode.Services.StatusService.returnList(StatusService.java:80)
            at com.paularagones.moode.Services.StatusService.access$100(StatusService.java:24)
            at com.paularagones.moode.Services.StatusService$2.onNext(StatusService.java:72)
            at com.paularagones.moode.Services.StatusService$2.onNext(StatusService.java:59)
            at rx.Observable$30.onNext(Observable.java:8069)
            at rx.observers.SafeSubscriber.onNext(SafeSubscriber.java:139)
            at rx.internal.operators.OperatorObserveOn$ObserveOnSubscriber.pollQueue(OperatorObserveOn.java:202)
            at rx.internal.operators.OperatorObserveOn$ObserveOnSubscriber$2.call(OperatorObserveOn.java:162)
            at rx.internal.schedulers.ScheduledAction.run(ScheduledAction.java:55)
            at android.os.Handler.handleCallback(Handler.java:615)
            at android.os.Handler.dispatchMessage(Handler.java:92)
            at android.os.Looper.loop(Looper.java:137)
            at android.app.ActivityThread.main(ActivityThread.java:4745)
            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:786)
            at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:553)
            at dalvik.system.NativeStart.main(Native Method)
03-11 11:48:19.685  15148-15148/com.paularagones.moode E/ChartService﹕ returnResults
03-11 11:48:19.685  15148-15148/com.paularagones.moode E/Moode-StatusFragment﹕ onReturnList
03-11 11:48:19.685  15148-15148/com.paularagones.moode E/ChartFragment﹕ onReturnResults
03-11 11:48:19.693  15148-15148/com.paularagones.moode E/StatusRecyclerAdapter﹕ onCreateViewHolder
03-11 11:48:19.697  15148-15148/com.paularagones.moode E/StatusRecyclerAdapter﹕ onBindViewHolder
03-11 11:48:19.709  15148-15148/com.paularagones.moode E/AndroidRuntime﹕ FATAL EXCEPTION: main
    java.lang.ClassCastException: com.paularagones.moode.Models.Result cannot be cast to com.paularagones.moode.Models.Status
            at com.paularagones.moode.Adapters.StatusRecyclerAdapter.onBindViewHolder(StatusRecyclerAdapter.java:101)
            at com.paularagones.moode.Adapters.StatusRecyclerAdapter.onBindViewHolder(StatusRecyclerAdapter.java:28)
            at android.support.v7.widget.RecyclerView$Adapter.onBindViewHolder(RecyclerView.java:5277)
            at android.support.v7.widget.RecyclerView$Adapter.bindViewHolder(RecyclerView.java:5310)
            at android.support.v7.widget.RecyclerView$Recycler.getViewForPosition(RecyclerView.java:4568)
            at android.support.v7.widget.RecyclerView$Recycler.getViewForPosition(RecyclerView.java:4461)
            at android.support.v7.widget.LinearLayoutManager$LayoutState.next(LinearLayoutManager.java:1962)
            at android.support.v7.widget.LinearLayoutManager.layoutChunk(LinearLayoutManager.java:1371)
            at android.support.v7.widget.LinearLayoutManager.fill(LinearLayoutManager.java:1334)
            at android.support.v7.widget.LinearLayoutManager.onLayoutChildren(LinearLayoutManager.java:563)
            at android.support.v7.widget.RecyclerView.dispatchLayout(RecyclerView.java:2847)
            at android.support.v7.widget.RecyclerView.onLayout(RecyclerView.java:3145)
            at android.view.View.layout(View.java:13754)
            at android.view.ViewGroup.layout(ViewGroup.java:4362)
            at android.support.v4.view.ViewPager.onLayout(ViewPager.java:1627)
            at android.view.View.layout(View.java:13754)
            at android.view.ViewGroup.layout(ViewGroup.java:4362)
            at android.support.design.widget.CoordinatorLayout.layoutChild(CoordinatorLayout.java:1034)
            at android.support.design.widget.CoordinatorLayout.onLayoutChild(CoordinatorLayout.java:744)
            at android.support.design.widget.ViewOffsetBehavior.onLayoutChild(ViewOffsetBehavior.java:42)
            at android.support.design.widget.AppBarLayout$ScrollingViewBehavior.onLayoutChild(AppBarLayout.java:1180)
            at android.support.design.widget.CoordinatorLayout.onLayout(CoordinatorLayout.java:757)
            at android.view.View.layout(View.java:13754)
            at android.view.ViewGroup.layout(ViewGroup.java:4362)
            at android.widget.FrameLayout.onLayout(FrameLayout.java:448)
            at android.view.View.layout(View.java:13754)
            at android.view.ViewGroup.layout(ViewGroup.java:4362)
            at android.widget.LinearLayout.setChildFrame(LinearLayout.java:1649)
            at android.widget.LinearLayout.layoutVertical(LinearLayout.java:1507)
            at android.widget.LinearLayout.onLayout(LinearLayout.java:1420)
            at android.view.View.layout(View.java:13754)
            at android.view.ViewGroup.layout(ViewGroup.java:4362)
            at android.widget.FrameLayout.onLayout(FrameLayout.java:448)
            at android.view.View.layout(View.java:13754)
            at android.view.ViewGroup.layout(ViewGroup.java:4362)
            at android.widget.LinearLayout.setChildFrame(LinearLayout.java:1649)
            at android.widget.LinearLayout.layoutVertical(LinearLayout.java:1507)
            at android.widget.LinearLayout.onLayout(LinearLayout.java:1420)
            at android.view.View.layout(View.java:13754)
            at android.view.ViewGroup.layout(ViewGroup.java:4362)
            at android.widget.FrameLayout.onLayout(FrameLayout.java:448)
            at android.view.View.layout(View.java:13754)
            at android.view.ViewGroup.layout(ViewGroup.java:4362)
            at android.view.ViewRootImpl.performLayout(ViewRootImpl.java:1866)
            at android.view.ViewRootImpl.performTraversals(ViewRootImpl.java:1687)
            at android.view.ViewRootImpl.doTraversal(ViewRootImpl.java:998)
            at android.view.ViewRootImpl$TraversalRunnable.run(ViewRootImpl.java:4212)
            at android.view.Choreographer$CallbackRecord.run(Choreographer.java:725)
            at android.view.Choreographer.doCallbacks(Choreographer.java:555)
            at android.view.Choreographer.doFrame(Choreographer.java:525)
            at android.view.Choreographer$FrameDisplayEventReceiver.run(Choreographer.java:711)
            at android.os.Handler.handleCallback(Handler.java:615)
            at android.os.Handler.dispatchMessage(Handler.java:92)
            at android.os.Looper.loop(Looper.java:137)
            at android.app.ActivityThread.main(ActivityThread.java:4745)
            at java.lang.reflect.Method.invokeNative(Native Method)

Ребята, вы знаете, что не так с моим кодом? Спасибо.


android android-fragments greenrobot-eventbus greenrobot-eventbus-3.0
person Paula Kristin    schedule 11.03.2016    source источник
comment
и как выглядят классы модели Result и Status? (код методов не имеет значения, только class ....... строка для обоих)   -  person Marcin Orlowski    schedule 11.03.2016
comment
@MarcinOrlowski спасибо, что указали на это. Я добавил необходимые модели.   -  person Paula Kristin    schedule 11.03.2016
comment
Какая точная версия EventBus?   -  person Marcin Orlowski    schedule 11.03.2016
comment
В отношении «липких» событий, как показано в этом ответе SO, упоминается, когда подписчик регистрируется с помощью registerSticky, также запускается доставка последнего закрепленного события - возможно, это следует учитывать.   -  person Bret    schedule 11.03.2016
comment
@MarcinOrlowski это eventBus 3.0   -  person Paula Kristin    schedule 12.03.2016
comment
@Брет, спасибо, я проверил. Это тот случай, когда мои последние события были запущены снова?   -  person Paula Kristin    schedule 14.03.2016
comment
Я бы посоветовал избегать липких, если это не абсолютно необходимо (на основе отброшенных событий из-за проблем со временем) - это одно - однако есть две другие вещи, на которые я хотел бы указать - я опубликую ответ, поэтому форматирование делает ответ понятным.   -  person Bret    schedule 14.03.2016

Ответы (1)


arrow_upward
4
arrow_downward

Первое, что я заметил, это то, что вы не удаляете липкое в своем методе onReturnResults(List<Result> results) ... не уверен, специально это или нет ... но подумал, что об этом следует упомянуть.

Главный момент, который я хотел подчеркнуть, касается наличия двух методов onEvent(), использующих один и тот же общий класс List<> — это вполне может быть проблемой — небольшое изменение кода и быстрый тест могут проверить это.

Создайте class для каждого List<>

public class Statuses
{
    List<Status> list;
    public Statuses(List<Status> data) { list = data; }
}

public class Results
{
    List<Result> list;
    public Results(List<Result> data) { list = data; }
}

Обновите сигнатуры методов onEvent():

public void onReturnList(Statuses statuses)
public void onReturnResults(Results results)

Обновите вызовы метода .postSticky():

eventBus.postSticky(new Statuses(statuses));
eventBus.postSticky(new Results(results));

Конечно, вам нужно будет получить доступ к спискам из своего члена класса (например):

foreach (var status in statuses.list)

а также

foreach (var result in results.list)

Если проблема, которую вы видите, просто исчезает после внесения этих изменений, то имеет смысл предположить, что EventBus greenrobot не учитывает тип, содержащийся в универсальном классе List, во время его вызовов отражения, наблюдаемых в стеке вызовов. Он может просто искать первую подходящую подпись, содержащую аргумент List; и когда есть 2 или более совпадений, правильный может просто не быть вызван.

Это догадки; так что вам придется попробовать и посмотреть, поможет ли это - но я успешно использовал EventBus greenrobot в проекте, использующем этот тип реализации, когда мне нужны списки классов, чтобы избежать двусмысленности.

person Bret    schedule 14.03.2016
comment
постараюсь проверить это позже и свяжусь с вами, если это работает или нет. - person Paula Kristin; 17.03.2016
comment
Рад это слышать. Не будете ли вы так любезны принять ответ? Спасибо. - person Bret; 13.05.2016