RecyclerViewの行内のハンドルボタンクリック


83

行のクリックを処理するために次のコードを使用しています。(ソース

static class RecyclerTouchListener implements RecyclerView.OnItemTouchListener {

    private GestureDetector gestureDetector;
    private ClickListener clickListener;

    public RecyclerTouchListener(Context context, final RecyclerView recyclerView, final ClickListener clickListener) {
        this.clickListener = clickListener;
        gestureDetector = new GestureDetector(context, new GestureDetector.SimpleOnGestureListener() {
            @Override
            public boolean onSingleTapUp(MotionEvent e) {
                return true;
            }

            @Override
            public void onLongPress(MotionEvent e) {
                View child = recyclerView.findChildViewUnder(e.getX(), e.getY());
                if (child != null && clickListener != null) {
                    clickListener.onLongClick(child, recyclerView.getChildPosition(child));
                }
            }
        });
    }

    @Override
    public boolean onInterceptTouchEvent(RecyclerView rv, MotionEvent e) {

        View child = rv.findChildViewUnder(e.getX(), e.getY());
        if (child != null && clickListener != null && gestureDetector.onTouchEvent(e)) {
            clickListener.onClick(child, rv.getChildPosition(child));
        }
        return false;
    }

    @Override
    public void onTouchEvent(RecyclerView rv, MotionEvent e) {
    }
}

ただし、各行に削除ボタンを設定したい場合は、これは機能します。これでそれを実装する方法がわかりません。

OnClickリスナーを機能する(行を削除する)削除ボタンにアタッチしましたが、行全体でonclickを起動します。

単一のボタンがクリックされた場合に行全体のクリックを回避する方法について、誰かが私を助けてくれますか?

ありがとう。

回答:


128

これは、recyclerView内で複数のonClickイベントを処理する方法です。

編集:コールバックを含むように更新されました(他のコメントで言及されているように)。でを使用しWeakReferenceて、ViewHolder潜在的なメモリリークを排除しました。

インターフェースの定義:

public interface ClickListener {

    void onPositionClicked(int position);
    
    void onLongClicked(int position);
}

次に、アダプタ:

public class MyAdapter extends RecyclerView.Adapter<MyAdapter.MyViewHolder> {
    
    private final ClickListener listener;
    private final List<MyItems> itemsList;

    public MyAdapter(List<MyItems> itemsList, ClickListener listener) {
        this.listener = listener;
        this.itemsList = itemsList;
    }

    @Override public MyViewHolder onCreateViewHolder(ViewGroup parent, int viewType) {
        return new MyViewHolder(LayoutInflater.from(parent.getContext()).inflate(R.layout.my_row_layout), parent, false), listener);
    }

    @Override public void onBindViewHolder(MyViewHolder holder, int position) {
        // bind layout and data etc..
    }

    @Override public int getItemCount() {
        return itemsList.size();
    }

    public static class MyViewHolder extends RecyclerView.ViewHolder implements View.OnClickListener, View.OnLongClickListener {

        private ImageView iconImageView;
        private TextView iconTextView;
        private WeakReference<ClickListener> listenerRef;

        public MyViewHolder(final View itemView, ClickListener listener) {
            super(itemView);

            listenerRef = new WeakReference<>(listener);
            iconImageView = (ImageView) itemView.findViewById(R.id.myRecyclerImageView);
            iconTextView = (TextView) itemView.findViewById(R.id.myRecyclerTextView);

            itemView.setOnClickListener(this);
            iconTextView.setOnClickListener(this);
            iconImageView.setOnLongClickListener(this);
        }

        // onClick Listener for view
        @Override
        public void onClick(View v) {

            if (v.getId() == iconTextView.getId()) {
                Toast.makeText(v.getContext(), "ITEM PRESSED = " + String.valueOf(getAdapterPosition()), Toast.LENGTH_SHORT).show();
            } else {
                Toast.makeText(v.getContext(), "ROW PRESSED = " + String.valueOf(getAdapterPosition()), Toast.LENGTH_SHORT).show();
            }
            
            listenerRef.get().onPositionClicked(getAdapterPosition());
        }


        //onLongClickListener for view
        @Override
        public boolean onLongClick(View v) {

            final AlertDialog.Builder builder = new AlertDialog.Builder(v.getContext());
            builder.setTitle("Hello Dialog")
                    .setMessage("LONG CLICK DIALOG WINDOW FOR ICON " + String.valueOf(getAdapterPosition()))
                    .setPositiveButton("OK", new DialogInterface.OnClickListener() {
                        @Override
                        public void onClick(DialogInterface dialog, int which) {

                        }
                    });

            builder.create().show();
            listenerRef.get().onLongClicked(getAdapterPosition());
            return true;
        }
    }
}

次に、アクティビティ/フラグメントで-実装できるものは何でも:Clicklistener-または必要に応じて匿名クラス:

MyAdapter adapter = new MyAdapter(myItems, new ClickListener() {
            @Override public void onPositionClicked(int position) {
                // callback performed on click
            }

            @Override public void onLongClicked(int position) {
                // callback performed on click
            }
        });

クリックされたアイテムを取得するには、ビューIDに一致しますievgetId()== whateverItem.getId()

このアプローチがお役に立てば幸いです。


1
おかげで、私は自分の実装にのみこのパターンを使用していました。しかし、問題は別のものでした。こちらをご覧くださいstackoverflow.com/questions/30287411/…–
Ashwani K 2015

1
「これ」はどこから引っ張られているのですか?コンテキストを一致させない限り、アダプター内にこれはありません。何らかの理由でそれを行うと、View.OnclickListenerもそれにキャストされます
Lion789 2016年

2
thisはそれ自体、ViewHolder(この場合は別の静的クラス)を参照しています-データがバインドされているビューホルダーにリスナーを設定していますonBindViewHolder()。これはアダプターのコンテキストとは関係ありません。あなたがどのような問題を抱えているのか正確にはわかりませんが、この解決策は問題なく機能します。
マークキーン2016年

1
@YasithaChinthakaandroid:background="?attr/selectableItemBackground"ビューのxmlでこの属性を設定してみましたか?
マークキーン2016年

2
立ち寄って感謝の気持ちを伝えたいだけです。このソリューションは非常に簡単に実行できます。
コードギアス2017年

51

私はそれを典型的に見つけます:

  • ボタンがいくつかあるので、複数のリスナーを使用する必要があります。
  • ロジックをアダプターやビューホルダーではなくアクティビティに含めたい。

したがって、@ mark-keenの答えはうまく機能しますが、インターフェイスがあると柔軟性が高まります。

public static class MyViewHolder extends RecyclerView.ViewHolder {

    public ImageView iconImageView;
    public TextView iconTextView;

    public MyViewHolder(final View itemView) {
        super(itemView);

        iconImageView = (ImageView) itemView.findViewById(R.id.myRecyclerImageView);
        iconTextView = (TextView) itemView.findViewById(R.id.myRecyclerTextView);

        iconTextView.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                onClickListener.iconTextViewOnClick(v, getAdapterPosition());
            }
        });
        iconImageView.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                onClickListener.iconImageViewOnClick(v, getAdapterPosition());
            }
        });
    }
}

onClickListenerがアダプターで定義されている場所:

public MyAdapterListener onClickListener;

public interface MyAdapterListener {

    void iconTextViewOnClick(View v, int position);
    void iconImageViewOnClick(View v, int position);
}

そして、おそらくコンストラクターを介して設定します。

public MyAdapter(ArrayList<MyListItems> newRows, MyAdapterListener listener) {

    rows = newRows;
    onClickListener = listener;
}

次に、アクティビティ内またはRecyclerViewが使用されている場所でイベントを処理できます。

mAdapter = new MyAdapter(mRows, new MyAdapter.MyAdapterListener() {
                    @Override
                    public void iconTextViewOnClick(View v, int position) {
                        Log.d(TAG, "iconTextViewOnClick at position "+position);
                    }

                    @Override
                    public void iconImageViewOnClick(View v, int position) {
                        Log.d(TAG, "iconImageViewOnClick at position "+position);
                    }
                });
mRecycler.setAdapter(mAdapter);

これは別の方法であり、私の答え(私が自分で使用しているものと同様のもの)の上に構築されていますonClickListenerが、静的にネストされたViewholderクラスにどのように渡していますか?何かが足りない場合を除いて、ViewHolderにどのように渡すかはわかりません。また、1つのインターフェースメソッドを使用する場合は、すべてを凝縮するLambda式を使用できます。
マークキーン

public MyAdapterListener onClickListener; 上記のコードのアダプターで定義され、アダプターコンストラクターで設定されるメンバー変数です。(または、上記には示されていませんが、setOnClickListenerのようなカスタムセッターを使用することもできます。)
LordParsley 2016

5
静的にネストされたクラス(ViewHolder)内からAdapterクラスのメンバー/インスタンス変数にどのようにアクセスしているかを尋ねていました。
マークキーン

マークのように、私はアダプターの中に入れ子にすることができませんでした。ネストを回避する方法についての私の回答を参照してください
Tony BenBrahim 2016年

@MarkKeen ..私が持っていたのとまったく同じ質問。
user2695433 2017年

6

後でガベージコレクションする必要のある余分なオブジェクト(つまりリスナー)を作成せず、アダプタークラス内にビューホルダーをネストする必要がないソリューションが必要でした。

ではViewHolderクラス

private static class MyViewHolder extends RecyclerView.ViewHolder implements View.OnClickListener {

        private final TextView ....// declare the fields in your view
        private ClickHandler ClickHandler;

        public MyHolder(final View itemView) {
            super(itemView);
            nameField = (TextView) itemView.findViewById(R.id.name);
            //find other fields here...
            Button myButton = (Button) itemView.findViewById(R.id.my_button);
            myButton.setOnClickListener(this);
        }
        ...
        @Override
        public void onClick(final View view) {
            if (clickHandler != null) {
                clickHandler.onMyButtonClicked(getAdapterPosition());
            }
        }

注意点:ClickHandlerインターフェースは定義されていますが、ここでは初期化されていないため、onClickメソッドにはこれまでに初期化されたという仮定はありません。

ClickHandlerインタフェースは、次のようになります。

private interface ClickHandler {
    void onMyButtonClicked(final int position);
} 

アダプターで、コンストラクターに「ClickHandler」のインスタンスを設定し、オーバーライドonBindViewHolderして、ビューホルダーの「clickHandler」を初期化します。

private class MyAdapter extends ...{

    private final ClickHandler clickHandler;

    public MyAdapter(final ClickHandler clickHandler) {
        super(...);
        this.clickHandler = clickHandler;
    }

    @Override
    public void onBindViewHolder(final MyViewHolder viewHolder, final int position) {
        super.onBindViewHolder(viewHolder, position);
        viewHolder.clickHandler = this.clickHandler;
    }

注:viewHolder.clickHandlerがまったく同じ値で複数回設定される可能性があることは知っていますが、これはnullや分岐をチェックするよりも安価であり、メモリコストはなく、追加の命令だけです。

最後に、アダプターを作成すると、次のClickHandlerようにインスタンスをコンストラクターに渡す必要があります。

adapter = new MyAdapter(new ClickHandler() {
    @Override
    public void onMyButtonClicked(final int position) {
        final MyModel model = adapter.getItem(position);
        //do something with the model where the button was clicked
    }
});

ここでadapterはメンバー変数であり、ローカル変数ではないことに注意してください


回答ありがとうございます:)ここに追加したいことの1つは、yourmodel.get(position)ではなくadapter.getItem(position)を使用しないことです
KhubaibRaza19年

5

すでにリサイクル業者のタッチリスナーがあり、ビューホルダーでボタンのタッチイベントを個別に処理するのではなく、その中のすべてのタッチイベントを処理したい場合は、別のソリューションを追加したかっただけです。この適応バージョンのクラスが行う重要なことは、アイテムコンテナではなく、タップされたときにonItemClick()コールバックでボタンビューを返すことです。次に、ビューがボタンであるかどうかをテストし、別のアクションを実行できます。ボタンを長押しすると、行全体を長押ししたと解釈されることに注意してください。

public class RecyclerItemClickListener implements RecyclerView.OnItemTouchListener
{
    public static interface OnItemClickListener
    {
        public void onItemClick(View view, int position);
        public void onItemLongClick(View view, int position);
    }

    private OnItemClickListener mListener;
    private GestureDetector mGestureDetector;

    public RecyclerItemClickListener(Context context, final RecyclerView recyclerView, OnItemClickListener listener)
    {
        mListener = listener;

        mGestureDetector = new GestureDetector(context, new GestureDetector.SimpleOnGestureListener()
        {
            @Override
            public boolean onSingleTapUp(MotionEvent e)
            {
                // Important: x and y are translated coordinates here
                final ViewGroup childViewGroup = (ViewGroup) recyclerView.findChildViewUnder(e.getX(), e.getY());

                if (childViewGroup != null && mListener != null) {
                    final List<View> viewHierarchy = new ArrayList<View>();
                    // Important: x and y are raw screen coordinates here
                    getViewHierarchyUnderChild(childViewGroup, e.getRawX(), e.getRawY(), viewHierarchy);

                    View touchedView = childViewGroup;
                    if (viewHierarchy.size() > 0) {
                        touchedView = viewHierarchy.get(0);
                    }
                    mListener.onItemClick(touchedView, recyclerView.getChildPosition(childViewGroup));
                    return true;
                }

                return false;
            }

            @Override
            public void onLongPress(MotionEvent e)
            {
                View childView = recyclerView.findChildViewUnder(e.getX(), e.getY());

                if(childView != null && mListener != null)
                {
                    mListener.onItemLongClick(childView, recyclerView.getChildPosition(childView));
                }
            }
        });
    }

    public void getViewHierarchyUnderChild(ViewGroup root, float x, float y, List<View> viewHierarchy) {
        int[] location = new int[2];
        final int childCount = root.getChildCount();

        for (int i = 0; i < childCount; ++i) {
            final View child = root.getChildAt(i);
            child.getLocationOnScreen(location);
            final int childLeft = location[0], childRight = childLeft + child.getWidth();
            final int childTop = location[1], childBottom = childTop + child.getHeight();

            if (child.isShown() && x >= childLeft && x <= childRight && y >= childTop && y <= childBottom) {
                viewHierarchy.add(0, child);
            }
            if (child instanceof ViewGroup) {
                getViewHierarchyUnderChild((ViewGroup) child, x, y, viewHierarchy);
            }
        }
    }

    @Override
    public boolean onInterceptTouchEvent(RecyclerView view, MotionEvent e)
    {
        mGestureDetector.onTouchEvent(e);

        return false;
    }

    @Override
    public void onTouchEvent(RecyclerView view, MotionEvent motionEvent){}

    @Override
    public void onRequestDisallowInterceptTouchEvent(boolean disallowIntercept) {

    }
}

次に、アクティビティ/フラグメントからそれを使用します。

recyclerView.addOnItemTouchListener(createItemClickListener(recyclerView));

    public RecyclerItemClickListener createItemClickListener(final RecyclerView recyclerView) {
        return new RecyclerItemClickListener (context, recyclerView, new RecyclerItemClickListener.OnItemClickListener() {
            @Override
            public void onItemClick(View view, int position) {
                if (view instanceof AppCompatButton) {
                    // ... tapped on the button, so go do something
                } else {
                    // ... tapped on the item container (row), so do something different
                }
            }

            @Override
            public void onItemLongClick(View view, int position) {
            }
        });
    }

1

onInterceptTouchEvent()クリックイベントを処理するときは、内部でtrueを返す必要があります。


1
こんにちは、これについてもっと詳しく説明していただけますか。次のコードを使用して、削除ボタンにバインドしますbtnDelete =(ImageButton)itemView.findViewById(R.id.btnDelete); btnDelete.setOnClickListener(new View.OnClickListener(){@ Override public void onClick(View view){remove(getLayoutPosition());}});
Ashwani K 2015

onTouchEvent()と同様に、戻り値は、イベントが処理されたかどうか、およびイベントが処理されなかった場合はイベントが行全体に渡されたことを示します。
Eliyahu Shwartz 2015年

0

最初に同様のエントリがあるかどうかを確認できます。サイズが0のコレクションを取得した場合は、新しいクエリを開始して保存します。

または

より専門的でより速い方法。クラウドトリガーを作成する(保存する前に)

この答えをチェックしてください くださいhttps://stackoverflow.com/a/35194514/1388852


0

getItemIdという名前のオーバーライドメソッドを配置するだけです。右クリック>生成>オーバーライドメソッド> getItemIdで取得します。このメソッドをAdapterクラスに配置します。

弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.