woshidan's loose leaf

ぼんやり勉強しています

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)

http://stackoverflow.com/questions/27416834/app-crashing-when-trying-to-use-recyclerview-on-android-5-0

これによると、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とログに出ます