11

Ich verwende einen RecyclerView mit einem einzelnen Zeilenlayout mit einem ImageView und einem TextView.Position von View in onCreateViewHolder erhalten

Ich möchte einen OnClickListener für die Ansicht und nicht für separate ViewHolder-Objekte implementieren. Wie kann ich die Position der Ansicht im Adapter bekommen?

Momentan lösche ich Kommentare bei Klick, aber ich kann die angeklickte Ansicht nicht auswählen. Ich habe ein TODO in der entsprechenden Zeile hinzugefügt.

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); 
     } 
    } 
} 
+0

Sie sollten wirklich stellen Sie Ihren OnClickListener in ViewHolder (endgültige Ansicht itemView) Konstruktor, in diesem Fall ViewHolder implementieren sollten verfügbar ist OnClickListener – pskink

Antwort

31

Sie können nicht verwenden, um die position Parameter von onBindViewHolder in einem Rückruf. Wenn oben ein neuer Artikel hinzugefügt wird, wird RecyclerView nicht erneut binden, so dass die Position veraltet ist. RecyclerView bietet stattdessen eine getAdapterPosition Methode auf dem 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; 
} 

Ich habe hinzugefügt position != RecyclerView.NO_POSITION Kontrolle, weil, wenn Artikel entfernt wird, wird RecyclerView die Ansicht ausgeblendet, so dass es nach wie vor durch den Benutzer, sondern seine Adapterposition NO_POSITION zurückkehren wird angeklickt werden kann.

+2

Was auch immer ich klicke, ich bekomme Position -1 (RecyclerView.NO_POSITION), was wird das Problem sein? – akki

-1

Überschreibung getItemViewType Methode:

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

Nun ist die Position auf Viewtype

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

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

Tun Sie dies nicht - es wird dafür sorgen, dass der Adapter für jedes Element in Ihrer Liste eine neue Ansicht erstellt, anstatt sie zu recyceln, da er denkt, dass es sich um verschiedene Typen handelt. – starkej2