SwipeRefreshLayoutの中にあるRecyclerViewにLayoutManagerをセットする前にタッチすると落ちる
SwipeRefreshLayoutの中にRecyclerViewのあるFragmentがあって、 データの読み込みを待ってから、Adapterなどをセットしようと考えていたら、 データが読み込まれる前にRecyclerViewの部分をタッチすると落ちてしまっていた。
E/AndroidRuntime: FATAL EXCEPTION: main java.lang.NullPointerException at android.support.v7.widget.RecyclerView.computeVerticalScrollOffset(RecyclerView.java:1613) at android.view.View.canScrollVertically(View.java:11214) at android.support.v4.view.ViewCompatICS.canScrollVertically(ViewCompatICS.java:35) at android.support.v4.view.ViewCompat$ICSViewCompatImpl.canScrollVertically(ViewCompat.java:1253) at android.support.v4.view.ViewCompat.canScrollVertically(ViewCompat.java:1695) at android.support.v4.widget.SwipeRefreshLayout.canChildScrollUp(SwipeRefreshLayout.java:646) at android.support.v4.widget.SwipeRefreshLayout.onInterceptTouchEvent(SwipeRefreshLayout.java:660) at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:1822)
これによると、RecyclerViewはLayoutManagerを設定していない状態でタッチすると落ちるみたいです。
ReyclerViewだけが置いてあっても特にデータやAdapterがなければレイアウト上に配置もされない*1ので、触っても落ちることはなく、ちょっと切り分けができてませんでした!!
単体だけあると配置されないのに、SwipeRefreshLayoutなどスクロールするViewGroupの中に置くと、Adapterなどを設定する前に配置されてしまうようです。
なので、いつもそうなんですが、SwipeRefreshLayoutなどのViewGroupの中に設置する場合は特に、 データの取得より前にとりあえずonActivityCreated()あたりで、LayoutManagerをセットしておいたほうがよさそうです。
// 触ると落ちるコードの例 package com.example.woshidan.donttouchrecyclerview; import android.os.Bundle; import android.support.design.widget.FloatingActionButton; import android.support.design.widget.Snackbar; import android.support.v7.app.AppCompatActivity; import android.support.v7.widget.RecyclerView; import android.support.v7.widget.Toolbar; import android.view.View; import android.view.Menu; import android.view.MenuItem; public class MainActivity extends AppCompatActivity { @Override protected void onCreate(Bundle savedInstanceState) { super.onCreate(savedInstanceState); setContentView(R.layout.activity_main); // RecylerViewにLayoutManagerも何もセットしない }
<?xml version="1.0" encoding="utf-8"?> <RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android" xmlns:tools="http://schemas.android.com/tools" xmlns:app="http://schemas.android.com/apk/res-auto" android:layout_width="match_parent" android:layout_height="match_parent" android:paddingLeft="@dimen/activity_horizontal_margin" android:paddingRight="@dimen/activity_horizontal_margin" android:paddingTop="@dimen/activity_vertical_margin" android:paddingBottom="@dimen/activity_vertical_margin" app:layout_behavior="@string/appbar_scrolling_view_behavior" tools:showIn="@layout/activity_main" tools:context="com.example.woshidan.donttouchrecyclerview.MainActivity"> <android.support.v4.widget.SwipeRefreshLayout android:id="@+id/swipeRefreshLayout" android:layout_width="match_parent" android:layout_height="match_parent"> <android.support.v7.widget.RecyclerView android:id="@+id/recyclerView" android:layout_width="match_parent" android:layout_height="match_parent"></android.support.v7.widget.RecyclerView> </android.support.v4.widget.SwipeRefreshLayout> <TextView android:text="Hello World!" android:layout_width="wrap_content" android:layout_height="wrap_content" /> </RelativeLayout>
*1:E/RecyclerView: No adapter attached; skipping layoutとログに出ます