Получение позиции View в onCreateViewHolder

Я использую RecyclerView с однострочным макетом с ImageView и TextView.

Я хочу реализовать OnClickListener для представления, а не для отдельных объектов ViewHolder. Как я могу получить позицию представления в адаптере?

Прямо сейчас я удаляю комментарии по клику, но не могу выбрать вид, по которому щелкнули. Я добавил TODO в соответствующую строку.

public class CommentAdapter extends RecyclerView.Adapter<CommentAdapter.ViewHolder> {

    /** List of Comment objects */
    private List<Comment> mCommentList;

    /** Database with Comment objects */
    private CommentsDataSource mDataSource;

    /**
     * Construcutor for CommentAdapter
     *
     * @param commentList   List of Comment objects
     * @param dataSource    Database with Comment objects
     */
    public CommentAdapter(List<Comment> commentList, CommentsDataSource dataSource) {
        this.mCommentList = commentList;
        this.mDataSource = dataSource;
    }

    /**
     * Add Comment objects to RecyclerView
     *
     * @param position  The position where the Comment object is added
     * @param comment   Comment Object
     */
    public void add(int position, Comment comment) {
        mCommentList.add(position, comment);
        notifyItemInserted(position);
    }

    /**
     * Remove Comment objects from RecyclerView
     *
     * @param comment Comment Object
     */
    public void remove(Comment comment) {
        int position = mCommentList.indexOf(comment);
        // Avoid double tap remove
        if (position != -1) {
            mCommentList.remove(position);
            mDataSource.deleteComment(comment);
            notifyItemRemoved(position);
        }
    }

    @Override
    public ViewHolder onCreateViewHolder(final ViewGroup parent, int viewType) {
        final View view = LayoutInflater.from(parent.getContext())
                .inflate(R.layout.single_line_row, parent, false);
        view.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                // TODO get position
                remove(mCommentList.get(getItemCount() - 1));
            }
        });
        return new ViewHolder(view);
    }

    @Override
    public void onBindViewHolder(ViewHolder holder, int position) {
        final Comment comment = mCommentList.get(position);
        holder.comment.setText(comment.getComment());
    }

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

    public static class ViewHolder extends RecyclerView.ViewHolder {

        /** ImageView icon */
        public ImageView icon;

        /** TextView comment */
        public TextView comment;

        /**
         * Constructor for ViewHolder
         *
         * @param itemView Layout for each row of RecyclerView
         */
        public ViewHolder(final View itemView) {
            super(itemView);
            icon = (ImageView) itemView.findViewById(R.id.icon);
            comment = (TextView) itemView.findViewById(R.id.comment);
        }
    }
}

android java android-recyclerview android-view android-viewholder
person Thomas Mohr    schedule 24.04.2015    source источник
comment
вам действительно следует установить свой OnClickListener в конструкторе ViewHolder (final View itemView), в этом случае ViewHolder должен реализовать OnClickListener   -  person pskink    schedule 24.04.2015

Ответы (3)


arrow_upward
49
arrow_downward

Вы не можете использовать параметр position для onBindViewHolder в обратном вызове. Если выше добавлен новый элемент, RecyclerView не будет перепривязывать ваш элемент, чтобы его позиция стала устаревшей. Вместо этого RecyclerView предоставляет метод getAdapterPosition для ViewHolder.

@Override
public ViewHolder onCreateViewHolder(final ViewGroup parent, int viewType) {
    final View view = LayoutInflater.from(parent.getContext())
            .inflate(R.layout.single_line_row, parent, false);
    final ViewHolder holder = new ViewHolder(view);
    view.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View v) {
            final int position = holder.getAdapterPosition();
            if (position != RecyclerView.NO_POSITION) {
                remove(mCommentList.get(position));
            }
        }
    });
    return holder;
}

Я добавил проверку position != RecyclerView.NO_POSITION, потому что, когда элемент удаляется, RecyclerView исчезает из представления, поэтому пользователь все еще может щелкнуть его, но его позиция адаптера вернет NO_POSITION.

person yigit    schedule 25.04.2015
comment
Что бы я ни нажимал, я получаю позицию -1 (RecyclerView.NO_POSITION), в чем может быть проблема? - person akki; 31.05.2017
comment
Имеет ли значение, если вы установите OnClickListener на view или holder? Сначала я связал его с держателем, а затем изменил его на представление, увидев этот ответ, но, похоже, на самом деле это не имело значения - по крайней мере, не с RecycleView, который не прокручивался (для этого недостаточно элементов). - person Neph; 03.03.2020

arrow_upward
1
arrow_downward

вы можете создать метод для обновления позиции в вашем классе.

в моем случае мне нужно прикрепить watcher и получить позицию для обновления arraylist. вот пример:

class DodolWatcher bla bla {
    private var position: Int = 0
    fun updatePosition(pos:Int)
    {
      position = pos
    }

    override fun onTextChanged(charSequence: CharSequence, i: Int, i2: Int, i3: Int) {
    Log.d("test", position.toString())
    }

}

и в вашем onCreateViewHolder вы можете прикрепить наблюдателя к edittext

 override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): RVPaymentMethodAdapter.ViewHolder {
     ....
     bla bla
     ....
     theWatcher = DodolWatcher() <-- this is the trick
     amount.addTextChangedListener(theWatcher)
 }

и вы сможете обновить позицию в своем bindViewHolder следующим образом:

override fun onBindViewHolder(viewHolder: RVPaymentMethodAdapter.ViewHolder, position: Int) {
     theWatcher.updatePosition(viewHolder.adapterPosition)  <-- this is the trick
 }
person Kakashi    schedule 30.07.2018

arrow_upward
-4
arrow_downward

переопределите метод getItemViewType в вашем адаптере:

@Override
public int getItemViewType(int position) {
    //...
    return position;
}

Теперь viewType - это позиция

 @Override
    public ViewHolder onCreateViewHolder(ViewGroup parent, int viewType) {
       int position=viewType; //position 
        //your code 

        View view = LayoutInflater.from(parent.getContext()).inflate(R.layout.item_recursive, parent, false);
        return new ViewHolder(view);
    }

Например

public class BrandBtnAdapter extends RecyclerView.Adapter<BrandBtnAdapter.MyViewHolder>
{

     //............

    @Override
    public int getItemViewType(int position)
    {
        //...
        return position;
    }

    @Override
    public BrandBtnAdapter.MyViewHolder onCreateViewHolder(ViewGroup parent, int viewType)
    {
        int position = viewType; //position 

        final View itemView = mInflater.inflate(mResource, parent, false);
        return new BrandBtnAdapter.MyViewHolder(itemView);
    }

}
person Hossein Hajizadeh    schedule 29.06.2017
comment
Не делайте этого — это заставит адаптер создать новое представление для каждого элемента в вашем списке, а не перерабатывать их, поскольку он будет думать, что все они разных типов. - person starkej2; 25.07.2017
comment
плохой способ сделать это. - person Irfan; 11.06.2019