OnSwipeRefreshLayout не прекращает обновление

Question

У меня есть OnSwipeRefreshLayout для RecyclerView во фрагменте. Фрагмент реализует OnRefreshListener. Я вызываю метод из фрагмента в качестве метода обновления. Я обнаружил, что он не устанавливает OnRefreshlistener, даже если нет ошибок. В приложении оно не перестанет обновляться, потому что OnRefreshListener никогда не устанавливается.

XML:

<android.support.v4.widget.SwipeRefreshLayout xmlns:android="http://schemas.android.com/apk/res/android"
android:id="@+id/SwipeRefresher"
android:layout_width="match_parent"
android:layout_height="match_parent">

<android.support.v7.widget.RecyclerView
    android:id="@+id/channel_list"
    android:scrollbars="vertical"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    />

</android.support.v4.widget.SwipeRefreshLayout>

Фрагмент:

public static class tabFragment extends Fragment implements SwipeRefreshLayout.OnRefreshListener {

    public static final String ARG_OBJECT = "object";
    public int tabPosition=0;
    RecyclerView mRecyclerView;
    RecyclerView.LayoutManager mLayoutManager;
    ChannelAdapter mAdapter;
    SwipeRefreshLayout mSwipeRefreshLayout;


    public void updateData(){
        mAdapter.retrieveData();
        mAdapter.notifyDataSetChanged();
    }

    @Override
    public void onCreate(@Nullable Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        savedInstanceState = getArguments();
        tabPosition = savedInstanceState.getInt(ARG_OBJECT);

    }

    public void refreshData(){
        mAdapter.retrieveData();
        mSwipeRefreshLayout.setRefreshing(false);
    }

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

        final View v = inflater.inflate(R.layout.page_main, container, false);
        mSwipeRefreshLayout = v.findViewById(R.id.SwipeRefresher);
        mSwipeRefreshLayout = new SwipeRefreshLayout(getContext());
        mSwipeRefreshLayout.setOnRefreshListener(this);
        mSwipeRefreshLayout.setColorSchemeColors(getResources().getColor(R.color.color_accent));


        mRecyclerView = v.findViewById(R.id.channel_list);
        mRecyclerView.setHasFixedSize(true);
        mLayoutManager = new LinearLayoutManager(this.getContext());
        mRecyclerView.setLayoutManager(mLayoutManager);
        mAdapter = new ChannelAdapter(tabPosition, new ChannelAdapter.OnItemClickListener() {
            @Override
            public void onItemClick(String channel, int tabPosition) {
                switch (tabPosition) {
                    case 0:
                        Intent intent = new Intent(getActivity(), Messenger.class);
                        intent.putExtra("currentChannel", channel);
                        getActivity().startActivity(intent);
                        break;
                    case 1:
                        //todo: display channel information probably
                        break;
                    case 2:
                        //request user info
                        break;
                }

            }
        });

        mRecyclerView.setAdapter(mAdapter);
        return v;
    }


    @Override
    public void onRefresh() {
        refreshData();
    }
}

Answer 1

просто прокомментируйте эту строку

mSwipeRefreshLayout = new SwipeRefreshLayout(getContext());

строка выше создает новый экземпляр SwipeRefreshLayout, поэтому mSwipeRefreshLayout будет ссылаться на другой объект, а не на mSwipeRefreshLayout, который есть в вашем фрагменте

Answer 2

Вы создаете отдельный объект

mSwipeRefreshLayout = new SwipeRefreshLayout(getContext());

, который не имеет ничего общего с макетом на экране, вместо этого используйте предыдущую ссылку, которая использует findViewById для получения ссылки из v вида, который будет отображаться как макет фрагмента, так что

    mSwipeRefreshLayout = v.findViewById(R.id.SwipeRefresher);
    // mSwipeRefreshLayout = new SwipeRefreshLayout(getContext()); remove this
    mSwipeRefreshLayout.setOnRefreshListener(this);